Only check for vars if there are vars in the trigger event

Author: iMattPro

src/Events/php_exporter.php

@@ -119,10 +119,13 @@ public function crawl_php_file($file)
 					$event_line = $i;
 					$this->set_current_event($this->get_event_name($event_line, false), $event_line);
 
-					// Find variables of the event
-					$arguments = $this->get_vars_from_array();
-					$doc_vars  = $this->get_vars_from_docblock();
-					$this->validate_vars_docblock_array($arguments, $doc_vars);
+					// Find variables of the event if it has them
+					if (strpos($this->file_lines[$event_line], 'compact('))
+					{
+						$arguments = $this->get_vars_from_array();
+						$doc_vars  = $this->get_vars_from_docblock();
+						$this->validate_vars_docblock_array($arguments, $doc_vars);
+					}
 				}
 				else
 				{

tests/php_exporter_test.php

@@ -52,15 +52,16 @@ public function extension_data()
 	/**
 	 * @dataProvider extension_data
 	 */
-	public function test_event_name($line, $content, $is_dispatch, $expected_name, $expected_vars, $expected_errors = [])
+	public function test_event_name($line, $file, $is_dispatch, $expected_name, $expected_vars, $expected_errors = [])
 	{
+		$content = file_get_contents($file);
 		$output   = new Output();
 		$exporter = new php_exporter($output, '');
-		$exporter->set_content(explode("\n", file_get_contents($content)));
+		$exporter->set_content(explode("\n", $content));
 
 		$name = $exporter->get_event_name($line, $is_dispatch);
 		$exporter->set_current_event($name, $line);
-		$vars = $expected_vars ? $exporter->get_vars_from_array(false) : [];
+		$vars = strpos($content, 'compact(') ? $exporter->get_vars_from_array(false) : [];
 
 		self::assertEquals($expected_name, $name);
 		self::assertEquals($expected_vars, $vars);