merge request id	169
Titel	Feature/ll 235 move player config to training
Beschreibung	+ Umstellung aus myCompetence übernommen + Minimal-Mode aus myCompetence übernommen + Fehler korrigiert, so dass in der Katalogsicht Sessions geschrieben wurden und in der Lernersicht nicht (war gerade verkehrt herum)
Status	merged
Created at	2020-03-27T06:58:25.089Z
Updated at	2020-04-02T14:21:55.871Z
Source branch	feature/LL-235-move-player-config-to-training
Target branch	develop
Author	Gregor Gabriel
Assignee	Sascha Immig

Discussion notes

sollten optionale Parameter nicht immer am Ende kommen? Sprich: $element, $training, $parent, $userBoughtTraining...	Sascha Immig	2020-03-27T08:39:33.209Z
Benötigen wir diese Properties am TrainingItem? könnten wir diese hier nicht sparen und an den aufrufenden Stellen - also bei der Übergabe an dne UCP und z.B. in der oben stehenden `TrainingItemResource` - direkt auf das Training zugreifen? Das erscheint mir so als Umweg...	Sascha Immig	2020-03-27T08:44:42.632Z
Statt node und item auszulesen, könenn wir da nicht direkt auf das Trianing zugreifen? (siehe vorherige Kommentare)	Sascha Immig	2020-03-27T08:46:24.286Z
100%, das dürfte so eigentlich gar nicht gehen bzw. sollte das der code checker nicht zulassen.	Sebastian Schlein	2020-03-27T09:09:17.512Z
changed this line in version 2 of the diff	Gregor Gabriel	2020-03-27T12:45:26.299Z
added 1 commit 424572b2 - adapted fuction signature, so that last parameter is optional now Compare with previous version	Gregor Gabriel	2020-03-27T12:45:26.648Z
Habe den Parameter jetzt optional gemacht. Der Grund warum ich die Parameter nicht vertausche ist, dass die Methoden-Signatur genau der aus dem AppServiceProvider entspricht und ich da auf jeden Fall rückwärtskompatibel sein wollte	Gregor Gabriel	2020-03-27T12:48:18.459Z
changed this line in version 3 of the diff	Gregor Gabriel	2020-03-30T07:56:13.568Z
changed this line in version 3 of the diff	Gregor Gabriel	2020-03-30T07:56:14.185Z
added 1 commit 81c6c6db - as player config belongst to training now, this information was removed from all other elements Compare with previous version	Gregor Gabriel	2020-03-30T07:56:14.659Z
das wird jetzt explizit nicht mehr von den Item gelesen, sondern kommt immer, wenn ein Training vorhanden ist vom Training	Gregor Gabriel	2020-03-30T07:58:04.100Z
das wird jetzt explizit nicht mehr von den Item gelesen, sondern kommt immer, wenn ein Training vorhanden ist vom Training	Gregor Gabriel	2020-03-30T07:58:28.752Z
merged	Sascha Immig	2020-04-02T14:21:55.923Z

Merge Commits

Title	Id	Author	Created at	Message
+ manually merged relevant code from mxCompetence	04f60a8eaf3c1eafa1c9b536d65fa11aba2c49f6	TUV\gabrielg	2020-03-26T16:21:25.000Z	+ manually merged relevant code from mxCompetence + added migration
adapted fuction signature, so that last parameter is optional now	424572b22a47197ea6feddf53bd8ef5e124f081d	TUV\gabrielg	2020-03-27T12:45:00.000Z	adapted fuction signature, so that last parameter is optional now
as player config belongst to training now, this information was removed from all other elements	81c6c6dba27936baf2e559544784e96f2563c26d	TUV\gabrielg	2020-03-30T07:55:49.000Z	as player config belongst to training now, this information was removed from all other elements
fixed deactivation of learn session usage - logic was flipped	c6fd3795d1e35e97df619d41929816cc7c676f44	TUV\gabrielg	2020-03-27T06:52:54.000Z	fixed deactivation of learn session usage - logic was flipped