Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[5.0] sort deleteFiles in update script #2870

Merged
merged 2 commits into from Aug 21, 2023

Conversation

heelc29
Copy link
Member

@heelc29 heelc29 commented Jul 25, 2023

Zusammenfassung der Änderungen

> Clear the lists of deleted files and folders and renamed files in script.php by old entries for updating from 3 to 4.

joomla/joomla-cms#40083

gelöschte Dateien nach Joomla-Version in script.php sortiert

Wo wird der Sprachstring angezeigt / Wie kann getestet werden

code review

@heelc29 heelc29 added this to the 5.0.0v1 milestone Jul 25, 2023
@heelc29 heelc29 self-assigned this Jul 25, 2023
@zero-24
Copy link
Member

zero-24 commented Aug 2, 2023

Hi @heelc29

wir können die Einträge nicht löschen, da wir nicht davon ausgehen können das ein Update von de-DE gemacht worden ist.
Ein mögliches Szenario wäre 3.10 -> 4.4 -> 5.0 ohne vorher das update für das sprachpaket zu installieren.

Der Core kann die Dateien löschen da es da ja garantiert ist das die script.php gelaufen ist wir können das aber nicht.

Ich würde hier vorschlagen in der script.php in dem array einen Kommentar zu schreiben der auf die alten Joomla 3 sprachdateien hinweist.

// Old files from Joomla 3 language packs

@heelc29 heelc29 changed the title [5.0] cleanup deleteFiles in update script [5.0] sort deleteFiles in update script Aug 5, 2023
@heelc29
Copy link
Member Author

heelc29 commented Aug 5, 2023

Ich würde hier vorschlagen in der script.php in dem array einen Kommentar zu schreiben der auf die alten Joomla 3 sprachdateien hinweist.

@zero-24 erledigt

@heelc29 heelc29 marked this pull request as ready for review August 13, 2023 17:19
@heelc29 heelc29 requested a review from a team August 13, 2023 17:37
@tecpromotion tecpromotion merged commit 8133c0a into joomlagerman:5.0-dev Aug 21, 2023
@heelc29 heelc29 deleted the 5.0/script/cleanup branch August 21, 2023 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

3 participants