prestashopcorp / trustedshops Goto Github PK

Here are a few comments about the box buyer protection:

• It takes too much space and can be really better designed. In example, if it has the same width as the other parts of the page, it would need less height to show all its content. The title should be reduced too.
• The checkbox is lost on a border
• The amounts are 0€. If this not caused by a bug, I wonder if it is really useful to show it in this case.
• I would have shown this box on a previous step, when the user still can modify his basket. In example with the hook displayShoppingCart

When I try to edit a certificate on PS 1.6, the panel dedicated to cron jobs is not properly shown.

On PrestaShop 1.5, the settings page of the module is not properly shown and should be fixed:

At the same time, it would be good to improve a little bit the design on PS 1.6.
In example, add a bow all around the introduction part, as you did for "Add trusted shops certificate" and "cronjobs configuration".

Hi!
I need to commit some changes to the TS module, but it seems I don't have any permission for that. Could you please grant us the pull access please.

regards
Julia, silbersaiten

I was trying the module to verify if everything is working, and I met the following error when I executed the cron task:

Fatal error: Class 'TSBuyerProtection' not found in /home/modules/www/public_html/trustedshops/modules/trustedshops/cron_garantee.php on line 25

The files TSCommon.php, RatingAlert.php and AbsTruestedShops.php are specific to PrestaShop and are not in the properly folder. They should be in /classes/.

On the order confirmation page, I think the messages could have a better place. On the current state, they are too intrusive:

You should show the popup or the the two buttons, but not both. Another modules than TrustedShops can be executed and shown on this page.

In order to respect the MVC architecture, the HTML in TSCommon.php code must not be written in PHP files. Please move it in new tpl files.