Rails 5中update方法无法更新记录，请求技术排查

让我们一步步定位你遇到的这个问题：明明日志显示COMMIT成功，但Transaction记录的status就是没更新。

1. 先搞清楚到底为什么没保存成功 #

你的控制器里直接调用了@transaction.update_attributes(transactions_params)，但这个方法在保存失败时只会返回false，不会抛出异常，所以你根本不知道它是不是真的成功了。先修改控制器代码，加上错误日志和反馈：

def update
  if @transaction.update(transactions_params) # update是update_attributes的别名，用法一致
    redirect_to admins_transactions_path, notice: 'Transaction approved successfully!'
  else
    # 把错误信息打印到日志，方便排查
    Rails.logger.error "Transaction update failed: #{@transaction.errors.full_messages.join(', ')}"
    redirect_to admins_transactions_path, alert: 'Failed to approve transaction. Check logs for details.'
  end
end

重新测试后，查看rails的日志（log/development.log），就能看到具体的失败原因了——这是最关键的第一步。

2. 检查模型的验证规则 #

很可能是Transaction模型里的验证阻止了更新。比如如果模型里有类似validates :amount, presence: true的规则，当你只更新status时，其他字段（比如amount）如果为空或者不符合验证，保存就会失败。

你可以直接在Rails控制台里测试，快速验证：

# 启动控制台：rails c
tx = Transaction.find(1)
tx.update(status: 'approved') # 执行更新操作
puts tx.errors.full_messages # 打印所有错误信息

如果这里有错误，就对应调整验证规则——比如给验证加上on: :create，只在创建时验证，或者允许更新时只修改status。

3. 排查单表继承(STI)的影响 #

你的Transaction表有type字段，这是Rails单表继承（STI）的默认字段。如果这条记录的type值不是Transaction，而是某个子类（比如WithdrawalTransaction），那用Transaction.find_by(id: params[:id])会返回子类的实例，子类可能有额外的验证或回调，导致更新失败。

你可以在控制台里查看这条记录的实际类型：

Transaction.find(1).class.name

如果是子类，建议用对应的类来查找记录，比如：

# 在get_transaction_id方法里修改
@transaction = params[:type].constantize.find_by(id: params[:id])
# 或者先判断类型再查找
tx = Transaction.find_by(id: params[:id])
@transaction = tx.becomes(tx.type.constantize)

4. 检查模型的回调方法 #

如果Transaction模型里有before_update、around_update这类回调，有可能在保存前修改了status值，或者直接阻止了保存。比如：

before_update :reset_status_if_invalid

def reset_status_if_invalid
  self.status = 'pending' unless some_condition_is_met
end

这种情况下，你设置的approved会被回调改回pending，导致看起来没更新。可以暂时注释掉所有回调，测试是否能成功更新，来排查是不是回调的问题。

5. 确认数据库层面的问题 #

虽然概率较低，但也可以检查：

• 数据库的status字段是不是被设置为只读（比如数据库触发器、约束）
• 数据库连接是否正常，有没有出现提交后回滚的情况（可以查看数据库的日志）

总结 #

最可能的原因是验证失败或者回调覆盖了status值，先从第一步添加错误日志开始，就能快速定位问题了。

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