【3/8】ヴァルハラ島ステージ追加 #2636

概要

https://docs.google.com/spreadsheets/d/13P64IG2HBr76pQ-Dz33_qiapTRv16vBVMfn-iNrjF8o/edit#gid=0
ヴァルハラ島ステージの追加
専用コンパスの追加
先行デプロイするため、導線のON/OFF対応
商会の勝利の証は確認用の仮設定（仕様確定後対応）

マージPR

• 7197:【10/5】ヴァルハラ島
• 7210:【画像】ヴァルハラ島で使う画像のみをいれた
• 7219:【作業中】ヴァルハラ 冒険画面UI追加・コンパス使用処理など
• 7273:ヴァルハラ告知の画像を追加しました。

告知画像は告知作成時に再度マージ

• 7756:【3/9実装】システムメッセージのアイコン変更

実装時に除外していたヴァルハラ島対応をマージ

• 8638:【12/28本番UP】冒険変速対応（冒険とヴァルハラ島）

実装時に除外していたヴァルハラ島対応をマージ

• 8753:【1/7】冒険ページ、魂ページ、アクセ選択ページのダブルタップ制御

実装時に除外していたヴァルハラ島対応をマージ

• 8775:【1/11】冒険ページ踏んだら倍速保存期間更新

実装時に除外していたヴァルハラ島対応をマージ

• 8989:【4/14】チーム系アイテムのロック処理

マージではなく、ヤマダ仕様への対応

その他対応

• Card_pieceモデルを参照する関数をSubFunc.incに追加（rzst版はCard_pieceモデルがないため）
• ヴァルハラ告知IDの変数化
• ヴァルハラ表示のON/OFF
• カードアルバムで連携先なしエラー対応

DB更新内容

【3/8】ヴァルハラ島ステージ追加 jolly/jrye-db#319

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

カードデータ(master_card_data)のcsvがないので、開発管理画面からカードデータインポートしてください。
ヴァルハラのコンパスを使用してヴァルハラ島をプレイ
必勝フラッグが正しく動作するか
ヴァルハラ表示をOFFにした際に、ヴァルハラ関連の表示がないか

コンフリクト

public_html/template/Mission.tplの{$_parts['stage_desc_image']}でコンフリクトが発生
developマージ時に対応

対応

• [x] Argument 1 passed to lib\models\Battle_reward::get_next_point_reward() must be of the type int, null given, called in /home/web/jr-web/lib/SubFunc.inc on line 45639
• [x] log_stage_changeの向き先を $_dbc[37]から $_dbc[38]

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

@wada
たまたまかもしれませんが、valhalaのブランチに切り替える前にブラックスライムの巣穴にいるデータでvalhalaのブランチに切り替えて冒険をしているとブラックスライムの巣穴から抜け出せなくなったので確認してみていただきたいです。

上記の問題はvalhalaブランチではブラックスライムの巣穴がリリースされていないからでした。masterブランチをマージしたら問題なく抜け出せました。

@wada
ソースの原因箇所を絞り込めていませんが、
天下無双開催中にヴァルハラの島に入り海賊バトルを行うと以下のエラーが出ますので確認お願いします。
イベント報酬対応周りっぽいです。

Argument 1 passed to lib\models\Battle_reward::get_next_point_reward() must be of the type int, null given, called in /home/web/jr-web/lib/SubFunc.inc on line 45639
Type: TypeError

Filename: /lib/models/Battle_reward.php

Line Number: 100

Backtrace:

File: /lib/SubFunc.inc
Line: 45639
Function: get_next_point_reward

File: /lib/ajax/battle_player.inc
Line: 419
Function: S_CheckNextBattleEventItem

@wada
ヴァルハラの島でエインヘリャル討伐時に紅玉が減らず、戦魂石が加算されない時がありました。
現象の再現頻度は高く、再現方法は以下の通りです。

冒険ページを開いて1回目エインヘリャル討伐する。
この時は紅玉が消費され、戦魂石が加算される
↓
リロードせずに続けて冒険をする。
2回目以降エインヘリャルを討伐する
紅玉が消費され、戦魂石が加算されない。
↓
上記の状態で何回か進めるとページが読み込み中が続きフリーズ。

クライフの獲得した勝利の証の個数表示も同様の問題が確認されました。
ページ読み込んで初回は加算されますが、2回目以降表示上加算されていません。

@wada
S_EscapeUserValhalla関数内のlog_stage_changeの向き先を $_dbc[37]から $_dbc[38]に修正をお願いいたします。

@wada
SubFunc_valhalla.inc 関数内のS_UserStageChangeLogでもlog_stage_changeの向き先を $_dbc[37]から $_dbc[38]に修正をお願いいたします。

@wada
[疑問ベース]
ヴァルハラの島での海賊バトル発生率が体感かなり高めですが仕様でしょうか？仕様書を確認しましたが、海賊バトルの発生率についての記載は見当たらず。

@wada
csv/master_card.csv に旋風の術士クライフのデータが見つかりませんので追加お願いいたします。

@wada
ヴァルハラの島でエインヘリャル討伐時のエラーを共有しましたが、
log_stage_changeの対応を入れたら再現しなくなりました。

@kouho
> ヴァルハラの島での海賊バトル発生率が体感かなり高めですが仕様でしょうか？仕様書を確認しましたが、海賊バトルの発生率についての記載は見当たらず。

ヤマダ版の仕様確定後に再度確認していただければと思います。

> csv/master_card.csv に旋風の術士クライフのデータが見つかりませんので追加お願いいたします。

ヤマダ版仕様で実装が決まれば追加しますので、その際に再度確認お願いします。

@kouho
> Argument 1 passed to lib\models\Battle_reward::get_next_point_reward() must be of the type int, null given, called in /home/web/jr-web/lib/SubFunc.inc on line 45639

上記エラーについて、ヴァルハラではイベント系の加算や表示を行わないので、賞金首の処理を弾くようにしました。

> log_stage_changeの向き先を $_dbc[37]から $_dbc[38]

関数を使用しました。

@wada
> 上記エラーについて、ヴァルハラではイベント系の加算や表示を行わないので、賞金首の処理を弾くようにしました。

> 関数を使用しました。

処理自体は確認OKですが、イベント系の加算を行わないことについてイベントポイントも増えない仕様なのかどうか確認していただきたいです。

@kouho
> 処理自体は確認OKですが、イベント系の加算を行わないことについてイベントポイントも増えない仕様なのかどうか確認していただきたいです。

ヤマダ版の仕様は決まっていないので確認は不要です。

パイレーツパスについて

リリースまで時間がないため、仕様が決まるまでに一通りの実装を済ませようと計画しています。
そのため、このPRでは機能のマージ漏れがないか等を確認していただきたいです。
仕様が確定しましたら、その際は別PRとして対応しますので確認お願いいたします。

リリースまでのスケジュール

• 機能マージ

2/22 ~ 3/24

• ヤマダ仕様調整

3/27 ~ 29

• 管理画面マージ

3/30 ~ 4/5

• デバッグ

4/3 ~ 4/14

• リリース

4/18

@wada
確認OKです