PR

復刻ガチャ #438

wada opened this at 2020-11-18T15:27:19; closed

Description

opened by wada at 2020-11-18T15:27:19; updated 2023-08-28T15:13:20

issue #409

Comment

#8343 by wada at 2020-11-18T15:30:48; updated 2020-11-18T15:30:48

@iwamoto
レビューお願いします。
変更箇所についてはissue、仕様書を確認お願いします。
特に、過去購入者のstep1リセット、step2-4リセットボーナスを重点的に

他に以下の確認お願いします。
・「復刻一の市」表記がないか
・注意事項に違和感がないか(実装されていない機能の表記がないか
・「ボックスガチャ」表記が残っていないか

Comment

#8344 by wada at 2020-11-18T15:33:49; updated 2020-11-18T15:33:49

マスタは以下の更新が必要です。
・master_gacha
・master_gacha_item

Comment

#8346 by iwamoto at 2020-11-18T19:16:18; updated 2020-11-18T19:16:18

あ、今回の復刻ガチャは gacha_style = 3 でしょうか?

master_gacha更新しましたが、対象のロロノアガチャはgacha_style = 2でしたので、
上記のエラーが出ていたと思われます…

Comment

#8349 by iwamoto at 2020-11-18T21:03:13; updated 2020-11-18T21:03:13

@wada
こちら3点ご確認お願い致します。

#過去購入者ではありません
1.STEP1&STEP5のリセット時に、「伝説の海賊の魂」を入手した旨のメッセージがでた(実際には配布されていない)
2.リセット可能時に、ガチャのリザルトで表示されるSTEP数がおかしい

3.各リセット毎の全カード枚数が画像と異なります(master_gacha_item.tb反映済み) 画像が正?

こちらは問題ないと思います。
*注意事項にある、「☆ボックスガチャとベルガチャのラインナップは全く異なります。」こちらに問題なければです

・「復刻一の市」表記がないか
・注意事項に違和感がないか(実装されていない機能の表記がないか
・「ボックスガチャ」表記が残っていないか

Comment

#8350 by wada at 2020-11-18T23:23:01; updated 2020-11-18T23:23:01

@iwamoto
確認ありがとうございます。

1,2のStep表示について修正しました。
3は仕様確認するので、レビューでは無視してもらって良いです。

注意事項のボックスガチャは復刻ガチャに修正しました。

Comment

#8351 by iwamoto at 2020-11-19T00:14:28; updated 2020-11-19T00:14:28

@wada
こちら修正分も確認しました。
大丈夫かと思います。

Comment

#8352 by wada at 2020-11-19T10:55:50; updated 2020-11-19T10:55:50

TOPのバナー表示と酒場のバナー位置調整してマージします。

wada merge at 2020-11-19T11:40:37: 復刻ガチャ
wada close at 2020-11-19T11:40:37: Close

commit comment

#26 by iwamoto at 2020-11-18T19:05:56; public_html/template/Bar-GachaReprint2.tpl / old:343 new:None
643148d6bfe7737dabb76e727540ab36e7bac18b

ここから育成講座の対応がありますが、動作しましたでしょうか?

$_temp['lecture_list'] = null; 

になっていることで発生しているようです。(おそらく)

#ガチャ酒場から復刻ガチャへ移動しようとしたときにエラー
2020-11-18 18:28:26.970934: {"ip":"172.18.0.1","uri":"/?M=Bar&A=Gacha&gacha_id=122","ua":"Mozilla/5.0 (Linux; Android 6.0.1; Moto G (4)) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/86.0.4240.193 Mobile Safari/537.36","req":{"M":"Bar","A":"Gacha","gacha_id":"122"},"code":0,"func":"{closure}","trace":"Bar-GachaReprint(515)/Bar-GachaReprint(515)/Bar-Gacha(542)/index(521)","msg":"Use of undefined constant bring_chair - assumed 'bring_chair' (this will throw an Error in a future version of PHP)"}
#Bar-Gacha.inc
379line  $_temp['lecture_list'] = S_GetLectureCardList();

私の環境下では
S_GetLectureCardList() -> return = null
になっています 

他に更新が必要でしたでしょうか?

Commit range

base17623baec28b3ba3053e6d4361ca0e5e95d58cd4
head643148d6bfe7737dabb76e727540ab36e7bac18b
base branchjolly/jr-web:develop
head branchjolly/jr-web:gacha_409

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed36 filesView patch
statusfile
modifiedlib/models/Gacha_payment.php
addedlib/models/gacha/Reprint_step.php
modifiedlib/public/Bar-Gacha.inc
modifiedlib/public/Bar-GachaInfo.inc
modifiedlib/public/Bar-GachaReset.inc
modifiedlib/public/Bar-GachaResetResult.inc
modifiedlib/public/Bar-GachaResult.inc
modifiedpublic_html/img/gacha/122/banner_gacha122.jpg
modifiedpublic_html/img/gacha/122/banner_gacha122_2.jpg
deletedpublic_html/img/gacha/122/banner_gacha122_reprint.jpg
modifiedpublic_html/img/gacha/122/gacha122.jpg
modifiedpublic_html/img/gacha/122/gacha122_2.jpg
modifiedpublic_html/img/gacha/122/gacha122_card_info18.jpg
modifiedpublic_html/img/gacha/122/gacha122_card_info23.jpg
modifiedpublic_html/img/gacha/122/gacha122_card_info71.jpg
modifiedpublic_html/img/gacha/122/gacha122_card_info93.jpg
modifiedpublic_html/img/gacha/122/gacha122_info.jpg
modifiedpublic_html/img/gacha/122/gacha122_info2.jpg
modifiedpublic_html/img/gacha/122/gacha122_info3.jpg
addedpublic_html/img/gacha/122/gacha122_info4.jpg
modifiedpublic_html/img/gacha/122/gacha122_title.jpg
addedpublic_html/img/gacha/122/gacha122_title2.jpg
addedpublic_html/img/gacha/stepboxinfo/122/info1.jpg
addedpublic_html/img/gacha/stepboxinfo/122/info2.jpg
addedpublic_html/img/gacha/stepboxinfo/122/info3.jpg
addedpublic_html/img/gacha/stepboxinfo/122/info4.jpg
addedpublic_html/img/gacha/stepboxinfo/122/info5.jpg
addedpublic_html/img/gacha/stepboxinfo/122/info6.jpg
modifiedpublic_html/img/gacha/stepboxinfo_info2.jpg
addedpublic_html/img/gacha/stepboxinfo_info2_2.jpg
modifiedpublic_html/template/Bar-Default.tpl
modifiedpublic_html/template/Bar-GachaInfo.tpl
modifiedpublic_html/template/Bar-GachaReprint2.tpl
modifiedpublic_html/template/Bar-GachaResetResult.tpl
modifiedpublic_html/template/Bar-GachaResultReprint2.tpl
modifiedpublic_html/template/Default.tpl
Patch body is stored as a separate file to keep this page lightweight.

Review comment

#26 by iwamoto at 2020-11-18T19:05:56; public_html/template/Bar-GachaReprint2.tpl / old:343 new:None
643148d6bfe7737dabb76e727540ab36e7bac18b

ここから育成講座の対応がありますが、動作しましたでしょうか?

$_temp['lecture_list'] = null; 

になっていることで発生しているようです。(おそらく)

#ガチャ酒場から復刻ガチャへ移動しようとしたときにエラー
2020-11-18 18:28:26.970934: {"ip":"172.18.0.1","uri":"/?M=Bar&A=Gacha&gacha_id=122","ua":"Mozilla/5.0 (Linux; Android 6.0.1; Moto G (4)) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/86.0.4240.193 Mobile Safari/537.36","req":{"M":"Bar","A":"Gacha","gacha_id":"122"},"code":0,"func":"{closure}","trace":"Bar-GachaReprint(515)/Bar-GachaReprint(515)/Bar-Gacha(542)/index(521)","msg":"Use of undefined constant bring_chair - assumed 'bring_chair' (this will throw an Error in a future version of PHP)"}
#Bar-Gacha.inc
379line  $_temp['lecture_list'] = S_GetLectureCardList();

私の環境下では
S_GetLectureCardList() -> return = null
になっています 

他に更新が必要でしたでしょうか?