search

agileware-jp / redmine_work_days

Redmine Work Days Plugin
18 stars 8 forks source link

RedmineWorkDaysのCSVインポートの不具合を修正する #21

Closed y-imaida closed 6 years ago

y-imaida commented 6 years ago

概要

Issue #17 への対応として入力ファイルの拡張子がcsvではない場合、「CSVから読み込む」ボタン押下後にinternal errorが発生する事象を解消することが目的です。

変更内容

issue #17 で指摘されている内容そのままになりますが、rest_days_controller.rbを修正しました。 入力ファイルの拡張子がcsvではない場合にrenderメソッドでindex画面を表示する前に@rest_day変数を初期化する処理を追加しました。 この変更により入力ファイルが不正であることを通知するフラッシュを含む以下のようなindex画面が表示されるようになります。

2018-04-03 13 56 57

備考

Redmineチケット https://insidemine.agileware.jp/issues/9659

maimai77 commented 6 years ago

@y-imaida テストを追加してください

y-imaida commented 6 years ago

ご指摘いただきありがとうございました。 テスト追加後にご連絡します。

y-imaida commented 6 years ago

@maimai77 CSVインポートに関するテストを追加しましたので確認をお願いします。

追加したテストの内容は以下のとおりです。