Details des Tickets
Beschreibung,Kommentare und Anhänge
| Ist das so korrekt? ich würde von `getProgressStatusCodeAttribute()` immer einen String erwarten. |
Sascha Immig |
2022-11-30T16:07:17.475Z |
| Den Kommentar würde ich gerne behalten |
Sascha Immig |
2022-11-30T16:07:38.317Z |
| Kann der Kommentar `# $this->training->minimal_mode;` nicht weg? |
Sascha Immig |
2022-11-30T16:08:25.146Z |
| kann hier ebenfalls so viel im `@return` zurückgegeben werden? |
Sascha Immig |
2022-11-30T16:09:49.462Z |
| So wie ich das sehe, läuft jedes `@return` auf einen String hinaus... |
Sascha Immig |
2022-11-30T16:12:34.212Z |
| s.o. |
Sascha Immig |
2022-11-30T16:14:23.019Z |
| jetzt wäre doch ein guter Zeitpunkt das direkt zu entfernen, oder? |
Sascha Immig |
2022-11-30T16:17:29.044Z |
| m.E. nicht korrekt, die Funktion macht nicht mehr das gleiche wie vorher.
vorher: Wenn es einen Namen und eine Anrede gibt, die Anrede mit `display_name` konkatenieren.
jetzt: Wenn es einen Namen gibt, wird dieser mit `display_name`konkateniert. Gibt es eine Anred wird der Name wieder überschrieben...
Zudem ist `if ($this->salutatio) {` falsch, hier fehlt ein 'n' |
Sascha Immig |
2022-11-30T16:24:01.944Z |
| ist etwas gegen `is_null()` einzuwenden? |
Sascha Immig |
2022-11-30T16:25:26.179Z |
| Kommentar umformulieren, aber inhaltlich behalten: Wichtig ist, dass - wenn mehr als ein `TrainingProgress` gefunden wird, der mit dem höchsten Fortschritt zurückgegben wird. |
Sascha Immig |
2022-11-30T16:27:46.256Z |
| Kommentar behalten |
Sascha Immig |
2022-11-30T16:28:20.027Z |
| wenn `$this->is_active` `false` ist, wurde vorher immer `false` zurückgegeben. Ungeachtet etwaige temporärer inaktivität.
Jetzt wird zunächst die temporäre Inaktivität geprüft. Dies kann zu anderen Resultaten führen. |
Sascha Immig |
2022-11-30T16:31:06.499Z |
| Kommentar umformulieren, aber Essenz behalten. |
Sascha Immig |
2022-11-30T16:32:10.928Z |
| Warum raus, woher? |
Sascha Immig |
2022-12-01T13:39:34.552Z |
| changed this line in version 2 of the diff |
Sascha Immig |
2022-12-01T15:12:11.174Z |
added 4 commits
- c99b8435 - updated findClassroomTrainingUsers() to force…
- d41d9c2e - fixed functon of getSalutatedNameAttribute()
- 889452cb - replaced `=== null` with `is_null()`
- 7687e464 - fixed getActiveAttribute()
Compare with previous version |
Sascha Immig |
2022-12-01T15:12:12.930Z |
| TODO hinzugefügt |
Sascha Immig |
2022-12-01T15:25:12.812Z |
| declared string as return |
Sascha Immig |
2022-12-01T15:46:48.758Z |
| added TODO comment |
Sascha Immig |
2022-12-01T15:49:35.549Z |
| changed this line in version 3 of the diff |
Sascha Immig |
2022-12-01T15:51:32.837Z |
| changed this line in version 3 of the diff |
Sascha Immig |
2022-12-01T15:51:33.621Z |
| changed this line in version 3 of the diff |
Sascha Immig |
2022-12-01T15:51:33.840Z |
| changed this line in version 3 of the diff |
Sascha Immig |
2022-12-01T15:51:34.867Z |
added 5 commits
- 46529faa - Restored comment and added TODOs
- eae12389 - force return of string
- a5a31aaa - declared return type of Session
- 304f3963 - declared return value of string
- a5130ae6 - added TODO
Compare with previous version |
Sascha Immig |
2022-12-01T15:51:37.085Z |
| added 1 commit
Compare with previous version |
Sascha Immig |
2022-12-01T16:46:20.341Z |
| added TODO comment |
Sascha Immig |
2022-12-01T16:51:02.148Z |
| Kommentar wiederhergestellt |
Sascha Immig |
2022-12-01T16:52:08.860Z |
| entfernt |
Sascha Immig |
2022-12-01T16:53:13.385Z |
| Kommentar wiederhergestellt |
Sascha Immig |
2022-12-01T16:56:51.168Z |
| Kommentar wiederhergestellt |
Sascha Immig |
2022-12-01T16:57:26.484Z |
| done |
Sascha Immig |
2022-12-01T16:59:19.285Z |
| scheint aus dem alten TNTSearch scout zu stammen. Da wir auf Meili umgestellt haben, dürfte es kein Problem machen, dass das entfernt wird. |
Sascha Immig |
2022-12-01T17:04:04.764Z |
| resolved all discussions |
Sascha Immig |
2022-12-01T17:04:05.494Z |
| changed this line in version 5 of the diff |
Sascha Immig |
2022-12-01T17:08:24.619Z |
| changed this line in version 5 of the diff |
Sascha Immig |
2022-12-01T17:08:24.817Z |
| changed this line in version 5 of the diff |
Sascha Immig |
2022-12-01T17:08:25.855Z |
| resolved all discussions |
Sascha Immig |
2022-12-01T17:08:27.082Z |
added 1 commit
- 1e7f2377 - restored some comments since they contain valuable information for quick…
Compare with previous version |
Sascha Immig |
2022-12-01T17:08:27.527Z |
| Title |
Id |
Author |
Created at |
Message |
| restored some comments since they contain valuable information for quick… |
1e7f2377794c2c30870b47e5360b2be653f437a1 |
TUV\immigs |
2022-12-01T17:08:15.000Z |
restored some comments since they contain valuable information for quick reference on the related function.
|
| declared return value of string |
304f39632ecf630164e8533696262a16f87a6f2f |
TUV\immigs |
2022-12-01T15:50:31.000Z |
declared return value of string
|
| Restored comment and added TODOs |
46529faa4ec1be918ab62aba907e9db47490ef64 |
TUV\immigs |
2022-12-01T15:25:54.000Z |
Restored comment and added TODOs
|
| LL-1301 | Refactored all training related models |
5e6fc73c37cf255625e5dc2b8040e3dcc73fb017 |
Sebastian Majchrzak |
2022-11-28T12:33:47.000Z |
LL-1301 | Refactored all training related models
|
| fixed getActiveAttribute() |
7687e464a1b0fc261044c1fb7aeac70407a507f0 |
TUV\immigs |
2022-12-01T14:59:10.000Z |
fixed getActiveAttribute()
|
| Added TODOs |
7ba30999e7381baa9f944d4a3f98ed944713b460 |
TUV\immigs |
2022-12-01T16:46:10.000Z |
Added TODOs
|
| replaced `=== null` with `is_null()` |
889452cb0bc94a0142e95d0bfc87cebc62e315d0 |
TUV\immigs |
2022-12-01T14:58:22.000Z |
replaced `=== null` with `is_null()`
|
| added TODO |
a5130ae623ec2b96992ed1ff7b90d1e4344772bd |
TUV\immigs |
2022-12-01T15:50:42.000Z |
added TODO
|
| declared return type of Session |
a5a31aaa69d00902864f4d20abe245348927897a |
TUV\immigs |
2022-12-01T15:38:10.000Z |
declared return type of Session
|
| updated findClassroomTrainingUsers() to force… |
c99b8435647c2fda8c46b6614070875cd11ca36f |
TUV\immigs |
2022-12-01T14:13:02.000Z |
updated findClassroomTrainingUsers() to force Illuminate\Database\Eloquent\Collection as return value
|
| LL-1301 | Refactoring from Training.php und User.php models |
ccf4beb5c58c53ecce35a5d2f26abe3b922e81ef |
Sebastian Majchrzak |
2022-11-28T09:09:39.000Z |
LL-1301 | Refactoring from Training.php und User.php models
|
| fixed functon of getSalutatedNameAttribute() |
d41d9c2e7533aabd67dc2595f2d9adae4d772152 |
TUV\immigs |
2022-12-01T14:33:52.000Z |
fixed functon of getSalutatedNameAttribute()
|
| force return of string |
eae12389281b1a7f8a4a0887c2da51e8d27c358b |
TUV\immigs |
2022-12-01T15:37:45.000Z |
force return of string
|