Details des Tickets

Beschreibung,Kommentare und Anhänge

merge request id 492
TitelLL-908-further-cms-status
BeschreibungGregor, kannst du es testen und Sascha übergeben? Er hatte mich darum gebeten. Dieser ablauf ist mit ihm besprochen und freigegeben beim löschen und archiviren. z.B. falls die Seite 'sommeraktion-2019' gelöscht wird: Falls es diese redirects gibt sommeraktion-2018 --> sommeraktion-2019 sommeraktion-2017 --> sommeraktion-2019 würde diese dann automatisch nach '/' also zur Startseite gehen, also entsprechend: sommeraktion-2018 --> / sommeraktion-2017 --> / Falls es diese redirects gibt sommeraktion-2019 --> sommeraktion sommeraktion-2018 --> sommeraktion-2019 Dann würde entsprechend: sommeraktion-2019 --> sommeraktion sommeraktion-2018 --> sommeraktion daraus.
Statusmerged
Created at 2021-09-24T14:30:30.815Z
Updated at 2021-09-29T13:05:46.176Z
Source branchfeature/LL-908-further-cms-status
Target branchdevelop
AuthorMarkus Gerlach
AssigneeGregor Gabriel

Discussion notes

besser: `$redirectToSlug->update(['new_url' => $slugRedirect]);` denn wir haben das Objekt ja schon noch besser: ```javascript if (Redirect::where('new_url', $slug)->count() > 0) { // get redirect from slug if exist else use / to homepage $slugRedirect = optional(Redirect::where('old_url', $slug)->first())->new_url ?? '/'; Redirect::where('new_url', $slug)->update(['new_url' => $slugRedirect]); } ```` Gregor Gabriel 2021-09-27T07:31:58.202Z
changed this line in version 2 of the diff Markus Gerlach 2021-09-27T10:34:13.080Z
resolved all discussions Markus Gerlach 2021-09-27T10:34:13.151Z
added 1 commit
  • b9bebd46 - LL-908-further-cms-status
Compare with previous version		 Markus Gerlach 2021-09-27T10:34:13.397Z
der alte Code hat auch den Fall behandelt, dass *status* null ist und hat dann den Status draft gesetzt. Das ist jetzt entfallen. Kann das nicht passieren? Gregor Gabriel 2021-09-27T10:59:15.543Z
Mandante-Urls müsssen hier noch zusätzlich geprüft werden Gregor Gabriel 2021-09-27T11:38:26.401Z
Genau Standard ist nun immer draft. Markus Gerlach 2021-09-27T11:51:20.247Z
generell ist es korrekt den redirect zu löschen, aber was ist mit folgendem Soderfall: 1. lege Page A mit Slug "go-a" als 'veröffentlicht' an 2. archiviere Page A 3. lege Page B mit Slug "go-a" als 'veröffentlicht' an 4. lege eine Weiterleitung "super-go-a" => "go-a" an 5. lösche nun Page A - damit müsste "super-go-a" letztlich auf "/" verweisen Gregor Gabriel 2021-09-27T12:11:01.831Z
added 1 commit
  • feac86d7 - LL-908-further-cms-status
Compare with previous version		 Markus Gerlach 2021-09-27T15:55:31.657Z
Danke für die Ausführung: In dem Fall sollte die Weiterleitung natürlich weiterhin so bleiben. Somit binde ich noch die Prüfung auf eine Veröfftlichte Seite mit dem Slug ein. Top - gut erkannt Markus Gerlach 2021-09-28T07:53:32.962Z
Was ist in folgendem Fall? es existiert die Seite "Heisseste Nummer" => "heisseste-nummer" und der Mandant "Heisseste Nummer 1" => "heisseste-nummer-1" Jetzt willst du noch eine Seite mit dem Slug "heisseste-nummer" anlegen. Die gibt's aber schon genau 1x also ist $count 1. Es wird also statt dessen versucht den Slug "heisseste-nummer-1" anzulegen, aber den gibt's auch schon :-( Gregor Gabriel 2021-09-28T08:05:03.086Z
changed this line in version 4 of the diff Markus Gerlach 2021-09-29T06:55:56.525Z
changed this line in version 4 of the diff Markus Gerlach 2021-09-29T06:55:56.775Z
added 1 commit
  • c5b26340 - LL-908-further-cms-status
Compare with previous version		 Markus Gerlach 2021-09-29T06:55:56.915Z
added 1 commit
  • ae7487e5 - LL-908-further-cms-status
Compare with previous version		 Markus Gerlach 2021-09-29T07:44:41.993Z
Wird nun berücksichtigt Markus Gerlach 2021-09-29T07:55:03.538Z
changed this line in version 6 of the diff Gregor Gabriel 2021-09-29T11:34:04.895Z
added 1 commit
  • 58fd3382 - made sure, that free slug gets no counter, e.g. if "ssv" is free, but "ssv-2021" is not free
Compare with previous version		 Gregor Gabriel 2021-09-29T11:34:05.043Z
resolved all discussions Gregor Gabriel 2021-09-29T11:52:58.720Z
merged Gregor Gabriel 2021-09-29T13:05:46.369Z
mentioned in commit 8108ee94ef71aa747157c7202171af22fa00ec1a Gregor Gabriel 2021-09-29T13:05:46.877Z

Merge Commits

Title Id Author Created at Message
LL-908-further-cms-status 0f7d38a19859550e2658a7f21f6fd83ae44c335c markusgerlach 2021-09-24T14:15:00.000Z LL-908-further-cms-status
made sure, that free slug gets no counter, e.g. if "ssv" is free, but "ssv-2021" is not free 58fd3382b0bfa8b58e533d82fecb96f2e351abb3 TUV\gabrielg 2021-09-29T11:33:58.000Z made sure, that free slug gets no counter, e.g. if "ssv" is free, but "ssv-2021" is not free
LL-908-further-cms-status ae7487e5456b73c69f254626339518dccfe8c096 markusgerlach 2021-09-29T07:44:38.000Z LL-908-further-cms-status
LL-908-further-cms-status b9bebd46dd20c55417c896557ac18bcfea50a0d6 markusgerlach 2021-09-27T10:33:52.000Z LL-908-further-cms-status
LL-908-further-cms-status c5b26340db7f163c6c56c505be69e2f7904b3967 markusgerlach 2021-09-29T06:55:52.000Z LL-908-further-cms-status
LL-908-further-cms-status feac86d7d766644dc0fb6a1eea670263039a8d40 markusgerlach 2021-09-27T15:48:31.000Z LL-908-further-cms-status