Details des Tickets

Beschreibung,Kommentare und Anhänge

merge request id 168
TitelLL-242 bugfix treeview
BeschreibungAktueller Fix: * maximale Tiefe des Tree = 2 * Kapitel können genrell nicht mehr aus der Root-Ebene verschoben werden (ist relevant, weil die Tiefe de Tree bei leeren Kapiteln nicht griff) * beim Ablegen wird geprüft ob das Ziel ein Content ist; dann wird das Verschieben unterbunden Der Treeview-Selber hat einige Probleme: * beim Verschieben wird der pathTo geprüft; der ermittelte Knoten stimmt aber nicht immer. Beim Versuch ein Item in seinen nachfolgenden Knoten zu verschieben, wurde das (zu verschiebende) Item selber als Ziel ermittelt (weil sich die Positionen tauschen). Dieser Fehler konnte abgefanen werden, indem dann der Ziel-Index manuell erhöht wird. * wird ein Item ans Ende der Liste veschoben und hat der vormalige letzte Knoten Kindeelemente, erkennt der Tree das Ziel nicht und man kann nicht ziehen. Hierfür gibt es noch keinen Fix, da hierfür die Komponente selber angepasst werden muss.
Statusmerged
Created at 2020-03-26T09:21:00.180Z
Updated at 2020-04-02T15:08:27.925Z
Source branchfeature/LL-242-Bugfix-treeview
Target branchdevelop
AuthorSascha Immig
AssigneeGregor Gabriel

Discussion notes

added 2 commits
  • 0e602520 - fixed style
  • 81551340 - WIP: changed item assignment and added translations [skip-ci]
Compare with previous version		 Sascha Immig 2020-03-26T09:34:37.733Z
marked as a **Work In Progress** from 81551340df2ac486ae88cc3ad889b3ba4d9f11f6 Sascha Immig 2020-03-26T09:34:37.891Z
unmarked as a **Work In Progress** Sascha Immig 2020-03-26T09:44:34.093Z
changed title from **{-WIP: Feature/ll -}242 bugfix treeview** to **{+LL-+}242 bugfix treeview** Sascha Immig 2020-03-26T09:44:34.124Z
changed the description Sascha Immig 2020-03-26T09:44:34.143Z
added 1 commit
  • a8a331d4 - added placeholder
Compare with previous version		 Sascha Immig 2020-03-26T09:57:33.171Z
müsste dass nich fa-desktop anstatt fa-deskto sein? Gregor Gabriel 2020-03-27T09:28:10.263Z
ich korrigiere das gleich Gregor Gabriel 2020-03-27T10:40:48.843Z
resolved all discussions Gregor Gabriel 2020-03-27T10:40:50.293Z
changed this line in version 4 of the diff Gregor Gabriel 2020-03-27T10:43:07.203Z
resolved all discussions Gregor Gabriel 2020-03-27T10:43:07.259Z
added 1 commit
  • be6bd75c - fixed some little problems
