[BUGFIX] Install tool: Check for broken ext uninstalls correct ext 36/23336/5
authorChristoph Buchli <cbuchli@snowflake.ch>
Mon, 26 Aug 2013 20:16:15 +0000 (22:16 +0200)
committerWouter Wolters <typo3@wouterwolters.nl>
Tue, 27 Aug 2013 11:17:23 +0000 (13:17 +0200)
"Check for broken extensions" in important actions of the install
tool renders uninstall buttons for every broken extension it found.

Currently only the last extension is uninstalled, no matter which
button is used. The patch fixes this by adding a class to identify
the extension in question to each button and adapts the javascript.

Resolves: #51368
Releases: 6.2
Change-Id: I7b754aca58e262dbd726452f816fe4dfaa6cf54d
Reviewed-on: https://review.typo3.org/23336
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Stefan Neufeind
Reviewed-by: Susanne Moog
Tested-by: Susanne Moog
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
typo3/sysext/install/Resources/Public/Javascript/Install.js

index d24ccd5..b447fb8 100644 (file)
@@ -143,8 +143,9 @@ function handleCheckExtensionsSuccess() {
                                        var extension = extensions[i];
                                        var unloadButton = $('<button />', {
                                                text: 'Uninstall '+ $.trim(extension),
+                                               "class": $.trim(extension),
                                                click: function(e) {
-                                                       uninstallExtension($.trim(extension));
+                                                       uninstallExtension($(this).attr('class'));
                                                        e.preventDefault();
                                                        return false;
                                                }
@@ -230,4 +231,4 @@ function clearCache() {
                url: location.href + '&install[controller]=ajax&install[action]=clearCache',
                cache: false
        });
-}
\ No newline at end of file
+}