mediawiki.special.upload.js: Remove spinner in .always() instead of .done()
authorFomafix <fomafix@googlemail.com>
Fri, 15 Apr 2016 04:43:04 +0000 (04:43 +0000)
committer[[mw:User:Fomafix]] <gerritpatchuploader@gmail.com>
Fri, 15 Apr 2016 04:43:04 +0000 (04:43 +0000)
This removes the spinner even when the API request fails.

Change-Id: Ic0f08312b9a1137772f5b6734256e7bd48d20fcf

resources/src/mediawiki.special/mediawiki.special.upload.js

index 8c89ed9..945bfb0 100644 (file)
@@ -74,8 +74,9 @@
                                if ( result.query ) {
                                        resultOut = result.query.pages[ result.query.pageids[ 0 ] ].imageinfo[ 0 ];
                                }
-                               $spinnerDestCheck.remove();
                                uploadWarning.processResult( resultOut, uploadWarning.nameToCheck );
+                       } ).always( function () {
+                               $spinnerDestCheck.remove();
                        } );
                },
 
                                prop: 'text',
                                pst: true
                        } ).done( function ( result ) {
-                               $spinnerLicense.remove();
                                uploadLicense.processResult( result, license );
+                       } ).always( function () {
+                               $spinnerLicense.remove();
                        } );
                },