Fixes #6086, in piwikApi angularjs object, make sure we use and add abort()...

Fixes #6086, in piwikApi angularjs object, make sure we use and add abort() method to new promises returned by angularjs promise methods.

plugins/CoreHome/angularjs/common/services/piwik-api.js

@@ -90,28 +90,29 @@ angular.module('piwikApp.service').factory('piwikApi', function ($http, $q, $roo
 
         // we can't modify requestPromise directly and add an abort method since for some reason it gets
         // removed after then/finally/catch is called.
-        var request = {
-            then: function () {
-                requestPromise.then.apply(requestPromise, arguments);
-                return this;
-            },
-
-            'finally': function () {
-                requestPromise['finally'].apply(requestPromise, arguments);
-                return this;
-            },
-
-            'catch': function () {
-                requestPromise['catch'].apply(requestPromise, arguments);
-                return this;
-            },
-
-            abort: function () {
-                deferred.reject();
-                return this;
-            }
+        var addAbortMethod = function (to) {
+            return {
+                then: function () {
+                    return addAbortMethod(to.then.apply(to, arguments));
+                },
+
+                'finally': function () {
+                    return addAbortMethod(to['finally'].apply(to, arguments));
+                },
+
+                'catch': function () {
+                    return addAbortMethod(to['catch'].apply(to, arguments));
+                },
+
+                abort: function () {
+                    deferred.reject();
+                    return this;
+                }
+            };
         };
 
+        var request = addAbortMethod(requestPromise);
+
         allRequests.push(request);
 
         return request;

plugins/CoreHome/angularjs/common/services/piwik-api_spec.js

@@ -46,8 +46,11 @@ describe('piwikApiClient', function () {
             method: "SomePlugin.action"
         }).then(function (response) {
             firstThenDone = true;
+
+            return "newval";
         }).then(function (response) {
             expect(firstThenDone).to.equal(true);
+            expect(response).to.equal("newval");
 
             done();
         }).catch(function (ex) {