Ticket #29085

村開始時刻を村建て以降に編集すると定時開始設定が異常動作してしまう。

Open Date: 2012-07-21 19:00 Last Update: 2012-07-29 23:48

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

Details

村編集で開始時間を変更しても自動開始時刻が村建て時のまま編集に連動せず、規定時刻以前に村が始まったり、初日が規定時間より短くなったりする。 (たとえば、18:30 に設定して村を建てて、19時自動開始に変更すると 18:30に村が開始して初日が30分になってしまう)

#29006 も参照のこと。

Ticket History (3/12 Histories)

2012-07-21 19:00 Updated by: mikari
  • New Ticket "村開始時刻を村建て移行に編集すると定時開始設定が異常動作してしまう。" created
2012-07-21 19:03 Updated by: mikari
  • Details Updated
2012-07-21 19:13 Updated by: mikari
Comment

……問題は、

$vil->{'nextchargedt'} = $nextupdate; 喉補給時間の更新と、ガードタイム(LIMIT_TIMEEDIT)の導入をどうするか、ですね(#29006 参照) 一応SWでは運用上のクレームは来ていませんが、村建て実績が少ないので何とも言えないところです。

28日までに何か問題ありそうな運用フローのご指摘がなかったらガードタイム導入とチャージタイムの更新を実装をしようと思いますので、ご意見よろしくお願いします。 担当変更しておきます。

2012-07-24 01:15 Updated by: cepterg
Comment

喉補給の更新ですが、ここは「開始時刻になったけど定員に満たなかったので24時間延長しました」って時に連動して補給するなら、
nextupdatedtと一緒に更新するのがいいんでしょうね。

LIMIT_TIMEEDITは、開始ギリギリの編集を許さない仕様ということでしょうか?
configでのデフォルト値が0なら、現在の物語オリジナルと同じということですよね。
こちらはバグ修正というより、どちらかというと機能追加っぽいイメージですね。

2012-07-24 21:26 Updated by: mikari
Comment

では、喉は一緒に動かしても問題なさそうですね。 (ちょろちょろと時間を編集してしまうと、陽炎現象が起こってしまいますが/笑 そういうフローはまずないでしょうしね)

LIMIT_TIMEEDIT は #29006 のコメントに「設定ミス確認の時間をとるため」とある通り、編成ミス等を確認するための猶予時間を確保するという意図があります。

ギリギリで編集してもいいけど、その場合は「時間までにしっかり見直して主導で開始してください」という主張を込めています。

薔薇下さんで編成ミスしてしまったみたいな事件まれにあったりするのみてるので、そういうのの対策として有用と考えて SchwarzWeißに導入したものですね。

 

>こちらはバグ修正というより、どちらかというと機能追加っぽいイメージですね。

確かに。 チケット分割したほうがいいかな?

2012-07-24 21:27 Updated by: mikari
Comment

s/主導/手動/

2012-07-26 06:36 Updated by: cepterg
Comment

LIMIT_TIMEEDIT の目的については了解です、言い方が悪くてすみませんでした。

こちらは更新時刻問題とは別で、国によって導入するかしないか選べる新機能になりそうなので、
導入作業は同時に実施でも、チケット分けた方がいいかもしれないですね。

LIMIT_TIMEEDITの値を定義するconfig.plとかも同時に修正になるのでしょうか?

2012-07-26 23:49 Updated by: mikari
Comment

言い方がと言うのは何かありましたでしょうか?

もし返信が何か責めてるように思われたのならごめんなさい。

 

>LIMIT_TIMEEDITの値を定義するconfig.plとかも同時に修正になるのでしょうか?

はい、そうなります。 初期値は 0で 2.00と同じ動作とする予定です。 別にチケット切っておきますね(ぺこ

2012-07-28 16:07 Updated by: cepterg
Comment

自分の「開始ギリギリの編集を許さない仕様」って言い方だと、
メリットについて全然述べられてなかったなぁ、と反省しました(^^;
かえってお気遣いさせてしまってすみません。

別にチケット切っていただいてありがとうございます。確認しました。

2012-07-29 11:04 Updated by: mikari
  • Summary Updated
Comment

実質同じだから何も問題ないけど、あれ?

それはそうとして、ローカルで黒白からマージしました。 動作確認したらコミットしますね。

2012-07-29 23:48 Updated by: mikari
  • Resolution Update from None to Fixed
  • Status Update from Open to Closed
  • Ticket Close date is changed to 2012-07-29 23:48
Comment

コミットしました、クローズします。

Attachment File List

Edit

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