Fix fatal error on dynamic method names in assignments

src/Tests/Tests/epv_test_validate_php_functions.php

@@ -399,11 +399,8 @@ private function validateMethodCalls(Node $node) {
         }
 		else if (isset($node->expr)
 			&& $node->expr instanceof Node\Expr\MethodCall
-			&& !($node->expr->name instanceof Variable
-				|| $node->expr->name instanceof PropertyFetch
-				|| $node->expr->name instanceof Concat
-				|| $node->expr->name instanceof Node\Scalar\Encapsed
-				|| $node->expr->name instanceof Node\Expr\Ternary
+			&& ($node->expr->name instanceof Node\Identifier
+				|| $node->expr->name instanceof Node\Name
 			)
 		)
 		{

tests/epv_test_validate_php_functions_test.php

@@ -171,6 +171,18 @@ public function test_usage_variable() {
 		$tester->validateFile($file);
 	}
 
+	public function test_usage_dynamic_method_name_in_assignment() {
+		$output = $this->getOutputMock();
+		$output->expects(self::exactly(0))
+			->method('addMessage')
+		;
+
+		$file = $this->getLoader()->loadFile('tests/testFiles/dynamic_method_call.php');
+
+		$tester = new epv_test_validate_php_functions(false, $output, '/a/b/', 'epv/test', false, '/a/');
+		$tester->validateFile($file);
+	}
+
 	private function getLoader()
 	{
 		return new FileLoader(new Output(), false, '.', '.');

tests/testFiles/dynamic_method_call.php

@@ -0,0 +1,17 @@
+<?php
+/**
+ * @package phpBB Extension - Test
+ */
+
+namespace epv\test;
+
+class dynamic_method_call
+{
+	public function run($methods, $step)
+	{
+		$removed = $this->store->{$methods[$step]}(self::DELETE_STEP);
+		$other = $this->store->{$methods['a']}();
+		$named = $this->store->delete_step();
+		return $removed;
+	}
+}