PR

【6/16】ステップアップガチャ「ギョクリン」 #869

iwamoto opened this at 2021-06-09T19:53:08; closed

Description

opened by iwamoto at 2021-06-09T19:53:08; updated 2024-10-09T10:14:28

【6/16】ステップアップガチャ「ギョクリン」 #815

web : ``gacha_0616``
db : ``gacha_0616``

覚醒ボーナス+20変更のため、パラメータとボーナス設定

マスタ更新

・master_gacha
・master_gacha_item
・master_card
・master_card_skill
・master_composition_bonus

テストスクリプト
localで


今回から追加&変更
・ステップアップガチャのページを本家仕様へ
・合わせてラインナップページも本家仕様へ

*反映した場合、すでに開催中ガチャの一部画像を修正する必要があります
iwamoto change_title at 2021-06-10T13:55:58: [WIP]【6/16】ステップアップガチャ「ギョクリン」 【6/16】ステップアップガチャ「ギョクリン」

Comment

#11450 by iwamoto at 2021-06-10T14:08:22; updated 2021-06-10T14:08:22

@wada @kouho
こちらレビューをお願い致します。

Comment

#11451 by wada at 2021-06-10T14:34:35; updated 2021-06-10T14:34:35

@iwamoto
ガチャページのギョクリン画像は修正待ち?

<img src="https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/16233032506271GwFXlkQUc" width="300">

iwamoto add_label at 2021-06-10T15:15:43: レビュー待ち

Comment

#11453 by wada at 2021-06-10T15:19:29; updated 2021-06-10T15:19:29

スキル詳細のLegendの文字色が黄色になっているので、ガチャページと統一して欲しいです。
ボタンがこちらの環境だと重なっちゃいます。

<img src="https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/1623305481863urRkvRm0sM" width="300">

Comment

#11454 by iwamoto at 2021-06-10T15:23:48; updated 2021-06-10T15:24:12

@wada
あー、画像を戻し忘れています、修正依頼はまだです。

合わせて素戔嗚尊とヒメカのほうも一応依頼しておきます

Comment

#11455 by wada at 2021-06-10T15:32:21; updated 2021-06-10T15:32:21

@iwamoto
了解ですー

ラインナップのタブ数をgacha_styleで判定しているので、アイテムが入っていない復刻ガチャ(gacha_style=3)の場合、リストが表示されませんでした。

