PR

【3/5】島テーマ(SP効果個別倍率対応) #3938

kouho opened this at 2025-02-27T18:51:25; closed

Description

opened by kouho at 2025-02-27T18:51:25; updated 2025-03-07T17:53:56

概要

https://docs.google.com/spreadsheets/d/1P7hJzFAMugrLuT52Oc_rkEaksWh4SYmTt34qFVG2Er8/edit?gid=2120632986#gid=2120632986

備考

  • SP効果 個別倍率変更に対応

$_df['event_build_early_bonus_object']['sp_effect']が1より大きい場合に個別設定と判定

  • Main.tplのSP効果周りをループ処理でまとめる
  • 画像PRは別途作成

TODO

  • Main.tplのエメラルドのアクセ直書き部分

Comment

#39156 by kouho at 2025-02-27T18:53:58; updated 2025-02-27T18:53:58

@wada
レビューお願いいたします

kouho change_title at 2025-02-27T18:54:56: 【3/5】島テーマ 【3/5】島テーマ(SP効果個別倍率対応)

Comment

#39169 by wada at 2025-02-28T11:25:25; updated 2025-02-28T11:25:25

@kouho
建築ページの集金情報で追加個数が1個になっています。

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

Comment

#39171 by wada at 2025-02-28T11:29:18; updated 2025-02-28T11:29:18

@kouho
集金ページにはアクセ3倍の記載がありませんでした。

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

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

Comment

#39176 by kouho at 2025-02-28T14:56:28; updated 2025-02-28T14:59:35

@wada
以下修正いたしました。
また合わせて以下の対応も入れております。
> 建築ページの集金情報で追加個数が1個になっています。
> 集金ページにはアクセ3倍の記載がありませんでした。

  • 集金ページと集金情報ページのSP効果表示周りのデザインを統合(集金ページをベース)
  • 黄さんからの要望で島テーマイベントお知らせへのリンクは?ボタンから飛べるように修正
  • 集金できる建物しかない時にもSP効果を表示
  • Main.tpl等へのエメラルドのアクセ直書きをBuildEventConfigに移動

Comment

#39177 by kouho at 2025-02-28T15:13:03; updated 2025-02-28T15:13:03

@wada
こちらが不要という話でしたので削除しました。
> 黄さんからの要望で島テーマイベントお知らせへのリンクは?ボタンから飛べるように修正

Comment

#39182 by kouho at 2025-02-28T16:43:16; updated 2025-02-28T16:43:16

@wada
修正しました
> 名前の後ろに空白が入っています。

変数名を修正しました
> 少し違和感があるので、問題がなければ下の変数と合わせるようにsp_build_nameにしたいです。

jsの更新をなくしましたので不要な修正でした。
> この変数はどこで使用されていますか?
> 今回キャッシュ対応が必要ですか?

Comment

#39183 by kouho at 2025-03-03T11:02:56; updated 2025-03-03T11:02:56

@wada
bonus_ratioをsp_effectにいれる形に修正しました
> 1~3はbonus_ratioと同じなので、1を設定しなくても大丈夫ですか?
> $value>1だとsp_effectをbonus_ratioと同じ値の場合に1としたときに、SP効果だけ3倍表示になり、ユーザを誤認させる要素になりえます。

Comment

#39185 by wada at 2025-03-03T12:10:19; updated 2025-03-03T12:10:19

@kouho
Subfunc.incS_GetUserBuildGoldDetail()でsp_effectの不等号の比較値がタイプによって異なります。

