Description
概要
2~4回目の価格が300から100に変更になったため、画像表示を修正
ショップのバナーは受け取り中非表示に修正
2~4回目の価格が300から100に変更になったため、画像表示を修正
ショップのバナーは受け取り中非表示に修正
@kouho
レビューお願いします。
@wada
価格の表示不整合が起きているように見えますので確認をお願いいたします。
以下が原因に見えます。
buy_countで参照
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
修正しました。
> 価格の表示不整合が起きているように見えますので確認をお願いいたします。
@wada
対応ありがとうございます。確認OKです。
@kouho
タブの挙動がおかしくなっていたので修正しました。
@kouho
確認ありがとうございます。修正しました。
@kouho
修正しました。
@wada
対応ありがとうございます。確認OKです
@wada
ここの判定は逆ではないでしょうか?
@wada
同じ対応をQuest-LoginBonus.incにも入れたほうがいいと思います
$pass = S_LoadModelClass('Monthly_pass')->get_template_data($user_info['user_no']);
$_temp = array_merge($_temp, $pass);
@wada
参照が$_temp['pass']['buy_limit_flg'] が正しいように見えます。
@wada
参照が$_temp['pass']['buy_limit_flg'] が正しいように見えます。
@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']) {
| base | fe7f5d83fb8170f25052d2a13bb4c160cf73c3f4 |
|---|---|
| head | 2de5ab77abf43155897167edc2817d56bca05b06 |
| base branch | jolly/jr-web:master |
| head branch | jolly/jr-web:monthly_pass_2 |
| status | file |
|---|---|
| modified | lib/class/controller/JrControllerSeasonPass.class.php |
| modified | lib/models/Monthly_pass.php |
| modified | lib/public/Quest-MonthlyPass.inc |
| modified | lib/public/Quest-TeamWarDetail.inc |
| added | public_html/img/monthlypass/pass_banner_100.png |
| added | public_html/img/monthlypass/pass_banner_100_first.png |
| added | public_html/img/shop/itembanner_86_100.jpg |
| modified | public_html/template/Quest-LoginBonus.tpl |
| modified | public_html/template/Quest-MonthlyPass.tpl |
| modified | public_html/template/Quest-TeamWarDetail.tpl |
| modified | public_html/template/SeasonPass-Home.tpl |
| modified | public_html/template/Shop.tpl |
| modified | public_html/template/parts_reward_tabs.tpl |
@wada
ここの判定は逆ではないでしょうか?
@wada
同じ対応をQuest-LoginBonus.incにも入れたほうがいいと思います
$pass = S_LoadModelClass('Monthly_pass')->get_template_data($user_info['user_no']);
$_temp = array_merge($_temp, $pass);
@wada
参照が$_temp['pass']['buy_limit_flg'] が正しいように見えます。
@wada
参照が$_temp['pass']['buy_limit_flg'] が正しいように見えます。
@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']) {