User tests: Successful: Unsuccessful:
The installer for components and plugins already automatically delete files, which are no longer present in the extension manifest. The module installer doesn't do that for some reason.
This PR proposes to add this functionality as I think it's a helpful feature for extension developers. Otherwise one has to manage a script file to delete no longer used files.
Added a property to track old installed files for updates.
This allows the parseFiles() method to automatically remove deleted files.
The Same code is already used in plugin and component adapter.
Install a module which used to have a file or folder that got deleted. Make sure it's an update (the module was already installed before).
You can fake such a module by taking any module and remove a file or folder line in the manifest and then install it.
The removed files or folder in the manifest are still present on the webserver after the update
The removed files or folder in the manifest are deleted on the webserver after the update
Please select:
Documentation link for docs.joomla.org:
No documentation changes for docs.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
| Category | ⇒ | Libraries |
| Labels |
Added:
PR-5.4-dev
|
||
| Labels |
Added:
Feature
|
||
| Category | Libraries | ⇒ | Repository Administration com_admin SQL |
As it is a new feature, this PR has to be rebased to 6.1-dev.
Done, but I think I broke the CI and you need to restart it :-)
Sorry for that.
| Title |
|
||||||
| Category | Repository Administration com_admin SQL | ⇒ | Libraries |
| Labels |
Added:
PR-6.1-dev
Removed: PR-5.4-dev |
||
As it is a new feature, this PR has to be rebased to 6.1-dev.
Done, but I think I broke the CI and you need to restart it :-) Sorry for that.
@Bakual The easiest way to fix that is to commit a change. I've allowed myself to apply a necessary change (use __DEPLOY_VERSION__) so we did not need a dummy change for that.
Perfect, thanks!
As it is a new feature, this PR has to be rebased to 6.1-dev.