[fix] 保健所提出書類のテンプレートを総務局現行様式に修正 by batcho0428 · Pull Request #1996 · NUTFes/group-manager-2

batcho0428 · 2026-01-21T06:45:22Z

対応Issue

resolve #1991

概要

総務局の保健所提出様式に合わせて並び替えと調理工程・平面図を追加
未実装だった、従業員名簿の学生・学外・教職員の区分判別を実装
管理者画面を改修に合わせて表示を編集
印刷後に扱いやすいように出力日時・ページ番号を追加
仕入れ日の表示を実装
PDFアウトラインの有効化

実装詳細

調理工程を追加
平面図を追加
並びを現行フォーマットに合わせて変更
従業員名簿の区分を学籍番号で判別するように実装
学籍番号88888888なら学外、99999999なら教職員、それ以外なら学生と表示するように実装（申請資料に合わせて）
仕入れ日が実施日-1日から申請データを参照するように変更
ヘッダーとフッターを追加　ヘッダーの追加に合わせてprint_controllerも修正
各種表示に　調理工程・平面図を追加
h1に団体名を表示し、各項目をh2で表示するように修正
PDFアウトラインを有効化し、タグで自動生成されるように実装
スタイルの微修正

画面スクリーンショット等

テスト項目

備考

提供予定数量が平均化されて表示されているが、正しいか総務に確認する必要があるかもしれない

Copilot

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copilot

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 12 comments.

Copilot · 2026-01-22T13:57:53Z

api/app/views/print_pdf/output_health_office_documents.pdf.erb

+            <h3><%= fp.name %></h3>
+            <div class="detail-area">
+              <% if cpo %>
+                <%= cpo.tent.present? ? simple_format(cpo.tent) : '調理行程が未登録です。' %><br>


「調理行程」は誤字です。正しくは「調理工程」です。

Suggested change

<%= cpo.tent.present? ? simple_format(cpo.tent) : '調理行程が未登録です。' %><br>

<%= cpo.tent.present? ? simple_format(cpo.tent) : '調理工程が未登録です。' %><br>

これは直して

Copilot · 2026-01-22T13:57:54Z

api/app/controllers/print_pdf_controller.rb

+
+        send_data pdf.to_pdf,
+                  filename: "#{output_file_name}.pdf",
+                  # disposition: "inline", # ダウンロードせず表示する


コメントアウトされたコード（# disposition: "inline", # ダウンロードせず表示する）が残っています。必要ない場合は削除すべきです。

api/app/views/print_pdf/output_health_office_documents_pdf.css

Copilot · 2026-01-22T13:57:54Z

api/app/views/print_pdf/output_health_office_documents.pdf.erb

+              <td>
+                <% pd = purchase_list.purchase_date %>
+                <% formatted_pd = if pd.respond_to?(:strftime)
+                  pd.strftime("%-m/%-d")
+                elsif pd.present?
+                  begin
+                    Date.parse(pd.to_s).strftime("%-m/%-d")
+                  rescue
+                    pd.to_s
+                  end
+                else
+                  ""
+                end %>
+                <%= formatted_pd %>


仕入れ日の表示ロジックが複雑すぎて、メンテナンスが難しいです。また、Date.parse の rescue ブロックで例外をキャッチしていますが、エラーハンドリングが不十分です。この複雑な日付フォーマットロジックはヘルパーメソッドに切り出すことを推奨します。

Copilot · 2026-01-22T13:57:54Z

api/app/views/print_pdf/output_health_office_documents.pdf.erb

-      <span>営業許可施設で調理された既製食品を現地で盛り付けるものは、「既製食品」欄に◯を記入</span>
    </div>
-    <div class="page-break"></div>
+      <div class="page-break" style="page-break-after: always;"></div>


インラインスタイル（style="page-break-after: always;"）が使用されていますが、これはCSSファイルで定義されている .page-break クラスと重複しています。CSSクラスのみを使用するべきです。

