OpenCart 4.x模块编辑链接未带module_id致重复新增数据问题

问题现象 #

开发的Testimonials模块包含height和status字段，安装后编辑已有模块实例时，保存操作会在oc_module表新增一行数据，而非更新原有记录。排查发现模块列表页的编辑按钮URL未携带module_id，示例链接：

http://localhost/op4/admin/index.php?route=extension/testimonials/module/testimonials&user_token=2e7a4d8fad2e1c4339e9c01bc83d707a

问题分析 #

1. 核心原因：编辑页面未获取到module_id参数，导致save方法判断为新增操作，调用addModule而非editModule。
2. 代码隐患：控制器index方法中$module_info变量存在作用域问题，仅在isset($this->request->get['module_id'])分支内定义，外部访问会触发未定义错误。

解决方案 #

1. 修复模块列表页编辑按钮的module_id传递 #

确保模块实例列表页的编辑按钮生成包含module_id的完整URL：

• OpenCart默认会为Extensions > Modules页面的模块实例自动生成带module_id的编辑链接，需确认模块已正确安装且oc_module表中存在对应实例记录。
• 若自定义了模块列表展示逻辑，需手动拼接module_id参数，示例代码：

  $this->url->link('extension/testimonials/module/testimonials', 'user_token=' . $this->session->data['user_token'] . '&module_id=' . $module['module_id'])
  

2. 修复控制器变量作用域问题 #

修改extensions/testimonial/admin/controller/module/testimonials.php的index方法，提前初始化$module_info变量：

public function index(): void {
    $this->load->language('extension/testimonials/module/testimonials');

    $this->document->setTitle($this->language->get('heading_title'));
    $this->load->model('setting/setting');
    $data['breadcrumbs'] = [];
    // 提前初始化变量，避免作用域报错
    $module_info = [];

    // 面包屑生成代码保持不变...

    if (isset($this->request->get['module_id'])) {
        $this->load->model('setting/module');
        $module_info = $this->model_setting_module->getModule($this->request->get['module_id']);
    }

    // 后续字段赋值逻辑保持不变...
}

3. 确认保存逻辑正确性 #

当前save方法的分支判断逻辑是正确的，需确保其保留：

if (!$json) {
    $this->load->model('setting/module');
    if (!isset($this->request->get['module_id'])) {
        $this->model_setting_module->addModule('module_testimonials', $this->request->post);
    } else {
        $this->model_setting_module->editModule($this->request->get['module_id'], $this->request->post);
    }

    $json['success'] = $this->language->get('text_success');
}

验证步骤 #

1. 进入Extensions > Modules页面，点击Testimonials模块实例的编辑按钮，确认URL中包含module_id参数。
2. 修改模块参数后保存，检查oc_module表，确认原有记录被更新而非新增。

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