jenkins-bot has submitted this change and it was merged.

Change subject: Remove unused wikibase.RepoApi.removeSitelink
......................................................................


Remove unused wikibase.RepoApi.removeSitelink

Change-Id: I484d36b98f5ec002b158e79b6d60a4cd3b95dbd1
---
M lib/resources/wikibase.RepoApi/wikibase.RepoApi.js
M lib/tests/qunit/wikibase.RepoApi/wikibase.RepoApi.tests.js
2 files changed, 0 insertions(+), 27 deletions(-)

Approvals:
  Hoo man: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/lib/resources/wikibase.RepoApi/wikibase.RepoApi.js 
b/lib/resources/wikibase.RepoApi/wikibase.RepoApi.js
index 30da8e3..20f3a43 100644
--- a/lib/resources/wikibase.RepoApi/wikibase.RepoApi.js
+++ b/lib/resources/wikibase.RepoApi/wikibase.RepoApi.js
@@ -465,18 +465,6 @@
        },
 
        /**
-        * Removes a sitelink of an item via the API.
-        *
-        * @param {String} id entity id
-        * @param {Number} baseRevId revision id
-        * @param {String} site the site of the link
-        * @return {jQuery.Promise}
-        */
-       removeSitelink: function( id, baseRevId, site ) {
-               return this.setSitelink( id, baseRevId, site, '' );
-       },
-
-       /**
         * Set the required options and parameters for a repo call from a 
client, if needed
         *
         * @since 0.4
diff --git a/lib/tests/qunit/wikibase.RepoApi/wikibase.RepoApi.tests.js 
b/lib/tests/qunit/wikibase.RepoApi/wikibase.RepoApi.tests.js
index c115730..8f47671 100644
--- a/lib/tests/qunit/wikibase.RepoApi/wikibase.RepoApi.tests.js
+++ b/lib/tests/qunit/wikibase.RepoApi/wikibase.RepoApi.tests.js
@@ -440,21 +440,6 @@
                } );
 
                testrun.queue( qkey, function() {
-                       api.removeSitelink(
-                               entity.id, entity.lastrevid, 
data.sitelinks.dewiki.site
-                       ).done( function( response ) {
-
-                               assert.equal(
-                                       response.success,
-                                       1,
-                                       'Removed site link.'
-                               );
-
-                               testrun.dequeue( qkey );
-                       } ).fail( onFail );
-               } );
-
-               testrun.queue( qkey, function() {
                        api.getEntities( entity.id, 'sitelinks' ).done( 
function( response ) {
 
                                assert.deepEqual(

-- 
To view, visit https://gerrit.wikimedia.org/r/160422
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I484d36b98f5ec002b158e79b6d60a4cd3b95dbd1
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: master
Gerrit-Owner: Adrian Lang <[email protected]>
Gerrit-Reviewer: Hoo man <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to