【3/9】新大陸イベント #644

https://docs.google.com/spreadsheets/d/19S1QtrWe-CergmrtozIIk6OMJnD3dM186M8oUWH5kEQ/edit#gid=873412819

変更箇所

• 証追加

NewIslandConfigに以下の定義を追加してます。

$card_piece_reward = array(
	'type' => PresentBoxContent::TYPE_CARD_PIECE,
	'present_id' => 1,
	'item_name' => '【宮廷騎士カール】勝利の証',
);

証追加 備考

test_newisland_event.phpで
$full_name = PresentNameConst::getFullName($item_type, $item_id, $count);
証追加の際にエラーが出たため以下の定義を追加しました。
これから先証追加のたびに追加していいくかどうか

PresentNameConst.class.php

// type:14(証)
14 => array(
        1 => "【宮廷騎士カール】勝利の証",
),

• 達成報酬

フックモンスターと限定カードと証がループ報酬に追加

達成報酬 備考

証の性質上、告知の達成報酬のページに個数表示をのせたほうがいいかと思いましたが、
他の部分と絡みもあるかと思いましたので報酬名だけにしています。
(表現の統一性、test_newisland_event.phpのチェック時の挙動)

• ランキング報酬

最低報酬が1001位からになり、上位の報酬が小刻みになりました。

• モンスター経験値

Razest開催時のものより減少してます

その他

~~遺跡と神殿の表記修正をする場合は画像修正が必要~~
告知に森(遺跡)、館(神殿)の表示があるので画像修正は不要でした

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

PresentNameConstの勝利の証について
・二重管理になるので、基本的にdbから取得する方が良いです
・テストスクリプトで名前エラーが出た場合は、エラーメッセージにアイテム名をつけて確認できるようお願いします
・名前エラーはNOTICEなので、内容さえ把握できれば特に解決する必要はないと思います
・もし厳密にチェックする場合は、アプリ側と同じ方法で名前チェックをお願いします

勝利の証の画像が出ていない箇所があるので確認お願いします。

!スクリーンショット 2021-03-04 11

!スクリーンショット 2021-03-04 11

!スクリーンショット 2021-03-04 11

テストスクリプトで、GKカードが1枚しか表示されない

@kouho
確認しました。上記修正をお願いします。

確認ありがとうございます
修正します

@kouho
修正内容確認しました。問題無いです。

修正済み

• 証の画像表示周りの修正
• 3600 * 1000 ローカル用設定を開発に上げてしまった分の修正

未対応&未修正

• PresentNameConstの勝利の証について
• テストスクリプトで、GKカードが1枚しか表示されない

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

ローカルでテストバッチが実行できないため、
・PresentNameConstの勝利の証について
の対応を上げていなかったのですが、先ほどあげました。
こちらも確認していただけませんか？

@kouho
テストスクリプトのために、アプリ側を更新するのは、あまりお勧めできませんので、dbアクセスに関しては、test_newisland_event.phpに実装した方が良いです。

masterデータによる厳密なチェックもなくて大丈夫です。
エラーメッセージにアイテム名を追加すれば確認できます。

#test_newisland_event.php
return "【NOTICE】アイテムの設定に問題がある可能性があります。確認してください。";
↓
return "【NOTICE】アイテムの設定に問題がある可能性があります。確認してください。{$item_name}";

ここの3600 * 1000は1000時間という事であってますか？