PR

【3/28】新PFの表示切り分け(告知・ガチャ) #3979

wada opened this at 2025-03-24T10:09:33; closed

Description

opened by wada at 2025-03-24T10:09:33; updated 2025-04-04T15:23:49

概要

PFで表示を切り分ける
※このPRでは告知とガチャの既存切り分けが対象。以降のPRで残り対応予定
※platform_typeを使用せず取得している箇所(lib/model/dao/master_gachaなど)は別PRで対応
既存カラムのplatform_typeを使用

スクリプト、管理画面側は別途対応

platform_type設定

対象テーブル(カラムは既設定)

official_info
official_info_event
master_gacha
master_item_gacha
master_skill_ball_gacha
master_monthly_skillball_gacha

対象テーブルのplatform_type設定

1: colopl(未使用)
2: dgame(未使用)
3: 全PF
4: yamada
5: gesoten

既存設定、今後の運用に影響が出ないように、
現在設定している3を全PF、4以降を各PFのtypeとして使用

更新内容

sqlクエリのplatform_type指定をGlobalConfigメソッド使用
告知取得処理は、sqlクエリのplatform_type指定を削除していたため追加
酒場のセットアップ、転生カードガチャ取得をplatform_type指定に変更

その他更新

TOPページ、イベントページで掲載されていないガチャバナーがあったため掲載対応

テスト方法、確認した内容

各PF(yamada, gesoten)で、対象テーブルのplatform_type変更によって表示内容が変わるか
PF切り替え方法は別PR参照(【3/21】新PFの判定追加 #3972)

Comment

#39578 by wada at 2025-03-25T09:51:02; updated 2025-03-25T09:51:02

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

Comment

#39694 by kouho at 2025-03-28T15:03:26; updated 2025-03-28T15:03:26

@wada

  • 以下の箇所に既存のプラットフォーム判定が残っておりましたので確認をお願いいたします。
  • public/Bar-SPGachaCheck.inc: if( $_df['PLATFORM'] == 'd-game' && $_temp['free_flg'] == 0 && !($_temp['gacha_ticket'] && $_temp['type'] == 1 && $_temp['ticket_flg']) ){
  • public/Bar-GachaExecFree.inc: if($_df['PLATFORM'] == 'yamada')
  • public/Bar-SetupGachaExec.inc: if( $_df['PLATFORM'] != 'd-game' ){
  • public/Bar-GachaExec.inc: if( $_df['PLATFORM'] != 'd-game' || !isset($_REQUEST['pid']) ){
  • models/Bar_model.php: if($_df['PLATFORM'] == 'd-game'){

Bar-SpecialGachaは本家マージで使用しないと思いますがどうしましょう?

  • public/Bar-SpecialGachaResult.inc:
  • public/Bar-SpecialGacha.inc
  • public/Bar-SpecialGachaCheck.inc
  • public/Bar-SpecialGachaExec.inc

不明

  • public/Bar-SelectGoldResult.inc
kouho add_label at 2025-03-28T15:03:30: コメントあり

Comment

#39708 by wada at 2025-03-28T16:20:32; updated 2025-04-03T17:45:43

@kouho
> 以下の箇所に既存のプラットフォーム判定が残っておりましたので確認をお願いいたします。

今回のPF切り替えには影響のない範囲なので対応していません。

> Bar-SpecialGachaは本家マージで使用しないと思いますがどうしましょう?

今回は対応しません。
・yamada対応自体入っていない
・巨獣ガチャや転生カードガチャのように、ヤマダゲームでリリースする際に対応

> 不明

そうですね?

kouho add_label at 2025-03-31T10:28:12: 皇甫 LGTM
kouho delete_label at 2025-03-31T10:28:49: 皇甫 LGTM
kouho add_label at 2025-04-03T17:46:53: 皇甫 LGTM

Comment

#39806 by kouho at 2025-04-03T17:47:04; updated 2025-04-03T17:47:04

@wada
了解です。確認OKです

wada merge at 2025-04-04T15:23:36: 【3/28】新PFの表示切り分け(告知・ガチャ)
wada close at 2025-04-04T15:23:36: Close
wada delete_branch at 2025-04-04T15:23:49: new_platform_2

Commit range

basedc20875324348361f697977a0b06e0de4ffc06ed
heada08699bd2de80b2742778341eb3992ce4a443931
base branchjolly/jr-web:master
head branchjolly/jr-web:new_platform_2

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed22 filesView patch
statusfile
modifiedlib/SubFunc.inc
modifiedlib/ajax/default_info.inc
modifiedlib/ajax/default_list.inc
modifiedlib/conf/class/GlobalConfig.class.php
modifiedlib/models/Gacha_skillball.php
modifiedlib/models/dao/master_gacha.php
modifiedlib/models/dao/master_item_gacha.php
modifiedlib/models/dao/master_skill_ball_gacha.php
modifiedlib/public/Bar-Default.inc
modifiedlib/public/Default-TypeList.inc
modifiedlib/public/Default.inc
modifiedlib/public/Event.inc
modifiedlib/public/Main.inc
modifiedlib/public/Raid-Ranking.inc
modifiedlib/public/Ranking-DailyEventTeam.inc
modifiedlib/public/Ranking-Event.inc
modifiedlib/public/Ranking-EventIsland.inc
modifiedlib/public/Ranking-EventLeague.inc
modifiedpublic_html/template/Bar-Default.tpl
modifiedpublic_html/template/Default.tpl
modifiedpublic_html/template/Event.tpl
modifiedpublic_html/template/Main.tpl
Patch body is stored as a separate file to keep this page lightweight.