Shopware 5自动代码审核不支持finally块的问题咨询

我之前也碰到过Shopware自动审核的类似奇怪规则，咱们一步步来解决这个问题：

• 先确认审核的核心关注点
  Shopware 5.3.7本身要求的最低PHP版本就是5.5.38，try/catch/finally是这个版本完全支持的合法语法，所以驳回大概率不是语法问题，而是自动审核的代码规范检测误判。你可以先检查finally块里的代码：

  • 有没有不符合Shopware编码规范的内容，比如变量命名不遵循驼峰/下划线规则、未初始化的变量调用
  • 有没有在finally里执行了Shopware禁止的操作，比如直接操作核心数据表而不使用官方ORM，或者调用了未授权的核心API

• 准备申诉的关键证据
  向Shopware申诉时，一定要把这些信息说清楚：

  1. 你的插件在本地Shopware 5.3.7环境运行完全正常，没有任何报错
  2. 包含finally块的代码在PHP 5.5.38及以上版本中语法完全合法
  3. 附上你的finally代码片段，说明它的作用（比如资源清理、连接关闭这类必要操作），举个例子：

     try {
         $dbConnection = Shopware()->Db()->getConnection();
         $dbConnection->executeUpdate($updateSql);
     } catch (\PDOException $e) {
         Shopware()->Container()->get('logger')->error('数据库操作失败：' . $e->getMessage());
         throw new \RuntimeException('操作失败，请稍后重试');
     } finally {
         // 确保数据库连接正常释放
         if (isset($dbConnection)) {
             $dbConnection->close();
         }
     }
     

  强调finally是PHP标准语法，用于必要的资源清理，属于代码最佳实践，不应该被驳回。

• 排查隐性风险点
  有时候自动审核会误判一些潜在风险，你可以自查：

  • finally块里有没有抛出未捕获的异常？虽然语法合法，但可能触发审核的风险检测
  • 有没有在finally里修改了Shopware的核心全局变量？这类操作可能被审核系统标记为风险行为

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