チケット一覧のページャーのリンクがおかしくなるバグの修正。

yandod / candycane

a port of Redmine to CakePHP from Ruby on Rails

http://my.candycane.jp

510 stars 127 forks source link

チケット一覧のページャーのリンクがおかしくなるバグの修正。 #222

Closed tsukarooohi closed 11 years ago

tsukarooohi commented 11 years ago

バグを確認したのはチケット一覧でステータスの絞り込みを行った後です。

一度目は問題なくページ移動出来るんですがそのタイミングでページャーのリンクがおかしくなりページ移動が出来なくなります。

修正後確認事項ページャーの動作確認 View/Helper/CandyHelper.phpのテストが問題なく通る事

以上です。宜しければマージの方宜しくお願い致します。

coveralls commented 11 years ago

Coverage remained the same when pulling 8d52ed336690619202bf3377a0751ed98e52f625 on tsukarooohi:fix into f3220889428b824df12898bc17b913de7308ac48 on yandod:master.

tsukarooohi commented 11 years ago

申し訳ありません。元々 View/Helper/CandyHelper.php に修正した pagination_links_full のテスト内容がなかったのですね。また確認ミスでした。

一度他のテストなどを見て今後は間違いなく修正箇所のテストも通る状態にしてプルリクエストを出したいと思います。

yandod commented 11 years ago

そんなに怖がらなくてもいいですよー。 Web側のテストがもともとないのでユニットレベルでバグの部分を再現できるならいいですね。

そろそろSeleniumもやりたいのですがそれはまた別の話ですね。