Chyba parseOutput

[jan-stanek]

Ve verzi 3.0.0 alpha 1 je problém s parsováním output, pokud neobsahuje tag <...Output> - např.:

Undefined property: stdClass::$UserDetailOutput

Další problém je v typu, pokud se vrací $result (stdClass místo array)

...
$output = $result->{$fname . 'Output'};
if (!isset($output)) {
    return $result; //neobsahuje $fname.Output
}
...

[fmasa]

Další problém je v typu, pokud se vrací $result (stdClass místo array)

Máš na mysli situaci, kdy prázdný výsledek vrací prázdnou stdClass místo pole, nebo něco jiného? To by každopádně stálo za to také opravit. Takhle je potřeba všude kontrolovat, že se nevrátila prázdná stdClass místo pole

[jan-stanek]

Tady:

Skautis/src/Wsdl/WebService.php

Line 163 in 48a2e95

return $result; //neobsahuje $fname.Output

Zatím jsem neměl čas zjistit, jestli stačí změnit na return [$result];

[JindrichPilar]

@jan-stanek Děkuji za vyzkoušení alpha verze a za bug report.

[JindrichPilar]

@jan-stanek #91 byl mergnut a vydal jsem v3.0.0-alpha.2. Tím by tato issue měla být vyřešena.

[sinacek]

@JindrichPilar @jan-stanek Můžu to tady zavřít a vydáme 3.0, nastavím to jako default branch?

[jan-stanek]

teď už si vůbec nevzpomínám, u SRS používáme pořád 2.x, můžu večer zkusit přepnout

[jan-stanek]

@sinacek tak nakonec jsem to nevyzkoušel, protože skautis/nette vyžaduje alpha 1, budeš dělat release i u něj?

[sinacek]

@sinacek tak nakonec jsem to nevyzkoušel, protože skautis/nette vyžaduje alpha 1, budeš dělat release i u něj?

udělal jsem https://github.com/skaut/SkautisNette/releases/tag/v3.0.0-alpha.1

[JindrichPilar]

@sinacek

vydáme 3.0, nastavím to jako default branch?

S 3.0 cekam na #97, aby v byla v WebServiceInterface metoda vracejici promise.
Nevim jak na tom @fmasa je, chtel se do toho pustit.

[jan-stanek]

skautis/nette pořád závisí na skautis/skautis 3.0.0.alpha.1, ne 3.0.0.alpha.2, stáhnul jsem to ručně, ale skautis/nette s tím není kompatibilní - mezi 3.0.0.alpha.1 a 3.0.0.alpha.2 se měnily namespacy

[sinacek]

skautis/nette pořád závisí na skautis/skautis 3.0.0.alpha.1, ne 3.0.0.alpha.2, stáhnul jsem to ručně, ale skautis/nette s tím není kompatibilní - mezi 3.0.0.alpha.1 a 3.0.0.alpha.2 se měnily namespacy

Tak ono těch změn tak je více než jen namespacy a to teďka rychle neupravím :/

[jan-stanek]

vydány nové verze skautisnette i skautis, které jsou mezi sebou kompatibilní a obsahují #91