Suggested change

<div class="page-break" style="page-break-after: always;"></div>

<div class="page-break"></div>

Copilot · 2026-01-22T13:57:55Z

api/app/views/print_pdf/output_health_office_documents.pdf.erb

-                <%= fes_date.date %><br>
+            </div>
+        <% end %>
+      <% else %>


空の else ブロックが存在しています。このブロックは削除するか、または販売品が未登録の場合のメッセージを表示すべきです。

Suggested change

<% else %>

<% else %>

<p>販売品が未登録です。</p>

71行目の <% if group.food_products.any? %>かな？
これならendで終わってもよさそう

Copilot · 2026-01-22T13:57:55Z

api/app/views/print_pdf/output_health_office_documents.pdf.erb

+          <td class="Belonging"><%= group.name %></td>
+          <td class="Name"><%= group.user.name %></td>
+          <td class="Category">
+          <% student_id = (group.user.user_detail&.student_id || 0).to_i %>


safe navigation operator（&.）を使用しているにもかかわらず、nil の場合のデフォルト値として 0 を設定しています。|| 0 は safe navigation operator で nil が返された場合にも適用されますが、より明示的に &.student_id.to_i または dig(:user_detail, :student_id)&.to_i || 0 のようにすべきです。

Copilot · 2026-01-22T13:57:56Z

api/app/controllers/print_pdf_controller.rb

+  def print_pdf_with_header_footer(template_name, style_name, output_file_name, type)
+    respond_to do |format|
+      format.pdf do
+        html = render_to_string template: "print_pdf/#{template_name}"
+        pdf_options = {
+          page_size: 'A4',
+          encoding: 'UTF-8',
+          margin_top: '25mm',
+          margin_bottom: '25mm',
+          margin_left: '10mm',
+          margin_right: '10mm',
+          header_spacing: 5,
+          footer_spacing: 5,
+          header_left: '技大祭実行委員会　総務局',
+          header_line: false,
+          footer_center: '[page] / [topage]',
+          footer_right: "#{Time.now.getlocal('+09:00').strftime('%Y/%-m/%-d %-H:%M')} Group Manager",
+          footer_line: false,
+          footer_font_size: 10,
+          header_font_size: 10,
+          outline: true
+        }
+
+        pdf_options[:orientation] = 'Landscape' if type == 'Landscape'
+        pdf = PDFKit.new(html, pdf_options)
+
+        pdf.stylesheets << Rails.root.join("app/views/print_pdf/#{style_name}.css").to_s
+
+        send_data pdf.to_pdf,
+                  filename: "#{output_file_name}.pdf",
+                  # disposition: "inline", # ダウンロードせず表示する
+                  type: 'application/pdf'
+      end
+    end
+  end


新しいメソッド print_pdf_with_header_footer が追加されていますが、既存の print_pdf メソッドとコードが大部分重複しています。DRY原則に違反しており、メンテナンスが困難です。共通部分を抽出して、ヘッダー・フッターのオプションをパラメータとして渡すようにリファクタリングすべきです。

これそうだねー
with_header_footerに切り替えるなら既存のprint_pdfを削除しちゃってもいいと思うんだけどprint_pdfを使う時ってある？

@hikahana
物品貸出表・参加団体情報とかで使われてるっぽいです。
共通化して集約したほうがいいですかね？

Copilot · 2026-01-22T13:57:56Z

api/app/views/print_pdf/output_health_office_documents.pdf.erb

+          <% student_id = (group.user.user_detail&.student_id || 0).to_i %>
+          <% if student_id == 88888888 %>
+            学外
+          <% elsif student_id == 99999999 %>
+            教職員
+          <% else %>
+            学生
+          <% end %>
+          </td>
+        </tr>
+        <% if group.sub_rep %>
+          <tr>
+            <td class="No">2</td>
+            <td class="Belonging"><%= group.name %></td>
+            <td class="Name"><%= group.sub_rep.name %></td>
+            <td class="Category">
+              <% sub_rep_student_id = (group.sub_rep&.student_id || 0).to_i %>
+              <% if sub_rep_student_id == 88888888 %>
+                学外
+              <% elsif sub_rep_student_id == 99999999 %>
+                教職員
+              <% else %>
+                学生
              <% end %>
            </td>
