PR

【即日】土日に戦争クエストを2個表示 #1016

wada opened this at 2021-08-02T14:40:40; closed

Description

opened by wada at 2021-08-02T14:40:40; updated 2025-03-13T11:40:37

概要

デイリークエストの戦争参加を土日に2つ表示する必要があった
デイリークエストに戦争参加があったら、土日だけ同じ条件のクエストを追加
user_daily_quest.border_keyで昼夜(12/21)を区別
クエスト表示時に、タイトルの後に時間を追記

例:戦争に参加しよう(21時)

テスト方法、確認した内容

マスタデータ反映後、master_yamada_campaignstart_dateend_date を変更
(開催中のキャンペーンは期間外にしておく)

当日ログインしている場合

  • user_daily_quest の当日データを削除
  • dau_login の当日データを削除
  • user_info.reset_timeを昨日以前の時間に変更

ログイン処理後にデイリークエストに追加されます
土日のチェック時は、S_GetTeamWarPeriodsが12時と21時のデータを返すように調整してください

戦争の確認

デバッグツールから戦争を開催して、出撃 or 防衛
参加して時にuser_daily_quest.border_keydate('H')が一致しないとカウントされないので、S_CountUpUserDailyQuestの処理を調整してください

Comment

#12636 by wada at 2021-08-02T14:41:26; updated 2021-08-02T14:41:26

@iwamoto @kouho
デイリークエストの戦争土日対応が抜けていました。
レビューをお願いします。

Comment

#12648 by kouho at 2021-08-02T15:43:37; updated 2021-08-02T15:43:37

@wada
12時と21時の区別が対応されているのを確認しました。OKです。
すみません、前のレビューの時に仕様を注意してみるべきでした。

kouho add_label at 2021-08-02T15:43:44: 皇甫 LGTM

Comment

#12656 by iwamoto at 2021-08-02T16:08:35; updated 2021-08-02T16:08:35

@wada
表示は問題ないと思いますが
出撃 & 防衛等を行った際にクエストがクリアにならないとおもいます。
(現在詳細確認中です)

Comment

#12660 by kouho at 2021-08-02T16:26:39; updated 2021-08-02T16:26:39

@wada @iwamoto
念のためこちらで確認した手順を記載しておきます。

1. ログイン状態をリセット
2. S_GetTeamWarPeriodsの if (in_array(date("N", time()), [6, 7])) { をコメントアウト
3. ログインしてデイリークエストに12時と21時の戦争が表示されることを確認
4. 戦争を管理画面で開始
5. jrye.team_dataのwar_idに対応するjrye_teamWar.teamWar_infoのyear_mon_dayを2021080212に修正
6. S_CountUpUserDailyQuestの $add_where = sprintf("AND border_key = %d", date('H')); $add_where = sprintf("AND border_key = %d", 12); に修正
7. 戦争で防衛コマンドを実行してデイリークエストのクリアとヤマダポイントの受け取りを確認
8. jrye.team_dataのwar_idに対応するjrye_teamWar.teamWar_infoのyear_mon_dayを2021080221に修正
9. S_CountUpUserDailyQuestの $add_where = sprintf("AND border_key = %d", date('H')); $add_where = sprintf("AND border_key = %d", 21); に修正
10. 戦争で出撃コマンドを実行してデイリークエストのクリアとヤマダポイントの受け取りを確認

Comment

#12669 by iwamoto at 2021-08-02T16:44:31; updated 2021-08-02T16:44:31

@wada
失礼しました、こちら確認できました。

iwamoto add_label at 2021-08-02T16:44:37: 岩本 LGTM
wada merge at 2021-08-02T17:45:03: 【即日】土日に戦争クエストを2個表示
wada close at 2021-08-02T17:45:03: Close

Commit range

base8cd64662c9c5e2841f17bf989ebc23d7787cb4c9
head12f9de51a3afe443f9571c54ca1f9240a9aa86f0
base branchjolly/jr-web:develop
head branchjolly/jr-web:yamada_pt_8

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed2 filesView patch
statusfile
modifiedlib/SubFunc.inc
modifiedlib/models/Yamada_point_quest.php
Patch body is stored as a separate file to keep this page lightweight.