Details des Tickets
Beschreibung,Kommentare und Anhänge
| Zur besseren Lesbarkeit bitte refactoren, sodass erst die Variable befüllt wird, anschließend auf null prüfen und danach prüfen, ob ein Mandant existiert. Jetzt ist der Code zwar in einer Zeile, aber ich hab 2 Minuten gebraucht um zu verstehen, was hier passiert – einfache "=" zeichen bitte niemals innerhalb von if-Abfragen gebrauchen, das ist ganz schlechter Stil |
Sebastian Schlein |
2021-08-31T07:42:47.657Z |
| Auch hier bitte die Zuweisung nicht in der if-Abfrage machen. Stattdessen wäre es so möglich:
```
if (session()->has('sso_redirect')) {
return redirect(session()->pull('sso_redirect'));
}
```
`pull` entfernt das element auch direkt und daher wird forget nicht benötigt. |
Sebastian Schlein |
2021-08-31T07:47:09.020Z |
| So sollte es auch oben gemacht werden ;-) |
Sebastian Schlein |
2021-08-31T07:47:44.422Z |
| Ich kann nicht nachvollziehen, wieso hier ein helper erstellt wurde. Die Funktion scheint an das Client-Model zu gehören. Desweiteren verstehe ich nicht was sie macht und wozu sie gebraucht wird.
In den Notifications sollte der client immerhin vorliegen `$notifiable->client` da $notifiable immer ein User ist und so müsste `notifiable->client->hash_id` bereits das gewünschte Ergebnis haben. Wenn das einen Fehler wirft, sollte sich das $notifiable zu einem User casten lassen. |
Sebastian Schlein |
2021-08-31T07:57:39.658Z |
| Wofür wird das genau gebraucht? Wenn es sich nicht um eine env-variable handelt, die überschreiben werden können muss, ist dies meist ein Zeichen dafür, dass es als Konstante an einem Model besser aufgehoben wäre, beispielsweise als `Client::SlUG` oder ähnliches |
Sebastian Schlein |
2021-08-31T08:00:20.605Z |
| changed this line in version 2 of the diff |
Richard Klings |
2021-08-31T10:43:50.610Z |
| changed this line in version 2 of the diff |
Richard Klings |
2021-08-31T10:43:50.895Z |
| changed this line in version 2 of the diff |
Richard Klings |
2021-08-31T10:43:51.255Z |
added 1 commit
- 86976795 - LL-791: Refactoring des Codes
Compare with previous version |
Richard Klings |
2021-08-31T10:43:51.390Z |
| Angepasst |
Richard Klings |
2021-08-31T10:44:25.605Z |
| Angepasst |
Richard Klings |
2021-08-31T10:44:37.750Z |
| so gelassen |
Richard Klings |
2021-08-31T10:44:51.489Z |
| Entsprechend der slack-diskussion umgesetzt |
Richard Klings |
2021-08-31T10:45:24.346Z |
| Nun keine Konstante mehr |
Richard Klings |
2021-08-31T10:45:36.112Z |
| resolved all discussions |
Sebastian Schlein |
2021-08-31T11:36:15.727Z |
| changed this line in version 3 of the diff |
Richard Klings |
2021-08-31T12:02:24.507Z |
| resolved all discussions |
Richard Klings |
2021-08-31T12:02:24.892Z |
added 1 commit
- 9f86023e - LL-791: nicht gebrauchte helper funktion entfernt.
Compare with previous version |
Richard Klings |
2021-08-31T12:02:25.192Z |
| merged |
Sascha Immig |
2021-08-31T12:22:14.295Z |
| mentioned in commit 10777da85c34410e8990472375287a2360807c9c |
Sascha Immig |
2021-08-31T12:22:14.436Z |
| Title |
Id |
Author |
Created at |
Message |
| LL-791: Im SSO-Fall wird der Deeplink explizit durchgereicht. |
2d12c3da9b792d68715019e2dc4e3bbd5c16b517 |
Richard Klings |
2021-08-30T09:23:09.000Z |
LL-791: Im SSO-Fall wird der Deeplink explizit durchgereicht.
|
| LL-791: Refactoring des Codes |
869767954503069e7e04b263567836bee707357f |
Richard Klings |
2021-08-31T10:43:27.000Z |
LL-791: Refactoring des Codes
|
| LL-971 - for all unauthentificated client users, that call internal links,… |
9dfffa3c1acf50ab0b6ffded963f6cca50d82636 |
TUV\gabrielg |
2021-08-27T11:16:49.000Z |
LL-971 - for all unauthentificated client users, that call internal links, client login page is displayed not general login page
|
| LL-791: nicht gebrauchte helper funktion entfernt. |
9f86023e52fde353ea04d86fb24659e999befddf |
Richard Klings |
2021-08-31T12:01:48.000Z |
LL-791: nicht gebrauchte helper funktion entfernt.
|