Ticket #29005

村が複数回開始できる

Open Date: 2012-07-15 19:19 Last Update: 2012-07-24 00:58

Reporter:
Owner:
Status:
Closed
Component:
(None)
Priority:
5 - Medium
Severity:
5 - Medium
Resolution:
Fixed
File:
None

Details

村開始後にもう一度村開始操作ができてしまう。 (村開始操作が行われた時点で再度1日目からとなる)

実際のオペレーションでは開始方法を審問や人狼BBS方式に設定しているときに手動で村開始をしようと確認フォームを出し、実際にボタン操作をする前に村の開始条件(参加人数・時刻等)が満たされて開始してしまっていた場合等に発症する。

Ticket History (3/10 Histories)

2012-07-15 19:19 Updated by: mikari
  • New Ticket "村が複数回開始できる" created
2012-07-17 21:56 Updated by: mikari
2012-07-21 00:10 Updated by: cepterg
Comment

手動村開始ボタンをダブルクリックしてしまった際に発生した問題に対処した時のログです。

-------------------------------- 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);
2012-07-21 10:41 Updated by: mikari
  • Owner Update from (None) to cepterg
Comment

そっち? 一番お膝元だね、なるほど。

こっちのローカルブランチでは 何故か存在を忘れ去られているファイル(笑 に sow.cgi のコード移して、そっちでチェックするようにしてる。

sow.cgi 的にはそっちの方でチェックする流れじゃないかって思うけど、どーかな……。 ちょっと待ってて、ご飯食べたらリモートにブランチ切って入れておくのでチェックしてみてもらっていい?

StartSession で二重チェックするのはありと思うのでせぷGさんのコードも一緒に入れるのもいいかな(turnチェックは、unless isproplgue() オススメです)

チェックして問題ないようなら master への投入よろしくです。 担当者変えておくね~。

2012-07-21 15:43 Updated by: mikari
Comment

作業ブランチに転送しておきましたのでご確認よろしくお願いします。 http://sourceforge.jp/projects/sow/scm/git/develop/commits/184ba3b279916d62af173e3b6599e50821c3ea53

2012-07-22 06:42 Updated by: cepterg
Comment

作業ブランチ拝見しました。
開始チェックのバリデーションでチェックすれば警告メッセージも表示できるし、いいですね。
念のためcommit.plにもということであれば、こちらのファイルは私が修正します。
同じく、時間を見つけて作業ブランチに入れてみます。

2012-07-22 09:22 Updated by: mikari
Comment

cloneしたリポジトリでブランチ切って、w_mikari をマージしてしまえばいいと思います。

開始チェックは、……あっと、Perlだから。

  1. return unless ($vil->isprologue());
でおっけなはず。

Ruby だと、nil/false 以外は真と判断されるので数値の戻り値は比較演算で真偽値にしないといけないので、その作法で書いてました――そもそもRubyだったら is_xxx の戻り値自体を真偽値で返しますけど、それはそうとして(笑

次のリリース前にリファクタリングしたほうがいいですね、これ。

 

では、クローズまでよろしくお願いします。

2012-07-23 02:54 Updated by: cepterg
Comment

すみません、拝見する前にmasterからブランチ切ってコミットしてしまいました。
w_cepterg リビジョン 6855f8d9cfbee27d1ab1895fb510b1e02b3538af

マージ前にご確認ください。よろしくお願いします。

2012-07-23 22:16 Updated by: mikari
Comment

master に統合しておきましたのでpullしといてね。 二重開始できないことは確認。

そちらでも確認できたらクローズお願いします。

そ言えば廃村ボタンも二重押し(=廃村即エピ終了)できてしまうのも見つけたけど、ここもチェックしたほうがいいのかな? ほっぽいといてもよさそうだけど、もし気になるならチケットお願いします。

2012-07-24 00:58 Updated by: cepterg
  • Resolution Update from None to Fixed
  • Status Update from Open to Closed
  • Ticket Close date is changed to 2012-07-24 00:58
Comment

統合とご確認ありがとうございました。クローズします。
廃村ボタンも同じなんですね、手元でも確認してみます。

Attachment File List

No attachments

Edit

You are not logged in. I you are not logged in, your comment will be treated as an anonymous post. » Login