質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.51%

  • JavaScript

    16490questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • Ruby on Rails

    7299questions

    Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

  • Ruby on Rails 4

    2440questions

    Ruby on Rails4はRubyによって書かれたオープンソースのウェブフレームワークです。 Ruby on Railsは「設定より規約」の原則に従っており、効率的に作業を行うために再開発を行う必要をなくしてくれます。

フラグの「一括」更新

受付中

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 301

innjera

score 100

実現したいこと

以下画面の承諾、拒否のフラグをクリックで選択し、『申し込みのフラグを更新する』を押すと、フラグが一括更新される機能を実装したのですが、『申し込みのフラグを更新する』をクリックしても、controller記載のflash.notice = 'レッスン申込者のフラグを更新しました。'は表示されるのですが、フラグは更新されません。

実装にあたりRails : フォームオブジェクトJavaScriptを利用しています。

誤りをご指摘頂きたく宜しくお願い致します。

イメージ説明

申し込みのためのフォームオブジェクト

class User::EntriesForm
  include ActiveModel::Model

  attr_accessor :product, :approved, :not_approved, :canceled, :not_canceled

  def initialize(product)
    @product = product
  end

  def assign_attributes(params)
    @approved = params[:approved]
    @not_approved = params[:not_approved]
    @canceled = params[:canceled]
    @not_canceled = params[:not_canceled]
  end

  def save
    approved_entry_ids = @approved.split(':').map(&:to_i)
    not_approved_entry_ids = @not_approved.split(':').map(&:to_i)
    canceled_entry_ids = @canceled.split(':').map(&:to_i)
    not_canceled_entry_ids = @not_canceled.split(':').map(&:to_i)

    ActiveRecord::Base.transaction do
      @product.entries.where(id: approved_entry_ids).
        update_all(approved: true)
      @product.entries.where(id: not_approved_entry_ids).
        update_all(approved: false)
      @product.entries.where(id: canceled_entry_ids).
        update_all(canceled: true)
      @product.entries.where(id: not_canceled_entry_ids).
        update_all(canceled: false)
    end
  end
end

テンプレート

<table class="table">
  <thead>
    <tr>
      <th>#</th>
      <th>ユーザーネーム</th>
      <th>承諾</th>
      <th>拒否</th>
    </tr>
  </thead>
  <tbody>
    <tr>
      <th scope="row">1</th>
      <td>erika</td>
      <td>
         <input type='checkbox', class='approved', data-entry-id='1'>
     </td>
     <td>
        <input type='checkbox', class='canceled', data-entry-id='1'>
    </td>
    </tr>
  </tbody>
  <tbody>
    <tr>
      <th scope="row">2</th>
      <td>chimaki</td>
      <td>
         <input type='checkbox', class='approved', data-entry-id='2'>
     </td>
     <td>
        <input type='checkbox', class='canceled', data-entry-id='2'>
    </td>
    </tr>
  </tbody>
  <tbody>
    <tr>
      <th scope="row">3</th>
      <td>sotako</td>
      <td>
         <input type='checkbox', class='approved', data-entry-id='3'>
     </td>
     <td>
        <input type='checkbox', class='canceled', data-entry-id='3'>
    </td>
    </tr>
  </tbody>
  <tbody>
    <tr>
      <th scope="row">4</th>
      <td>ruffy</td>
      <td>
         <input type='checkbox', class='approved', data-entry-id='4'>
     </td>
     <td>
        <input type='checkbox', class='canceled', data-entry-id='4'>
    </td>
    </tr>
  </tbody>
</table>
<div class = 'button-wrapper'>
  <form class="new_form" id="new_form" action="/user/products/1/entries" accept-charset="UTF-8" method="post"><input name="utf8" type="hidden" value="&#x2713;" /><input type="hidden" name="_method" value="patch" /><input type="hidden" name="authenticity_token" value="se6s0O8G5KRrknZrFG79P8IIkOH7BrU79SYHeLSkcvh2VGyiLCpMNuzDl8GfBTLi6RR+egO/yWfElgG15xU37A==" />
  <input type="hidden" name="form[approved]" id="form_approved" />
  <input type="hidden" name="form[not_approved]" id="form_not_approved" />
  <input type="hidden" name="form[canceled]" id="form_canceled" />
  <input type="hidden" name="form[not_canceled]" id="form_not_canceled" />
  <button name="button" type="button" id="update-entries-button" class="btn btn-outline-primary">申し込みのフラグを更新する</button>
</form></div>

