Novawave

we have just got the Classified mod too is there any way that the Feedback could show in there maybe?.. a sidebar would be cool.. If there is any Addons we can use please could you let us know.

Yes, classifieds integration is available. I can provide the code via PM.

My problem is when you submit feedback there is an error message. The feedback still goes through but it happens every time. Also if you mod a feedback same thing happens.

Please check http://forums.novawave.ca/topic/11409-bad-notify-key-error/

Simply upload the new files and overwrite. There is currently no scripted upgrade required.

I'm not sure that these two things are exactly related. There is a version of the script that may fix this issue.

Please PM me and I'll get you a copy of it later today.

Sorry, new features development has to be put on hold. I just have too heavy of a workload at the moment, so I'll be focusing on bug fixes, updates for new IPB versions, and similar support.

If anyone has any mods to this script that add new features, I can offer a refund of your purchase price for their inclusion in future releases. PM me.

The notification issue has several reported fixes, from recaching, saving notify preferences, or if those don't work there is an easy way to disable notifications now. Again, it doesn't affect everyone. All of the reported bugs have been fixed, or a fix exists.

I haven't had time to put out version 1.1, but the mod will definitely be compatible with IPB 3.2.

The system seems to be working now, but there seems to be no confirmation page when adding feedback. It just goes to a blank page.

That's not working as intended. I briefly uploaded a version of the nonotify script with the redirect accidentally commented out. You may have it, please upload the latest version again.

http://fearless-assassins.com/forum/index.php?app=feedback&u=3446

When you click on 'Testing' it moves you to forum index page while it should move you to that topic and/or post.

Your feedback URLs aren't being entered into the DB properly. Try the script I PMed you.

It's a tricky thing to implement without being restrictive. Say a person starts a topic, and completes a deal with someone who PMed them. Who gets to leave/receive feedback? Fake feedback could still be left by people posting in each other's topics. The trader has to take some diligence in verifying someone's feedback, imo.

However, every forum is different, and I could possibly guide you to mod the script to fit your needs.

Please PM me the URL to your forum, and I'll write the code which should work for it.

Please see http://forums.novawave.ca/topic/11409-bad-notify-key-error/

Check around line 200 to edit the output link structure, near:

if (ctype_digit($row['link'])) {

Oh, I missed your edit. I'll try and find the time to do this for you shortly.

Ah, I hadn't realized you were asking for directions. Do you know PHP at all? It would be a matter of changing _getTidFromUrl() in /admin/applications_addon/other/feedback/modules_public/view/view.php

I could look into writing it if you aren't sure how. Doing an str_replace() specific to your forum's URL, then just outputting the ID substring would be the easiest way.

Yes, you're certainly allowed to make modifications to the app. The license has the details: http://novascripts.com/feedback_license.php

It'd be great to see what you come up with!

Yes, under the install directions there is an optional code edit to show the feedback scores next to posts.

We'll I've edited the code and deleted the "index.php?/' from the url before the topic id is extraced and now i can paste the full url. It's a dirty hack but, hey it works. Mabe this kind of sanitizing should be there by default ...

Interesting that this is the case, because the code is the same as IPB itself uses to extract ids in the moderation tools. Nonetheless, I'll make incremental update, 1.0.6 which should fix this.

Would it be possible to include this as options in next release? I would like to see custom role support if possible so that members can leave feedback under 'General' if user is not trader, buyer or seller. Will help by good amount.

Also, in each page load how many queries it uses? Just curious on performance of application.

I'm not sure about the feasibility of automating this without a big rewrite, but I'll look into it.

If you have IPB set to output debug info, the number of queries will be displayed on the bottom of the page.

I am getting below error, while giving feedback to someone. Using the new version from download.

Fatal error: Uncaught exception 'Exception' with message 'BAD_NOTIFY_KEY'

Please see http://forums.novawave.ca/topic/11409-bad-notify-key-error/

Also, I have FURL enabled. Why it's using simple URL structure then?

Under Search Engine Optimization settings in the ACP, enable 'Redirect to new friendly URL format'.

Right now there's no way to do this without coding. Seller, Buyer, etc are just language strings based on the value of the 'type' column in the database. You would need to edit /admin/applications_addon/other/feedback/modules_public/view/view.php, under the '//who are you?' comment, and the leave feedback skin template.

Let me know if you're interested in modifying this and need help.

Hi there, did you manage to find the error? I have had to disable the system because it still doesnt work.

Sorry, I should have made it more clear I was talking directly to you a few posts back. I made a revision on Feb 11 to 1.0.5 which should fix the error you were experiencing. Please download and upload again. If you're still having problems let me know.

When did you last download the script? If it was before Feb. 3, download the newest copy at http://novascripts.com or

Of course you can, my username is PooM :)