-            <td>
-              <% @fes_dates.select { |fes_date| fes_date.fes_year_id == group.fes_year_id && fes_date.days_num == 1 }.each do |fes_date| %>
-                <%= fes_date.date %><br>
-              <% end %>
-              <% @fes_dates.select { |fes_date| fes_date.fes_year_id == group.fes_year_id && fes_date.days_num == 2 }.each do |fes_date| %>
-                <%= fes_date.date %><br>
+          </tr>
+        <% end %>
+        <% group.employees.each_with_index do |employee, index| %>
+          <tr>
+            <td class="No"><%= group.sub_rep ? index +3 : index +2 %></td>
+            <td class="Belonging"><%=group.name %></td>
+            <td class="Name"><%=employee.name %></td>
+            <td class="Category">
+              <% employee_student_id = (employee&.student_id || 0).to_i %>
+              <% if employee_student_id == 88888888 %>
+                学外
+              <% elsif employee_student_id == 99999999 %>
+                教職員
+              <% else %>
+                学生
              <% end %>


学籍番号による区分判別ロジック（88888888=学外、99999999=教職員）がマジックナンバーとして3箇所に重複しています。この判別ロジックをモデルメソッドまたはヘルパーメソッドに切り出すべきです。コードの重複が多く、メンテナンス性が低下しています。

これできそう？マジックナンバーは対応しておきたい。適当にcursorに投げたの載せとく。
user_details.rb

# 学籍番号の特殊値（マジックナンバー） STUDENT_ID_EXTERNAL = 88888888 # 学外 STUDENT_ID_STAFF = 99999999 # 教職員

※新規ファイル作ってる。別にどこかに書けばいいと思う。
application_helper.rb

# frozen_string_literal: true module ApplicationHelper # student_idからユーザータイプ（学外/教職員/学生）を判定して返す def user_category_label(student_id) return '学外' if student_id == UserDetail::STUDENT_ID_EXTERNAL return '教職員' if student_id == UserDetail::STUDENT_ID_STAFF '学生' end end

そしたらerbの中身これだけで良くなるらしい

<%= user_category_label((group.user.user_detail&.student_id || 0).to_i) %>

Copilot · 2026-01-22T13:57:56Z

api/app/views/print_pdf/output_health_office_documents.pdf.erb

+  <% else %>
  <% end %>


空の else ブロックが存在しています。団体がない場合のメッセージを表示するか、このブロックを削除すべきです。

コントローラー側でgroup.exists()で存在確認をしているからこのerbでgroupあるかどうかのifチェックは不要になる！
のでこのifとめっちゃしたにあるend消してネスト減らそう！！！

hikahana · 2026-01-23T08:19:13Z

これ正解が見たいから出力したpdfくれると嬉しい

hikahana · 2026-01-23T08:28:00Z

2020だと白紙で出力されて他の年度にしたらpdf開けなかった。
また後でちゃんと見ます

hikahana

コードレビューと動作確認だけ
正式なファイルになっているかどうかはあとむになげまーす
@Atomic-Bombs

hikahana · 2026-01-25T06:02:48Z

api/app/controllers/print_pdf_controller.rb

