Salamek/PplMyApi

Chybějící povinné pole v CreatePackages

Closed this issue · 15 comments

Obchoďák stále není spokojen a vymýšlí hlouposti v metode CreatePackages chybí tyto věci:

<Flags> <MyApiFlag> <Code>SL</Code> <Value>1</Value> </MyApiFlag> </Flags>

Dále vícekusé balíky - i když balík není vícekusý a nebudeme je používat, podle dokumentace je to pole povinné: PackageSet a MasterPackNumber

Omlouvám se, Flags jsem přehlédl.

Takže se to týká jen MasterPackNumber, nové pole povinné od 21.8.2021 podle dokumentace

V případě sady balíků - číslo nosného balíku/nadřazeného (první ze sady nebo výběrčí dobírky).

V případě typu balíku s typem „dobírka“ je nosný balík (označený jako 1 / Y) zatížen celkovou částkou. Ostatní jsou
označeny pořadím/celkový počet (2 / Y ; 3 / Y ; atd) a dále hodnotou dobírky „0“

@MichalSkoula hmm mam to implementovane, ale nez to releasnu mam jeden dotaz, chapu to spravne ze PackageSet je povinny v zasilanych datech jen kdyz mam vice baliku?

(V example datech v doc je PackageSet vynechan, ale byl bych rad kdyby to potvrdil nekdo z PPL)

Pokud to tak je tak bych z PackageSet udelal novou tridu protoze mit $masterPackageNumber, $packagePosition a $packageCount v Package zacina byt trochu divne :D

No vlastne posilat tohle je trochu divne:

'PackageSet' => [
      'MasterPackNumber' => null // Je null ok? nebo tam ma byt cislo toho jednoho baliku?
      'PackageInSetNr' => 1,
      'PackagesInSet' => 1
  ],

chtějí string, dal bych tam "1/1"

ale napsal jsem do PPL, dám vědět

Obchoďák totiž sám psal, že vícekusové balíky používat nebudeme, takže nepotřebují ani testovací data ke kontrola. Současně ale dopsal, ať si doplníme MasterPackageNumber. Je to celé zmatené.

@MichalSkoula MasterPackNumber ma byt cislo hlavniho baliku v sade baliku, jde mi o to co tam posilat kdyz se jedna jen o jeden balik, pripadne zda li muzu cele info v pripade jednoho baliku vynechat a pouzit jen

'PackageSet' => null

samozrejme na stitky musim dat 1/1 atp.

@MichalSkoula a jestli te muzu poprosit jestli se muzes zeptat nekoho v PPL na tohle?
#62

Na strane 38 dokumentace je duplicitni return code a ostatni return cody v rade jsou asi spatne...

napsal jsem jim

PackageSet je teda povinný pro všechny balíky. Poslal ještě obrázek jak to má vypadat u jednokusového balíku: https://1drv.ms/u/s!AoyEPGbZnge3q69zpUSPDPV9yN8rzw?e=a65me1

@MichalSkoula takze masterPackageNumber == PackageNumber ok

po upgradu dostávám tento error

An uncaught Exception was encountered

Type: ParseError

Message: syntax error, unexpected ')', expecting variable (T_VARIABLE)

Filename: /var/www/html/vendor/salamek/ppl-my-api/src/Model/Package.php

Line Number: 105

(PHP 7.4, verze 2.1.2 funguje ok)

@MichalSkoula jo uz to vidim, je tam carka na vic ktera neni problem v PHP 8.1 co mam na localu ale je problem v 7.4

@MichalSkoula melo by byt opraveno v 2.2.1

perfekt, díky