PR

【3/21】蒼玉召喚の書交換 #2662

wada opened this at 2023-03-17T14:34:58; closed

Description

opened by wada at 2023-03-17T14:34:58; updated 2023-03-23T09:38:35

概要

蒼玉交換 jolly/jr-cs-issue#338

画面作成してからだとレビュー時間が足りないので機能実装のみ
交換ログは蒼玉消費ログlog_user_blue_ballで代用

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

以下のURLにアクセスして召喚の書が増えるか、レイドが出現するか

#bb_change_numは交換したい蒼玉の数
/?M=RaidEvent&A=ExecBlueBallChange&bb_change_num=2

TODO

  • [x] 交換画面
  • [x] ページデザイン調整

Comment

#26409 by wada at 2023-03-17T14:36:27; updated 2023-03-17T14:36:27

@kouho
画面作成が間に合わなそうなので、先に機能面のレビューをお願いします。

画面作成はデバッグまでに間に合えばこのPRに追加します。

Comment

#26416 by kouho at 2023-03-17T17:16:19; updated 2023-03-17T17:16:19

@wada
ありがとうございます。
NewiIslandConfigとRaidMonsterConfigの期間修正を巻き込んでしまっているので確認お願いいたします

wada change_title at 2023-03-20T15:14:26: 【3/20】蒼玉召喚の書交換 【3/21】蒼玉召喚の書交換

Comment

#26435 by wada at 2023-03-20T15:36:23; updated 2023-03-22T09:47:50

@kouho
交換ページ作成できたので、コミットしました。

対応箇所

メインページ

  • 召喚の書アイコンの遷移先を交換ページに変更(レイド出現中は遷移しない)

イベントモーダル内のアイコンは蒼玉召喚を間違う可能性があるため遷移しない

  • イベントモーダルに所持蒼玉を追加
  • イベントモーダルの所持蒼玉リカバリで、期限付きの個数をまとめて表示

~~現在の仕様ではどちらかが必ず0なので~~

  • イベントモーダルに交換ページへのボタンリンクを設置

アイテム使用

  • イベント出現中以外でも蒼玉リカバリ(期限付き含む)使用可能
  • アイテム使用後ページに所持蒼玉表示
  • チーム未所属ユーザはリカバリを使用できない

交換ページ

  • 蒼玉2個と召喚の書1個の交換対応
  • 蒼玉2個未満は冒険への導線を表示
  • 召喚の書が20個になりレイドが出現したらレイドへの導線を表示

複数個交換は、実装や確認に時間がかかるため対応していません。
できそうならdevelopマージ後に対応してもらって大丈夫です。

デバッグ日に合わせて、締め切りを3/21に変更しました。

Comment

#26439 by kouho at 2023-03-20T16:04:05; updated 2023-03-20T16:04:05

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

log_team_raid_itemへのログの入り方が気になります。
レイド出現中に蒼玉を拾った場合log_team_raid_itemに元々入っていなかったのが今は入るようになっているように見えますので確認していただきたいです・

Comment

#26444 by kouho at 2023-03-20T18:32:04; updated 2023-03-20T18:32:04

@wada
指摘箇所以外は確認OKです

Comment

#26446 by wada at 2023-03-20T18:49:36; updated 2023-03-20T18:49:36

@kouho
> log_team_raid_itemへのログの入り方が気になります。
レイド出現中に蒼玉を拾った場合log_team_raid_itemに元々入っていなかったのが今は入るようになっているように見えますので確認していただきたいです・

どの箇所でしょうか?

Comment

#26460 by kouho at 2023-03-22T16:11:36; updated 2023-03-22T16:15:31

@wada
> log_team_raid_itemへのログの入り方が気になります。
レイド出現中に蒼玉を拾った場合log_team_raid_itemに元々入っていなかったのが今は入るようになっているように見えますので確認していただきたいです・
>
どの箇所でしょうか?

すみません、これについては月曜日に伝えたように勘違いでした。

> $_df['MEMCACHE_NAME']を呼んでいるのでglobalに$_dfも定義お願いします。
> $_df['MEMCACHE_NAME']を呼んでいるのでglobalに$_dfも定義お願いします。
> 種類の違う期限付き蒼玉リカバリをまとめた個数表示するなら、
$_temp['blueball_item'][ItemConst::ID_BLUEBALL_RECOVERY_WED]['num'] += $weekday_item_manager->getAmount($item_no);
にした方が良いと思います。

これらの対応はどうしましょう?レイド実装の一環としてこちらで入れたほうが良いでしょうか?

Comment

#26464 by wada at 2023-03-22T17:44:35; updated 2023-03-22T17:44:35

@kouho
以下対応しました。
> $_df['MEMCACHE_NAME']を呼んでいるのでglobalに$_dfも定義お願いします。
$_df['MEMCACHE_NAME']を呼んでいるのでglobalに$_dfも定義お願いします。
種類の違う期限付き蒼玉リカバリをまとめた個数表示するなら、
$_temp['blueball_item'][ItemConst::ID_BLUEBALL_RECOVERY_WED]['num'] += $weekday_item_manager->getAmount($item_no);
にした方が良いと思います。

kouho add_label at 2023-03-22T18:31:40: 皇甫 LGTM

Comment

#26476 by kouho at 2023-03-22T18:31:53; updated 2023-03-22T18:31:53

@wada
ありがとうございます。確認OKです

wada merge at 2023-03-23T09:38:35: 【3/21】蒼玉召喚の書交換
wada close at 2023-03-23T09:38:35: Close

Commit range

base47f3077a4b694b721422c17bb677895b4f922d5f
headb45110e876c2edd767a22cf1da945075d5818f82
base branchjolly/jr-web:develop
head branchjolly/jr-web:blue_ball_change

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed12 filesView patch
statusfile
modifiedlib/SubFunc.inc
modifiedlib/ajax/item_result.inc
modifiedlib/class/controller/JrControllerRaidEvent.class.php
modifiedlib/conf/class/RaidMonsterConfig.class.php
modifiedlib/models/Raid_event.php
addedlib/models/dao/log_team_raid_item.php
modifiedlib/models/dao/team_data.php
modifiedlib/public/Main.inc
modifiedpublic_html/js/JOLLYROGER.js
modifiedpublic_html/template/Item.tpl
modifiedpublic_html/template/Main.tpl
addedpublic_html/template/RaidEvent-ConfirmBlueBallChange.tpl
Patch body is stored as a separate file to keep this page lightweight.