+  def print_pdf_with_header_footer(template_name, style_name, output_file_name, type)
+    respond_to do |format|
+      format.pdf do
+        html = render_to_string template: "print_pdf/#{template_name}"
+        pdf_options = {
+          page_size: 'A4',
+          encoding: 'UTF-8',
+          margin_top: '25mm',
+          margin_bottom: '25mm',
+          margin_left: '10mm',
+          margin_right: '10mm',
+          header_spacing: 5,
+          footer_spacing: 5,
+          header_left: '技大祭実行委員会　総務局',
+          header_line: false,
+          footer_center: '[page] / [topage]',
+          footer_right: "#{Time.now.getlocal('+09:00').strftime('%Y/%-m/%-d %-H:%M')} Group Manager",
+          footer_line: false,
+          footer_font_size: 10,
+          header_font_size: 10,
+          outline: true
+        }
+
+        pdf_options[:orientation] = 'Landscape' if type == 'Landscape'
+        pdf = PDFKit.new(html, pdf_options)
+
+        pdf.stylesheets << Rails.root.join("app/views/print_pdf/#{style_name}.css").to_s
+
+        send_data pdf.to_pdf,
+                  filename: "#{output_file_name}.pdf",
+                  # disposition: "inline", # ダウンロードせず表示する
+                  type: 'application/pdf'
+      end
+    end
+  end


これそうだねー
with_header_footerに切り替えるなら既存のprint_pdfを削除しちゃってもいいと思うんだけどprint_pdfを使う時ってある？

hikahana · 2026-01-25T06:37:47Z

api/app/views/print_pdf/output_health_office_documents.pdf.erb

+  <% else %>
  <% end %>


コントローラー側でgroup.exists()で存在確認をしているからこのerbでgroupあるかどうかのifチェックは不要になる！
のでこのifとめっちゃしたにあるend消してネスト減らそう！！！

hikahana · 2026-01-25T06:40:19Z

api/app/views/print_pdf/output_health_office_documents.pdf.erb

+            <h3><%= fp.name %></h3>
+            <div class="detail-area">
+              <% if cpo %>
+                <%= cpo.tent.present? ? simple_format(cpo.tent) : '調理行程が未登録です。' %><br>


これは直して

hikahana · 2026-01-25T06:41:14Z

api/app/views/print_pdf/output_health_office_documents.pdf.erb

-                <%= fes_date.date %><br>
+            </div>
+        <% end %>
+      <% else %>


71行目の <% if group.food_products.any? %>かな？
これならendで終わってもよさそう

hikahana · 2026-01-25T06:50:28Z

api/app/views/print_pdf/output_health_office_documents.pdf.erb

+          <% student_id = (group.user.user_detail&.student_id || 0).to_i %>
+          <% if student_id == 88888888 %>
+            学外
+          <% elsif student_id == 99999999 %>
+            教職員
+          <% else %>
+            学生
+          <% end %>
+          </td>
+        </tr>
+        <% if group.sub_rep %>
+          <tr>
+            <td class="No">2</td>
+            <td class="Belonging"><%= group.name %></td>
+            <td class="Name"><%= group.sub_rep.name %></td>
+            <td class="Category">
+              <% sub_rep_student_id = (group.sub_rep&.student_id || 0).to_i %>
+              <% if sub_rep_student_id == 88888888 %>
+                学外
+              <% elsif sub_rep_student_id == 99999999 %>
+                教職員
+              <% else %>
+                学生
              <% end %>
            </td>
-            <td>
-              <% @fes_dates.select { |fes_date| fes_date.fes_year_id == group.fes_year_id && fes_date.days_num == 1 }.each do |fes_date| %>
-                <%= fes_date.date %><br>
-              <% end %>
-              <% @fes_dates.select { |fes_date| fes_date.fes_year_id == group.fes_year_id && fes_date.days_num == 2 }.each do |fes_date| %>
-                <%= fes_date.date %><br>
+          </tr>
+        <% end %>
+        <% group.employees.each_with_index do |employee, index| %>
+          <tr>
+            <td class="No"><%= group.sub_rep ? index +3 : index +2 %></td>
+            <td class="Belonging"><%=group.name %></td>
+            <td class="Name"><%=employee.name %></td>
+            <td class="Category">
+              <% employee_student_id = (employee&.student_id || 0).to_i %>
+              <% if employee_student_id == 88888888 %>
+                学外
+              <% elsif employee_student_id == 99999999 %>
+                教職員
+              <% else %>
+                学生
              <% end %>


