Make the whole data model immutable

Author: thiemowmde

README.md

@@ -11,6 +11,41 @@ at the heart of the [Wikibase software](http://wikiba.se/).
 ### 3.0.0 (alpha)
 
 * Removed cloning from `MultiTerm.getTexts`.
+* The whole data model is now immutable:
+** Removed `Claim.setMainSnak`
+** Removed `Claim.setQualifiers`
+** Removed `Entity.setFingerprint`
+** Removed `Fingerprint.removeAliases`
+** Removed `Fingerprint.removeAliasesFor`
+** Removed `Fingerprint.removeDescription`
+** Removed `Fingerprint.removeDescriptionFor`
+** Removed `Fingerprint.removeLabel`
+** Removed `Fingerprint.removeLabelFor`
+** Removed `Fingerprint.setAliases`
+** Removed `Fingerprint.setDescription`
+** Removed `Fingerprint.setLabel`
+** Made `Group.setItemContainer` private
+** Removed `GroupableCollection.addItem`
+** Removed `GroupableCollection.removeItem`
+** Removed `Item.addSiteLink`
+** Removed `Item.addStatement`
+** Removed `Item.removeSiteLink`
+** Removed `Item.removeStatement`
+** Removed `MultiTerm.setLanguageCode`
+** Removed `MultiTerm.setTexts`
+** Removed `Property.addStatement`
+** Removed `Property.removeStatement`
+** Removed `SiteLink.setBadges`
+** Removed `SnakList.getValidMoveIndices`
+** Removed `SnakList.merge`
+** Removed `SnakList.move`
+** Removed `SnakList.moveDown`
+** Removed `SnakList.moveUp`
+** Removed `Statement.setClaim`
+** Made `Statement.setRank` private
+** Removed `Statement.setReferences`
+** Removed `Term.setLanguageCode`
+** Removed `Term.setText`
 
 ### 2.0.1 (2016-01-27)
 

src/Claim.js

@@ -14,10 +14,20 @@
  * @param {wikibase.datamodel.SnakList|null} [qualifiers=new wikibase.datamodel.SnakList()]
  * @param {string|null} [guid=null] The Global Unique Identifier of this Claim. Can be null if this
  *        is a new Claim, not yet stored in the database and associated with some entity.
+ *
+ * @throws {Error} if parameter is not a Snak instance.
+ * @throws {Error} if parameter is not a SnakList instance.
  */
 var SELF = wb.datamodel.Claim = function WbDataModelClaim( mainSnak, qualifiers, guid ) {
-	this.setMainSnak( mainSnak );
-	this.setQualifiers( qualifiers || new wb.datamodel.SnakList() );
+	if( !( mainSnak instanceof wb.datamodel.Snak ) ) {
+		throw new Error( 'Main snak needs to be a Snak instance' );
+	}
+	if( qualifiers && !( qualifiers instanceof wb.datamodel.SnakList ) ) {
+		throw new Error( 'Qualifiers have to be a SnakList object' );
+	}
+
+	this._mainSnak = mainSnak;
+	this._qualifiers = qualifiers || new wb.datamodel.SnakList();
 	this._guid = guid || null;
 };
 
@@ -59,39 +69,13 @@ $.extend( SELF.prototype, {
 		return this._mainSnak;
 	},
 
-	/**
-	 * Overwrites the current main Snak.
-	 *
-	 * @param {wikibase.datamodel.Snak} mainSnak
-	 *
-	 * @throws {Error} if parameter is not a Snak instance.
-	 */
-	setMainSnak: function( mainSnak ) {
-		if( !( mainSnak instanceof wb.datamodel.Snak ) ) {
-			throw new Error( 'Main snak needs to be a Snak instance' );
-		}
-		this._mainSnak = mainSnak;
-	},
-
 	/**
 	 * @return {wikibase.datamodel.SnakList}
 	 */
 	getQualifiers: function() {
 		return this._qualifiers;
 	},
 
-	/**
-	 * @param {wikibase.datamodel.SnakList} qualifiers
-	 *
-	 * @throws {Error} if parameter is not a SnakList instance.
-	 */
-	setQualifiers: function( qualifiers ) {
-		if( !( qualifiers instanceof wb.datamodel.SnakList ) ) {
-			throw new Error( 'Qualifiers have to be a SnakList object' );
-		}
-		this._qualifiers = qualifiers;
-	},
-
 	/**
 	 * @param {*} claim
 	 * @return {boolean}

src/Entity.js

@@ -51,13 +51,6 @@ $.extend( SELF.prototype, {
 		return this._fingerprint;
 	},
 
-	/**
-	 * @param {wikibase.datamodel.Fingerprint} fingerprint
-	 */
-	setFingerprint: function( fingerprint ) {
-		this._fingerprint = fingerprint;
-	},
-
 	/**
 	 * Returns what type of Entity this is.
 	 *

src/Fingerprint.js

@@ -87,33 +87,6 @@ $.extend( SELF.prototype, {
 		return this._labels.hasItemForKey( languageCode );
 	},
 
-	/**
-	 * @param {string} languageCode
-	 * @param {wikibase.datamodel.Term} term
-	 */
-	setLabel: function( languageCode, term ) {
-		if ( term.getText() === '' ) {
-			this._labels.removeItemByKey( languageCode );
-		} else {
-			this._labels.setItem( languageCode, term );
-		}
-	},
-
-	/**
-	 * @param {string} languageCode
-	 * @param {wikibase.datamodel.Term} label
-	 */
-	removeLabel: function( languageCode, label ) {
-		this._labels.removeItem( languageCode, label );
-	},
-
-	/**
-	 * @param {string} languageCode
-	 */
-	removeLabelFor: function( languageCode ) {
-		this._labels.removeItemByKey( languageCode );
-	},
-
 	/**
 	 * @return {wikibase.datamodel.TermMap}
 	 */
@@ -146,33 +119,6 @@ $.extend( SELF.prototype, {
 		return this._descriptions.hasItemForKey( languageCode );
 	},
 
-	/**
-	 * @param {string} languageCode
-	 * @param {wikibase.datamodel.Term} term
-	 */
-	setDescription: function( languageCode, term ) {
-		if ( term.getText() === '' ) {
-			this._descriptions.removeItemByKey( languageCode );
-		} else {
-			this._descriptions.setItem( languageCode, term );
-		}
-	},
-
-	/**
-	 * @param {string} languageCode
-	 * @param {wikibase.datamodel.Term} description
-	 */
-	removeDescription: function( languageCode, description ) {
-		this._descriptions.removeItem( languageCode, description );
-	},
-
-	/**
-	 * @param {string} languageCode
-	 */
-	removeDescriptionFor: function( languageCode ) {
-		this._descriptions.removeItemByKey( languageCode );
-	},
-
 	/**
 	 * @return {wikibase.datamodel.MultiTermMap}
 	 */
@@ -205,60 +151,6 @@ $.extend( SELF.prototype, {
 		return this._aliases.hasItemForKey( languageCode );
 	},
 
-	/**
-	 * @param {string|wikibase.datamodel.MultiTermMap} languageCodeOrAliases
-	 * @param {wikibase.datamodel.MultiTerm} [aliases]
-	 *
-	 * @throws {Error} when passing a MultiTerm without a language code.
-	 * @throws {Error} when passing a MultiTermMap with a language code.
-	 * @throws {Error} when neither passing a MultiTerm nor a MultiTermMap object.
-	 */
-	setAliases: function( languageCodeOrAliases, aliases ) {
-		var languageCode;
-
-		if( typeof languageCodeOrAliases === 'string' ) {
-			languageCode = languageCodeOrAliases;
-		} else {
-			aliases = languageCodeOrAliases;
-		}
-
-		if( aliases instanceof wb.datamodel.MultiTerm ) {
-			if( !languageCode ) {
-				throw new Error( 'Language code the wb.datamodel.MultiTerm object should be set '
-					+ 'for needs to be specified' );
-			}
-			if ( aliases.isEmpty() ) {
-				this._aliases.removeItemByKey( languageCode );
-			} else {
-				this._aliases.setItem( languageCode, aliases );
-			}
-		} else if( aliases instanceof wb.datamodel.MultiTermMap ) {
-			if( languageCode ) {
-				throw new Error( 'Unable to handle language code when setting a '
-					+ 'wb.datamodel.MultiTermMap' );
-			}
-			this._aliases = aliases;
-		} else {
-			throw new Error( 'Aliases need to be specified as wb.datamodel.MultiTerm or '
-				+ 'wb.datamodel.MultiTermMap instance' );
-		}
-	},
-
-	/**
-	 * @param {string} languageCode
-	 * @param {wikibase.datamodel.MultiTerm} aliases
-	 */
-	removeAliases: function( languageCode, aliases ) {
-		this._aliases.removeItem( languageCode, aliases );
-	},
-
-	/**
-	 * @param {string} languageCode
-	 */
-	removeAliasesFor: function( languageCode ) {
-		this._aliases.removeItemByKey( languageCode );
-	},
-
 	/**
 	 * @return {boolean}
 	 */

src/Group.js

@@ -76,9 +76,9 @@ $.extend( SELF.prototype, {
 
 	/**
 	 * @param {wikibase.datamodel.GroupableCollection} groupableCollection
-	 *
 	 * @throws {Error} when passed GroupableCollection instance contains an item whose key does not
 	 *         match the key registered with the Group instance.
+	 * @private
 	 */
 	setItemContainer: function( groupableCollection ) {
 		if( !( groupableCollection instanceof wb.datamodel.GroupableCollection ) ) {

src/GroupableCollection.js

@@ -31,22 +31,6 @@ $.extend( SELF.prototype, {
 	 */
 	hasItem: util.abstractMember,
 
-	/**
-	 * Adds an item to the collection.
-	 * @abstract
-	 *
-	 * @param {*} item
-	 */
-	addItem: util.abstractMember,
-
-	/**
-	 * Removes an item from the collection.
-	 * @abstract
-	 *
-	 * @param {*} item
-	 */
-	removeItem: util.abstractMember,
-
 	/**
 	 * Returns whether the collection contains any items.
 	 * @abstract

src/Item.js

@@ -62,41 +62,13 @@ var SELF = wb.datamodel.Item = util.inherit(
 		return this._siteLinkSet;
 	},
 
-	/**
-	 * @param {wikibase.datamodel.SiteLink} siteLink
-	 */
-	addSiteLink: function( siteLink ) {
-		this._siteLinkSet.setSiteLink( siteLink );
-	},
-
-	/**
-	 * @param {wikibase.datamodel.SiteLink} siteLink
-	 */
-	removeSiteLink: function( siteLink ) {
-		this._siteLinkSet.removeSiteLink( siteLink );
-	},
-
 	/**
 	 * @return {wikibase.datamodel.StatementGroupSet}
 	 */
 	getStatements: function() {
 		return this._statementGroupSet;
 	},
 
-	/**
-	 * @param {wikibase.datamodel.Statement} statement
-	 */
-	addStatement: function( statement ) {
-		this._statementGroupSet.addStatement( statement );
-	},
-
-	/**
-	 * @param {wikibase.datamodel.Statement} statement
-	 */
-	removeStatement: function( statement ) {
-		this._statementGroupSet.removeStatement( statement );
-	},
-
 	/**
 	 * @return {boolean}
 	 */

src/MultiTerm.js

@@ -12,10 +12,20 @@
  *
  * @param {string} languageCode
  * @param {string[]} texts
+ *
+ * @throws {Error} when the language code is not a string or empty.
+ * @throws {Error} when texts is not an array.
  */
 var SELF = wb.datamodel.MultiTerm = function WbDataModelMultiTerm( languageCode, texts ) {
-	this.setLanguageCode( languageCode );
-	this.setTexts( texts );
+	if( typeof languageCode !== 'string' || languageCode === '' ) {
+		throw new Error( 'Language code has to be a non-empty string' );
+	}
+	if( !$.isArray( texts ) ) {
+		throw new Error( 'texts needs to be an array of strings' );
+	}
+
+	this._languageCode = languageCode;
+	this._texts = texts;
 };
 
 $.extend( SELF.prototype, {
@@ -38,37 +48,13 @@ $.extend( SELF.prototype, {
 		return this._languageCode;
 	},
 
-	/**
-	 * @param {string} languageCode
-	 *
-	 * @throws {Error} when the language code is not a string or empty.
-	 */
-	setLanguageCode: function( languageCode ) {
-		if( typeof languageCode !== 'string' || languageCode === '' ) {
-			throw new Error( 'Language code has to be a non-empty string' );
-		}
-		this._languageCode = languageCode;
-	},
-
 	/**
 	 * @return {string[]}
 	 */
 	getTexts: function() {
 		return this._texts;
 	},
 
-	/**
-	 * @param {string[]} texts
-	 *
-	 * @throws {Error} when texts is not an array.
-	 */
-	setTexts: function( texts ) {
-		if( !$.isArray( texts ) ) {
-			throw new Error( 'texts needs to be an array of strings' );
-		}
-		this._texts = texts;
-	},
-
 	/**
 	 * @param {*} multiTerm
 	 * @return {boolean}