#1は > 1
elseif($_df['event_build_early_bonus_object']['sp_effect'][1] > 1){
#3は > 0
elseif($_df['event_build_early_bonus_object']['sp_effect'][3] > 0){

Comment

#39188 by kouho at 2025-03-03T14:20:24; updated 2025-03-03T14:20:24

@wada
修正しました。
> 倍率0の時($_df['event_build_early_bonus_object']['sp_effect'][4] = 0)は表示されないように対応をお願いします。

$_df['event_build_early_bonus_object']['sp_effect'][3] > 0が誤っておりましたので修正しました
> Subfunc.incのS_GetUserBuildGoldDetail()でsp_effectの不等号の比較値がタイプによって異なります。

Comment

#39243 by wada at 2025-03-04T16:10:12; updated 2025-03-04T16:10:12

@kouho
確認OKです。

wada add_label at 2025-03-04T16:10:34: 和田 LGTM
kouho merge at 2025-03-07T10:30:13: 【3/5】島テーマ(SP効果個別倍率対応)
kouho close at 2025-03-07T10:30:13: Close
kouho commit at 2025-03-07T17:53:56: Revert "Merge pull request #3938 from jolly/event_build_10075" This reverts commit ec000828217d2c40547eeffe9a1279c417675e9d, reversing changes made to 30d4b9005fd6e31a27203843e4f3a7bc339fbd21. 11303bbdefca1cfd4cc18f097ae59e5f10cce605

commit comment

#277 by wada at 2025-02-28T16:11:43; lib/ajax/main_listGold.inc / old:None new:20
05fc6a202be061fce96e8c9dfffe41675c5419a5

@kouho
この変数はどこで使用されていますか?

commit comment

#278 by wada at 2025-02-28T16:12:54; lib/conf/define_y.inc / old:677 new:None
05fc6a202be061fce96e8c9dfffe41675c5419a5

@kou
1~3はbonus_ratioと同じなので、1を設定しなくても大丈夫ですか?

commit comment

#279 by wada at 2025-02-28T16:17:08; lib/conf/class/BuildEventConfig.class.php / old:None new:22
ced1c374de5ac8cb37ef5cfae6da5f7d61c68533

@kouho
名前の後ろに空白が入っています。

commit comment

#280 by wada at 2025-02-28T16:18:27; lib/conf/define_y.inc / old:None new:890
ced1c374de5ac8cb37ef5cfae6da5f7d61c68533

@kouho
少し違和感があるので、問題がなければ下の変数と合わせるようにsp_build_nameにしたいです。

commit comment

#281 by wada at 2025-02-28T16:30:18; public_html/template/Main.tpl / old:2307 new:None
05fc6a202be061fce96e8c9dfffe41675c5419a5

@kouho
$value>1だとsp_effectをbonus_ratioと同じ値の場合に1としたときに、SP効果だけ3倍表示になり、ユーザを誤認させる要素になりえます。

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

commit comment

#282 by wada at 2025-02-28T16:31:20; public_html/template/header_sp.tpl / old:None new:3
ced1c374de5ac8cb37ef5cfae6da5f7d61c68533

@kouho
今回キャッシュ対応が必要ですか?

commit comment

#283 by wada at 2025-03-03T12:01:25; public_html/template/Main.tpl / old:None new:2463
3d3530c1f206ed027ca3da7d81b136f746d5d82f

@kouho
倍率0の時($_df['event_build_early_bonus_object']['sp_effect'][4] = 0)は表示されないように対応をお願いします。

Commit range

base473becec702621db905c48607ef112b7f1e6f434
head05fc6a202be061fce96e8c9dfffe41675c5419a5
base branchjolly/jr-web:master
head branchjolly/jr-web:event_build_10075

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed8 filesView patch
statusfile
modifiedlib/SubFunc.inc
modifiedlib/ajax/main_listGold.inc
modifiedlib/conf/class/BuildEventConfig.class.php
modifiedlib/conf/define_y.inc
modifiedlib/exec/tool_jrce_status.php
modifiedlib/public/Build-Info.inc
modifiedpublic_html/template/Build-Info.tpl
modifiedpublic_html/template/Main.tpl
Patch body is stored as a separate file to keep this page lightweight.

Review comment

#277 by wada at 2025-02-28T16:11:43; lib/ajax/main_listGold.inc / old:None new:20
05fc6a202be061fce96e8c9dfffe41675c5419a5

@kouho
この変数はどこで使用されていますか?

Review comment

#278 by wada at 2025-02-28T16:12:54; lib/conf/define_y.inc / old:677 new:None
05fc6a202be061fce96e8c9dfffe41675c5419a5

@kou
1~3はbonus_ratioと同じなので、1を設定しなくても大丈夫ですか?

Review comment

#279 by wada at 2025-02-28T16:17:08; lib/conf/class/BuildEventConfig.class.php / old:None new:22
ced1c374de5ac8cb37ef5cfae6da5f7d61c68533

@kouho
名前の後ろに空白が入っています。

Review comment

#280 by wada at 2025-02-28T16:18:27; lib/conf/define_y.inc / old:None new:890
ced1c374de5ac8cb37ef5cfae6da5f7d61c68533

@kouho
少し違和感があるので、問題がなければ下の変数と合わせるようにsp_build_nameにしたいです。

Review comment

#281 by wada at 2025-02-28T16:30:18; public_html/template/Main.tpl / old:2307 new:None
05fc6a202be061fce96e8c9dfffe41675c5419a5

@kouho
$value>1だとsp_effectをbonus_ratioと同じ値の場合に1としたときに、SP効果だけ3倍表示になり、ユーザを誤認させる要素になりえます。

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

Review comment

#282 by wada at 2025-02-28T16:31:20; public_html/template/header_sp.tpl / old:None new:3
ced1c374de5ac8cb37ef5cfae6da5f7d61c68533

@kouho
今回キャッシュ対応が必要ですか?

Review comment

#283 by wada at 2025-03-03T12:01:25; public_html/template/Main.tpl / old:None new:2463
3d3530c1f206ed027ca3da7d81b136f746d5d82f

@kouho
倍率0の時($_df['event_build_early_bonus_object']['sp_effect'][4] = 0)は表示されないように対応をお願いします。