PR

【3/14】PF判定用の設定追加 #3959

wada opened this at 2025-03-10T10:13:25; closed

Description

opened by wada at 2025-03-10T10:13:25; updated 2025-03-14T15:23:11

概要

PF判定の設定をなるべく既存の設定のまま定義
(platform_typeは3を全体のままにして、4以降PFを追加していく)
各機能のPF切り分け実装時に使用

更新内容

  • PlatformConst

PFを判定する文字列、テーブルのtypeを定数として設定

  • GlobalConfig

PF定数はPlatformConstを使用(クラス内の設定は完全に移行してから削除)
現PFのテーブルtypeを取得するメソッド追加(例:yamada_types = [3,4])

使用例

#旧
case 'yamada': return 2020;
new PlatformType(3, '両方'),
#新
case PlatformConst::PLATFORM_YAMADA: return 2020;
new PlatformType(PlatformConst::PLATFORM_TYPE_ALL, '両方'),
#旧
if($_df['PLATFORM'] == 'yamada'){
#新
if(GlobalConfig::get()->isYamada()){

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

設定方法に問題がないか
他に良い方法はないか
運用中のシステムに影響はないか

Comment

#39358 by wada at 2025-03-10T10:23:02; updated 2025-03-10T10:23:02

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

kouho add_label at 2025-03-13T15:58:19: 皇甫 LGTM

Comment

#39435 by kouho at 2025-03-13T15:58:48; updated 2025-03-13T15:58:48

@wada
確認しました。問題ないと思います。

wada merge at 2025-03-14T15:23:07: 【3/14】PF判定用の設定追加
wada close at 2025-03-14T15:23:07: Close
wada delete_branch at 2025-03-14T15:23:11: new_platform

Commit range

baseec000828217d2c40547eeffe9a1279c417675e9d
head80ffc6c2c191c401236c1587127409a44fba52a6
base branchjolly/jr-web:master
head branchjolly/jr-web:new_platform

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed2 filesView patch
statusfile
addedlib/class/const/PlatformConst.class.php
modifiedlib/conf/class/GlobalConfig.class.php
Patch body is stored as a separate file to keep this page lightweight.