PR

【11/9】SP建物取り壊し時注意文言の修正と判定を修正 #1296

kouho opened this at 2021-11-09T10:21:54; closed

Description

opened by kouho at 2021-11-09T10:21:54; updated 2026-01-14T10:14:24

概要

SP建物取り壊しに関して表示と判定を修正するPRです。

  • 取り壊し時注意文言

この建物は現在取り壊せません。

この建物は〜〜〜まで取り壊せません。 (〜〜はSP効果終了時間)

  • 取り壊し可能かの判定
・コインガチャチケットスタンプorぱんだまをSP効果で集金時に獲得可能でSP効果期間中は取り壊せない
・集金時にパールのアクセを獲得可能で集金間隔が24時間以上の建物の場合はSP効果期間中は取り壊せない

・集金間隔の24時間制限を設けない
・集金時に通常の集金処理以外に別のアイテムが出る建物はSP効果期間中取り壊すことができない。
・島テーマのSP建物は取り壊し可能

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

  • SP期間中に対象のSP建物を取り壊そうとした場合の表示が正しいか
  • 上記の場合に取り壊しができないか

TODO

島テーマと一緒に反映できれば

Comment

#15078 by kouho at 2021-11-09T10:23:30; updated 2021-11-09T10:23:30

@wada @iwamoto
すみません、PR作れていませんでした。
島テーマ確認時にこちらも合わせてご確認ください。

Comment

#15118 by kouho at 2021-11-09T18:09:31; updated 2021-11-09T18:09:31

@wada
確認ありがとうございます。以下修正を行いましたのでご確認よろしくお願いいたします。

  • 倉庫枠拡張対応公開状態に修正
  • 日付けフォーマットの統一
  • build_selectBreak.incの修正

Comment

#15119 by wada at 2021-11-09T18:39:54; updated 2021-11-09T18:39:54

@kouho
修正確認しました。OKです

wada add_label at 2021-11-09T18:39:57: 和田 LGTM

Comment

#15123 by iwamoto at 2021-11-09T19:49:47; updated 2021-11-09T19:49:47

@kouho
上記変数名の他は問題ないと思います。

Comment

#15176 by kouho at 2021-11-11T17:05:19; updated 2021-11-11T17:06:55

@iwamoto
変数名を修正してグローバル変数をそのまま使うように修正しました。

@wada
SP建物の倉庫への収納時のダイアログ表示がわからなかったため確認ページに飛ばして、
確認ページでの表示を修正しました。

Comment

#15179 by wada at 2021-11-11T18:55:44; updated 2021-11-11T18:55:44

@kouho
対応ありがとうございます。

倉庫収納確認ページでSP建物のアイテム表記が変わっているので確認をお願いします。
また、確認ページでSP建物の注意文言を出すと、ユーザーが建物管理ページに戻れなくなるので、戻るためのボタンを追加して欲しいです。
<img src="https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/1636624375856xpppjIOuzt" width="300">

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

Comment

#15182 by kouho at 2021-11-12T10:34:14; updated 2021-11-12T10:34:14

@wada
確認ありがとうございます。指摘点を修正いたしました。
SP建物は例外処理を入れた建物以外はアイテムが×である前提でコードを書き直しています。

Comment

#15183 by wada at 2021-11-12T10:41:59; updated 2021-11-12T10:41:59

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

kouho merge at 2021-11-12T10:46:48: 【11/9】SP建物取り壊し時注意文言の修正と判定を修正
kouho close at 2021-11-12T10:46:48: Close
kouho refer at 2021-11-12T11:27:07: 1303:【11/12】リリース
kouho refer at 2021-11-12T11:29:04: 1304:【11/12】リリース

commit comment

#97 by wada at 2021-11-09T16:51:59; lib/ajax/build_selectBreak.inc / old:91 new:92
9ef8eff66c1b854707cee44ce8135bebf30484c8

@kouho
echoの後にexitが必要です。

commit comment

#98 by wada at 2021-11-09T16:56:50; lib/ajax/build_selectCheck.inc / old:None new:98
9ef8eff66c1b854707cee44ce8135bebf30484c8

@kouho
おそらくbonus_timeが抜けています。$object_name[$user_object['object_no']]['bonus_time']
日付のフォーマットについて、できる限りn/j(k) H:iで表示して欲しいです。

commit comment

#99 by wada at 2021-11-09T17:01:11; public_html/template/Build.tpl / old:None new:222
9ef8eff66c1b854707cee44ce8135bebf30484c8

@kouho
倉庫拡張が巻き戻っているので、公開状態に修正お願いします。

commit comment

#100 by iwamoto at 2021-11-09T19:49:07; lib/ajax/build_selectBreak.inc / old:None new:92
2a0ce1b4adf2ad619ceef3a92541204a9abfa0ff

@kouho
変数名の修正をお願い致します。 $$tmp_date_name;
build_selectCheck.incも同様にお願い致します

Commit range

base2bdb799afba45ed4d5f1e62f06df6dfae7e93dd3
head2a0ce1b4adf2ad619ceef3a92541204a9abfa0ff
base branchjolly/jr-web:develop
head branchjolly/jr-web:sp_build_event_object

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/ajax/build_imgBreak.inc
modifiedlib/ajax/build_selectBreak.inc
modifiedlib/ajax/build_selectCheck.inc
modifiedlib/class/building/BuildingStorageManager.class.php
modifiedlib/class/controller/JrControllerBuildingStorage.class.php
modifiedpublic_html/js/JOLLYROGER.js
modifiedpublic_html/template/Build.tpl
modifiedpublic_html/template/BuildingStorage-StoreConfirm.tpl
Patch body is stored as a separate file to keep this page lightweight.

Review comment

#97 by wada at 2021-11-09T16:51:59; lib/ajax/build_selectBreak.inc / old:91 new:92
9ef8eff66c1b854707cee44ce8135bebf30484c8

@kouho
echoの後にexitが必要です。

Review comment

#98 by wada at 2021-11-09T16:56:50; lib/ajax/build_selectCheck.inc / old:None new:98
9ef8eff66c1b854707cee44ce8135bebf30484c8

@kouho
おそらくbonus_timeが抜けています。$object_name[$user_object['object_no']]['bonus_time']
日付のフォーマットについて、できる限りn/j(k) H:iで表示して欲しいです。

Review comment

#99 by wada at 2021-11-09T17:01:11; public_html/template/Build.tpl / old:None new:222
9ef8eff66c1b854707cee44ce8135bebf30484c8

@kouho
倉庫拡張が巻き戻っているので、公開状態に修正お願いします。

Review comment

#100 by iwamoto at 2021-11-09T19:49:07; lib/ajax/build_selectBreak.inc / old:None new:92
2a0ce1b4adf2ad619ceef3a92541204a9abfa0ff

@kouho
変数名の修正をお願い致します。 $$tmp_date_name;
build_selectCheck.incも同様にお願い致します