rowe42 / lhm_animad_admin_html5

0 stars 6 forks source link

Reload für Liste, wenn loadCompleteData = true #238

Closed ejcsid closed 6 years ago

rowe42 commented 6 years ago

Ich habe es mir angesehen. Es funktioniert, allerdings gibt es einen Bug: Wenn man ein neues Element speichert, erscheint dieses nicht in der Liste. Grund dafür ist, dass wir zuerst zur Liste navigieren und dann speichern aufrufen (in animad-form-behavior). Aber auch wenn wir das umdrehen würden, bleibt es weiterhin eine Racing Condition: es hängt davon ab, ob das Speichern früher fertig ist als das Neuladen ankommt.

Ich merge das jetzt trotzdem mal. Aber wir müssen uns die ganze Sache mit dem Reload nochmal ansehen. Ich denke, wir dürfen das nicht an die Navigation binden, sondern müssen nach jedem save ein Custom Event auslösen, auf das die "interessierten Elemente" (z.B. eben die loadCompleteData-Liste) reagieren kann.

Ich mache mal ein Issue dafür, dann kann uns @xdoo auch seine Meinung dazu sagen.