Hello! I have a problem with this module. After somebody place an order, he get an email with the right pick pack pont name and address. After that in the same order is an another pick pack point name and address.
Unfortunatelly this problem is not continuous, so it is randomly.
It is the same with dev and the other version.
Can you help me? Thank you.

CommentFileSizeAuthor
#4 Target_address_changes-2216203.patch11.79 KBdj1999

Comments

dj1999’s picture

dj1999 commented
Status: Active » Postponed (maintainer needs more info)

Need more info to check this bug.

fox mulder’s picture

fox mulder commented

Hi dj!

We have same problem! I think I found the source:

The simplexml_load_file() loads the current list of ppp shops and commerce_pickpackpoints_get_option_list() stores the options in cache. The simplexml_load_file() returns a non associative array and when your module stores ppp data assigned to an order, it uses the numeric array keys returned by simplexml_load_file().

But when a new shop was added to, or an existing was removed from the validboltlista.xml than the numeric key ( which one is stored in db, and represents ppp shop data assigned to an order ) points to another shop ( shop ID, shop name, city, etc. ) after cron run.

Here is an old list of ppp shops ( the date can not be recognized ): http://honlap.hu/pages/kutyatapok/contents/static/23/doc/pppboltlista.xls

And here is the current: http://partner.pickpackpont.hu/stores/pppboltlista.xls

Your module uses values from column "A", but the first difference is in the row where the vaue of "A" is 11.

fox mulder’s picture

fox mulder commented
Status: Postponed (maintainer needs more info) » Active
dj1999’s picture

dj1999 commented
Status: Active » Needs review
StatusFileSize
new Target_address_changes-2216203.patch11.79 KB

Created a patch to using ajax instead normal JS. In this case able to using shop id in associative array. Please review it.

dj1999’s picture

dj1999 commented
Status: Needs review » Closed (fixed)

Posted to dev.