search

yandod / candycane

a port of Redmine to CakePHP from Ruby on Rails
http://my.candycane.jp
510 stars 127 forks source link

チケットのステータスでの絞り込み後にソートしてもステータスが引き継がれないバグ修正 #221

Closed tsukarooohi closed 11 years ago

tsukarooohi commented 11 years ago

初めまして。 勝手にプルリクエストしてよいものか分からなかったのですが、もし処理的にも問題なければマージして頂けると光栄です。

プルリクエスト自体が初めてなので何かしら失礼な事がありましたら申し訳ありません。

以上宜しくお願いします。

yandod commented 11 years ago

テストが壊れたようなのでちょっと確認しないといけませんね。

tsukarooohi commented 11 years ago

申し訳ありません。

基本なのにテストの確認をしていませんでした。 またテストも通るよう確認してプルリクエストさせていただくので宜しくお願いします。

yandod commented 11 years ago

テスト側の修正が必要になるケースもあるので、なにが正しい挙動かを検討する必要がありますね。

coveralls commented 11 years ago

Coverage Status

Coverage increased (+0.03%) when pulling 50161f522ebe756afc794cb1c5e9e680087c8d43 on tsukarooohi:Component-QueriesComponent into 3981ef96a935afb1d75a1e68ab26ac77cd24b845 on yandod:master.

yandod commented 11 years ago

テスト再実行したら通りましたね。マージしてみます。

tsukarooohi commented 11 years ago

ありがとうございます。 初めてだったので緊張しましたが良かったです。

これからもテスト含め挙動を確認しながら貢献出来ればと思います。

ありがとうございました。