Rails4.2.8+CarrierWave+MiniMagick+Jcrop图片上传跳转异常求助

Hey，我来帮你一步步排查这个跳转异常的问题——这种情况大多和控制器逻辑、路由配置或者上传器的处理流程有关，咱们从几个核心点入手：

1. 先检查UsersController的update动作逻辑 #

这是最可能出问题的地方，你需要确认update方法里有没有正确判断「用户上传了新图片」，并触发跳转到crop页面的逻辑。

比如正确的逻辑应该是这样：

def update
  if @user.update(user_params)
    # 检测是否有新图片上传，且图片存在
    if @user.picture_changed? && @user.picture.present?
      redirect_to crop_user_path(@user) # 跳转到裁剪页
    else
      redirect_to @user, notice: '用户信息更新成功'
    end
  else
    # 如果验证失败，才会跳回edit页面，这里要排查是不是验证没通过
    render :edit
  end
end

另外别忘了检查user_params里有没有允许picture参数，不然图片根本不会被保存，自然不会触发裁剪跳转：

private
def user_params
  params.require(:user).permit(:name, :email, :picture) # 必须包含picture
end

2. 确认Crop相关路由是否正确配置 #

打开config/routes.rb，检查有没有为users添加crop的member路由：

resources :users do
  member do
    get 'crop'       # 显示裁剪页面的路由
    patch 'update_crop' # 保存裁剪结果的路由
  end
end

运行rake routes（Rails 4.x用这个命令）确认crop_user_path是存在的，路径是否正确。如果路由没定义，redirect时会报错，但你这里是跳转到edit，所以大概率不是路由问题，但还是要确认下。

3. 验证update动作是否真的检测到图片变化 #

可以在update方法里加个日志，看看系统有没有识别到新图片上传：

def update
  if @user.update(user_params)
    logger.debug "图片是否有变化？#{@user.picture_changed?}" # 加这行日志
    if @user.picture_changed? && @user.picture.present?
      redirect_to crop_user_path(@user)
    else
      redirect_to @user, notice: '用户信息更新成功'
    end
  else
    render :edit
  end
end

然后查看log/development.log里的输出，如果显示图片是否有变化？false，那可能是这几个原因：

• 表单没加multipart: true属性（上传文件必须加这个），比如你的edit/show页面的表单：

  <%= form_for @user, html: { multipart: true } do |f| %>
    <!-- 其他字段 -->
    <%= f.file_field :picture %>
    <%= f.submit '更新图片' %>
  <% end %>
  

• 图片字段名不是picture（比如你用了avatar？），导致picture_changed?判断失效。

4. 排查是否因验证失败跳回edit页面 #

如果@user.update(user_params)返回false，会直接render :edit，这种情况通常是用户模型的其他验证没通过（比如name不能为空、email格式错误）。你可以在edit页面加错误提示，或者看日志里的验证信息：

<!-- 在edit.html.erb里加错误提示 -->
<% if @user.errors.any? %>
  <div id="error_explanation">
    <h2><%= pluralize(@user.errors.count, "error") %> prohibited this user from being saved:</h2>
    <ul>
      <% @user.errors.full_messages.each do |msg| %>
        <li><%= msg %></li>
      <% end %>
    </ul>
  </div>
<% end %>

如果看到错误信息，就知道是验证的问题，调整验证规则或者确保提交的参数符合要求即可。

5. 检查CarrierWave上传器的处理逻辑 #

打开picture_uploader.rb，确认没有在上传时自动执行裁剪动作——因为我们需要先上传原图，跳转到裁剪页面让用户选择区域后再处理。比如不要在上传器里写这样的代码：

# 不要这样写，会自动裁剪，跳过手动裁剪流程
process :crop

def crop
  # ... 裁剪逻辑
end

正确的做法是，把裁剪逻辑放在update_crop动作里，当用户在crop页面提交裁剪参数后，再调用上传器的crop方法生成缩略图。

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