yochiyochirb/kajaeru

自分自身に投票できないようにする

Closed this issue · 6 comments

yucao24hours commented

投票してる人（voting_member_id）と投票対象者（voted_member_id）が一緒の場合は、Member#index にリダイレクトさせて「自分自身には投票できません！」というメッセージを出す。

yucao24hours commented

そろそろ、不正行為を一括管理するモジュールみたいのが必要になってきそうかな。リファクタリング用の Issue が必要かも。

yucao24hours commented

とはいえ、この Issue では特に考えず実装していただいて大丈夫です！（わりと急ぎめのバグ対応なので）

yucao24hours commented

というか、コントローラが太ってきそうなので、モデルに実装するべきなのはしたほうがいいのかも。

yucao24hours commented

不正行為のバリエーションが増えてきた（もともとあったものが顕在化してきた）ので、 @to0526 さんや @katorie さんが作ってくださったコードをもとに、モデルの値を比較するような処理はモデルに移動させていこうと思っています。
また、redirect_root_with_alert メソッドも、そのときの不正行為にあわせて alert を出し分けられるようにしたいので、そのように書き換えてみたいなと思っていますが、いかがでしょうか〜？

katorie commented

はい:ok_woman:
とはいうものの、実際どういう感じになるのか理解が追いついていないのですが:sweat_drops: