PR

【2/27】別typeチュートリアル #4532

wada opened this at 2026-02-24T15:36:42; closed

Description

opened by wada at 2026-02-24T15:36:42; updated 2026-02-27T16:42:43

概要

仕様書
合体版シートを参照

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

各typeのチュートリアル動作確認(type=1,2,3)
新type(3)の更新処理過不足確認

TODO

  • [x] 不足している画像素材の反映
  • [x] URL書き換え対応(caseにidがない場合)

Comment

#44458 by wada at 2026-02-24T15:37:32; updated 2026-02-24T15:37:32

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

Comment

#44519 by kouho at 2026-02-27T09:37:21; updated 2026-02-27T09:37:21

@wada
public_html/img/tutorial/type3/image_17_1.jpg
public_html/img/tutorial/type3/image_17_2.jpg
public_html/img/tutorial/type3/image_17_3.jpg
public_html/img/tutorial/type3/image_17_4.jpg

上記の画像だけ荒いため、出力しなおせませんか?

kouho add_label at 2026-02-27T09:37:27: コメントあり

Comment

#44521 by kouho at 2026-02-27T09:57:54; updated 2026-02-27T09:57:54

@wada

海賊団加入 ( 重要 )

Tutorial-Type3-4.tpl (lines 21-24)
$team_noが未定義のため、海賊団に加入できていませんでした。

URL書き換え対応 ( 不具合 )

/?M=Tutorial&id=〜
idのURLを書き換えることで該当の部分を何度も実行できてしまいます。
例えば、チュートリアル突破後にブラウザバック等でチュートリアル/?M=Tutorial&id=18にアクセスすると、チュートリアルから抜け出せなくなりました。

Comment

#44522 by kouho at 2026-02-27T10:06:20; updated 2026-02-27T10:06:20

@wada

重複呼び出し ( Tutorial-Scroll.incでも同様 )

Tutorial-Type3.inc (line 124, 135) S_CheckPanelMissionStates関数が直前のS_SetTeamMember関数内でも呼ばれているた重複しているように見えます

Comment

#44524 by kouho at 2026-02-27T10:18:14; updated 2026-02-27T10:18:26

@wada

チーム満員時( 例外処理 )

踏むことはほとんどないかもしれませんが、、、
Tutorial-Type3.inc(line 115 )
exit()

S_redirect("/?M=Tutorial&id=5&error=1");
Tutorial-Type3-4.tpl 側で $_REQUEST['error'] を見て1行メッセージ表示

もしくはS_ErrorPage関数の利用でTodoのType3-5.tplを運用するまで対応できませんか?

Comment

#44526 by wada at 2026-02-27T11:07:45; updated 2026-02-27T11:07:45

@kouho
更新しました。また枚数が変更されたのでテンプレートを修正しました。

> 上記の画像だけ荒いため、出力しなおせませんか?

修正しました。

> 海賊団加入 ( 重要 )

TESTだと戻ることができたので判定を削除しました。
進む方向への書き換えは可能なので対応予定です。

> URL書き換え対応 ( 不具合 )

今動いている状態を正として、重複した状態ですがそのままにします。

> 重複呼び出し ( Tutorial-Scroll.incでも同様 )

チーム一覧にリダイレクトさせました。エラー表示は対応未定です。

> チーム満員時( 例外処理 )

Comment

#44527 by kouho at 2026-02-27T11:11:03; updated 2026-02-27T11:11:03

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

kouho add_label at 2026-02-27T11:11:09: 皇甫 LGTM
wada merge at 2026-02-27T15:11:35: 【2/27】別typeチュートリアル
wada close at 2026-02-27T15:11:35: Close
wada delete_branch at 2026-02-27T15:11:42: tutorial_type_3
wada commit at 2026-02-27T16:42:43: Revert "Merge pull request #4532 from jolly/tutorial_type_3" This reverts commit dec99526fe95266a364b1fc945e21097a50cff5f, reversing changes made to 5cf4775662b7c7cecd8d1d3a6de4342efcfe7135. 7001cfa8c0dadd6683aeb20adc15d1bcd95ef0d8

Commit range

basea865854ffbb98983c70b9bd1d33263ddf070dd78
head0cb3ff3d72aa3b2b99e167037ba896f1783519a2
base branchjolly/jr-web:master
head branchjolly/jr-web:tutorial_type_3

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed46 filesView patch
statusfile
modifiedlib/SubFunc.inc
modifiedlib/ajax/battle_player.inc
modifiedlib/ajax/mission_default.inc
modifiedlib/conf/class/TutorialConfig.class.php
addedlib/public/Tutorial-Type3.inc
modifiedlib/public/Tutorial.inc
addedpublic_html/img/tutorial/type3/image_10_1.jpg
addedpublic_html/img/tutorial/type3/image_11_1.jpg
addedpublic_html/img/tutorial/type3/image_12_1.jpg
addedpublic_html/img/tutorial/type3/image_13_1.jpg
addedpublic_html/img/tutorial/type3/image_14_1.jpg
addedpublic_html/img/tutorial/type3/image_15_1.jpg
addedpublic_html/img/tutorial/type3/image_16_1.jpg
addedpublic_html/img/tutorial/type3/image_17_1.jpg
addedpublic_html/img/tutorial/type3/image_17_2.jpg
addedpublic_html/img/tutorial/type3/image_17_3.jpg
addedpublic_html/img/tutorial/type3/image_1_0.png
addedpublic_html/img/tutorial/type3/image_1_1.jpg
addedpublic_html/img/tutorial/type3/image_2_1.jpg
addedpublic_html/img/tutorial/type3/image_3_1.jpg
addedpublic_html/img/tutorial/type3/image_4_1.jpg
addedpublic_html/img/tutorial/type3/image_6_1.png
addedpublic_html/img/tutorial/type3/image_6_2.jpg
addedpublic_html/img/tutorial/type3/image_7_1.jpg
addedpublic_html/img/tutorial/type3/list_item.png
addedpublic_html/img/tutorial/type3/mission_1.png
addedpublic_html/img/tutorial/type3/mission_2.png
modifiedpublic_html/js/JOLLYROGER_mission.js
modifiedpublic_html/js/jr_mission.min.js
modifiedpublic_html/template/Mission.tpl
addedpublic_html/template/Tutorial-Type3-1.tpl
addedpublic_html/template/Tutorial-Type3-10.tpl
addedpublic_html/template/Tutorial-Type3-11.tpl
addedpublic_html/template/Tutorial-Type3-12.tpl
addedpublic_html/template/Tutorial-Type3-13.tpl
addedpublic_html/template/Tutorial-Type3-14.tpl
addedpublic_html/template/Tutorial-Type3-15.tpl
addedpublic_html/template/Tutorial-Type3-16.tpl
addedpublic_html/template/Tutorial-Type3-17.tpl
addedpublic_html/template/Tutorial-Type3-2.tpl
addedpublic_html/template/Tutorial-Type3-3.tpl
addedpublic_html/template/Tutorial-Type3-4.tpl
addedpublic_html/template/Tutorial-Type3-5.tpl
addedpublic_html/template/Tutorial-Type3-6.tpl
addedpublic_html/template/Tutorial-Type3-7.tpl
modifiedpublic_html/template/header_sp.tpl
Patch body is stored as a separate file to keep this page lightweight.