Invision Community 4: SEO, prepare for v5 and dormant account notifications By Matt Monday at 02:04 PM
Novawave Posted February 9, 2011 Posted February 9, 2011 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.
poomv2 Posted February 10, 2011 Posted February 10, 2011 Thanks, that fixed the notification error. However, there is no message to say "User Feedback submitted succesfully", etc? It just takes me to a blank page. Not very intuitive for the user. When is 1.0.5 coming out? As there is no point in using the system unless you can use full topic URLs. Members are lazy, and will not want to put the effort in extracting the Topic ID from the URL. Thanks Steve :)
Novawave Posted February 10, 2011 Posted February 10, 2011 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.
poomv2 Posted February 10, 2011 Posted February 10, 2011 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. I have checked the error logs and they are set to view all level errors, no errors since yesterday. :unsure:
Novawave Posted February 10, 2011 Posted February 10, 2011 Please send me the link and I'll check it out.
poomv2 Posted February 10, 2011 Posted February 10, 2011 Thanks mate. The link to my board is... Www.civiclife.net Thanks :)
Novawave Posted February 10, 2011 Posted February 10, 2011 What is your account? Do you mind if I leave feedback for testing?
Fisherking Posted February 10, 2011 Posted February 10, 2011 (edited) done you steps. still the error. did delete line 310-319Fatal error: Uncaught exception 'Exception' with message 'BAD_NOTIFY_KEY' in /home/ekorn106/public_html/forum/admin/sources/classes/member/notifications.php:457 Stack trace: #0 /home/ekorn106/public_html/forum/admin/applications_addon/other/feedback/modules_public/view/view.php(319): notifications->sendNotification() #1 /home/ekorn106/public_html/forum/admin/applications_addon/other/feedback/modules_public/view/view.php(57): public_feedback_view_view->leave_feedback('2') #2 /home/ekorn106/public_html/forum/admin/sources/base/ipsController.php(285): public_feedback_view_view->doExecute(Object(ipsRegistry)) #3 /home/ekorn106/public_html/forum/admin/sources/base/ipsController.php(112): ipsCommand->execute(Object(ipsRegistry)) #4 /home/ekorn106/public_html/forum/admin/sources/base/ipsController.php(65): ipsController->handleRequest() #5 /home/ekorn106/public_html/forum/index.php(26): ipsController::run() #6 {main} thrown in /home/ekorn106/public_html/forum/admin/sources/classes/member/notifications.php on line 457 Edited February 11, 2011 by Kessler Removed source code
Novawave Posted February 11, 2011 Posted February 11, 2011 @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.
poomv2 Posted February 11, 2011 Posted February 11, 2011 What is your account? Do you mind if I leave feedback for testing? Of course you can, my username is PooM :)
AlexJ Posted February 11, 2011 Posted February 11, 2011 Of course you can, my username is PooM :) Is issue resolved? I am waiting for your post to buy this mod :P
Novawave Posted February 11, 2011 Posted February 11, 2011 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.
Jinkler Posted February 22, 2011 Posted February 22, 2011 I've just released an incremental update which should resolve this issue. No need to run the upgrader, just upload the changed files. How do I get hold of this update? I'm also having the [#1] Invalid deal topic error. Thanks.
Novawave Posted February 22, 2011 Posted February 22, 2011 When did you last download the script? If it was before Feb. 3, download the newest copy at http://novascripts.com or
poomv2 Posted February 22, 2011 Posted February 22, 2011 Hi there, did you manage to find the error? I have had to disable the system because it still doesnt work. Let me know Cheers Steve
Novawave Posted February 22, 2011 Posted February 22, 2011 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.
AlexJ Posted February 24, 2011 Posted February 24, 2011 How can I add more Role? Example: General? At present, I only see 3. Buyer, Seller, Trader.
Novawave Posted February 24, 2011 Posted February 24, 2011 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.
Cosmin Dumitru Posted February 24, 2011 Posted February 24, 2011 Hi, I've just purchased the script and I get the infamous [#1] Invalid deal topic error. Script version is 1.0.5 and URL structure is :http://forums.rgc.ro...rtul-steve-vai/ so nice URLS but still a '?' there ... Later edit: 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 ...
AlexJ Posted February 24, 2011 Posted February 24, 2011 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. 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.
AlexJ Posted February 24, 2011 Posted February 24, 2011 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' in /home/xxxxxx/public_html/forum/admin/sources/classes/member/notifications.php:457 Stack trace: #0 /home/xxxxxx/public_html/forum/admin/applications_addon/other/feedback/modules_public/view/view.php(319): notifications->sendNotification() #1 /home/xxxxxx/public_html/forum/admin/applications_addon/other/feedback/modules_public/view/view.php(57): public_feedback_view_view->leave_feedback('5470') #2 /home/xxxxxx/public_html/forum/admin/sources/base/ipsController.php(285): public_feedback_view_view->doExecute(Object(ipsRegistry)) #3 /home/xxxxxx/public_html/forum/admin/sources/base/ipsController.php(112): ipsCommand->execute(Object(ipsRegistry)) #4 /home/xxxxxx/public_html/forum/admin/sources/base/ipsController.php(65): ipsController->handleRequest() #5 /home/xxxxxx/public_html/forum/index.php(26): ipsController::run() #6 {main} thrown in /home/xxxxxx/public_html/forum/admin/sources/classes/member/notifications.php on line 4
AlexJ Posted February 25, 2011 Posted February 25, 2011 Other then that bad notify key error, there is one more issue. When you supply topic link, while giving someone reputation, it doesn't work as intended. Example: 1. Giving someone reputation. 2. +1 and put link in form. 3. Hit Enter. 4. User get reputation. 5. When you click on the reputation, it should take you to right topic link but it redirects to wrong link. http://fearless-assassins.com/forum/index.php?showtopic=0 Also, I have FURL enabled. Why it's using simple URL structure then?
Novawave Posted February 25, 2011 Posted February 25, 2011 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'.
Fluent Inc Posted March 2, 2011 Posted March 2, 2011 Is it possible to display the users actual feedback some how? Rather then just having a view link it would be useful to quickly see who actually has feedback to view and not.
Recommended Posts