Activate no-self-assign rule of eslint

sao/eslint.config.mjs

@@ -21,7 +21,6 @@ export default [
             },
         },
         rules: {
-            "no-self-assign": "off",
             "no-unexpected-multiline": "off",
             "no-unused-vars": "off",
             "no-useless-escape": "off",

sao/src/screen.js

@@ -1350,7 +1350,8 @@
             }
             return jQuery.when.apply(jQuery, deferreds).then(
                 () => this.set_tree_state().then(() => {
-                    this.current_record = this.current_record;
+                    var record = this.current_record
+                    this.current_record = record;
                     // set_cursor must be called after set_tree_state because
                     // set_tree_state redraws the tree
                     if (set_cursor) {

sao/src/tab.js

@@ -1660,12 +1660,7 @@
             this.board.reload();
         },
         record_message: function() {
-            var i, len;
-            var action;
-
-            len = this.board.actions.length;
-            for (i = 0, len=len; i < len; i++) {
-                action = this.board.actions[i];
+            for (const action of this.board.actions) {
                 action.update_domain(this.board.actions);
             }
         },