datakick

@Generaal, do you have debug mode enabled? If so, please turn this fu***g functionality off. It screws everything

@wakabayashi awesome. I assume one of the hooks you've mentioned will be there to discover all available rewardable actions in the system, and the second hook to inform your module that the action actually happened? Or do you have some other mechanism in mind?

@generaal said in [Free Module]Revws - Product Reviews On the review in FO, there don't come up some options to give some information on the review, after placed and approved the review. Options to vote/report abuse are available only for other visitors, not for review author. Review author see edit/delete review buttons instead. When I tried to delete some reviews in BO, I have to click serval times to change some settings and it's not possible to delete them even. I don't understand what exactly the problem is. Do you mean that the app does nothing when you click on delete button, of on review itself? If so, what browser do you use? Also, it's not possible to permanently delete review, you can only mark it as deleted. I'll add an option to permanently delete them in the future, or even better filter them out of the list. I'm not a big fan of permanent deletion

@Manisch I would need to see source code of the module. But I believe that the line you've mentioned starts with something like this: $this->context->controller->addJS(.... You will want to remove the whole line. Next, search for text 'google.maps' in *.js files of this module... and get rid of all references to google maps api... somehow...

also, if your server is on 2-level domain such as http://domain.com, then it will have no effect if you set up static server on subdomain, ie static.domain.com. Cookies will be shared. Cookieless request will be only if your primary server is on subdomain as well, ie www.domain.com

Did you clear cache? This feature doesn't have big impact. The http requests are still routed to the same server, the only difference is that these requests don't contain cookies. Which can save few hundreds bytes per request. It would be much more interesting if there was a mechanism to transfer static context to different server (say amazon s3), and when everything is synced then turn this media server automatically. It's probably hard to implement, because the static context in tb is not that static :)

@manisch that's not a good solution. This will probably create script tag in your page like this: <script type="text/javascript" src='httpen' /> that will result in http request to your domain, ie: http://www.domain.com/httpen. Request will return 404 not found, and you'll get error code in your console. Immediately after that you will find another couple of errors related to missing google maps library, because javascript code that utilize this library is still present...

@generaal, there a bug in json module. It tests whether productcomments module is installed, but it doesn't test if it's also enabled. So either uninstall productcomments completely, or set up json module to use different review module. From the code it seems like this should be part of configuration....

@lesley I don't think we have to complicate this by reserving urls, etc... the history log doesn't even have to be unique. Just find out which product used this link-rewrite last, and display that one. I'm sure this simple mechanism would display the 'right' product 95% of the time. I believe conflicts with link rewrite would rarely happen. And even when there's a conflict and we display different product than to which link originally point to, it's safe to assume this new product will be similar. It's definitely much better than showing 404, that one only cause bounce.

@lesley I think we are talking about two different things. I believe you are talking about changing schema or urls / routes under Preferences > SEO & URLs. I'm talking about changing link_rewrite on product page. I'm not sure which problem @30knees originally talked about.... Now, I also think it doesn't make any sense to automatically fix url when route schema changed. That would be crazy, and we would indeed have to use htaccess fille. Moreover, it's also not very important issue because no sane merchant will ever change routes on running shop with some history. But they can, and often will, change friendly url on individual products. At the moment, when you change friendly url on product, any link to this product will stop working. For example, I've changed friendly url on Candle to candle-1 -- url changed to /en/gifts/candle-1, and original url /en/gifts/candle now returns 404. Prestashop doesn't have this problem because they rely on product id as a primary identifier in URL, so they can change friendly urls how many times they want. It would be very easy to fix this issue. We just have to remember history of friendly urls for each product, and use this info when we can't find product. Performance impact is none for happy path, and only a single sql query for historical url. But added value is huge.

@davidp I disagree. First of all, table with 20000 records is not a big table, especially when it contains 2 columns only (product id and original url / link rewrite). This mechanism wouldn't have any negative performance impact on 'normal' page load, as it would be triggered only when product can't be found. Also, there can't be any redirect chain, only one redirect to current product url. Actually, we don't even have to do the redirect, we could simply render product page with canonical url meta tag in the header.

modules have their own repositories, these aren't really related to tb version at all. I was talking about fixes in the core

@movieseals is there a file CryptBlowfish in /home/publichtml/classes? If there is, try to delete cache/classindex.php

