Viestiminen käyttäjän kanssa ja tallentamisen pysäytys virheen sattuessa

[nmaarnio]

No description provided.

[LKajan]

Voisi vielä miettiä tarviiko oikeasti tallennusta keskeyttää kaikissa tapauksissa (ks. kommentti).

[LKajan]

Koko tätä tallentamisen keskeytyslogiikkaa tai ainakin näitä viestejä voisi miettiä selkeämmäksi. Nyt koko kaavan ja sen lapsi-kohteiden tallennus keskeytetään jos esim tässä kohtaa assosiaation poistaminen epäonnistuu. Eli esimerkiksi asiakirjojen tallentamista ei edes yritetä enää tehdä. Ja käyttäjälle ilmoitetaan vaan että assosiaatioiden poistaminen ei onnistunut, mutta ei sanota, että mitään muuta ei yritetä tämän jälkeen.

Ehkä tässä ei tarvitsisi edes returnia. Assosiaation poistamisen epäonnistuminen ei ole varsinainen este tallentaa muita kaavan lapsikohteita.
Sanoisin melkeinpä, että tallentamien pitäisi keskeyttää ainoastaan tapauksissa, joissa luodaan uusia kohteita ja parent-kohteen luominen epäonnistuu, jolloin tietokannassa ei ole kohdetta, jolle lapsikohde pitäisi tallentaa.

[nmaarnio]

Joo, ehdottomasti. En ollut koskaan kovin tyytyväinen miten tämä tuli toteutettua.

Taisin ajatella, että ideaalisti koko operaatio haluttaisiin rollBackata jos sattuu mikä tahansa virhe poistamisessa tai lisäämisessä, mutta hyvä jos ei. Eikö esim. kuitenkin assosiaation poistamisen epäonnistuminen voisi johtaa siihen, että jää viittaus johonkin kohteeseen tai määräysryhmään joka on jo poistettu?

[nmaarnio]

Nyt pitäisi toimia siten, että pysäyttää tallennukset/poistamiset vain jos täytyy (eli esim. jos kaavan tallennus epäonnistuu, ei koiteta tehdä mitään kaavaan linkittyville tiedoille, esim. asiakirjoille).

Tällä hetkellä on ehkä vähän hassua, että tulee success-viesti käyttäjälle, vaikka jokin osa tallennuksesta epäonnistui. Esim. jos kaavan tallentaminen onnistuu, mutta asiakirjan tallentaminen epäonnistuu, päällimäisenä tulee viesti kaavan tallentumisen onnistumisesta, ja tämän viestin alla on piilossa ilmoitus asiakirjan tallentamisen epäonnistumisesta.

[LKajan]

Tällä hetkellä on ehkä vähän hassua, että tulee success-viesti käyttäjälle, vaikka jokin osa tallennuksesta epäonnistui. Esim. jos kaavan tallentaminen onnistuu, mutta asiakirjan tallentaminen epäonnistuu, päällimäisenä tulee viesti kaavan tallentumisen onnistumisesta, ja tämän viestin alla on piilossa ilmoitus asiakirjan tallentamisen epäonnistumisesta.

Aloin nyt vielä miettimään, että tarviiko onnistumisista oikeastaan edes kertoa käyttäjälle varsinkin nyt kun tuo kursori muuttuu tallennettaessa. Nyt käyttäjä saa informaation, että tallennus käynnissä (kursori tiimalasina) ja kun tallennus valmistuu (kursori palautuu). Oletettavaa on, että tällöin tallennus olisi onnistunut. Riittäisikö tällöin, että vain virheistä viestitään?

[LKajan]

Dekoraattorillahan tää kannattikin tehdä, hyvä!
Tässä voisi vielä hyvin tapojen mukaisesti käyttää functools.wraps metodia, niin dekoraattoria käyttävien funktioiden signature pysyy samana.

Suggested change

	def use_wait_cursor(func):
	def wrapper(*args, **kwargs):
	from functools import wraps
	def use_wait_cursor(func):
	@wraps(func)
	def wrapper(*args, **kwargs):

[nmaarnio]

Poistin kaikki success-viestit paitsi JSON tallennukseen liittyvän, ehkä ilman niitä pärjää nyt ihan riittävän hyvin. Yleisesti olisin kuitenkin sitä mieltä, että success-viestejä kannattaa olla, mutta niihin voi paneutua tarkemmin tulevaisuudessa.