Compare with previous version		 Gregor Gabriel 2020-03-27T10:43:07.466Z
Wenn man ein neues Kapitel einfügt und es dann ganz nach oben schiebt, dann wird das vorletzte Kapitel (das dann das letzte ist) von dem nach oben geschobenen überschrieben und ist dann weg. Das Verschobene scheint 2x da zu sein Gregor Gabriel 2020-03-27T11:03:15.279Z
Hmmm... kann das sein, dass das vorher schon so war? Sascha Immig 2020-03-30T18:15:00.085Z
Grade getestet. Die Klasse des neuen Kapitels ist immer nested-item-2, Scheinbar wird darüber idenitifiziert.. :( Sascha Immig 2020-03-30T18:21:48.821Z
Ich hatte in diese Richtung getestet, das du Operatioen mit einem Index gemacht hast. Ob das vorher schon war habe ich jetzt nicht explizit getestet, aber ich denke nicht, dass es vorher schon so war, da das Test-Szenario eher Standard und nichts ungewöhnliches ist. Gregor Gabriel 2020-03-31T06:29:51.902Z
added 18 commits
  • be6bd75c...7c4b14c7 - 16 commits from branch develop
  • 52e0bfec - Merge branch 'develop' into feature/LL-242-Bugfix-treeview
  • 73baf88d - Merge branch 'develop' into feature/LL-242-Bugfix-treeview
Compare with previous version		 Sascha Immig 2020-04-01T12:36:18.040Z
added 2 commits
  • d87dab07 - corrected issue of all new chapters having the same ID;
  • 6efe9861 - corrected layout
Compare with previous version		 Sascha Immig 2020-04-02T07:59:39.894Z
Problem der Dopplet vergebenen ID behoben. Wen die UID deR Komponente vergeben wird, kann das nicht funktionieren. Jetzt wird die höchste ID ermittelt, inkrementiert und dann genutzt. läuft! Sascha Immig 2020-04-02T08:01:06.082Z
resolved all discussions Sascha Immig 2020-04-02T08:01:56.931Z
die Operationen mit dem Index greifen aber nur beim Verschieben... die Klasse stimmt bereits beim Erstellen nicht... ich gucke mal tiefer rein... hab das gestern aber auf develop (ohne meinen Kram) getestet und hatte das Problem auch... ~~im Übrigen erhalte ich bim Upload von neuen Content immer ein leeres Kapitel ohne Titel, das ich auch nicht bearbeiten kann... da ist also grundlegend was im argen~~ Sascha Immig 2020-04-02T08:02:52.380Z
Das hinzufügen der leeren Kapitel biem Upload von neuen Inhalten erfasse ich als separatne Task, da der nichts mit dem Drag & Drop zu tun hat. Sascha Immig 2020-04-02T08:03:04.137Z
resolved all discussions Gregor Gabriel 2020-04-02T14:39:53.050Z
added 9 commits
  • 6efe9861...ed60c6b8 - 8 commits from branch develop
  • 0bee94fa - Merge branch 'develop' of https://gitlab.learningsystem.de/lms/seventeen into…
Compare with previous version		 Gregor Gabriel 2020-04-02T14:39:53.293Z
enabled an automatic merge when the pipeline for 0bee94fa1cf04e5eab213054c05ab52a6bc429a5 succeeds Gregor Gabriel 2020-04-02T14:40:18.304Z
merged Gregor Gabriel 2020-04-02T15:08:28.005Z
mentioned in commit 6a7eb111aff40d9e0aaa4b022c7a77d564a1cfc0 Gregor Gabriel 2020-04-02T15:08:28.033Z

Merge Commits

Title Id Author Created at Message
WIP: Optimized Code. Chapters cannot be moved into any other element. if the… 045497158d4114a21ade8fe78d9bc8a427e3a48d TUV\immigs 2020-03-26T07:39:45.000Z WIP: Optimized Code. Chapters cannot be moved into any other element. if the target is a content, movin is prohibited, too. But currently the calculation of the node ist not working in all cases, preventing too much.
Merge branch 'develop' of https://gitlab.learningsystem.de/lms/seventeen into… 0bee94fa1cf04e5eab213054c05ab52a6bc429a5 TUV\gabrielg 2020-04-02T14:39:30.000Z Merge branch 'develop' of https://gitlab.learningsystem.de/lms/seventeen into feature/LL-242-Bugfix-treeview # Conflicts: # resources/views/admin/trainings/items/create.blade.php
fixed style 0e602520510002a268e12b46a7a95dcaa4624752 TUV\immigs 2020-03-26T09:33:36.000Z fixed style
added check for Index-confusion on dragin item to the next sibling. increased… 0f73b07ee920af26e3f615c3b33b57e1273f9d23 TUV\immigs 2020-03-26T07:40:15.000Z added check for Index-confusion on dragin item to the next sibling. increased index to get the "real" target. Attention: If the max depth will be increased, the code for determining the next sibling needs to be corrected to take into account, that the ParentPath migt be a more complex array.
WIP: changed config 2e85119bb72f7b734ea751e1a4e57550fe3c503d TUV\immigs 2020-03-26T07:39:09.000Z WIP: changed config
Merge branch 'develop' into feature/LL-242-Bugfix-treeview 52e0bfec7511e32e695c8b80eb11c8b09504be6e TUV\immigs 2020-03-31T10:27:08.000Z Merge branch 'develop' into feature/LL-242-Bugfix-treeview # Conflicts: # resources/lang/de.json
corrected layout 6efe98614bccb8ee89decebdef976928422aa0a2 TUV\immigs 2020-04-02T07:59:27.000Z corrected layout
Merge branch 'develop' into feature/LL-242-Bugfix-treeview 73baf88d485979a6d4f7a92a8606546ed9920d7b TUV\immigs 2020-04-01T12:35:43.000Z Merge branch 'develop' into feature/LL-242-Bugfix-treeview
WIP: changed item assignment and added translations [skip-ci] 81551340df2ac486ae88cc3ad889b3ba4d9f11f6 TUV\immigs 2020-03-26T09:34:00.000Z WIP: changed item assignment and added translations [skip-ci]
added placeholder a8a331d4acf701a49fd8ca230239ee93907cdcdc TUV\immigs 2020-03-26T09:57:20.000Z added placeholder
fixed some little problems be6bd75c9910797f23a9d1740d4eeb4ce7507544 TUV\gabrielg 2020-03-27T10:42:44.000Z fixed some little problems
corrected issue of all new chapters having the same ID; d87dab0725f98114a0cd14bdcb7524458b213f65 TUV\immigs 2020-04-02T07:56:39.000Z corrected issue of all new chapters having the same ID;