stoo2000 Posted May 23, 2014 Author Posted May 23, 2014 Typo: feedback_left Your feedback has been left on the users profile. Should be: feedback_left Your feedback has been left on the user's profile. Fixed Hi, found a word that may be hardcoded in the source code. * The word "N/A" at the user's profile Please refer to the attached file. Please let me know the files and lines to edit. Thanks. Fixed Kudos for having the iTrader converter in place. This saved me a lot of time, as I didn't have to write my own converter! However, the user profile rebuild was hardcoded to 50 profiles per batch. With a forum with almost 450,000 users, that would take us days to finish. Edited modules_admin/tools/rebuild.php and changed: public $perCycle = 50; to public $perCycle = 500; Suggestion for the next verion: this value to be configurable from inside the ACP. Thanks for the great work. For the amount of data to process in a loop, we need to consider underpowered systems, we want to avoid timing out. - Some converters also have a query per item that connects to a remote database. Just noticed you were actually on about the rebuild process, however it's quite similar we're avoiding a timeout. Re: N/A Hardcoded in: sources/cache.php and in: xml/feedback_root_templates.xml (3 times, i.e., profilePage, topMembers, and userPaneFeedback templates) Also, on the same XML file the following words are hardcoded, also three times, on the same templates: positive neutral negative I fixed here by creating these words and replacing their hardcoded versions with the correct variable. Thanks. positive/neutral/negative are class names, don't change those. It's due for a minor maintenance release, I'll update soon.
Gabriel Torres Posted May 23, 2014 Posted May 23, 2014 positive/neutral/negative are class names, don't change those. It's due for a minor maintenance release, I'll update soon. I noticed that and removed my previous message... Thanks for the prompt reply, and thank you for your hard work. I have a couple of suggestions, if I may. 1. On the user profile, on the table that list all transactions, each message ("note") has a link to the original topic that relates to the transaction. This is great. However, the link uses the regular index.php?showtopic=xxxxx format; I'd change that to include SEO, which is pretty easy to implement. 2. When you click on the "leave feedback" button on the classifieds page of a given product, you need to enter the link of the product manually on the form that shows up on the next page. Why the system simply doesn't carry over the item number and fills this field automatically for me? (I am using the Classifieds app and I am assuming the user will leave feedback to that particular product/transaction). Once again, thank you very much. Gabriel
stoo2000 Posted May 23, 2014 Author Posted May 23, 2014 I noticed that and removed my previous message... Thanks for the prompt reply, and thank you for your hard work. I have a couple of suggestions, if I may. 1. On the user profile, on the table that list all transactions, each message ("note") has a link to the original topic that relates to the transaction. This is great. However, the link uses the regular index.php?showtopic=xxxxx format; I'd change that to include SEO, which is pretty easy to implement. This can happen if you have rows for classifieds, but it isn't installed or active. {parse url="showtopic={$f['link']}" seotitle="{$f['title_seo']}" template="showtopic" base="public"} 2. When you click on the "leave feedback" button on the classifieds page of a given product, you need to enter the link of the product manually on the form that shows up on the next page. Why the system simply doesn't carry over the item number and fills this field automatically for me? (I am using the Classifieds app and I am assuming the user will leave feedback to that particular product/transaction). Once again, thank you very much. Gabriel I should be able to add something to take care of that.
stoo2000 Posted May 24, 2014 Author Posted May 24, 2014 1.4.2 is now in the Marketplace. What's New in Version 1.4.2 (See full changelog) Enhancement: Setting to prevent users from leaving more than one Feedback per deal Enhancement: Feedback data now part of the User Hover Card Enhancement: Removed app copyright Enhancement: SMF Trader 1.6 Converter (Beta - Needs testing) Enhancement: Clicking 'leave feedback' from a classifieds item now pre-fills the Feedback form Bugfix: Fixed a few language strings Tripp★, Maxxius and Gabriel Torres 3
Gabriel Torres Posted May 25, 2014 Posted May 25, 2014 Great. I've just upgraded, it is working like a charm. Thank you so much for taking my suggestions into consideration. However, I didn't understand your instructions for my question above: This can happen if you have rows for classifieds, but it isn't installed or active.
stoo2000 Posted May 25, 2014 Author Posted May 25, 2014 Great. I've just upgraded, it is working like a charm. Thank you so much for taking my suggestions into consideration. However, I didn't understand your instructions for my question above: I remembered after I posted that, that you've converted the data. This is because you've switched to 'classifieds' mode, the side affect of this is that we don't join on the topics table, the join is to the classifieds table instead. The result of this is that we don't have the 'title_seo' data for the feedback data associated with a topic. Hope that clears it up. Stuart
Gabriel Torres Posted May 26, 2014 Posted May 26, 2014 Ok, I hope you can change that in the future. Thanks.
Jυra Posted May 26, 2014 Posted May 26, 2014 How easy is it just to use the profile part of this and not the topic view or page parts? Settings?
stoo2000 Posted May 26, 2014 Author Posted May 26, 2014 How easy is it just to use the profile part of this and not the topic view or page parts? Settings? There are settings so you can choose where the feedback data is shown, PMs, Topics (and in which forums), Hover Card. In theory you could disable all of those, so it's just a tab on the users profile Jυra 1
Gabriel Torres Posted June 13, 2014 Posted June 13, 2014 Hi, stoo2000, There is a conflict with this module and Classifieds as explained below through my message exchange with Andy (Classifieds' developer): Gabriel: Hi,The price of the product is only displayed at the main/category list.When you click on an advert, there is no price listed whatsoever! So, our users are getting confused.Please advise how we could add a price field to be displayed or even the price on top of the images like it happens on the main list.Thanks,Gabriel. Andy: Hi Gabriel, The problem was a conflict with the trader feedback addon. This app also defines a css style called "price_box" which was overriding the classifieds one. As it had a declaration of display:none it was hiding the price element in the classifieds basic_info template. I've added an inline style in the classifieds template to make sure this is used.Please let me know if you require any further assistance.RegardsAndy
stoo2000 Posted June 13, 2014 Author Posted June 13, 2014 Hi, stoo2000, There is a conflict with this module and Classifieds as explained below through my message exchange with Andy (Classifieds' developer): Hi Gabriel, The stock CSS for the app does not define a .price_box rule I'm afraid, this may be a customisation that has been added to your style sheet. Thanks Stuart
Gabriel Torres Posted June 14, 2014 Posted June 14, 2014 Ops, yes, you are right... This was something I was playing with six months ago, now I remember... Sorry for bothering you and thanks for the clarification!
Jujulefooteux Posted June 22, 2014 Posted June 22, 2014 thx a lot for the updated !! Great Enhancement: Setting to prevent users from leaving more than one Feedback per deal :wub: :sorcerer:
IPBjard Posted July 2, 2014 Posted July 2, 2014 Hello, Can someone explain me how to increase feedback comment characters limit ? because at this moment limit is only 80 Characters and sometimes it is hard to write good feedback.
stoo2000 Posted July 7, 2014 Author Posted July 7, 2014 IPS4 Information This app will be updated for IPS4, providing that you have an active renewal you will be able to access the new version at no additional charge. Gabriel Torres 1
Daniel Ocio Posted July 14, 2014 Posted July 14, 2014 I add a bug/ticket support in your web. The convert tool is not working for me
Daniel Ocio Posted July 16, 2014 Posted July 16, 2014 HELP please http://ipb.silvesterwebdesigns.com/topic/254-error-in-conversion/
stoo2000 Posted July 17, 2014 Author Posted July 17, 2014 HELP please http://ipb.silvesterwebdesigns.com/topic/254-error-in-conversion/ Sorry for the delay, have replied to your ticket.
xtech Posted July 31, 2014 Posted July 31, 2014 Hi Stuart, i have the previous version. To upgrade i just need to upload the new files? P.S. - Just uploaded and upgraded: all ok! stoo2000 1
xtech Posted August 1, 2014 Posted August 1, 2014 Sometime after upgrading to the last version, a hook, the Editor Thems from Tom Christian (basicaly a ckeditor replacement) stopped working within the next 2-3 hours, so i had an error which prevented me to open any topic or post. It may be just a coincidence, but i got the following error: I am getting the following error on opening a post (every post): Fatal error: Call to a member function ipsEditorThemes() on a non-object in C:inetpubwwwroothooksipsEditorThemes_7322d94242545f574bc1243589850e65.php on line 15 The content of that file is: class ipsEditorThemes { public function __construct() { $this->registry = ipsRegistry::instance(); $this->memberData = $this->registry->member()->fetchMemberData(); $this->settings = $this->registry->fetchSettings(); } public function getOutput() { if ( strstr( ",{$this->settings['ipsEditorThemes_group']},", ",{$this->memberData['member_group_id']}," ) && ipsRegistry::$settings['ipsEditorThemes_enabled'] !== '0') { $html = ipsRegistry::getClass('output')->getTemplate('ipsthemes')->ipsEditorThemes(); } return $html; } } Tried to recache the skins, with no avail. May the upgrade to the last version of the trader feedback system may be involved with this? I had to disable the hook because i won't work now. I tried to recache skins, but it didn't solve the issue.
stoo2000 Posted August 1, 2014 Author Posted August 1, 2014 Sometime after upgrading to the last version, a hook, the Editor Thems from Tom Christian (basicaly a ckeditor replacement) stopped working within the next 2-3 hours, so i had an error which prevented me to open any topic or post. It may be just a coincidence, but i got the following error: I am getting the following error on opening a post (every post): Fatal error: Call to a member function ipsEditorThemes() on a non-object in C:inetpubwwwroothooksipsEditorThemes_7322d94242545f574bc1243589850e65.php on line 15 The content of that file is: class ipsEditorThemes { public function __construct() { $this->registry = ipsRegistry::instance(); $this->memberData = $this->registry->member()->fetchMemberData(); $this->settings = $this->registry->fetchSettings(); } public function getOutput() { if ( strstr( ",{$this->settings['ipsEditorThemes_group']},", ",{$this->memberData['member_group_id']}," ) && ipsRegistry::$settings['ipsEditorThemes_enabled'] !== '0') { $html = ipsRegistry::getClass('output')->getTemplate('ipsthemes')->ipsEditorThemes(); } return $html; } } Tried to recache the skins, with no avail. May the upgrade to the last version of the trader feedback system may be involved with this? I had to disable the hook because i won't work now. I tried to recache skins, but it didn't solve the issue. Not related, but possibly caused by the upgrader recaching. Follow the first part of this: http://www.invisionpower.com/support/kb/_/manually-rebuild-skins-and-languages-from-xml-r42 xtech 1
xtech Posted August 1, 2014 Posted August 1, 2014 Thank you Stoo :thumbsup: You've made it! Working fine now!
stoo2000 Posted August 11, 2014 Author Posted August 11, 2014 Can this work with SalesPortal as well? I'm not familiar with that app. Trader Feedback System in essence is a standalone app, it just has some added integration for Classifieds (if you use it)
NoGi Posted August 12, 2014 Posted August 12, 2014 It's this >one, an alternative to classifieds. If Trader Feedback works standalone it may still work out for me. Thanks.
Recommended Posts