Added conversion for setter args - fixed defs file again.

Author: yGuy

lib/migration-tool.js

@@ -169,7 +169,7 @@ AbstractMemberTransformation.prototype.detect = function (node, parent, stack) {
     var type = object.scopeInfo.type;
 
     type = getType(type, transformationTargets, oldMemberName, stack);
-
+    
     if (type.type === "?") {
       return null;
     }
@@ -239,6 +239,65 @@ function isReservedMember(keyName) {
       || keyName === "$static" || keyName === "$clinit";
 }
 
+/**
+ * @param {Object} config
+ * @constructor
+ * @extends AbstractTransformation
+ */
+function SetterArgTransformation(config) {
+  AbstractTransformation.call(this, config["memberRenamings"]);
+}
+SetterArgTransformation.prototype = Object.create(AbstractTransformation.prototype);
+
+/**
+ * @param {Node} node
+ * @param {Node} parent
+ * @return {AbstractResolver|null}
+ */
+SetterArgTransformation.prototype.detect = function detect(node, parent, stack) {
+  if (node.type === Syntax.Property && node.kind === "init" && stack[1].type === Syntax.ObjectExpression && stack[2].type === Syntax.NewExpression){
+    if (stack[2].scopeInfo && stack[2].scopeInfo.type){
+      var key = node.key.type === Syntax.Literal ? node.key.value : node.key.name;
+      var object = stack[2];
+      var type = object.scopeInfo.type;
+
+      var safe = type.type !== "?";
+      type = getType(type, this.transformationTargets, key, stack);
+      if (type){
+        let renames = this.transformationTargets[type.type];
+        if (renames){
+          if (renames[key]){
+            if (!safe){
+              return new ConfidentCommentResolver(this, "'" + memberToString(type.type, key) + "' has been renamed to '" + renames[key] + "'", true);
+            } else {
+              return new AutomaticResolver(this);
+            }
+          }
+        }
+      }
+    }
+  }
+};
+/**
+ * @param {Node} node
+ * @param {Node|null} parent
+ */
+SetterArgTransformation.prototype.perform = function perform(node, parent, stack) {
+  var key = node.key.type === Syntax.Literal ? node.key.value : node.key.name;
+  var object = stack[2];
+  var type = object.scopeInfo.type;
+
+  var newName = this.transformationTargets[getType(type, this.transformationTargets, key, stack).type][key];
+
+  if (node.key.type === Syntax.Literal) {
+    node.key.value = newName;
+  } else {
+    node.key.name = newName;
+  }
+};
+
+
+
 /**
  * @param {Object} config
  * @constructor
@@ -799,7 +858,7 @@ function ConfidentCommentResolver(transformation, comment, keepOldLine) {
 }
 ConfidentCommentResolver.prototype = Object.create(AbstractResolver.prototype);
 
-ConfidentCommentResolver.prototype.resolve = function (node, parent) {
+ConfidentCommentResolver.prototype.resolve = function (node, parent, stack) {
   commentsCount++;
 
   var rawComments = [].concat(
@@ -814,7 +873,7 @@ ConfidentCommentResolver.prototype.resolve = function (node, parent) {
 
   var comments = createCommentsAst(rawComments);
 
-  node = AbstractResolver.prototype.resolve.call(this, node, parent);
+  node = AbstractResolver.prototype.resolve.call(this, node, parent, stack);
 
   attachComments(node, comments);
 };
@@ -1041,7 +1100,8 @@ module.exports = function (files, finishedCallback, convertClassDeclarations) {
     new MemberRemovedTransformation(config),
     new TypeRemovedTransformation(config),
     new ChangeNamespaceTransformation(config),
-    new MemberRenameTransformation(config)
+    new MemberRenameTransformation(config),
+    new SetterArgTransformation(config),  
   ];
 
 

lib/transformer.js

@@ -110,9 +110,9 @@ function AbstractResolver(transformation) {
  * @param {Node} parent
  * @return {Node|undefined}
  */
-AbstractResolver.prototype.resolve = function (node, parent) {
+AbstractResolver.prototype.resolve = function (node, parent, stack) {
   return this.transformations.reduce(function (node, transformation) {
-    return transformation.perform(node, parent) || node;
+    return transformation.perform(node, parent, stack) || node;
   }, node);
 };