Archive / jolly / jr-web
PR

【10/x】アルバム機能の改修 #1199

iwamoto opened this at 2021-10-06T16:44:41; closed

Description

opened by iwamoto at 2021-10-06T16:44:41; updated 2025-09-30T13:58:51

概要

アルバム機能の改修(https://git.dev.d-es.co.jp/gitbucket/jolly/jr-rzst/issues/24)

カード詳細ページに「専用アイテム」の項目を追加
!6OKCcrYdNgAJ5oe1633504103_1633504173

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

「専用アイテム」項目について

  • 公開設定済みになっている強化アイテムのみ画像を表示
  • 未所持の場合はグレースケールで表示
  • 専用アイテム(専用ドリンク)、転生・神魔降臨カードについては表示されないこと

TODO

このPull Requestに対して、後続で発生する積み残し作業などがある場合は記載。

Comment

#14177 by iwamoto at 2021-10-06T17:14:25; updated 2021-10-06T17:14:25

@wada @kouho
お手すきの際にご確認お願いします。

iwamoto add_label at 2021-10-06T20:24:56: レビュー待ち

Comment

#14275 by wada at 2021-10-08T17:17:54; updated 2021-10-08T17:17:54

@iwamoto
確認しました。コメント箇所確認お願いします。

Comment

#14279 by kouho at 2021-10-08T18:53:47; updated 2021-10-08T18:53:47

@iwamoto
確認しました。和田さん指摘箇所以外OKかと思います。

Comment

#14362 by iwamoto at 2021-10-12T18:33:21; updated 2021-10-12T18:33:21

@wada @kouho
指摘箇所の$_temp['resurrection_after_card'][$_temp['card_no']]を削除して対応しました
確認をお願いします。

Comment

#14363 by kouho at 2021-10-12T18:59:09; updated 2021-10-12T18:59:09

@iwamoto
確認しました。問題ないと思います。

kouho add_label at 2021-10-12T18:59:12: 皇甫 LGTM

Comment

#14365 by wada at 2021-10-12T19:01:21; updated 2021-10-12T19:01:21

@iwamoto
確認しました。OKです

wada add_label at 2021-10-12T19:01:25: 和田 LGTM
iwamoto merge at 2021-10-18T10:47:51: 【10/x】アルバム機能の改修
iwamoto close at 2021-10-18T10:47:51: Close
iwamoto refer at 2021-10-21T16:43:02: 1243:【10/22】リリース

commit comment

#91 by wada at 2021-10-08T16:58:28; public_html/template/Card-AlbumDetail.tpl / old:None new:48
abed93a351df8212d2fe95d1c29ca01fc23f069f

@iwamoto
S_GetLimitBreakEffect の中で、$_temp['resurrection_after_card'][$_temp['card_no']] が0の時の処理もあったので、上のif文で0を弾くのが危険かもしれない。

Commit range

basef2b3efae589060843eec66636ba2fc5b2a7a7a9d
head5d84a4f0b7048448a7878338213f5d8a3fda40ec
base branchjolly/jr-web:develop
head branchjolly/jr-web:card_album_update

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed7 filesView patch
statusfile
modifiedlib/public/Card-Album.inc
modifiedlib/public/Card-AlbumDetail.inc
addedpublic_html/img/common/icon_list.png
addedpublic_html/img/common/icon_stock.png
modifiedpublic_html/js/JOLLYROGER.js
modifiedpublic_html/template/Card-Album.tpl
modifiedpublic_html/template/Card-AlbumDetail.tpl
Patch body is stored as a separate file to keep this page lightweight.

Review comment

#91 by wada at 2021-10-08T16:58:28; public_html/template/Card-AlbumDetail.tpl / old:None new:48
abed93a351df8212d2fe95d1c29ca01fc23f069f

@iwamoto
S_GetLimitBreakEffect の中で、$_temp['resurrection_after_card'][$_temp['card_no']] が0の時の処理もあったので、上のif文で0を弾くのが危険かもしれない。