PR

【2/4】カードテキスト省略表示修正と新大陸報酬GK期間修正 #1529

kouho opened this at 2022-02-04T11:12:12; closed

Description

opened by kouho at 2022-02-04T11:12:12; updated 2022-02-04T14:05:39

概要

新大陸の実装時にhtmlspecialcharsしたデータをcard_textに入れるようにしましたが、
省略表示のためカードテキストをトリミングした際に
htmlエンティティの途中で終了する場合に表示がおかしくなる問題が起きました。
表示側でhtmlspecialchars_decodeで対応をしています。

また新大陸報酬のGK期間が間違えていましたので併せて修正しています。

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

以下の表示を確認

  • カード詳細
  • デッキ編集
  • 酒場でのカード雇用

TODO

確認できしだい開発に反映

Comment

#17074 by kouho at 2022-02-04T11:13:16; updated 2022-02-04T11:13:16

@wada @iwamoto
イベント以外にも影響が出る可能性があるためPRを作成しました。確認お願いいたします。

Comment

#17077 by wada at 2022-02-04T12:07:40; updated 2022-02-04T12:07:40

@kouho
> 新大陸の実装時にhtmlspecialcharsしたデータをcard_textに入れるようにしましたが、

まずこの理由が知りたいです。

Comment

#17078 by kouho at 2022-02-04T12:11:47; updated 2022-02-04T12:11:47

@wada
記載が足りなくてすみません。Slackに書いたこちらの件の対応です。
https://scry.slack.com/archives/C0164SMGG8P/p1643178177040500

管理画面で一部カードのデータがエスケープ処理の関係でエクスポートしてインポートがうまくいかない問題が起きたためcpanelブランチの以下のコミットで修正しています。
念のため確認していただけましたら幸いです。
b533be1357 カードデータエクスポートの修正
問題が発生したカード 491 狂戦士ギース

Comment

#17079 by wada at 2022-02-04T13:18:24; updated 2022-02-04T13:18:24

@kouho
master_card_textにエスケープ処理された文字が含まれているため、今回の問題が発生したという事ですね。

個人的には、データが複数のルールに従っているのが違和感があるので、テーブルに格納しているデータを修正する方が良いと思いました。

master_card_textの強調ルール
"{text}"
""{text}""
\"{text}\"

git管理されているギースのmaster_card_text""になっているので、本番で同じ問題は発生しないと思われます。

Comment

#17082 by kouho at 2022-02-04T14:04:10; updated 2022-02-04T14:04:10

@wada
かしこまりました。カードテキスト省略表示修正の対応はRevertして新大陸報酬GK期間修正だけ反映させようと思います。

kouho merge at 2022-02-04T14:05:39: 【2/4】カードテキスト省略表示修正と新大陸報酬GK期間修正
kouho close at 2022-02-04T14:05:39: Close

Commit range

base5b42afa01410abf3336100ebedc52af2170878ec
head478a448dfc357f76cf17f8f654da0a6585a8cfe6
base branchjolly/jr-web:develop
head branchjolly/jr-web:event_newisland_1477

View commits

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