@pedalman there are no commits related to 1.0.3. Version 1.0.3 is just a tag, a bookmark in history of branch 1.0.x. Everything and anything committed after this point in time is part of upcoming version 1.0.4 Now, if there is some fix that's really important to you, and you are not willing to wait for next version, you can cherry-pick this commit. Start by creating your own branch git checkout -b branch1.0.3 1.0.3 This will create new branch named branch1.0.3 from commit associated with tag 1.0.3. It's like you have traveled back in time and have the same codebase as it was at the time 1.0.3 was released. Now, you can use git cherry-pick <commit-hash> where commit-hash is id of commit (from the future) you wish to merge to your 1.0.3 branch. Obviously, this can be a dangerous operation, as the commit can depend on code that didn't exists in 1.0.3 yet... so test it properly.

@slick_303 this option is available only when guest reviews are disabled. Maybe that's the problem...? Edit: also, I've found a bug in the code , so use this version instead

Definitely don't delete products, google juice is way too precious. I personally would go with redirect to similar product, or to parent category if no product is close enough alternative. This doesn't need to be such pain when you use mass update / inline editing tools.

@slick_303 yes, git repository is NOT a valid module. You need to build it first, and for that you need nodejs and gulp. I've described how to do it in this dev manual

@slick_303 said in [Free Module]Revws - Product Reviews: It would be nice that if a product didn't have a review, and you were not allowed to write one, that the review tab and button would not even show. Done, will be released in upcoming version. I've added new settings option that is available only when you disable guest reviews. You can now choose one of the three behaviors: show empty reviews tab -- this is current behavior. Review tab will be rendered with No customer reviews for the moment message hide reviews tab completely -- this is what you've requested. All references to reviews will be removed from product page show sign in button - I vaguely remember that someone requested this one. If there are no reviews yet then we will render Sign in to write a review button.

version 1.0.3 has been released. This is just a bunch of bugfixes that have been reported recently, no new functionality.

Thanks @slick_303 for sharing this. I think that date condition shouldn't be there at all, though. Number of unapproved reviews is a snapshot metric. We can't really create a sql to determine how many unapproved reviews were at any given time (or time interval). It doesn't make sense to filter on review date here. Different situation would be if we were interested in number of newly created reviews. That we can measure for any time interval. This date condition will filter out unapproved reviews that are older than from date. So the metric can say there are 1 unapproved review, but in fact when you click on the link you can see more. The same applies for original code for productcomments reviews.

@wakabayashi hmmm, productcomments module doesn't save emails for reviews created by guests, so when I import these reviews I don't have anything to store to email field. That will later cause this problem. I'll come with some solution. Meanwhile, you can edit file sql/migrate-productcomments.sql and replace COALESCE(`cust`.`email`, '') with COALESCE(`cust`.`email`, '[email protected]') And then reimport product comments reviews. This will associate guest reviews with '[email protected]'. Replace this email with your own, or use some throwaway

Also, maybe use dateupd field instead of dateadd, to count old reviews that needs to be re-approved

@slick_303 you didn't properly escape deleted and validated fields. Use this sql: (new DbQuery()) ->select('COUNT(*)') ->from('revws_review') ->where('`deleted` = 0') ->where('`validated` = 0') ->where('`date_add` BETWEEN "'.pSQL($params['date_from']).'" AND "'.pSQL($params['date_to']).'" ')

get rid of shop restriction and it should work

@slick_303 said in [Free Module]Revws - Product Reviews: @datakick Is there a way to change the css look of the js popups? You know, those grayish/rose colored guys... the js app depends on this library that uses jss instead of css. So it's not possible to extract css, but we can set palette for the components. I plan to add this option to settings page. I think normal users will prefer to set colors in UI instead of via css. It's definitely not a best way for power users, but they have an option to rebuild js app with their own settings if they wish. @luv said in [Free Module]Revws - Product Reviews: 2) In product list review displayed upon hover other wise hidden. How to do that. You can copy css file /modules/revws/views/css/front.css to /themes/<THEME>/css/modules/revws/revws.css and modify it as you see fit. For example you can try to add this to the end: ``` .revws-product-list { transition: opacity 200ms ease; opacity: 0; } .product-container:hover .revws-product-list { opacity: 1; } ``` 3) Can you add a feature in review display tab for customer who have made a purchase as "Verified Buyer" below name. (It is going to make review more trustworthy). A small feature but very powerful. Sure, nice idea. 4) I see in setting tap Review request. did not understand what it does. Does that send review request mail to the customer. If it is so, this module is a gem. If not please consider adding. Sorry, not a gem yet :) When logged-in customer goes to My Reviews in My Account, he will be asked to review recently purchased products. This option only limits max number of displayed products. Revws module will never contain ask by email functionality. But I'm (slowly) developing another module for this.