prestashopcorp/sellstrom

Sellstromship

Sellstrom Shipping Module for PrestaShop

---

Update October 31, 2014

• Current Module version: 0.1.0

Update November 28, 2014

Issues fixed:

• #7: minor security fix

Update November 17, 2014

Current Module version: 0.1.1. Issues fixed:

• #1: Checks missing in label.php
• #2: Comments for sellstrom.php
• #4: Removed the unused $session_id variable to avoid the Notices
• #5: Design on order page
• #6: Small design issue in settings page on PrestaShop 1.5

Update February 11, 2015

Current Module version: 0.1.2.

New functionality added:

• #1: Create Invoice
• #2: Add Insurance

Issues fixed:

• #8: Notices on add to cart with SellStrom

sellstrom's Issues

minor security fix

sellstrom.php:965: 0 AND name = ''.$carrier->name.'''; add pSQL()

Checks missing in label.php

In the file /label.php, you should add some tests before executing the requests.

In example:

if (Tools::getIsset('id_order') && Tools::getIsset('secure_key') && Tools::getIsset('id_tracking'))

Notice

When I am on the settings page, or on a order, I meet the following notice:

And when I am on an order page not related to sellstrom, I get these ones:

You should check in your hookDisplayOrderDetail if the concerned carrier is yours.

Comments for sellstrom.php

Here are a few comments for /sellstrom.php:

• function hookDisplayOrderDetail():
• function hookBackOfficeHeader():
• function hookHeader():
  • HTML code must be moved in a tpl file
• General: You should remove all commented HelperP::log(...) in you code for the release.
• l.833: What this test is supposed to do ?

Carrier list is not properly shown

When I try to purchase an order, the carrier list is broken when I have a carrier from sellstrom:

Design on order page

On the order details on PrestaShop 1.6, I have the following box from Sellstrom.

It seems to be not designed. Is it really what you expected ?

SellStrom panel should not appear on Order details page

On the Back Office, you made the following panel to create labels for the new orders:

But you should not display it when the carrier selected is not one of yours. Because if every module does not make this test, the page will be fulfilled by panels which won't be used.

Thanks

Small design issue in settings page on PrestaShop 1.5

When I try to configure the module on PrestaShop 1.5, I meet the following small issue:

As you can see, the text comes over the pictures.

Notices on "add to cart" with SellStrom

Hi,

When I try to add a product to the cart, I get the following list of notices.