これできそう？マジックナンバーは対応しておきたい。適当にcursorに投げたの載せとく。
user_details.rb

# 学籍番号の特殊値（マジックナンバー） STUDENT_ID_EXTERNAL = 88888888 # 学外 STUDENT_ID_STAFF = 99999999 # 教職員

※新規ファイル作ってる。別にどこかに書けばいいと思う。
application_helper.rb

# frozen_string_literal: true module ApplicationHelper # student_idからユーザータイプ（学外/教職員/学生）を判定して返す def user_category_label(student_id) return '学外' if student_id == UserDetail::STUDENT_ID_EXTERNAL return '教職員' if student_id == UserDetail::STUDENT_ID_STAFF '学生' end end

そしたらerbの中身これだけで良くなるらしい

<%= user_category_label((group.user.user_detail&.student_id || 0).to_i) %>

hikahana · 2026-01-25T06:50:45Z

api/app/views/print_pdf/output_health_office_documents.pdf.erb

+      <% else %>
      <% end %>


これもendで良さそう

batcho0428 · 2026-02-01T14:12:28Z

@hikahana
修正しました。
確認お願いします！

batcho0428 · 2026-02-01T14:13:23Z

@Atomic-Bombs
ファイル確認お願いします！
（写真が載っていないところは登録していないだけです。最後の稲葉テスト環境で確認お願いします。）
保健所提出書類（調理計画・調理工程・従事者・平面図） (1).pdf

Atomic-Bombs · 2026-02-02T05:12:26Z

@batcho0428
大丈夫だと思います！

hikahana

概ね良さそう

[fix] 保健所提出書類のテンプレートを総務局現行様式に修正

8ca97b9

github-actions bot assigned batcho0428 Jan 21, 2026

chore(rubocop): auto fix (-A) [skip ci]

08eee59

batcho0428 requested review from Atomic-Bombs, Copilot and hikahana January 21, 2026 07:51

Copilot started reviewing on behalf of batcho0428 January 21, 2026 14:20 View session

Copilot AI reviewed Jan 21, 2026

View reviewed changes

batcho0428 requested a review from Copilot January 22, 2026 13:52

Copilot started reviewing on behalf of batcho0428 January 22, 2026 13:52 View session

Copilot AI reviewed Jan 22, 2026

View reviewed changes

hikahana reviewed Jan 25, 2026

View reviewed changes

batcho0428 added 4 commits February 1, 2026 22:35

[fix] 誤字・余分なelseを削除

42cc5f0

[feat] 学籍番号による区分判定をヘルパー実装に変更

2d234a5

[feat] 学籍番号による判定をヘルパーによる処理に変更

55af2e2

[fix] ptint_pdf_controller.rbを修正してファイルのif-elseを削減

4ea350d

hikahana approved these changes Feb 2, 2026

View reviewed changes

	<%= cpo.tent.present? ? simple_format(cpo.tent) : '調理行程が未登録です。' %><br>
	<%= cpo.tent.present? ? simple_format(cpo.tent) : '調理工程が未登録です。' %><br>

	<div class="page-break" style="page-break-after: always;"></div>
	<div class="page-break"></div>

Conversation

batcho0428 commented Jan 21, 2026

対応Issue

概要

実装詳細

画面スクリーンショット等

テスト項目

備考

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Jan 22, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 22, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Copilot AI Jan 22, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 22, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 22, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 22, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 22, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 22, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 22, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

hikahana commented Jan 23, 2026

Uh oh!

hikahana commented Jan 23, 2026

Uh oh!

hikahana left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

batcho0428 commented Feb 1, 2026

Uh oh!

batcho0428 commented Feb 1, 2026

Uh oh!

Atomic-Bombs commented Feb 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Atomic-Bombs commented Feb 2, 2026 •

edited

Loading