PR

【9/9】ヤマダゲームマラソン #3675

kouho opened this at 2024-09-06T16:03:39; closed

Description

opened by kouho at 2024-09-06T16:03:39; updated 2024-09-11T10:42:21

概要

https://docs.google.com/spreadsheets/d/1bY1pwZahoStvSgnl8jt_0Z9ucXzYvxU0/edit?gid=132198363#gid=132198363

変更点

  • 新大陸ステージモンスター討伐ミッション
  • 新大陸以外のステージモンスター討伐ミッション
  • レイドモンスター攻撃

TODO

  • [x] 期限付きアイテムの報酬対応(対応済みだった)
  • [x] 告知用画像
kouho change_title at 2024-09-06T16:40:22: [WIP]【9/9】ヤマダゲームマラソン 【9/9】ヤマダゲームマラソン

Comment

#36451 by kouho at 2024-09-06T16:40:51; updated 2024-09-06T16:40:51

@wada
レビューお願いいたします

Comment

#36463 by wada at 2024-09-09T09:15:36; updated 2024-09-09T09:15:36

@kouho
バナー画像はどのPRに追加されますか?

Comment

#36464 by wada at 2024-09-09T09:17:52; updated 2024-09-09T09:17:52

@kouho
報酬受け取りページで、ぱんだまとコインガチャチケットスタンプも直接配布されることを記載お願いします。

Comment

#36465 by wada at 2024-09-09T09:20:27; updated 2024-09-09T09:20:27

@kouho
ゴールドパス報酬の9,16km報酬がコインガチャチケットではなく、ぱんだまになっています。

wada add_label at 2024-09-09T09:20:31: コメントあり

Comment

#36467 by wada at 2024-09-09T09:47:46; updated 2024-09-09T09:47:46

@kouho
S_SetUserQuestVarious()if($marathon->is_active()) 内に以下の処理も追加お願いします。

$marathon->update_mission_count($user_info['user_no'], $marathon::MISSION_TYPE_MONSTER, 1);

Comment

#36468 by wada at 2024-09-09T09:49:22; updated 2024-09-09T09:49:22

@kouho
S_SetUserQuestVarious() の新大陸反映は、MissionStageConst::getNewIslandStage() を使用してください。

Comment

#36479 by kouho at 2024-09-09T16:50:39; updated 2024-09-09T16:50:39

@wada
確認ありがとうございます。以下修正しました。

このPRに含めています。
> バナー画像はどのPRに追加されますか?

データ修正しました。コインガチャチケットスタンプにしています。
> ゴールドパス報酬の9,16km報酬がコインガチャチケットではなく、ぱんだまになっています。

記載しました。
> 報酬受け取りページで、ぱんだまとコインガチャチケットスタンプも直接配布されることを記載お願いします。

$this->user->get_artack_count()を使用するように修正しました。
> このif文に入るのはどのような時ですか?
for文の外で $i を参照するのは避けた方が良いと思います。

Comment

#36480 by kouho at 2024-09-09T17:13:13; updated 2024-09-09T17:13:13

@wada
以下も修正しました。
> S_SetUserQuestVarious() の if($marathon->is_active()) 内に以下の処理も追加お願いします。
>
$marathon->update_mission_count($user_info['user_no'], $marathon::MISSION_TYPE_MONSTER, 1);

> S_SetUserQuestVarious() の新大陸反映は、MissionStageConst::getNewIslandStage() を使用してください。

Comment

#36482 by wada at 2024-09-09T17:45:48; updated 2024-09-09T17:45:48

@kouho
ありがとうございます。確認oKです。

wada delete_label at 2024-09-09T17:45:51: コメントあり
wada add_label at 2024-09-09T17:45:52: 和田 LGTM
kouho merge at 2024-09-11T10:42:21: 【9/9】ヤマダゲームマラソン
kouho close at 2024-09-11T10:42:21: Close

commit comment

#262 by wada at 2024-09-09T09:35:17; lib/models/Raid_battle_model.php / old:None new:138
f3c8b9467dcdf4cde8ad31038f4ddb57351aa541

@kouho
このif文に入るのはどのような時ですか?
for文の外で $i を参照するのは避けた方が良いと思います。

Commit range

basea920ca703fd538679c2b1769169afd3005846d48
headd021c8fc882eaa88fbd4a2772f22dbd24d865e1c
base branchjolly/jr-web:master
head branchjolly/jr-web:marathon_5

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed13 filesView patch
statusfile
modifiedlib/SubFunc.inc
modifiedlib/models/Raid_battle_model.php
modifiedlib/models/Yamada_game_marathon.php
addedpublic_html/img/banner/Campaign/banner_yamada_game_marathon_5_1.jpg
addedpublic_html/img/banner/Campaign/banner_yamada_game_marathon_5_2.jpg
addedpublic_html/img/info_jrye/3665/info1.png
addedpublic_html/img/info_jrye/3665/info2.png
addedpublic_html/img/info_jrye/3665/info3.png
addedpublic_html/img/info_jrye/3665/info4.png
addedpublic_html/img/info_jrye/3665/info5.png
addedpublic_html/img/info_jrye/3665/info6.png
addedpublic_html/img/info_jrye/3665/info7.png
modifiedpublic_html/template/YamadaGame-MarathonRewawrds.tpl
Patch body is stored as a separate file to keep this page lightweight.

Review comment

#262 by wada at 2024-09-09T09:35:17; lib/models/Raid_battle_model.php / old:None new:138
f3c8b9467dcdf4cde8ad31038f4ddb57351aa541

@kouho
このif文に入るのはどのような時ですか?
for文の外で $i を参照するのは避けた方が良いと思います。