PR

【3/6】新人応援パス2回目画像対応 #4549

wada opened this at 2026-03-03T11:01:59; closed

Description

opened by wada at 2026-03-03T11:01:59; updated 2026-03-06T16:45:53

概要

2~4回目の価格が300から100に変更になったため、画像表示を修正
ショップのバナーは受け取り中非表示に修正

Comment

#44636 by wada at 2026-03-03T11:31:51; updated 2026-03-03T11:31:51

@kouho
レビューお願いします。

Comment

#44692 by kouho at 2026-03-06T10:23:57; updated 2026-03-06T10:23:57

@wada
価格の表示不整合が起きているように見えますので確認をお願いいたします。

以下が原因に見えます。

  • lib/models/Monthly_pass.php
  • get_banner_image_path
  • get_image_suffix

buy_countで参照

  • lib/models/shop/Monthly_pass.php
  • get_price

buy_count+1で参照

・価格の表示不整合

◼︎1回目
初回50コインであるが、以下のページで300コイン表示

  • 冒険応援パスページ

http://127.0.0.1:20080/?M=Quest&A=MonthlyPass

  • ショップページ

http://127.0.0.1:20080/?M=Shop

購入直前画面では50コイン表示
http://127.0.0.1:20080/?M=Shop#page_shop_select?id=86

kouho add_label at 2026-03-06T10:24:00: コメントあり

Comment

#44696 by wada at 2026-03-06T10:48:20; updated 2026-03-06T10:48:20

@kouho
修正しました。

> 価格の表示不整合が起きているように見えますので確認をお願いいたします。

kouho add_label at 2026-03-06T11:00:19: 皇甫 LGTM

Comment

#44702 by kouho at 2026-03-06T11:00:31; updated 2026-03-06T11:00:31

@wada
対応ありがとうございます。確認OKです。

Comment

#44705 by wada at 2026-03-06T12:15:39; updated 2026-03-06T12:15:39

@kouho
タブの挙動がおかしくなっていたので修正しました。

  • シーズンパスタブから週間パスタブに移動できない、バッジがつかない
  • 週間パスタブからログインボーナスタブに受け取り済みでもバッジがつく
  • 購入回数制限後に週間パスタブに遷移できる
kouho delete_label at 2026-03-06T13:09:13: 皇甫 LGTM

Comment

#44707 by wada at 2026-03-06T14:06:39; updated 2026-03-06T14:06:39

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

Comment

#44709 by wada at 2026-03-06T14:20:40; updated 2026-03-06T14:20:40

@kouho
修正しました。

Comment

#44710 by kouho at 2026-03-06T14:24:45; updated 2026-03-06T14:24:45

@wada
対応ありがとうございます。確認OKです

kouho add_label at 2026-03-06T14:24:50: 皇甫 LGTM
kouho delete_label at 2026-03-06T14:51:53: コメントあり
wada merge at 2026-03-06T16:45:47: 【3/6】新人応援パス2回目画像対応
wada close at 2026-03-06T16:45:47: Close
wada delete_branch at 2026-03-06T16:45:53: monthly_pass_2

commit comment

#336 by kouho at 2026-03-06T13:48:33; lib/class/controller/JrControllerSeasonPass.class.php / old:None new:369
f44293ae1d5e90f595c7eae611416908612267b0

@wada
ここの判定は逆ではないでしょうか?

commit comment

#337 by kouho at 2026-03-06T13:56:41; lib/public/Quest-TeamWarDetail.inc / old:None new:48
2de5ab77abf43155897167edc2817d56bca05b06

@wada
同じ対応をQuest-LoginBonus.incにも入れたほうがいいと思います
$pass = S_LoadModelClass('Monthly_pass')->get_template_data($user_info['user_no']);
$_temp = array_merge($_temp, $pass);

commit comment

#338 by kouho at 2026-03-06T13:57:40; public_html/template/Quest-LoginBonus.tpl / old:None new:389
f44293ae1d5e90f595c7eae611416908612267b0

