ProductChecker初期化での未定義イベント参照を削除

[machida]

adminでログインしたときの提出物一覧ページで発生したエラーに対応。

Summary by CodeRabbit

リリースノート

• リファクタリング
  • ボタン状態の管理フローを改善しました。内部的な処理の最適化により、より効率的な状態管理を実現しています。

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

app/javascript/product-checker.js内のinitProductChecker関数から直接的なDOM操作を削除しました。ボタンの状態管理はgenerateActionButtonとupdateButtonのフロー専用に統一されました。

Changes

内容 / ファイル	変更の説明
DOM操作の削除 app/javascript/product-checker.js	initProductChecker内でボタンの最初の子要素のクラス名とtextContentを直接操作していた処理を削除。ボタン状態はすべてgenerateActionButton/updateButtonフローで管理するよう統一。

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

• DOM操作ロジックの削除であり、1ファイルのみの変更
• 既存のgenerateActionButton/updateButtonフローでの管理に統一する単純な整理作業
• 機能的な挙動の変更ではなく、状態管理の責任分離

Possibly related PRs

• ProductChecker.jsx を素のJavaScriptに移行 #9264: 同じapp/javascript/product-checker.jsのProductChecker実装を修正しており、このPRで削除される直接DOM操作の削除が、当該PRで導入されたより広範な移行の一部を構成している。

Suggested reviewers

• komagata
• kitarou888

Poem

🐰✨ ボタンのお手入れ、きれいきれい
DOM操作を片付けて
generateActionButtonに任せて
責任分離で、スッキリ爽快
うさぎも満足、ホップホップホップ！🌿

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	PR説明は概要のセクションのみで、必須テンプレートの多くのセクション（Issue番号、変更確認方法、Screenshots）が不足しています。	テンプレートに従い、Issue番号、詳細な変更確認方法、変更前後のスクリーンショットを追加してください。

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	PRのタイトルは、削除された未定義イベント参照に関する変更を正確に説明しており、変更内容と適切に関連しています。
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix-product-checker.js

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between dae9279 and 60076c5.

📒 Files selected for processing (1)

• app/javascript/product-checker.js (0 hunks)

💤 Files with no reviewable changes (1)

• app/javascript/product-checker.js

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: build_and_test

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}