Feeling Lucky
?
[#24162] - [4.0] - unistall extension fix
- Closed
- 14 Mar 2019
- Medium
- Build: 4.0-dev
- # 24162
- Diff
- alikon:patch-81
User tests: Successful: Unsuccessful:
Pull Request for Issue #24155 .
Summary of Changes
use the correct method parameter
Testing Instructions
install an extension (i.e com_patchtester)
uninstall the previous one
Expected result
Extension is uninstalled correctly
Actual result
Error 'Too few arguments' passed to the function 'onExtensionBeforeUninstall' of the new plugin 'extension/finder/finder.php'
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End Plugins |
SharkyKZ
- comment
- 11 Mar 2019
Test this scenario:
View #__finder_terms_common table. See it has some words for en language.
Try to uninstall en-GB (core) language. The installation will fail because it can't be uninstalled.
View #__finder_terms_common again. The data is now gone.
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2019-03-14 11:32:52 |
| Closed_By | ⇒ | alikon | |
| Labels |
Added:
?
|
||
This isn't right. The plugin should only run after uninstallation. See PR #24163 for alternative.