Details des Tickets
Beschreibung,Kommentare und Anhänge
| `public function scopePluckForOptions($query): Collection`
Mit dem Update wird `scopePluckForOptions` auf `Illuminate\Support\Collection` festgelegt, statt zusätzlich `Illuminate\Database\Eloquent\Collection` zu erlauben... Ich begrüße die Reduktion der Rückgabewerte!
Warum nehmen wir nicht `Illuminate\Database\Eloquent\Collection`? diese ist von `Illuminate\Support\Collection` abgeleitet:
```
class Collection extends BaseCollection implements QueueableCollection
``` |
Sascha Immig |
2024-01-16T16:07:31.077Z |
| Das zweite Thema, was mit auffällt, was aber eher ein Task für ein separates Ticket / einen separaten Branch ist:
Haben wir beide Collections im Code, heißt zumeinst eine Einfach `Collection` und die zweite dann `EloquentCollection` oder `SupportCollection`, je nach dem, welche zuerst da war...
Ist nur eine der Beiden da, heißt sie immer `Collection` und wir können raten... Ich weiß, es gibt jede Menge Unterstützung in der IDE, aber eine Namenskonvention hier fände ich "schön"..
Vorschlag: wir Orientieren uns an `Illuminate\Database\Eloquent\Collection` und nennen `Illuminate\Support\Collection` immer `BaseCollection`... Was hälst du davon ,@gabrielg? |
Sascha Immig |
2024-01-16T16:08:19.505Z |
| Möglicherweise ist hier das Blade nicht i.O., wenn diese Texte nicht benötigt werden. Nochmals prüfen @immigs. |
Sascha Immig |
2024-01-16T16:19:05.731Z |
| s.o. |
Sascha Immig |
2024-01-16T16:19:14.923Z |
| Möglicherweise ist hier das Blade nicht i.O., wenn dieser Text nicht benötigt werden. der Text sollte meines Wissens als Quelle erscheinen, wenn die Buchung für eine Einzelperson vorgenommen wurde.
Nochmals prüfen @immigs . |
Sascha Immig |
2024-01-16T16:20:14.415Z |
| Bitte kurze Rücksprache, was die Änderung bewirkt. Kann das aktuell nicht einschätzen. |
Sascha Immig |
2024-01-16T16:21:07.778Z |
| resolved all discussions |
Sascha Immig |
2024-01-17T09:21:26.685Z |
| merged |
Sascha Immig |
2024-01-17T09:22:25.072Z |
| mentioned in commit 7012dc737f1eee5dae116cecb08b2aa86da8d5b8 |
Sascha Immig |
2024-01-17T09:22:25.327Z |
| Title |
Id |
Author |
Created at |
Message |
| readded necessary loop keys |
1185be7320e3df766fdf2fd8210458e0f2ffc275 |
TUV\gabrielg |
2024-01-10T17:46:49.000Z |
readded necessary loop keys
|
| applied suggested corrections from ide (partly they were already fixed but appeared again) |
1e4c77772dcfa5d130bff56cf46d52e57ebc9b93 |
TUV\gabrielg |
2024-01-10T13:09:42.000Z |
applied suggested corrections from ide (partly they were already fixed but appeared again)
|
| fixed some small errors |
5163d7a2469e3430bb2f67b57789352607a07729 |
TUV\gabrielg |
2024-01-10T17:37:38.000Z |
fixed some small errors
|
| renamed loop variables to different names in nested template loops to make code… |
7f9a8c11f5531c5d96e675dba65072bd92ee9bb4 |
TUV\gabrielg |
2024-01-11T08:33:01.000Z |
renamed loop variables to different names in nested template loops to make code eazy understandable and express, that code is intended as it is
|
| fixed errors found by "qodana" PHP-Storm Plugin |
c1079b6b2cce31ac12a950ec8ea75d2a1f5a5393 |
TUV\gabrielg |
2024-01-10T16:40:07.000Z |
fixed errors found by "qodana" PHP-Storm Plugin
|