@ckeditor/ckeditor5-engine 35.4.0 → 36.0.0

package/src/model/operation/insertoperation.js

@@ -1,5 +1,5 @@
 /**
- * @license Copyright (c) 2003-2022, CKSource Holding sp. z o.o. All rights reserved.
+ * @license Copyright (c) 2003-2023, CKSource Holding sp. z o.o. All rights reserved.
  * For licensing, see LICENSE.md or https://ckeditor.com/legal/ckeditor-oss-license
  */
 /**
@@ -15,42 +15,21 @@ import Element from '../element';
 import { CKEditorError } from '@ckeditor/ckeditor5-utils';
 /**
  * Operation to insert one or more nodes at given position in the model.
- *
- * @extends module:engine/model/operation/operation~Operation
  */
 export default class InsertOperation extends Operation {
     /**
      * Creates an insert operation.
      *
-     * @param {module:engine/model/position~Position} position Position of insertion.
-     * @param {module:engine/model/node~NodeSet} nodes The list of nodes to be inserted.
-     * @param {Number|null} baseVersion Document {@link module:engine/model/document~Document#version} on which operation
+     * @param position Position of insertion.
+     * @param nodes The list of nodes to be inserted.
+     * @param baseVersion Document {@link module:engine/model/document~Document#version} on which operation
      * can be applied or `null` if the operation operates on detached (non-document) tree.
      */
     constructor(position, nodes, baseVersion) {
         super(baseVersion);
-        /**
-         * Position of insertion.
-         *
-         * @readonly
-         * @member {module:engine/model/position~Position} module:engine/model/operation/insertoperation~InsertOperation#position
-         */
         this.position = position.clone();
         this.position.stickiness = 'toNone';
-        /**
-         * List of nodes to insert.
-         *
-         * @readonly
-         * @member {module:engine/model/nodelist~NodeList} module:engine/model/operation/insertoperation~InsertOperation#nodeList
-         */
         this.nodes = new NodeList(_normalizeNodes(nodes));
-        /**
-         * Flag deciding how the operation should be transformed. If set to `true`, nodes might get additional attributes
-         * during operational transformation. This happens when the operation insertion position is inside of a range
-         * where attributes have changed.
-         *
-         * @member {Boolean} module:engine/model/operation/insertoperation~InsertOperation#shouldReceiveAttributes
-         */
         this.shouldReceiveAttributes = false;
     }
     /**
@@ -61,16 +40,12 @@ export default class InsertOperation extends Operation {
     }
     /**
      * Total offset size of inserted nodes.
-     *
-     * @returns {Number}
      */
     get howMany() {
         return this.nodes.maxOffset;
     }
     /**
      * Creates and returns an operation that has the same parameters as this operation.
-     *
-     * @returns {module:engine/model/operation/insertoperation~InsertOperation} Clone of this operation.
      */
     clone() {
         const nodes = new NodeList([...this.nodes].map(node => node._clone(true)));
@@ -80,8 +55,6 @@ export default class InsertOperation extends Operation {
     }
     /**
      * See {@link module:engine/model/operation/operation~Operation#getReversed `Operation#getReversed()`}.
-     *
-     * @returns {module:engine/model/operation/moveoperation~MoveOperation}
      */
     getReversed() {
         const graveyard = this.position.root.document.graveyard;
@@ -134,9 +107,8 @@ export default class InsertOperation extends Operation {
     /**
      * Creates `InsertOperation` object from deserilized object, i.e. from parsed JSON string.
      *
-     * @param {Object} json Deserialized JSON object.
-     * @param {module:engine/model/document~Document} document Document on which this operation will be applied.
-     * @returns {module:engine/model/operation/insertoperation~InsertOperation}
+     * @param json Deserialized JSON object.
+     * @param document Document on which this operation will be applied.
      */
     static fromJSON(json, document) {
         const children = [];

package/src/model/operation/markeroperation.js

@@ -1,5 +1,5 @@
 /**
- * @license Copyright (c) 2003-2022, CKSource Holding sp. z o.o. All rights reserved.
+ * @license Copyright (c) 2003-2023, CKSource Holding sp. z o.o. All rights reserved.
  * For licensing, see LICENSE.md or https://ckeditor.com/legal/ckeditor-oss-license
  */
 /**
@@ -7,57 +7,23 @@
  */
 import Operation from './operation';
 import Range from '../range';
-/**
- * @extends module:engine/model/operation/operation~Operation
- */
 export default class MarkerOperation extends Operation {
     /**
-     * @param {String} name Marker name.
-     * @param {module:engine/model/range~Range|null} oldRange Marker range before the change.
-     * @param {module:engine/model/range~Range|null} newRange Marker range after the change.
-     * @param {module:engine/model/markercollection~MarkerCollection} markers Marker collection on which change should be executed.
-     * @param {Boolean} affectsData Specifies whether the marker operation affects the data produced by the data pipeline
+     * @param name Marker name.
+     * @param oldRange Marker range before the change.
+     * @param newRange Marker range after the change.
+     * @param markers Marker collection on which change should be executed.
+     * @param affectsData Specifies whether the marker operation affects the data produced by the data pipeline
      * (is persisted in the editor's data).
-     * @param {Number|null} baseVersion Document {@link module:engine/model/document~Document#version} on which operation
+     * @param baseVersion Document {@link module:engine/model/document~Document#version} on which operation
      * can be applied or `null` if the operation operates on detached (non-document) tree.
      */
     constructor(name, oldRange, newRange, markers, affectsData, baseVersion) {
         super(baseVersion);
-        /**
-         * Marker name.
-         *
-         * @readonly
-         * @member {String}
-         */
         this.name = name;
-        /**
-         * Marker range before the change.
-         *
-         * @readonly
-         * @member {module:engine/model/range~Range|null}
-         */
         this.oldRange = oldRange ? oldRange.clone() : null;
-        /**
-         * Marker range after the change.
-         *
-         * @readonly
-         * @member {module:engine/model/range~Range}
-         */
         this.newRange = newRange ? newRange.clone() : null;
-        /**
-         * Specifies whether the marker operation affects the data produced by the data pipeline
-         * (is persisted in the editor's data).
-         *
-         * @readonly
-         * @member {Boolean}
-         */
         this.affectsData = affectsData;
-        /**
-         * Marker collection on which change should be executed.
-         *
-         * @private
-         * @member {module:engine/model/markercollection~MarkerCollection}
-         */
         this._markers = markers;
     }
     /**
@@ -68,16 +34,12 @@ export default class MarkerOperation extends Operation {
     }
     /**
      * Creates and returns an operation that has the same parameters as this operation.
-     *
-     * @returns {module:engine/model/operation/markeroperation~MarkerOperation} Clone of this operation.
      */
     clone() {
         return new MarkerOperation(this.name, this.oldRange, this.newRange, this._markers, this.affectsData, this.baseVersion);
     }
     /**
      * See {@link module:engine/model/operation/operation~Operation#getReversed `Operation#getReversed()`}.
-     *
-     * @returns {module:engine/model/operation/markeroperation~MarkerOperation}
      */
     getReversed() {
         return new MarkerOperation(this.name, this.newRange, this.oldRange, this._markers, this.affectsData, this.baseVersion + 1);
@@ -118,9 +80,8 @@ export default class MarkerOperation extends Operation {
     /**
      * Creates `MarkerOperation` object from deserialized object, i.e. from parsed JSON string.
      *
-     * @param {Object} json Deserialized JSON object.
-     * @param {module:engine/model/document~Document} document Document on which this operation will be applied.
-     * @returns {module:engine/model/operation/markeroperation~MarkerOperation}
+     * @param json Deserialized JSON object.
+     * @param document Document on which this operation will be applied.
      */
     static fromJSON(json, document) {
         return new MarkerOperation(json.name, json.oldRange ? Range.fromJSON(json.oldRange, document) : null, json.newRange ? Range.fromJSON(json.newRange, document) : null, document.model.markers, json.affectsData, json.baseVersion);

package/src/model/operation/mergeoperation.js

@@ -1,5 +1,5 @@
 /**
- * @license Copyright (c) 2003-2022, CKSource Holding sp. z o.o. All rights reserved.
+ * @license Copyright (c) 2003-2023, CKSource Holding sp. z o.o. All rights reserved.
  * For licensing, see LICENSE.md or https://ckeditor.com/legal/ckeditor-oss-license
  */
 /**
@@ -18,51 +18,29 @@ import { CKEditorError } from '@ckeditor/ckeditor5-utils';
  * {@link ~MergeOperation#targetPosition}. All nodes from the merged element are moved to {@link ~MergeOperation#targetPosition}.
  *
  * The merged element is moved to the graveyard at {@link ~MergeOperation#graveyardPosition}.
- *
- * @extends module:engine/model/operation/operation~Operation
  */
 export default class MergeOperation extends Operation {
     /**
      * Creates a merge operation.
      *
-     * @param {module:engine/model/position~Position} sourcePosition Position inside the merged element. All nodes from that
+     * @param sourcePosition Position inside the merged element. All nodes from that
      * element after that position will be moved to {@link ~#targetPosition}.
-     * @param {Number} howMany Summary offset size of nodes which will be moved from the merged element to the new parent.
-     * @param {module:engine/model/position~Position} targetPosition Position which the nodes from the merged elements will be moved to.
-     * @param {module:engine/model/position~Position} graveyardPosition Position in graveyard to which the merged element will be moved.
-     * @param {Number|null} baseVersion Document {@link module:engine/model/document~Document#version} on which operation
+     * @param howMany Summary offset size of nodes which will be moved from the merged element to the new parent.
+     * @param targetPosition Position which the nodes from the merged elements will be moved to.
+     * @param graveyardPosition Position in graveyard to which the merged element will be moved.
+     * @param baseVersion Document {@link module:engine/model/document~Document#version} on which operation
      * can be applied or `null` if the operation operates on detached (non-document) tree.
      */
     constructor(sourcePosition, howMany, targetPosition, graveyardPosition, baseVersion) {
         super(baseVersion);
-        /**
-         * Position inside the merged element. All nodes from that element after that position will be moved to {@link ~#targetPosition}.
-         *
-         * @member {module:engine/model/position~Position} module:engine/model/operation/mergeoperation~MergeOperation#sourcePosition
-         */
         this.sourcePosition = sourcePosition.clone();
         // This is, and should always remain, the first position in its parent.
         this.sourcePosition.stickiness = 'toPrevious';
-        /**
-         * Summary offset size of nodes which will be moved from the merged element to the new parent.
-         *
-         * @member {Number} module:engine/model/operation/mergeoperation~MergeOperation#howMany
-         */
         this.howMany = howMany;
-        /**
-         * Position which the nodes from the merged elements will be moved to.
-         *
-         * @member {module:engine/model/position~Position} module:engine/model/operation/mergeoperation~MergeOperation#targetPosition
-         */
         this.targetPosition = targetPosition.clone();
         // Except of a rare scenario in `MergeOperation` x `MergeOperation` transformation,
         // this is, and should always remain, the last position in its parent.
         this.targetPosition.stickiness = 'toNext';
-        /**
-         * Position in graveyard to which the merged element will be moved.
-         *
-         * @member {module:engine/model/position~Position} module:engine/model/operation/mergeoperation~MergeOperation#graveyardPosition
-         */
         this.graveyardPosition = graveyardPosition.clone();
     }
     /**
@@ -73,9 +51,6 @@ export default class MergeOperation extends Operation {
     }
     /**
      * Position before the merged element (which will be deleted).
-     *
-     * @readonly
-     * @type {module:engine/model/position~Position}
      */
     get deletionPosition() {
         return new Position(this.sourcePosition.root, this.sourcePosition.path.slice(0, -1));
@@ -83,9 +58,6 @@ export default class MergeOperation extends Operation {
     /**
      * Artificial range that contains all the nodes from the merged element that will be moved to {@link ~MergeOperation#sourcePosition}.
      * The range starts at {@link ~MergeOperation#sourcePosition} and ends in the same parent, at `POSITIVE_INFINITY` offset.
-     *
-     * @readonly
-     * @type {module:engine/model/range~Range}
      */
     get movedRange() {
         const end = this.sourcePosition.getShiftedBy(Number.POSITIVE_INFINITY);
@@ -93,16 +65,12 @@ export default class MergeOperation extends Operation {
     }
     /**
      * Creates and returns an operation that has the same parameters as this operation.
-     *
-     * @returns {module:engine/model/operation/mergeoperation~MergeOperation} Clone of this operation.
      */
     clone() {
         return new MergeOperation(this.sourcePosition, this.howMany, this.targetPosition, this.graveyardPosition, this.baseVersion);
     }
     /**
      * See {@link module:engine/model/operation/operation~Operation#getReversed `Operation#getReversed()`}.
-     *
-     * @returns {module:engine/model/operation/splitoperation~SplitOperation}
      */
     getReversed() {
         // Positions in this method are transformed by this merge operation because the split operation bases on
@@ -175,9 +143,8 @@ export default class MergeOperation extends Operation {
     /**
      * Creates `MergeOperation` object from deserilized object, i.e. from parsed JSON string.
      *
-     * @param {Object} json Deserialized JSON object.
-     * @param {module:engine/model/document~Document} document Document on which this operation will be applied.
-     * @returns {module:engine/model/operation/mergeoperation~MergeOperation}
+     * @param json Deserialized JSON object.
+     * @param document Document on which this operation will be applied.
      */
     static fromJSON(json, document) {
         const sourcePosition = Position.fromJSON(json.sourcePosition, document);

package/src/model/operation/moveoperation.js

@@ -1,5 +1,5 @@
 /**
- * @license Copyright (c) 2003-2022, CKSource Holding sp. z o.o. All rights reserved.
+ * @license Copyright (c) 2003-2023, CKSource Holding sp. z o.o. All rights reserved.
  * For licensing, see LICENSE.md or https://ckeditor.com/legal/ckeditor-oss-license
  */
 /**
@@ -14,42 +14,24 @@ import { CKEditorError, compareArrays } from '@ckeditor/ckeditor5-utils';
 /**
  * Operation to move a range of {@link module:engine/model/item~Item model items}
  * to given {@link module:engine/model/position~Position target position}.
- *
- * @extends module:engine/model/operation/operation~Operation
  */
 export default class MoveOperation extends Operation {
     /**
      * Creates a move operation.
      *
-     * @param {module:engine/model/position~Position} sourcePosition
-     * Position before the first {@link module:engine/model/item~Item model item} to move.
-     * @param {Number} howMany Offset size of moved range. Moved range will start from `sourcePosition` and end at
+     * @param sourcePosition Position before the first {@link module:engine/model/item~Item model item} to move.
+     * @param howMany Offset size of moved range. Moved range will start from `sourcePosition` and end at
      * `sourcePosition` with offset shifted by `howMany`.
-     * @param {module:engine/model/position~Position} targetPosition Position at which moved nodes will be inserted.
-     * @param {Number|null} baseVersion Document {@link module:engine/model/document~Document#version} on which operation
+     * @param targetPosition Position at which moved nodes will be inserted.
+     * @param baseVersion Document {@link module:engine/model/document~Document#version} on which operation
      * can be applied or `null` if the operation operates on detached (non-document) tree.
      */
     constructor(sourcePosition, howMany, targetPosition, baseVersion) {
         super(baseVersion);
-        /**
-         * Position before the first {@link module:engine/model/item~Item model item} to move.
-         *
-         * @member {module:engine/model/position~Position} module:engine/model/operation/moveoperation~MoveOperation#sourcePosition
-         */
         this.sourcePosition = sourcePosition.clone();
         // `'toNext'` because `sourcePosition` is a bit like a start of the moved range.
         this.sourcePosition.stickiness = 'toNext';
-        /**
-         * Offset size of moved range.
-         *
-         * @member {Number} module:engine/model/operation/moveoperation~MoveOperation#howMany
-         */
         this.howMany = howMany;
-        /**
-         * Position at which moved nodes will be inserted.
-         *
-         * @member {module:engine/model/position~Position} module:engine/model/operation/moveoperation~MoveOperation#targetPosition
-         */
         this.targetPosition = targetPosition.clone();
         this.targetPosition.stickiness = 'toNone';
     }
@@ -67,8 +49,6 @@ export default class MoveOperation extends Operation {
     }
     /**
      * Creates and returns an operation that has the same parameters as this operation.
-     *
-     * @returns {module:engine/model/operation/moveoperation~MoveOperation} Clone of this operation.
      */
     clone() {
         return new MoveOperation(this.sourcePosition, this.howMany, this.targetPosition, this.baseVersion);
@@ -79,21 +59,19 @@ export default class MoveOperation extends Operation {
      * inside the same parent but {@link module:engine/model/operation/moveoperation~MoveOperation#targetPosition targetPosition}
      * is after {@link module:engine/model/operation/moveoperation~MoveOperation#sourcePosition sourcePosition}.
      *
-     *		 vv              vv
-     *		abcdefg ===> adefbcg
-     *		     ^          ^
-     *		     targetPos	movedRangeStart
-     *		     offset 6	offset 4
-     *
-     * @returns {module:engine/model/position~Position}
+     * ```
+     *  vv              vv
+     * abcdefg ===> adefbcg
+     *      ^          ^
+     *      targetPos  movedRangeStart
+     *      offset 6   offset 4
+     *```
      */
     getMovedRangeStart() {
         return this.targetPosition._getTransformedByDeletion(this.sourcePosition, this.howMany);
     }
     /**
      * See {@link module:engine/model/operation/operation~Operation#getReversed `Operation#getReversed()`}.
-     *
-     * @returns {module:engine/model/operation/moveoperation~MoveOperation}
      */
     getReversed() {
         const newTargetPosition = this.sourcePosition._getTransformedByInsertion(this.targetPosition, this.howMany);
@@ -166,9 +144,8 @@ export default class MoveOperation extends Operation {
     /**
      * Creates `MoveOperation` object from deserilized object, i.e. from parsed JSON string.
      *
-     * @param {Object} json Deserialized JSON object.
-     * @param {module:engine/model/document~Document} document Document on which this operation will be applied.
-     * @returns {module:engine/model/operation/moveoperation~MoveOperation}
+     * @param json Deserialized JSON object.
+     * @param document Document on which this operation will be applied.
      */
     static fromJSON(json, document) {
         const sourcePosition = Position.fromJSON(json.sourcePosition, document);

package/src/model/operation/nooperation.js

@@ -1,5 +1,5 @@
 /**
- * @license Copyright (c) 2003-2022, CKSource Holding sp. z o.o. All rights reserved.
+ * @license Copyright (c) 2003-2023, CKSource Holding sp. z o.o. All rights reserved.
  * For licensing, see LICENSE.md or https://ckeditor.com/legal/ckeditor-oss-license
  */
 /**
@@ -13,8 +13,6 @@ import Operation from './operation';
  * In most cases this operation is a result of transforming operations. When transformation returns
  * {@link module:engine/model/operation/nooperation~NoOperation} it means that changes done by the transformed operation
  * have already been applied.
- *
- * @extends module:engine/model/operation/operation~Operation
  */
 export default class NoOperation extends Operation {
     get type() {
@@ -22,16 +20,12 @@ export default class NoOperation extends Operation {
     }
     /**
      * Creates and returns an operation that has the same parameters as this operation.
-     *
-     * @returns {module:engine/model/operation/nooperation~NoOperation} Clone of this operation.
      */
     clone() {
         return new NoOperation(this.baseVersion);
     }
     /**
      * See {@link module:engine/model/operation/operation~Operation#getReversed `Operation#getReversed()`}.
-     *
-     * @returns {module:engine/model/operation/nooperation~NoOperation}
      */
     getReversed() {
         return new NoOperation(this.baseVersion + 1);

package/src/model/operation/operation.js

@@ -1,5 +1,5 @@
 /**
- * @license Copyright (c) 2003-2022, CKSource Holding sp. z o.o. All rights reserved.
+ * @license Copyright (c) 2003-2023, CKSource Holding sp. z o.o. All rights reserved.
  * For licensing, see LICENSE.md or https://ckeditor.com/legal/ckeditor-oss-license
  */
 /**
@@ -7,86 +7,31 @@
  */
 /**
  * Abstract base operation class.
- *
- * @abstract
  */
 export default class Operation {
     /**
      * Base operation constructor.
      *
-     * @param {Number|null} baseVersion Document {@link module:engine/model/document~Document#version} on which operation
+     * @param baseVersion Document {@link module:engine/model/document~Document#version} on which operation
      * can be applied or `null` if the operation operates on detached (non-document) tree.
      */
     constructor(baseVersion) {
-        /**
-         * {@link module:engine/model/document~Document#version} on which operation can be applied. If you try to
-         * {@link module:engine/model/model~Model#applyOperation apply} operation with different base version than the
-         * {@link module:engine/model/document~Document#version document version} the
-         * {@link module:utils/ckeditorerror~CKEditorError model-document-applyOperation-wrong-version} error is thrown.
-         *
-         * @member {Number|null}
-         */
         this.baseVersion = baseVersion;
-        /**
-         * Defines whether operation is executed on attached or detached {@link module:engine/model/item~Item items}.
-         *
-         * @readonly
-         * @member {Boolean} #isDocumentOperation
-         */
         this.isDocumentOperation = this.baseVersion !== null;
-        /**
-         * {@link module:engine/model/batch~Batch Batch} to which the operation is added or `null` if the operation is not
-         * added to any batch yet.
-         *
-         * @member {module:engine/model/batch~Batch|null} #batch
-         */
         this.batch = null;
-        /**
-         * Operation type.
-         *
-         * @readonly
-         * @member {String} #type
-         */
-        /**
-         * Creates and returns an operation that has the same parameters as this operation.
-         *
-         * @method #clone
-         * @returns {module:engine/model/operation/operation~Operation} Clone of this operation.
-         */
-        /**
-         * Creates and returns a reverse operation. Reverse operation when executed right after
-         * the original operation will bring back tree model state to the point before the original
-         * operation execution. In other words, it reverses changes done by the original operation.
-         *
-         * Keep in mind that tree model state may change since executing the original operation,
-         * so reverse operation will be "outdated". In that case you will need to transform it by
-         * all operations that were executed after the original operation.
-         *
-         * @method #getReversed
-         * @returns {module:engine/model/operation/operation~Operation} Reversed operation.
-         */
-        /**
-         * Executes the operation - modifications described by the operation properties will be applied to the model tree.
-         *
-         * @protected
-         * @method #_execute
-         */
     }
     /**
      * Checks whether the operation's parameters are correct and the operation can be correctly executed. Throws
      * an error if operation is not valid.
      *
      * @internal
-     * @protected
-     * @method #_validate
      */
     _validate() {
     }
     /**
      * Custom toJSON method to solve child-parent circular dependencies.
      *
-     * @method #toJSON
-     * @returns {Object} Clone of this object with the operation property replaced with string.
+     * @returns Clone of this object with the operation property replaced with string.
      */
     toJSON() {
         // This method creates only a shallow copy, all nested objects should be defined separately.
@@ -101,8 +46,6 @@ export default class Operation {
     }
     /**
      * Name of the operation class used for serialization.
-     *
-     * @type {String}
      */
     static get className() {
         return 'Operation';
@@ -110,9 +53,8 @@ export default class Operation {
     /**
      * Creates Operation object from deserilized object, i.e. from parsed JSON string.
      *
-     * @param {Object} json Deserialized JSON object.
-     * @param {module:engine/model/document~Document} doc Document on which this operation will be applied.
-     * @returns {module:engine/model/operation/operation~Operation}
+     * @param json Deserialized JSON object.
+     * @param doc Document on which this operation will be applied.
      */
     static fromJSON(json, document) {
         return new this(json.baseVersion);

package/src/model/operation/operationfactory.js

@@ -1,5 +1,5 @@
 /**
- * @license Copyright (c) 2003-2022, CKSource Holding sp. z o.o. All rights reserved.
+ * @license Copyright (c) 2003-2023, CKSource Holding sp. z o.o. All rights reserved.
  * For licensing, see LICENSE.md or https://ckeditor.com/legal/ckeditor-oss-license
  */
 /**
@@ -28,16 +28,13 @@ operations[SplitOperation.className] = SplitOperation;
 operations[MergeOperation.className] = MergeOperation;
 /**
  * A factory class for creating operations.
- *
- * @abstract
  */
 export default class OperationFactory {
     /**
      * Creates an operation instance from a JSON object (parsed JSON string).
      *
-     * @param {Object} json Deserialized JSON object.
-     * @param {module:engine/model/document~Document} document Document on which this operation will be applied.
-     * @returns {module:engine/model/operation/operation~Operation}
+     * @param json Deserialized JSON object.
+     * @param document Document on which this operation will be applied.
      */
     static fromJSON(json, document) {
         return operations[json.__className].fromJSON(json, document);

package/src/model/operation/renameoperation.js

@@ -1,5 +1,5 @@
 /**
- * @license Copyright (c) 2003-2022, CKSource Holding sp. z o.o. All rights reserved.
+ * @license Copyright (c) 2003-2023, CKSource Holding sp. z o.o. All rights reserved.
  * For licensing, see LICENSE.md or https://ckeditor.com/legal/ckeditor-oss-license
  */
 /**
@@ -13,40 +13,23 @@ import { CKEditorError } from '@ckeditor/ckeditor5-utils';
  * Operation to change element's name.
  *
  * Using this class you can change element's name.
- *
- * @extends module:engine/model/operation/operation~Operation
  */
 export default class RenameOperation extends Operation {
     /**
      * Creates an operation that changes element's name.
      *
-     * @param {module:engine/model/position~Position} position Position before an element to change.
-     * @param {String} oldName Current name of the element.
-     * @param {String} newName New name for the element.
-     * @param {Number|null} baseVersion Document {@link module:engine/model/document~Document#version} on which operation
+     * @param position Position before an element to change.
+     * @param oldName Current name of the element.
+     * @param newName New name for the element.
+     * @param baseVersion Document {@link module:engine/model/document~Document#version} on which operation
      * can be applied or `null` if the operation operates on detached (non-document) tree.
      */
     constructor(position, oldName, newName, baseVersion) {
         super(baseVersion);
-        /**
-         * Position before an element to change.
-         *
-         * @member {module:engine/model/position~Position} module:engine/model/operation/renameoperation~RenameOperation#position
-         */
         this.position = position;
         // This position sticks to the next node because it is a position before the node that we want to change.
         this.position.stickiness = 'toNext';
-        /**
-         * Current name of the element.
-         *
-         * @member {String} module:engine/model/operation/renameoperation~RenameOperation#oldName
-         */
         this.oldName = oldName;
-        /**
-         * New name for the element.
-         *
-         * @member {String} module:engine/model/operation/renameoperation~RenameOperation#newName
-         */
         this.newName = newName;
     }
     /**
@@ -58,15 +41,13 @@ export default class RenameOperation extends Operation {
     /**
      * Creates and returns an operation that has the same parameters as this operation.
      *
-     * @returns {module:engine/model/operation/renameoperation~RenameOperation} Clone of this operation.
+     * @returns Clone of this operation.
      */
     clone() {
         return new RenameOperation(this.position.clone(), this.oldName, this.newName, this.baseVersion);
     }
     /**
      * See {@link module:engine/model/operation/operation~Operation#getReversed `Operation#getReversed()`}.
-     *
-     * @returns {module:engine/model/operation/renameoperation~RenameOperation}
      */
     getReversed() {
         return new RenameOperation(this.position.clone(), this.newName, this.oldName, this.baseVersion + 1);
@@ -119,9 +100,8 @@ export default class RenameOperation extends Operation {
     /**
      * Creates `RenameOperation` object from deserialized object, i.e. from parsed JSON string.
      *
-     * @param {Object} json Deserialized JSON object.
-     * @param {module:engine/model/document~Document} document Document on which this operation will be applied.
-     * @returns {module:engine/model/operation/attributeoperation~AttributeOperation}
+     * @param json Deserialized JSON object.
+     * @param document Document on which this operation will be applied.
      */
     static fromJSON(json, document) {
         return new RenameOperation(Position.fromJSON(json.position, document), json.oldName, json.newName, json.baseVersion);