Novawave Posted March 2, 2011 Share Posted March 2, 2011 Yes, under the install directions there is an optional code edit to show the feedback scores next to posts. Link to comment Share on other sites More sharing options...
davvowl Posted March 2, 2011 Share Posted March 2, 2011 great mod ! mikkee reported earlier, that by going to Members -> Notification Defaults and saving the page, it seems to fix the notification key error. had that problem too, and solved itself the same way (just thought i'd share i got a suggestion here http://community.invisionpower.com/files/file/3103-classifieds/ im going to use this mod for my forum's classifieds. the links looks like this : websiteurl/forum/classifieds/item/12345-item-name/ so its basically the same as board topics furl's is there a way i can tweak your otherwise perfect mod to use this ? --- and you should contact the guys doing ip.classifieds and work something out together ;) Link to comment Share on other sites More sharing options...
Novawave Posted March 2, 2011 Share Posted March 2, 2011 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! Link to comment Share on other sites More sharing options...
davvowl Posted March 2, 2011 Share Posted March 2, 2011 directions please ? im new with ipb 3x hooks and stuff what do i need to change so your mod accepts classifieds urls ? (see my previous post) thanks edit damn. the feedbacks arent showing in the topic view. i did add m.feedback_score, any ideas ? Link to comment Share on other sites More sharing options...
Novawave Posted March 2, 2011 Share Posted March 2, 2011 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. davvowl 1 Link to comment Share on other sites More sharing options...
davvowl Posted March 2, 2011 Share Posted March 2, 2011 ok thx for direction imma look into it right now. will it also replace the links in the feedback list ? websiteurl/forum/index.php?app=feedback&u=X edit: wow, that php is a bit confusing, im a little rusty here. i havent done any serious php in some years, back when it wasnt object oriented :P edit2: yeah, would definitely appreciate some help here (understand: do it for me, pretty please :)) Link to comment Share on other sites More sharing options...
Novawave Posted March 2, 2011 Share Posted March 2, 2011 (edited) 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. Edited March 2, 2011 by Novawave davvowl 1 Link to comment Share on other sites More sharing options...
davvowl Posted March 2, 2011 Share Posted March 2, 2011 awesome, thanks in advance. i'm sure more people would love to use traderfeedback along with classifieds. you gotta work something out, it makes perfect sense to me :P Link to comment Share on other sites More sharing options...
PECarter Posted March 3, 2011 Share Posted March 3, 2011 I get this as well...Fatal error: Uncaught exception 'Exception' with message 'BAD_NOTIFY_KEY' in /home/mainelys/public_html/forums/admin/sources/classes/member/notifications.php:457 Stack trace: #0 /home/mainelys/public_html/forums/admin/applications_addon/other/feedback/modules_public/view/view.php(319): notifications->sendNotification() #1 /home/mainelys/public_html/forums/admin/applications_addon/other/feedback/modules_public/view/view.php(57): public_feedback_view_view->leave_feedback('152') #2 /home/mainelys/public_html/forums/admin/sources/base/ipsController.php(285): public_feedback_view_view->doExecute(Object(ipsRegistry)) #3 /home/mainelys/public_html/forums/admin/sources/base/ipsController.php(112): ipsCommand->execute(Object(ipsRegistry)) #4 /home/mainelys/public_html/forums/admin/sources/base/ipsController.php(65): ipsController->handleRequest() #5 /home/mainelys/public_html/forums/index.php(26): ipsController::run() #6 {main} thrown in /home/mainelys/public_html/forums/admin/sources/classes/member/notifications.php on line 457 Link to comment Share on other sites More sharing options...
Novawave Posted March 3, 2011 Share Posted March 3, 2011 Please see http://forums.novawave.ca/topic/11409-bad-notify-key-error/ Link to comment Share on other sites More sharing options...
davvowl Posted March 5, 2011 Share Posted March 5, 2011 any news ? :) my community needs a feedback system :P Link to comment Share on other sites More sharing options...
Novawave Posted March 5, 2011 Share Posted March 5, 2011 Please PM me the URL to your forum, and I'll write the code which should work for it. davvowl 1 Link to comment Share on other sites More sharing options...
davvowl Posted March 5, 2011 Share Posted March 5, 2011 link sent, thanks alot Link to comment Share on other sites More sharing options...
davvowl Posted March 7, 2011 Share Posted March 7, 2011 mad props to nathan for top-notch support ! :cool: Link to comment Share on other sites More sharing options...
Jinkler Posted March 10, 2011 Share Posted March 10, 2011 Can I make a feature suggestion? Is it possible to tie-in more closely somehow the topic that feedback is being left for? At the moment, it's simply based on trust. I'm wondering if it can be adapted so you can only leave specific feedback if you particpated in that topic, and you were a topic starter. Link to comment Share on other sites More sharing options...
Novawave Posted March 10, 2011 Share Posted March 10, 2011 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. Link to comment Share on other sites More sharing options...
Jinkler Posted March 10, 2011 Share Posted March 10, 2011 Thanks, modding it will be far beyond my skills I'm afraid! Link to comment Share on other sites More sharing options...
poomv2 Posted March 18, 2011 Share Posted March 18, 2011 Hi there Bit of an update... 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. Is this an error or is it a feature that hasn't been added yet? I think it's quite important from a users point of view. Let me know. Cheers Steve Link to comment Share on other sites More sharing options...
AlexJ Posted March 18, 2011 Share Posted March 18, 2011 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. Link to comment Share on other sites More sharing options...
Novawave Posted March 19, 2011 Share Posted March 19, 2011 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. Link to comment Share on other sites More sharing options...
Maxxius Posted March 28, 2011 Share Posted March 28, 2011 Were notification errors fixed in this mod yet? Is mod stable now? When the new version is coming out (like 1.1.0)? I really hesitate to purchase this mod because of these bugs that keep popping out for people? :ermm: Also will the mod be upgraded to work with IPB 3.2.0 when it comes out? Link to comment Share on other sites More sharing options...
Novawave Posted March 28, 2011 Share Posted March 28, 2011 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. Link to comment Share on other sites More sharing options...
Zenithlt Posted April 1, 2011 Share Posted April 1, 2011 Sorry, I spoke too soon. I forgot that the values for positive and negative aren't stored in the member row, only the aggregate score. It would be extremely inefficient to calculate this in the hook in every post. That's a good idea. I'll implement it next release. PMed it to you. So what about my request? Link to comment Share on other sites More sharing options...
Novawave Posted April 1, 2011 Share Posted April 1, 2011 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. Maxxius 1 Link to comment Share on other sites More sharing options...
Jinkler Posted May 11, 2011 Share Posted May 11, 2011 Hi, Could you help me pinpoint an error? I've made a change to my forums. Now, when an item has been sold, I'm using the "multi-moderation" feature to move it to an archive forum. This has had the knock-on effect of somehow stopping feedback from being left, everyone now gets: [#1] Invalid deal topic Any ideas? Thanks Link to comment Share on other sites More sharing options...
Recommended Posts