Add options class for table update, fix put in client

[romanbracinik]

[tomasfejfar]

LGTM. Viděl jsem funkční testy.

[zajca]

Ale to je přece BC

[tomasfejfar]

No je to BC break v klientu. To, že to posílalo parametry v getu je bug resp. "nedokumentované náhodně funkční chování", ne kontakt. (resp. tak to beru já)

[ErikZigo]

Tohle je oprava v klientovi, nebo měníte i něco na API?

[tomasfejfar]

V API je nedokumentovaná "feature" (spíš bug), že to tahá ty hodnoty postupně ze všech $_GET, $_POST, $_SERVER, $_ENV, etc. Takže se nepoznalo, že to ten klient posílá getem i když by to měl být post. Takže je to fix jen v klientovi. API zůstává jak bylo. A dokonce je pořád možné tam poslat místo PUTu get. Resp. poslat parametry PUTu jako GET params v URL. Myslím, že s tím souvisí KBC-272

[ErikZigo]

Pokud je to jen fix v klientovi tak je to v pohodě.

[romanbracinik]

Jojo v connection sa nemenilo nic, funguje to tam uz historicky tak, ze to mozes poslat jak v GET tak ako ako hodnoty v PUTe. Takze API sa chova uplne rovnako.

[zajca]

Každopádně by se měla po tomto vydat verze 11, protože je to BC v klientu.

// kdyby bylo mergnuté #350 tak už řve.

[romanbracinik]

Jo to som za vydat novu verziu. Dam to este raz na review mal som nejaky konflikt po rebase tak pre istotu.

[tomasfejfar]

Přijde mi to stejné. A první dataset testů mi prošel. Takže LGTM

[zajca]

Co ti přijde stejné?
změnila se signatura metody z public function updateTable($tableId, $options) na public function updateTable(TableUpdateOptions $options) to je BC. Otázka jestly by se s novou major verzí neměli stejným způsobem změnit i další signatury metod ať používájí Options objekty (neprocházel jsem teda kolik jich teď je).

[tomasfejfar]

Jo, já už to chápu jak to myslíš - že je to zbytečný BC break! No, to máš asi pravdu.

[tomasfejfar]

Zbytečný BC break

[romanbracinik]

Tak este jeden pokus :) Upravil som len tie parametre

[tomasfejfar]

Viděl jsem projít testy.