Changeset: Deprecate oldLen() and newLen() functions

ether · Nov 5, 2021 · 65c56ea · 65c56ea
1 parent 1a7106a
commit 65c56ea
Show file tree

Hide file tree

Showing 4 changed files with 18 additions and 10 deletions.
diff --git a/src/node/handler/PadMessageHandler.js b/src/node/handler/PadMessageHandler.js
@@ -651,11 +651,12 @@ const handleUserChanges = async (socket, message) => {
 
     const prevText = pad.text();
 
-    if (Changeset.oldLen(changeset) !== prevText.length) {
+    if (Changeset.unpack(changeset).oldLen !== prevText.length) {
       socket.json.send({disconnect: 'badChangeset'});
       stats.meter('failedChangesets').mark();
-      throw new Error(`Can't apply USER_CHANGES ${changeset} with oldLen ` +
-                      `${Changeset.oldLen(changeset)} to document of length ${prevText.length}`);
+      throw new Error(
+          `Can't apply USER_CHANGES ${changeset} with oldLen ` +
+          `${Changeset.unpack(changeset).oldLen} to document of length ${prevText.length}`);
     }
 
     try {

diff --git a/src/static/js/Changeset.js b/src/static/js/Changeset.js
@@ -278,18 +278,26 @@ class Changeset {
 /**
  * Returns the required length of the text before changeset can be applied.
  *
+ * @deprecated Use `Changeset.unpack(cs).oldLen` instead.
  * @param {string} cs - String representation of the Changeset
  * @returns {number} oldLen property
  */
-exports.oldLen = (cs) => Changeset.unpack(cs).oldLen;
+exports.oldLen = (cs) => {
+  warnDeprecated('Changeset.oldLen(cs) is deprecated; use Changeset.unpack(cs).oldLen instead');
+  return Changeset.unpack(cs).oldLen;
+};
 
 /**
  * Returns the length of the text after changeset is applied.
  *
+ * @deprecated Use `Changeset.unpack(cs).newLen` instead.
  * @param {string} cs - String representation of the Changeset
  * @returns {number} newLen property
  */
-exports.newLen = (cs) => Changeset.unpack(cs).newLen;
+exports.newLen = (cs) => {
+  warnDeprecated('Changeset.newLen(cs) is deprecated; use Changeset.unpack(cs).newLen instead');
+  return Changeset.unpack(cs).newLen;
+};
 
 /**
  * Parses a string of serialized changeset operations.

diff --git a/src/static/js/ace2_inner.js b/src/static/js/ace2_inner.js
@@ -1474,11 +1474,10 @@ function Ace2Inner(editorInfo, cssManagers) {
   };
 
   const doRepApplyChangeset = (changes, insertsAfterSelection) => {
-    Changeset.unpack(changes).validate();
+    const cs = Changeset.unpack(changes).validate();
 
-    if (Changeset.oldLen(changes) !== rep.alltext.length) {
-      const errMsg = `${Changeset.oldLen(changes)}/${rep.alltext.length}`;
-      throw new Error(`doRepApplyChangeset length mismatch: ${errMsg}`);
+    if (cs.oldLen !== rep.alltext.length) {
+      throw new Error(`doRepApplyChangeset length mismatch: ${cs.oldLen}/${rep.alltext.length}`);
     }
 
     const editEvent = currentCallStack.editEvent;

diff --git a/src/static/js/changesettracker.js b/src/static/js/changesettracker.js
@@ -183,7 +183,7 @@ const makeChangesetTracker = (scheduler, apool, aceCallbacksProvider) => {
       let cs = null;
       if (toSubmit) {
         submittedChangeset = toSubmit;
-        userChangeset = Changeset.identity(Changeset.newLen(toSubmit));
+        userChangeset = Changeset.identity(Changeset.unpack(toSubmit).newLen);
 
         cs = toSubmit;
       }