Refactor assignExpression -> assignExpressionAtPath

src/ast/nodes/Identifier.js

@@ -25,8 +25,8 @@ export default class Identifier extends Node {
 	}
 
 	bindAssignmentAtPath ( path, expression ) {
-		if ( this.variable && path.length === 0 ) {
-			this.variable.assignExpression( expression );
+		if ( this.variable ) {
+			this.variable.assignExpressionAtPath( path, expression );
 		}
 	}
 

src/ast/nodes/shared/FunctionNode.js

@@ -7,9 +7,9 @@ export default class FunctionNode extends Node {
 		const thisVariable = this.scope.findVariable( 'this' );
 
 		if ( withNew ) {
-			thisVariable.assignExpression( UNKNOWN_OBJECT_LITERAL );
+			thisVariable.assignExpressionAtPath( [], UNKNOWN_OBJECT_LITERAL );
 		} else {
-			thisVariable.assignExpression( UNKNOWN_ASSIGNMENT );
+			thisVariable.assignExpressionAtPath( [], UNKNOWN_ASSIGNMENT );
 		}
 	}
 

src/ast/scopes/Scope.js

@@ -25,7 +25,7 @@ export default class Scope {
 		if ( this.variables[ name ] ) {
 			const variable = this.variables[ name ];
 			variable.addDeclaration( identifier );
-			options.init && variable.assignExpression( options.init );
+			options.init && variable.assignExpressionAtPath( [], options.init );
 		} else {
 			this.variables[ name ] = new LocalVariable( identifier.name, identifier, options.init || UNDEFINED_ASSIGNMENT );
 		}

src/ast/variables/GlobalVariable.js

@@ -14,8 +14,6 @@ export default class GlobalVariable extends Variable {
 		if ( reference.isReassignment ) this.isReassigned = true;
 	}
 
-	assignExpression () {}
-
 	hasEffectsWhenCalled () {
 		return !pureFunctions[ this.name ];
 	}

src/ast/variables/LocalVariable.js

@@ -21,12 +21,12 @@ export default class LocalVariable extends Variable {
 		Array.from( this.assignedExpressions ).forEach( expression => expression.bindCall( callOptions ) );
 	}
 
-	addReference () {}
-
-	assignExpression ( expression ) {
-		this.assignedExpressions.add( expression );
-		this.isReassigned = true;
-		Array.from( this.calls ).forEach( callOptions => expression.bindCall( callOptions ) );
+	assignExpressionAtPath ( path, expression ) {
+		if ( path.length === 0 ) {
+			this.assignedExpressions.add( expression );
+			this.isReassigned = true;
+			Array.from( this.calls ).forEach( callOptions => expression.bindCall( callOptions ) );
+		}
 	}
 
 	getName ( es ) {

src/ast/variables/NamespaceVariable.js

@@ -19,8 +19,6 @@ export default class NamespaceVariable extends Variable {
 		this.name = node.name;
 	}
 
-	assignExpression () {}
-
 	includeVariable () {
 		const hasBeenIncluded = super.includeVariable();
 		if ( hasBeenIncluded ) {

src/ast/variables/Variable.js

@@ -5,6 +5,10 @@ export default class Variable {
 
 	addCall () {}
 
+	addReference () {}
+
+	assignExpressionAtPath () {}
+
 	getName () {
 		return this.name;
 	}