PR

【5/16】スキルチェンジで本来つけられない技玉をつけられる問題を修正 #1825

kouho opened this at 2022-05-13T18:49:05; closed

Description

opened by kouho at 2022-05-13T18:49:05; updated 2022-05-16T18:45:03

概要

本来覚えることができないスキルを覚えることができる #1822

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

修正前確認

アイテム一覧→スキルチェンジャー→変更するSKILL SLOTを選択→入れ替える技玉を選択した後の画面
(ページ上部のスキル変更でいう4の工程)でURLのafter_item_noを所持している別の技玉のidに修正してリロード→はい

技玉の使用条件を無視してスキルを覚えれます

ユーザー技玉のidを知る方法
アイテムページの技玉一覧で技玉を選択してURLのno

修正後確認

修正前と同じ手順で問題が起きないことの確認
カウンタースキル所持カードも防御スキルをつけられないことの確認

TODO

Comment

#19021 by kouho at 2022-05-13T18:49:47; updated 2022-05-13T18:49:47

@wada
確認お願いいたします

Comment

#19044 by wada at 2022-05-16T14:39:37; updated 2022-05-16T14:40:51

@kouho
カウンタースキルをスキルチェンジで防御スキルに変更しようとするとアイテム一覧に飛ばされます。
通常の攻撃スキルは入れ替え可能
<img src="https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/1652679525484nVO6YwSwNg" width="300">

Comment

#19045 by kouho at 2022-05-16T15:07:31; updated 2022-05-16T15:07:31

@wada
確認ありがとうございます。カウンタースキルをスキルチェンジで防御スキルに変更できるように修正しました。

Comment

#19054 by wada at 2022-05-16T16:18:45; updated 2022-05-16T16:21:50

@kouho
確認しました。
防御スキルがある状態での、カウンタースキルの技玉入れ替えも確認しますか?

Comment

#19055 by kouho at 2022-05-16T16:25:04; updated 2022-05-16T16:25:04

@wada
カウンター系が今の所専用技玉で1種類のみなのでそちらは確認しなくても大丈夫かと思います。

Comment

#19056 by wada at 2022-05-16T16:31:32; updated 2022-05-16T16:31:32

@kouho
了解しました。確認OKです

wada add_label at 2022-05-16T16:31:36: 和田 LGTM
kouho merge at 2022-05-16T16:46:18: 【5/16】スキルチェンジで本来つけられない技玉をつけられる問題を修正
kouho close at 2022-05-16T16:46:18: Close
kouho refer at 2022-05-16T18:45:03: 1830:【リリース】技玉修正・リクエスト表示修正

Commit range

base96a8e76eb077a1c2b714b20ee64f1617942ad461
headf767b042fccda9105d65176a260cf7de7cedbc29
base branchjolly/jr-web:develop
head branchjolly/jr-web:hotfix_item_change

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed3 filesView patch
statusfile
modifiedlib/SubFunc.inc
modifiedlib/public/Item-ChangeSkillCheck.inc
modifiedlib/public/Item-ChangeSkillResult.inc
Patch body is stored as a separate file to keep this page lightweight.