PR

【10/17 確認出来次第】アルバム修正。ヤマトコラボ対応 #3156

kouho opened this at 2023-10-16T18:36:15; closed

Description

opened by kouho at 2023-10-16T18:36:15; updated 2023-10-17T16:45:52

概要

カードアルバムでヤマトコラボのカードを見ようとするとエラーが起きていたため修正用のプルリク。
配列側の対応ではなく、関数側で対応しています。

2023-10-15 19:43:36.512366: {"ip":"10.41.0.78","uri":"/?M=Card&A=AlbumDetail&car
d=1494&property=&p=0&name_text=&rare=&gacha_style=0&year=0","ua":"Mozilla/5.0 (M
acintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Versi
on/16.6 Safari/605.1.15","req":{"M":"Card","A":"AlbumDetail","card":"1494","prop
erty":"","p":"0","name_text":"","rare":"","gacha_style":"0","year":"0"},"code":0
,"func":"{closure}","trace":"Card-AlbumDetail(395)/

Comment

#31065 by kouho at 2023-10-16T18:36:37; updated 2023-10-16T18:36:37

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

Comment

#31070 by wada at 2023-10-17T10:55:40; updated 2023-10-17T10:55:54

@kouho
汎用関数に手を入れる場合、確認する範囲が広すぎるので、
すぐに反映するのであれば、ページ固有の処理を修正した方が良いです。

$tags = $skill_friends['skill_relation'][$skill_no]['tag_name'] ?? [];

/* ↓↓↓ */

//タグ名が付いているスキルは、関連カードを取得
if ($tags = $skill_friends['skill_relation'][$skill_no]['tag_name']) {
	$tags = S_EvenArray($tags);
	foreach ($tags as $tag) {
		...
	}
}

Comment

#31081 by kouho at 2023-10-17T14:13:14; updated 2023-10-17T14:13:14

@wada
確認ありがとうございます。ページ側処理の修正で対応いたしました。

Comment

#31090 by wada at 2023-10-17T16:11:06; updated 2023-10-17T16:11:06

@kouho
確認OKです。

wada add_label at 2023-10-17T16:11:10: 和田 LGTM
kouho merge at 2023-10-17T16:45:52: 【10/17 確認出来次第】アルバム修正。ヤマトコラボ対応
kouho close at 2023-10-17T16:45:52: Close

Commit range

base3e6f899a1e2f057c19b61a3a37278e21cd79b9b5
head2c19dfbad39f43e0331ddc6802b1667bfafc8864
base branchjolly/jr-web:master
head branchjolly/jr-web:fix_album

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/public/Card-AlbumDetail.inc
Patch body is stored as a separate file to keep this page lightweight.