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

cleanup folders in install script #9639

Merged
merged 5 commits into from
Feb 29, 2024

Conversation

Ruud68
Copy link
Contributor

@Ruud68 Ruud68 commented Feb 29, 2024

Pull Request for Issue # .

Summary of Changes

added preflight folder cleanup. it is safe to remove folders in preflight as these will be then filled with the update or installation of the extension.
ASSUMPTION (!) here is that the site owner did not do any modifications / added files to the folders that are being deleted

It checks for the installed version and when that is above 6.0.0 and below 6.3.0 then the folders are handled.

Testing Instructions

build package with this PR in it and install it, there should not be any errors and everything should be working as it should.
All Kunena provided folders should now be exactly like in the install package without any left over files from previous versions that where not removed.

…el / vi… (Kunena#9637)

* Refactor log clean view into modal making log controller / model / view obsolete

* Refactor log clean view into modal making log controller / model / view obsolete
@xillibit
Copy link
Member

xillibit commented Feb 29, 2024

You should exclude the following files because the user could have added things in it :

\src\site\template\aurelia\assets\scss\custom.scss
\src\site\template\aurelia\config\params.ini
\src\media\kunena\core\js\customckeditor_config.js (the user can personnalize the name of this file in template config it should start with custom)

@Ruud68
Copy link
Contributor Author

Ruud68 commented Feb 29, 2024

Removed the aurelia directory,
what is the installation path of the customckeditor_config.js file? As I cannot find this on my dev station

@xillibit
Copy link
Member

It's \joomla-51-dev-kunena-63-dev\media\kunena\core\js\

@Ruud68
Copy link
Contributor Author

Ruud68 commented Feb 29, 2024

Ok, this PR doesn't remove anything in the [site]/media folder

@xillibit xillibit merged commit edd8241 into Kunena:K6.3 Feb 29, 2024
4 checks passed
@Ruud68 Ruud68 deleted the cleanup-folders-preflight branch March 1, 2024 08:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants