Details des Tickets
Beschreibung,Kommentare und Anhänge
| assigned to @gabrielg |
Sascha Immig |
2021-12-02T14:43:42.531Z |
| kann man die obigen 4 Zeilen nicht einfach als
`Redirect::where('old_url', $oldUrl)->delete();`
schreiben? Er löscht doch eh nur das, was die Query liefert. |
Gregor Gabriel |
2021-12-02T14:55:23.128Z |
| kann man die oberen 6 Zeilen nicht so schreiben:
`$slugRedirect = optional(Redirect::where('old_url', $slug)->first())->new_url ?? '/';`
`if (! Redirect::where('old_url', $slug)->count()) {`
` Redirect::create(['old_url' => $slug, 'new_url' => $slugRedirect]);`
`}` |
Gregor Gabriel |
2021-12-02T15:28:04.852Z |
| Generelle Frage?
Brauchen wir eine Migration die Redirects auf Pages die nicht mehr veröffentlicht sind auf '/' umbiegt oder löscht? |
Gregor Gabriel |
2021-12-02T15:37:18.925Z |
| Der Code sieht ganz ordentlich aus. Ich bin mir nicht immer ganz sicher das ich die Anforderungen verstehe. Falls der Code auf myCopetence übernommen wurde, und dort das Verhalten wie im Ticket beschrieben ist umgesetzt wurde passt das generell von meiner Seite.
Die folgenden 2 Anforderungen verstehe ich nicht so recht:
1. Page update() Änderungen an Redirects nur falls es keine andere Page mit Published gibt
2. Page update() Redirects ändern auch wenn der Status auf Entwurf gesetzt wird |
Gregor Gabriel |
2021-12-02T15:42:08.127Z |
| Top funktioniert :) |
Markus Gerlach |
2021-12-03T07:22:04.905Z |
| "Brauchen wir eine Migration die Redirects auf Pages die nicht mehr veröffentlicht sind auf '/' umbiegt oder löscht?"
Sinnvol wäre eine Änderung zu "/". Es sollte noch geprüft werden ob es eine aktive Clientseite gibt. |
Markus Gerlach |
2021-12-03T07:34:28.334Z |
| ja geht auch |
Markus Gerlach |
2021-12-03T08:17:54.290Z |
| https://gitlab.learningsystem.de/lms/seventeen/merge_requests/529#note_4797
1. Da dies bei Page update nur abgefragt wird, falls der alte Status 'PUBLISHED' ist ist dies überflüssig. Macht Sinn wenn deleteUrlFromRedirects() von wo anders aufgerufen wird z.B. delete().
2. Indem Fall ist die Seite nicht erreichbar also sollte es auch kein Redirect zur Seite geben. |
Markus Gerlach |
2021-12-03T10:44:22.429Z |
| changed this line in version 2 of the diff |
Markus Gerlach |
2021-12-03T10:51:02.276Z |
added 1 commit
- 89ad5e9f - LL-1043-page-redirects-bug-optimization
Compare with previous version |
Markus Gerlach |
2021-12-03T10:51:02.539Z |
| resolved all discussions |
Gregor Gabriel |
2021-12-07T10:13:56.534Z |
| merged |
Gregor Gabriel |
2021-12-07T10:15:01.081Z |
| mentioned in commit 684e0d063d1534dfc3e602b90715c45b3f0771a0 |
Gregor Gabriel |
2021-12-07T10:15:01.215Z |