Nelze přidat neregistrovaného závodníka do štafety

[janpalas]

Nepodařilo se mi zadat závodníka, který není registrován, na soupisku libovolné štafety - program vždy spadnul. Závodníka jsem chtěl nejdříve přidat v sekci "Závodníci" - vyplnil jsem údaje, dal uložit a konec běhu programu.

• Toto chování jsme pozorovali na ČPŠ 25.-26.9.2021
• Přikládám zálohu 20210926-1500.zip

[xkenia]

Ano, nejde to, jelikoz pri pridavani zavodnika/useku je mozne vybrat pouze z registrovanych zavodniku.

Jsou 2 workaroundy:

1. dat tam registrovaneho zavodnika, ktery urcite nepobezi a prepsat mu udaje
2. do tabulky registration (nebo tak nejak) pridat rucne tohoto zavodnika (pak nejspis restart programu, tohle jeste nevim, co presne pomuze) a zavodnika/usek je mozne pridat

[arnost00]

Dialog na priadavani useku do stafety je rozsiren o pridani neregistrovaneho zavodnika

[fvacek]

zda se mi, ze je to naimplementovano v jinem pull requestu #673

[arnost00]

zda se mi, ze je to naimplementovano v jinem pull requestu #673

Ano. Je to v ramci toho pull requestu.

[paukert]

Štafety jsem nikdy nepořádal, takže nevím, jaký je přesně workflow, ale pokud to chápu správně, tak v záložce závodníci mohu upravovat pouze detaily závodníků, kteří již jsou (byli?) k nějaké štafetě přiřazeni. I přes to by ale podle mě bylo dobré, aby program nespadl, pokud se budu snažit přidat nového závodníka z tohoto rozhraní, a pro uživatele, kteří se o to budou pokoušet, bych klidně asi zobrazil nějakou radu, jak nového neregistrovaného závodníka přidat.

[arnost00]

Normalne si myslim ze tuto zalozku ve stafetach ani nemusis otevrit. Ale je pravda ze padat by to nemelo.
Protoze otevrit a pouzit ji muzes.

[arnost00]

Moje veskere pokusy s opravou vedly maximalne k tomu, ze ve stafetech to pridalo zaznam jak do competitors tak i runs.
Coz je vec kterou si myslim ze nechceme, do runs by to nemelo nic pridavat.
Cim dal vic si myslim, ze by toto tlacitko pri stafetach nemelo zobrazit dialog vyberu zavodnika, ale pouze info, aby jsi zavodnika pridal ve casti stafety.

[paukert]

Cim dal vic si myslim, ze by toto tlacitko pri stafetach nemelo zobrazit dialog vyberu zavodnika, ale pouze info, aby jsi zavodnika pridal ve casti stafety.

To by podle mě bylo také validní řešení, ale o štafetách toho opravdu moc nevím, takže bych asi ještě počkal na vyjádření někoho dalšího.

[fvacek]

zacal bych tim padanim ,co musim udelat aby to spadlo?

[paukert]

zacal bych tim padanim ,co musim udelat aby to spadlo?

Otevřít si nějaký závod štafet (třeba ten z této issue) a ze záložky competitors zkusit přidat nového závodníka (při kliknutí na Save / Save and next program spadne).

[arnost00]

zacal bych tim padanim ,co musim udelat aby to spadlo?

Presne jak pise @paukert .
A spadne to proto, ze ve stafetach nema zavodnik kategorii. Registrovany kvuli tomu v Save projde (protoze se mu tipne nejaka kategorie do disabloveneho comboboxu).
V nekterych castech dialogu se kontroluje ze jsou to stafety a neni potreba mit vyplnenu kategorii, ale pri Save vnitrni model nejak narazi na to ze neni a vyhodi vyjimku, ktera se nikde nechytne a sestreli to.

V tomto ohledu by se melo zvazit zda ma cenu vyhazovat v kodu vyjimky, ktere nechytame. A zacit je chytat, nebo prestat pouzivat. Ale prehled pres cele QE nemam, abych dokazal rict ktera cesta by byla vhodnejsi.

[fvacek]

V tomto ohledu by se melo zvazit zda ma cenu vyhazovat v kodu vyjimky, ktere nechytame. A zacit je chytat, nebo prestat pouzivat. Ale prehled pres cele QE nemam, abych dokazal rict ktera cesta by byla vhodnejsi.

QE pouziva neodchycene vyjimky jako fatal error, ve srovnani s qFatal() to ma tu vyhodu, ze pokud to neni fatalni chyba, ale neco, co se da zachranit, lze tu vyjjimku odchytnout. Pravdepodobne to bude i tento pripad.

[paukert]

Když už se začalo více řešit logování (#672) – možná bychom mohli pro snadnější identifikaci problému, chytat všechny výjimky na nejvyšší úrovni, vytvořit log a vyhazovat je znovu.

[fvacek]

Když už se začalo více řešit logování (#672) – možná bychom mohli pro snadnější identifikaci problému, chytat všechny výjimky na nejvyšší úrovni, vytvořit log a vyhazovat je znovu.

tohle se presne deje: https://github.com/Quick-Event/quickbox/blob/eb547d1c21895e2034d2656f877a47daf834f989/libqf/libqfcore/src/core/exception.cpp#L38

[arnost00]

@fvacek dostanes se na to padani ? Nebo to vyresime zablokovanim spousteni tohoto dialogu pri stafetach ?
Vycistil bych z toho dialogu handlovani stafet a nechal ho pouze pro jednotlivce.

[fvacek]

jj ,mrknu na to

[fvacek]

fixed in 2.4.5