PR

【2/25】新大陸報酬テーブル対応 #1565

wada opened this at 2022-02-18T17:16:19; closed

Description

opened by wada at 2022-02-18T17:16:19; updated 2022-03-14T11:54:35

概要

イベント報酬テーブル作成 #1471

新大陸configのランキング設定削除
報酬画像関数のサイズ調整追加
・jsの画像表示をフェイル名渡しに調整
ランキングページの各順位閾値が1つずれていた
・閾値と同順位なら表示しない
デバッグ、管理ツールの報酬テーブル対応

ブランチ

web: island_reward_table
db: island_reward_table

dbブランチの報酬データはレビュー用のデータなので、開発にはマージしません。

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

達成報酬

メインページイベントモーダル
冒険中表示(画面表示、報酬獲得時)
報酬内容の正誤

ランキング報酬

user_event_new_island を本番からインポートして以下のスクリプトを実行して確認してください。

ランキング集計

ENVIRONMENT=develop PF=yamada HTTP_HOST=127.0.0.1 IS_DOCKER=true php event_ranking_event_island.inc

ランキング報酬配布

ENVIRONMENT=develop PF=yamada HTTP_HOST=127.0.0.1 IS_DOCKER=true php tool_event_newisland_present.inc

TODO

管理画面インポートページ

wada add_label at 2022-02-21T14:49:36: デベロップへマージ
wada change_title at 2022-02-21T14:50:00: [WIP]新大陸報酬テーブル対応 【2/25】新大陸報酬テーブル対応
wada add_label at 2022-02-21T14:50:03: レビュー待ち

Comment

#17484 by wada at 2022-02-21T14:50:53; updated 2022-02-21T14:50:53

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

Comment

#17531 by kouho at 2022-02-24T13:57:35; updated 2022-02-24T13:57:35

@wada
ゲーム内動作は問題ないと思います。
お知らせに達成報酬をコピペする用の test_newisland_event.php実行後のoutput/event_newisland_html.txtの期限付きアイテムの表示が一部おかしくなっていました。
紅玉リカバリ【日曜まで】×1×1
必勝のフラッグ【日曜まで】×1×1

Comment

#17534 by iwamoto at 2022-02-24T16:37:57; updated 2022-02-24T16:37:57

@wada
一部達成報酬のアイテムにおいてアイテム名が表示されないものがあります
現状はコンパスが表示されていないです。

island_reward.phpにあるget_point_rewards()
PresentNameConst::getFullNameから取得されていますが、コンパスは未登録のため取得できていませんでした。

他は問題ないと思います。

Comment

#17536 by wada at 2022-02-24T16:57:32; updated 2022-02-24T16:57:32

@kouho
テスト対応しました。
その他問題あれば、イベント実装時に修正してもらって大丈夫です。

Comment

#17537 by wada at 2022-02-24T17:00:02; updated 2022-02-24T17:00:02

@iwamoto
テスト用のデータには、item_nameが入っていませんでしたが、
PresentNameConstで取得できない名前は、カードや強化アイテムのようにテーブルデータ作成時に登録します。

Comment

#17543 by iwamoto at 2022-02-24T17:20:54; updated 2022-02-24T17:20:54

@wada
承知しました。それであれば大丈夫です。
問題ないと思います。

iwamoto add_label at 2022-02-24T17:20:59: 岩本 LGTM
kouho add_label at 2022-02-24T17:46:37: 皇甫 LGTM

Comment

#17552 by kouho at 2022-02-24T17:47:29; updated 2022-02-24T17:47:29

@wada
対応ありがとうございます。かしこまりました。確認OKです。

wada merge at 2022-02-28T10:53:06: 【2/25】新大陸報酬テーブル対応
wada close at 2022-02-28T10:53:06: Close
wada refer at 2022-03-03T14:23:41: 1591:[WIP]新大陸報酬テーブル
wada refer at 2022-03-14T11:54:35: 1627:【3/17】新大陸報酬テーブル対応

Commit range

base0ab09409d56cb5117d4e21478e89ad4c17171ab3
head2cdd5a5f28242ca0b6d3965a13e1de1425d25326
base branchjolly/jr-web:develop
head branchjolly/jr-web:island_reward_table

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed21 filesView patch
statusfile
modifiedcpanel_html/template/header.tpl
addedcpanel_html/template/m_new_island_event_spec.tpl
modifiedlib/SubFunc.inc
modifiedlib/ajax/main_default.inc
modifiedlib/class/const/RewardConst.class.php
modifiedlib/conf/class/NewIslandConfig.class.php
addedlib/cpanel/m_new_island_event_spec.inc
modifiedlib/cpanel/m_new_island_result.inc
modifiedlib/exec/event_ranking_event_island.inc
modifiedlib/exec/tool_event_newisland_point.inc
modifiedlib/exec/tool_event_newisland_present.inc
addedlib/models/Island_reward.php
modifiedlib/models/Newisland_event_model.php
modifiedlib/models/dao/master_event_reward.php
modifiedlib/public/Main.inc
modifiedlib/public/Ranking-EventIsland.inc
modifiedpublic_html/js/JOLLYROGER_mission.js
modifiedpublic_html/template/Battle.tpl
modifiedpublic_html/template/Mission.tpl
modifiedpublic_html/template/Ranking-EventIsland.tpl
modifiedtool/event/test_newisland_event.php
Patch body is stored as a separate file to keep this page lightweight.