Details des Tickets

Beschreibung,Kommentare und Anhänge

merge request id 989
TitelFeature/ll 872 code review user model
BeschreibungDas Ticket war sehr lange offen und es gab viel Input! + geprüft und nicht verändert wurden die Mutatoren, die ab Larvel9 anders erstellt werden. Wobei sich herausstellt, dass wir gar fast keine Mutatoren als Mutatoren nutzen, sondern als Weg komplexe Model-Berechnungen in eine Funktion zu packen und die Ergebisse einfach in die JSON-Repräsentation des Objektes zu bekommen. Hier sollte in Zukunft abgewogen werden, ob ein wirklicher Nutzen vorliegt, wenn die Funktion am Objekt als Attribut erscheint. Wenn nicht sollte sie auch nur als Funktion aufgerufen werden. + fillable wurde überarbeitet + Datetime-Felder wurden in $cast deklariert + nicht genutzte DB-Felder wurden entfernt
Statusmerged
Created at 2024-07-05T07:00:25.809Z
Updated at 2024-09-24T13:22:59.450Z
Source branchfeature/LL-872-code-review-user-model
Target branchdevelop
AuthorGregor Gabriel
AssigneeSascha Immig

Discussion notes

added 163 commits
  • 1e2a5f80...616c8dda - 162 commits from branch develop
  • c76faa6c - Merge branch 'develop' of https://gitlab.learningsystem.de/lms/seventeen into…
Compare with previous version
Gregor Gabriel 2024-08-08T13:17:41.278Z
added 164 commits
  • c76faa6c...7c5fd249 - 163 commits from branch develop
  • 564183bf - Merge branch 'develop' of https://gitlab.learningsystem.de/lms/seventeen into…
Compare with previous version
Gregor Gabriel 2024-09-24T10:55:47.072Z
added 1 commit
  • 84c99ebe - fixed 3 phpstan problems
Compare with previous version
Gregor Gabriel 2024-09-24T12:17:32.271Z
merged Sascha Immig 2024-09-24T13:22:59.524Z
mentioned in commit ecb16d2ebfc481dd753e6b535c7771b6d6dedfa3 Sascha Immig 2024-09-24T13:22:59.550Z

Merge Commits

Title Id Author Created at Message
casting following fields to datetime: confirmed_at & password_changed_at &… 1e2a5f80f9b68c66cf0f77b9888acaec0a988812 TUV\gabrielg 2024-07-04T11:33:02.000Z casting following fields to datetime: confirmed_at & password_changed_at & last_login_at & finished_onboarding_at
users.is_superadmin in not fillable any more 227fe7fc17abdf4ceb3e556c698c00cd3733bdbc TUV\gabrielg 2024-07-04T10:20:11.000Z users.is_superadmin in not fillable any more
Merge branch 'develop' of https://gitlab.learningsystem.de/lms/seventeen into… 564183bf224c41294ccb58e880e3ab8b57086087 TUV\gabrielg 2024-09-24T10:55:38.000Z Merge branch 'develop' of https://gitlab.learningsystem.de/lms/seventeen into feature/LL-872-code-review-user-model
removes 'text' from fillable 58f2cd0e6952c0572e7d6bf2f07b05157aeffe91 TUV\gabrielg 2024-07-03T16:50:38.000Z removes 'text' from fillable
run php-cs-fixere on migrations folder 6e88f9e665a97df3a9234a48a59975af4f314ed6 TUV\gabrielg 2024-07-03T14:39:21.000Z run php-cs-fixere on migrations folder
fixed 3 phpstan problems 84c99ebe799f4a11ee829a8c88494d9e07dae2e1 TUV\gabrielg 2024-09-24T12:17:26.000Z fixed 3 phpstan problems
users.last_login_at removed from fillable 8b9f94a7e8915a0aca2c7b91e42049e39228b201 TUV\gabrielg 2024-07-03T14:37:54.000Z users.last_login_at removed from fillable
removed unused field users.is_rating_allowed from users table 907d9286ae289ba5b1d017013cadf7dde03df491 TUV\gabrielg 2024-07-03T14:38:53.000Z removed unused field users.is_rating_allowed from users table
users.api_token is not fillable any more 96d26041eef607c7bc19bdb281161ee58750d097 TUV\gabrielg 2024-07-03T16:06:52.000Z users.api_token is not fillable any more
removed field users.birth_day from database a9a538a936e40c712481c8d5dd869fe8604fdce5 TUV\gabrielg 2024-07-03T16:52:11.000Z removed field users.birth_day from database
Merge branch 'develop' of https://gitlab.learningsystem.de/lms/seventeen into… c76faa6ca6ab479af0fcc0f30dafc89ec80b9821 TUV\gabrielg 2024-08-08T13:17:28.000Z Merge branch 'develop' of https://gitlab.learningsystem.de/lms/seventeen into feature/LL-872-code-review-user-model # Conflicts: # app/Models/User.php
refactored user settigs admin ui - replaced checkboxes by x-switch component -… fd80d0ef2fa5474938e12d95835feb52a7e2e613 TUV\gabrielg 2024-07-04T09:35:47.000Z refactored user settigs admin ui - replaced checkboxes by x-switch component - thus user controller does not need handle checkbox values separately