Salamek/PplMyApi

DepoCode need update request

dariuszrup opened this issue · 5 comments

DepoCode need update request. Currently the system only supports ranges 1 to 24. Request to add support for the following: 41, 44, 46, 48, 61, 62, 65, 66, 67, 96, 98, T2 or rework the setDepoCode method so that it does not throw exception

@dariuszrup is there any official source for these depo codes?

Yes, please see what official information I got from PPL:

"From the error I think that there is trouble with wrong mapping of depocodes. I see that there is used part of basic code from GitHub developer Salamek. This person is doing theyr own solution which is not approved from PPL and theyr solution has some errors. This looks like one of them.

Salamek has own script for label creating and this error reflect incomplete list of depo codes, because we are depoCode with values which is written but there are missing another one like 41, 44, 46, 48, 61, 62, 65, 66, 67, 96, 98, T2

In this case there is need to do some modification of code, or add depots for list. I'm sorry but I don't know where and what to do because this is not our product and person Salamek is not cooperate with our company.

Thank you for your understanding"

@dariuszrup should be fixed in 2.5.2

PS:
Also it is "nice" to read how PPL staff is throwing blame to my direction...while i maintain a PHP library for them for free...without any support from them... 😞

In this case there is need to do some modification of code, or add depots for list. I'm sorry but I don't know where and what to do because this is not our product and person Salamek is not cooperate with our company.

They know very well where to reach me, they know that this project exists and there is my email right on my profile if they want some private communication... also i have asked them about cooperation and they did not respond, you can see that in #62 (comment)

Oki thank you for your reply. I can prepare a proposal to fix the code tomorrow

oki I see you already added the fix -> thanks