if(in_array($_temp['gacha_master'][1]['gacha_style'],array(2,3,4))){
	//復刻ガチャならアイテムあり
	$_parts['tab_4'] = <<<END_OF_HTML

Comment

#11456 by kouho at 2021-06-10T17:07:04; updated 2021-06-10T17:07:04

@iwamoto
動作面では和田さんが指摘している箇所以外大丈夫だと思いますが、
Notice Errorが大量なので可能なら修正していただけましたら幸いです。

PHP   1. {main}() /home/web/jr-web/public_html/index.php:0
PHP   2. require() /home/web/jr-web/public_html/index.php:533
PHP Notice:  Undefined offset: 240 in /home/web/jr-web/lib/public/Bar-Gacha.inc on line 309
PHP Stack trace:
PHP   1. {main}() /home/web/jr-web/public_html/index.php:0
PHP   2. require() /home/web/jr-web/public_html/index.php:533
PHP Notice:  Undefined offset: 4 in /home/web/jr-web/lib/public/Bar-Gacha.inc on line 310
PHP Stack trace:
 PHP   1. {main}() /home/web/jr-web/public_html/index.php:0
 PHP   2. require() /home/web/jr-web/public_html/index.php:533
 PHP Notice:  Undefined variable: merge_after_card_no in /home/web/jr-web/lib/public/Bar-Gacha.inc on line 477
 PHP Stack trace:
 PHP   1. {main}() /home/web/jr-web/public_html/index.php:0
 PHP   2. require() /home/web/jr-web/public_html/index.php:533
 PHP Notice:  Undefined variable: limit_break_after_card_no in /home/web/jr-web/lib/public/Bar-Gacha.inc on line 495
 PHP Stack trace:
 PHP   1. {main}() /home/web/jr-web/public_html/index.php:0
 PHP   2. require() /home/web/jr-web/public_html/index.php:533
 PHP Notice:  Trying to access array offset on value of type null in /home/web/jr-web/lib/public/Bar-Gacha.inc on line 495
 PHP Stack trace:
 PHP   1. {main}() /home/web/jr-web/public_html/index.php:0
 PHP   2. require() /home/web/jr-web/public_html/index.php:533
 PHP Notice:  Undefined index:  in /home/web/jr-web/lib/public/Bar-Gacha.inc on line 495
 PHP Stack trace:
 PHP   1. {main}() /home/web/jr-web/public_html/index.php:0
 PHP   2. require() /home/web/jr-web/public_html/index.php:533
 PHP Notice:  Undefined variable: merge_after_card_no in /home/web/jr-web/lib/public/Bar-Gacha.inc on line 477
 PHP Stack trace:
 PHP   1. {main}() /home/web/jr-web/public_html/index.php:0
 PHP   2. require() /home/web/jr-web/public_html/index.php:533
 PHP Notice:  Undefined variable: limit_break_after_card_no in /home/web/jr-web/lib/public/Bar-Gacha.inc on line 491
PHP   1. {main}() /home/web/jr-web/public_html/index.php:0
PHP   2. require() /home/web/jr-web/public_html/index.php:533
PHP Notice:  Undefined offset: 22 in /home/web/jr-web/lib/public/Bar-Gacha.inc on line 144
PHP Stack trace:
PHP   1. {main}()

Comment

#11462 by iwamoto at 2021-06-11T09:02:45; updated 2021-06-11T09:06:04

@wada
ギョクリン他画像も頂き反映しました。
スキル詳細のLegendの文字色&ボタンの重なりを修正しました。
ラインナップのリスト表示も修正しました。

@kouho
ガチャ関連のnotice_error対応しました

Comment

#11466 by wada at 2021-06-11T11:08:39; updated 2021-06-11T11:08:39

@iwamoto
確認しました。OKです。

wada add_label at 2021-06-11T11:08:44: 和田 LGTM
kouho commit at 2021-06-11T11:10:54: 酒場notice error対応 #869 b5cb3aadeb5f6e63fc363a472d9b62c7058f6bee
kouho add_label at 2021-06-11T11:16:14: 皇甫 LGTM

Comment

#11470 by kouho at 2021-06-11T11:16:28; updated 2021-06-11T11:16:28

@iwamoto
確認しました。OKかと思います。

iwamoto merge at 2021-06-14T11:02:34: 【6/16】ステップアップガチャ「ギョクリン」
iwamoto close at 2021-06-14T11:02:34: Close
iwamoto commit at 2021-06-16T10:20:01: 酒場notice error対応 #869 bbb8d1f1a1dd223e75b6be69935e3d41b59d0241

Commit range

base8717d0b6d53308835acd1287cea66a7f2a5de62f
heade4457c89b313026ff350d4496cb9d5a32b612cfd
base branchjolly/jr-web:develop
head branchjolly/jr-web:gacha_0616

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed51 filesView patch
statusfile
modifiedlib/SubFunc.inc
modifiedlib/conf/class/GachaConfig.class.php
modifiedlib/conf/define_y.inc
modifiedlib/public/Bar-Default.inc
modifiedlib/public/Bar-Gacha.inc
modifiedlib/public/Bar-GachaExec.inc
modifiedlib/public/Bar-GachaList.inc
modifiedlib/public/Bar-GachaResult.inc
modifiedlib/public/Bar-SPGacha.inc
modifiedpublic_html/img/gacha/139/gacha139_info.jpg
modifiedpublic_html/img/gacha/139/gacha139_info2.jpg
modifiedpublic_html/img/gacha/139/gacha139_info3.jpg
modifiedpublic_html/img/gacha/139/gacha139_info4.jpg
modifiedpublic_html/img/gacha/143/banner_gacha143.jpg
modifiedpublic_html/img/gacha/143/banner_gacha143_2.jpg
addedpublic_html/img/gacha/143/composition_bonus.jpg
modifiedpublic_html/img/gacha/143/gacha143.jpg
modifiedpublic_html/img/gacha/143/gacha143_2.jpg
modifiedpublic_html/img/gacha/143/gacha143_card_info240.jpg
modifiedpublic_html/img/gacha/143/gacha143_card_info281.jpg
modifiedpublic_html/img/gacha/143/gacha143_card_info312.jpg
modifiedpublic_html/img/gacha/143/gacha143_card_info366.jpg
modifiedpublic_html/img/gacha/143/gacha143_info.jpg
modifiedpublic_html/img/gacha/143/gacha143_info2.jpg
modifiedpublic_html/img/gacha/143/gacha143_info3.jpg
modifiedpublic_html/img/gacha/143/gacha143_info4.jpg
modifiedpublic_html/img/gacha/143/gacha143_title.jpg
addedpublic_html/img/gacha/143/gacha143_title2.jpg
modifiedpublic_html/img/gacha/147/gacha147_info.jpg
modifiedpublic_html/img/gacha/147/gacha147_info2.jpg
modifiedpublic_html/img/gacha/147/gacha147_info3.jpg
modifiedpublic_html/img/gacha/147/gacha147_info4.jpg
addedpublic_html/img/info_jrye/CharmInfo/143/info2-2.jpg
addedpublic_html/img/info_jrye/CharmInfo/143/info2.jpg
addedpublic_html/img/info_jrye/CharmInfo/143/info3-2.jpg
addedpublic_html/img/info_jrye/CharmInfo/143/info3.jpg
addedpublic_html/img/info_jrye/CharmInfo/143/info4-2.jpg
addedpublic_html/img/info_jrye/CharmInfo/143/info4.jpg
addedpublic_html/img/info_jrye/CharmInfo/143/info5-2.jpg
addedpublic_html/img/info_jrye/CharmInfo/143/info5.jpg
modifiedpublic_html/js/JOLLYROGER.js
modifiedpublic_html/js/jr_main.min.js
modifiedpublic_html/template/Bar-Default.tpl
modifiedpublic_html/template/Bar-GachaExec.tpl
modifiedpublic_html/template/Bar-GachaList.tpl
modifiedpublic_html/template/Bar-GachaRatioReprint.tpl
modifiedpublic_html/template/Bar-GachaResult.tpl
modifiedpublic_html/template/Bar-Gacha_old.tpl
modifiedpublic_html/template/Bar-SPGachaList.tpl
modifiedpublic_html/template/Bar-SPGachaResult.tpl
modifiedpublic_html/template/Bar-SPGacha_117.tpl
Patch body is stored as a separate file to keep this page lightweight.