@wada
参照が$_temp['pass']['buy_limit_flg'] が正しいように見えます。

commit comment

#339 by kouho at 2026-03-06T13:58:06; public_html/template/Quest-TeamWarDetail.tpl / old:None new:125
f44293ae1d5e90f595c7eae611416908612267b0

@wada
参照が$_temp['pass']['buy_limit_flg'] が正しいように見えます。

commit comment

#340 by kouho at 2026-03-06T14:10:40; lib/models/Monthly_pass.php / old:None new:108
f44293ae1d5e90f595c7eae611416908612267b0

@wada
バナー画像の表示仕様にもよりますが、is_buy_count_limit関数と判定を統一するなら、buy_limit_countのemptyの判定を入れていた他だけませんか?
if( !empty($_df['monthly_pass']['buy_limit_count'] ) && $buy_count >= $_df['monthly_pass']['buy_limit_count']) {

Commit range

basefe7f5d83fb8170f25052d2a13bb4c160cf73c3f4
head2de5ab77abf43155897167edc2817d56bca05b06
base branchjolly/jr-web:master
head branchjolly/jr-web:monthly_pass_2

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/class/controller/JrControllerSeasonPass.class.php
modifiedlib/models/Monthly_pass.php
modifiedlib/public/Quest-MonthlyPass.inc
modifiedlib/public/Quest-TeamWarDetail.inc
addedpublic_html/img/monthlypass/pass_banner_100.png
addedpublic_html/img/monthlypass/pass_banner_100_first.png
addedpublic_html/img/shop/itembanner_86_100.jpg
modifiedpublic_html/template/Quest-LoginBonus.tpl
modifiedpublic_html/template/Quest-MonthlyPass.tpl
modifiedpublic_html/template/Quest-TeamWarDetail.tpl
modifiedpublic_html/template/SeasonPass-Home.tpl
modifiedpublic_html/template/Shop.tpl
modifiedpublic_html/template/parts_reward_tabs.tpl
Patch body is stored as a separate file to keep this page lightweight.

Review comment

#336 by kouho at 2026-03-06T13:48:33; lib/class/controller/JrControllerSeasonPass.class.php / old:None new:369
f44293ae1d5e90f595c7eae611416908612267b0

@wada
ここの判定は逆ではないでしょうか?

Review comment

#337 by kouho at 2026-03-06T13:56:41; lib/public/Quest-TeamWarDetail.inc / old:None new:48
2de5ab77abf43155897167edc2817d56bca05b06

@wada
同じ対応をQuest-LoginBonus.incにも入れたほうがいいと思います
$pass = S_LoadModelClass('Monthly_pass')->get_template_data($user_info['user_no']);
$_temp = array_merge($_temp, $pass);

Review comment

#338 by kouho at 2026-03-06T13:57:40; public_html/template/Quest-LoginBonus.tpl / old:None new:389
f44293ae1d5e90f595c7eae611416908612267b0

@wada
参照が$_temp['pass']['buy_limit_flg'] が正しいように見えます。

Review comment

#339 by kouho at 2026-03-06T13:58:06; public_html/template/Quest-TeamWarDetail.tpl / old:None new:125
f44293ae1d5e90f595c7eae611416908612267b0

@wada
参照が$_temp['pass']['buy_limit_flg'] が正しいように見えます。

Review comment

#340 by kouho at 2026-03-06T14:10:40; lib/models/Monthly_pass.php / old:None new:108
f44293ae1d5e90f595c7eae611416908612267b0

@wada
バナー画像の表示仕様にもよりますが、is_buy_count_limit関数と判定を統一するなら、buy_limit_countのemptyの判定を入れていた他だけませんか?
if( !empty($_df['monthly_pass']['buy_limit_count'] ) && $buy_count >= $_df['monthly_pass']['buy_limit_count']) {