Is issue resolved? I am waiting for your post to buy this mod :P

Should be fine now. In the NoNotify version of the script, I accidentally cut off the redirection after feedback is left for a user. I've fixed it in the download.

If you were affected, you can redownload, or PM me for the working copy of the affected file.

@Fisherking, I reported your post because it contained the code to the mod. Keep in mind it is a paid mod.

Regarding your question, feel free to PM me your modified code. Or, you can use the version of the mod contained in the *NoNotify.zip.

What is your account? Do you mind if I leave feedback for testing?

Please send me the link and I'll check it out.

1.0.5 is out. If you are getting a blank page, that means another error is occurring. Is feedback being left even with the blank page? Check your error logs, or turn error displaying on if you can.

Is there possibility such that only staff or admins can add feedback?

Any group permissions?

Yes, there are full group permissions.

Entering the full URL doesn't work...

However, if you enter the topic code (not very intuitive for members), it brings up this error as soon as you click Submit. But it does add the Feedback successfully.

Ideally, it needs to be a full URL of the topic, as this is what members will be trying to do.

Fatal error: Uncaught exception 'Exception' with message 'BAD_NOTIFY_KEY' in /home/civiclif/public_html/board/admin/sources/classes/member/notifications.php:457 Stack trace: #0


/home/civiclif/public_html/board/admin/applications_addon/other/feedback/modules_public/view/view.php(319): notifications->sendNotification() #1


/home/civiclif/public_html/board/admin/applications_addon/other/feedback/modules_public/view/view.php(57): public_feedback_view_view->leave_feedback('32391') #2


/home/civiclif/public_html/board/admin/sources/base/ipsController.php(285): public_feedback_view_view->doExecute(Object(ipsRegistry)) #3 


/home/civiclif/public_html/board/admin/sources/base/ipsController.php(112): ipsCommand->execute(Object(ipsRegistry)) #4 


/home/civiclif/public_html/board/admin/sources/base/ipsController.php(65): ipsController->handleRequest() #5 


/home/civiclif/public_html/board/index.php(26): ipsController::run() #6 {main} thrown in 


/home/civiclif/public_html/board/admin/sources/classes/member/notifications.php on line 457

The bad notify key error is unrelated to the topic id error. The topic url problem should be fixed in 1.0.5. If you have the bad notify error, please use the version of the script contained in the *_nonotify.zip, or see http://novatechforums.com/topic/11409-bad-notify-key-error/.

I laso have the 1.0.5-1 but when I delete all files on server, re upload and choose install it says 1.0!!!!!

This is fine. The incremental updates don't require the IPB update script to run, so the version still shows as 1.0. If you installed 1.0.5, you have that version.

Hi there

Yes I am sure I have uploaded the updates. The file date is yesterdays. Any ideas?

Really want this up and running asap! :)

Does this also occur when just entering in a number, without the full URL?

installed aftert the book, all permission are set, however I return to the index clicking that feedback link in the menu!!!!

Can you PM me a link, so I can look into it.

Hi there

I have re-uploaded the "upload" folder of the new version and I am still getting the same issue.

Am I doing something wrong?

Let me know.

Cheers

Steve

Are you sure /admin/applications_addon/other/feedback/modules_public/view/view.php's modification date is yesterday? What I did in this update is replace the new topic extractor a few posts back.

If you already purchased the mod how do you download the update? Its wanting me to buy it again.

PM me your account email, I'll make sure you're in the customer's list. Otherwise, you can grab it from http://novascripts.com.