製作ステータスと注文ステータスの変更を連動させたい

Question

前提・実現したいこと

注文ステータスを入金確認に変更して更新した際、
製作ステータスを製作待ちに連動して変更したい

発生している問題・エラーメッセージ

連動して変更されず、そのまま更新される(エラー等はなし)

該当のソースコード

     <tr>
        <th>注文ステータス</th>
        <td>
        <% if @order.order_status == 1 %>
          <%= form.select :production_status == 1, class: "production_status-select" %>
        <% else %>
          <%= form_with model: @order, url: admin_order_path, method: :patch, local: true do |f| %>
            <%= f.select :order_status, Order.order_statuses.keys, class: "order_status" %>
            <%= f.submit "更新" , class: "btn btn-outline-success" %>
          <% end %>
        <% end %>
        </td>
      </tr>

省略

製作ステータス
 <tbody>
      <% @ordered_products.each do |ordered_product| %>
        <tr>
          <td><%= ordered_product.product.name %></td>
          <td><%= ordered_product.product.add_tax_price %></td>
          <td><%= ordered_product.quantity %></td>
          <td><%= ordered_product.product.add_tax_price * ordered_product.quantity %></td>
            <%= form_with model: ordered_product, url: admin_order_detail_path, method: :patch, local: true do |f| %>
              <td><%= f.select :production_status, OrderedProduct.production_statuses.keys, class: "production_status-select" %></td>
              <td><%= f.submit "更新", class: "btn btn-outline-success" %></td>
            <% end %>
        </tr>
      <% end %>
    </tbody>

  enum order_status: {
     "入金待ち":0, "入金確認":1, "製作中":2, "発送準備中":3, "発送済み":4
  }

  enum production_status: {
     製作不可:0,製作待ち:1,製作中:2,製作終了:3
  }

class Admin::OrdersController < ApplicationController

  # before_action :authenticate_admin

  def show
    @customer = Customer.find(params[:id])
    @order = Order.find(params[:id])
    @product = Product.find(params[:id])
    @ordered_products = @order.ordered_products
  end

  def update
    @order = Order.find(params[:id])
    @order.update(order_params)
    redirect_to request.referer
  end


  private

  def order_params
    params.require(:order).permit(:order_status)
  end

end

試したこと

①if文をviewに追加

 <% if @order.order_status == 1%>
          <%= @ordered_products.production_status == 1 %>
        <% else %>
          <%= form_with model: @order, url: admin_order_path, method: :patch, local: true do |f| %>
            <%= f.select :order_status, Order.order_statuses.keys, class: "order_status" %>
            <%= f.submit "更新" , class: "btn btn-outline-success" %>
          <% end %>
        <% end %>
```

結果：変化なし

②updateにif文追加して条件分岐
更新をsaveした際に、製作ステータスに変更する
→undifined method "production_status"
そもそも全ての更新で変わってしまうから不向きかも

````
  def update
    @order = Order.find(params[:id])
    @ordered_products = @order.ordered_products
     if @order.update(order_params)
       @ordered_products.order.production_status == 1
       redirect_to request.referer
     else
       redirect_to request.referer
     end
  end

```

上記試しましたが解消しませんでした。
ご確認をよろしくお願い致します。

Answer 1

「②updateにif文追加して条件分岐」 でよいのでは？
Order と OderProductの関連部分が載っていないので、仮に1:1だとすると

if @order.update(order_params)
  @ordered.order_product.update(status: 1) if @order.status == 1
  redirect_to request.referer
else

@order.status が 0 → 1 になったときにのみ、か
すると if の前に
status_change_to_1 = @order.status_was == 0 and @order.status == 1
とでもしておいて
@ordered.order_product.update(status: 1) if status_change_to_1
ですね。
1:多なら
@ordered.order_products.update_all(status: 1)

ただし、2つ

1. enum してるから、1 ではだめで "入金確認" などにする必要あるかも
2. Orderが成功しても OrderProduct 変更が成功しなかった場合は全体を取りやめないと駄目でしょうから、Transactionをかけた方がよいです。（これは別の話になるので、質問するなら別立てで）

Comments

[nao0725]

ありがとうございます！1：多なので@ordered.order_products.update_all(status: 1)
を使用しました。