Ace upgrade from 1.2.9 -> 1.3.1 breaks c9 about pylon

[nightwing]

just replacing actiontracker with normal ace undo manager should fix the issue

diff --git a/plugins-client/ext.code/code.js b/plugins-client/ext.code/code.js
@@ -25,6 +25,7 @@ var settings = require("ext/settings/settings");
 var themes = require("ext/themes/themes");
 var markupSettings = require("text!ext/code/settings.xml");
 var editors = require("ext/editors/editors");
+var UndoManager = require("ace/undomanager").UndoManager;
 
 require("ace/config").setDefaultValue("renderer", "vScrollBarAlwaysVisible", true);
 
@@ -217,7 +218,7 @@ module.exports = ext.register("ext/code/code", {
             doc.acedoc = doc.acesession.getDocument();
             doc.acesession.c9doc = doc;
 
-            doc.acesession.setUndoManager(actiontracker);
+            doc.acesession.setUndoManager(new UndoManager());
 
             if (doc.isInited && doc.state)
                  _self.setState(doc, doc.state);

also in the current version scrolling into view is broken, for that to work change commands.js to

diff --git a/plugins-client/ext.commands/commands.js b/plugins-client/ext.commands/commands.js
@@ -108,11 +108,12 @@ module.exports = ext.register("ext/commands/commands", apf.extend(
                 args: args
             };
 
-            if (cloud9config.debug) {
-                var retvalue = this._emit("exec", execEvent);
-            } else try {
+            if (editor && editor.$nativeCommands) {
+                var retvalue = editor.$nativeCommands.exec(command, editor, args);
+            }
+            else {
                 var retvalue = this._emit("exec", execEvent);
-            } catch (e) {}
+            }
 
 
             if (retvalue !== false && e) {

[exsilium]

@nightwing - Thanks so much for this, I actually started looking into the apf.actiontracker to see how to fix the missing/new functions. Much appreciated! 🍻 🙇