OpenCart v1.5.6.4适配PHP7时遇Function name must be a string错误的解决方法

这个问题我在帮客户迁移OpenCart老版本到PHP7环境时遇到过，核心原因是PHP7对变量函数的解析优先级和PHP5存在差异，导致原来的写法触发了语法解析歧义。

错误原因分析 #

你箭头指向的代码行$listener[0]->$listener[1]($event);在PHP5中，解析器会默认先识别$listener[1]作为方法名，然后调用$listener[0]对象的对应方法。但PHP7调整了变量解析规则，它会先尝试解析$listener[0]->$listener，再去取这个结果的[1]索引——显然$listener[0]->$listener不是数组，自然就会抛出"Function name must be a string"的致命错误。

两种可行的修复方案 #

方案1：用大括号明确方法名的解析顺序

直接给方法名变量加上大括号，强制PHP先解析$listener[1]作为方法名：

public function notify(sfEvent $event){
 foreach ($this->getListeners($event->getName()) as $listener){
     $listener[0]->{$listener[1]}($event); // 这里加了大括号
 }
 return $event;
}

这是最简洁的修复方式，大括号明确告诉PHP：$listener[1]是要调用的方法名称。

方案2：提前将方法名赋值给临时变量

如果觉得大括号的写法不够直观，可以先把方法名存到一个临时变量里，再调用：

public function notify(sfEvent $event){
 foreach ($this->getListeners($event->getName()) as $listener){
     $methodName = $listener[1];
     $listener[0]->$methodName($event);
 }
 return $event;
}

这种方式可读性更强，尤其是在复杂的变量函数调用场景下，能避免解析歧义。

额外说明 #

OpenCart v1.5.6.4是比较老的版本，除了这个问题，你可能还会碰到其他PHP7兼容性问题（比如mysql_*函数被移除、__autoload被弃用等），建议修复完这个问题后，全面检查代码中的PHP7不兼容语法。

内容的提问来源于stack exchange，提问作者vtdev