#上記部分のruby記述
<div class = 'button-wrapper'>
  <%= form_for User::EntriesForm.new(product: @product), as: 'form', url: [:entries, :user, @product], html: { method: :patch} do |f| %>
  <%= f.hidden_field :approved %>
  <%= f.hidden_field :not_approved %>
  <%= f.hidden_field :canceled %>
  <%= f.hidden_field :not_canceled %>
  <%= button_tag '申し込みのフラグを更新する', type:'button', id:'update-entries-button', class: "btn btn-outline-primary"%>
  <% end %>
</div>


ルーティング

resources :products do
  patch :entries, on: :member
end


申し込みのフラグを更新するボタンを押された時の処理を記述したJavaScript

$(function() {
  return $('div.button-wrapper').on('click', 'button#update-entries-button', function() {
    var approved, canceled, not_approved, not_canceled;
    approved = [];
    not_approved = [];
    canceled = [];
    not_canceled = [];
    $('table.entries input.approved').each(function() {
      if ($(this).prop('checked')) {
        return approved.push($(this).data('entry-id'));
      } else {
        return not_approved.push($(this).data('entry-id'));
      }
    });
    $('#form_approved').val(approved.join(':'));
    $('#form_not_approved').val(not_approved.join(':'));
    $('table.entries input.canceled').each(function() {
      if ($(this).prop('checked')) {
        return canceled.push($(this).data('entry-id'));
      } else {
        return not_canceled.push($(this).data('entry-id'));
      }
    });
    $('#form_canceled').val(canceled.join(':'));
    $('#form_not_canceled').val(not_canceled.join(':'));
    return $('div.button-wrapper form').submit();
  });
});

コントローラー

def entries
    entries_form = User::EntriesForm.new(Product.find(params[:id]))
    entries_form.assign_attributes(params[:form])
    entries_form.save
    flash.notice = 'レッスン申込者のフラグを更新しました。'
    redirect_to [current_user, :user_product]
  end
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • moke

    2017/02/07 16:14

    大変申し訳ないですが、railsから脱線しているようです。これを動くようにもできますが、railsに従って書き直す方が早いような気がします。

    キャンセル

回答 1

0

回答になっていなくてすみません
このまま修正するかrailsに沿って作り直すか
決めていただきたいと思います

今更ですが一括更新はcocoonやnested_form
を使うと楽ですよー。(本当はactivemodelを使うのがベストですが)
コード記述が三分の一になりますし
セキュリティも高められます。
jsも最小限の記述になります。

レール(rails)に乗った人生なんてくそっくらえと言うならこの脱線したコードを
脱線したまま動くようお手伝いしますが。
railsの恩恵をほとんど受けれないと思います。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/02/07 17:00

    コメント有難うございます。

    Railsからはみ出したいタイプですので、このまま修正したいと考えております。
    というのも、若干イレギュラーなこのやり方で動かせると、何かと応用効くかな、と考えております
    もし、お気付きのミスあればご指摘頂けますと大変有難いです。

    cocoon、nested_formアドバイス有難うございます。これらは別途勉強してみます!

    キャンセル

  • 2017/02/07 17:14

    えーっと、はみ出しと脱線は違いますよ。
    はみ出しは修正してレールに戻せますし知識があればかえって良くなることもありますが
    脱線は破綻しています。レールのない荒野を走っているようなものです。
    じゃあなぜrailsを使うんだろうと言う話になるレベルなんですが
    それでもいいならこのまま回答を考えます。

    キャンセル

  • 2017/02/07 23:15

    有難うございます、ご指摘の通りにRailsのセオリー通りに行うことが望ましいと思います。
    一方で、コードや実現したい子を見直して考えてもみたのですが、本件、そこまで脱線している様にも思えず、もう少しこの仕様でできないか、あがいてみることにします!!

    キャンセル

  • 2017/02/09 01:00

    一応コードは全部追わせていただきましたが、私が未熟なため問題が見つけられませんでした。
    これ以上となるとやはり環境を再現して、post時のparamやjavascriptの内容をdebugしないとわかりません、ちょっとそこまで時間が割けないので、innjeraさん自身でdebugをしてみてください。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.51%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • JavaScript

    16490questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • Ruby on Rails

    7299questions

    Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

  • Ruby on Rails 4

    2440questions

    Ruby on Rails4はRubyによって書かれたオープンソースのウェブフレームワークです。 Ruby on Railsは「設定より規約」の原則に従っており、効率的に作業を行うために再開発を行う必要をなくしてくれます。