村が複数回開始できる
手動村開始ボタンをダブルクリックしてしまった際に発生した問題に対処した時のログです。
-------------------------------- lib/commit.pl --------------------------------
index 249c796..e428048 100644
@@ -8,6 +8,8 @@ sub StartSession {
my $textrs = $sow->{'textrs'};
my $pllist = $vil->getpllist();
+ return if ($vil->{'turn'} > 0); # 開始済み
+
# 確定待ち発言の強制確定
&FixQueUpdateSession($sow, $vil);
そっち? 一番お膝元だね、なるほど。
こっちのローカルブランチでは 何故か存在を忘れ去られているファイル(笑 に sow.cgi のコード移して、そっちでチェックするようにしてる。
sow.cgi 的にはそっちの方でチェックする流れじゃないかって思うけど、どーかな……。 ちょっと待ってて、ご飯食べたらリモートにブランチ切って入れておくのでチェックしてみてもらっていい?
StartSession で二重チェックするのはありと思うのでせぷGさんのコードも一緒に入れるのもいいかな(turnチェックは、unless isproplgue() オススメです)
チェックして問題ないようなら master への投入よろしくです。 担当者変えておくね~。
作業ブランチに転送しておきましたのでご確認よろしくお願いします。 http://sourceforge.jp/projects/sow/scm/git/develop/commits/184ba3b279916d62af173e3b6599e50821c3ea53
作業ブランチ拝見しました。
開始チェックのバリデーションでチェックすれば警告メッセージも表示できるし、いいですね。
念のためcommit.plにもということであれば、こちらのファイルは私が修正します。
同じく、時間を見つけて作業ブランチに入れてみます。
すみません、拝見する前にmasterからブランチ切ってコミットしてしまいました。
w_cepterg リビジョン 6855f8d9cfbee27d1ab1895fb510b1e02b3538af
マージ前にご確認ください。よろしくお願いします。
master に統合しておきましたのでpullしといてね。 二重開始できないことは確認。
そちらでも確認できたらクローズお願いします。
そ言えば廃村ボタンも二重押し(=廃村即エピ終了)できてしまうのも見つけたけど、ここもチェックしたほうがいいのかな? ほっぽいといてもよさそうだけど、もし気になるならチケットお願いします。
統合とご確認ありがとうございました。クローズします。
廃村ボタンも同じなんですね、手元でも確認してみます。
村開始後にもう一度村開始操作ができてしまう。 (村開始操作が行われた時点で再度1日目からとなる)
実際のオペレーションでは開始方法を審問や人狼BBS方式に設定しているときに手動で村開始をしようと確認フォームを出し、実際にボタン操作をする前に村の開始条件(参加人数・時刻等)が満たされて開始してしまっていた場合等に発症する。