Archive / jolly / jr-web
PR

【今日】限定船の強化アイコンの表示 #1195

wada opened this at 2021-10-04T14:13:43; closed

Description

opened by wada at 2021-10-04T14:13:43; updated 2025-09-29T10:32:30

概要

港のドッグ一覧で、限定船が強化可能なのに、アイコン（上矢印）が表示されていなかった
強化条件が素材ではない場合、素材数のチェックをしないよう修正

修正前
<img src="https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/16333242957385YkSsAEQN0" width="100">
↓
修正後
<img src="https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/1633324205713cWWmIR05Dc" width="100">

Comment

#14127 by wada at 2021-10-04T14:14:25; updated 2021-10-04T14:14:25

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

Comment

#14128 by iwamoto at 2021-10-04T14:41:28; updated 2021-10-04T14:41:28

@wada
確認しました。
ヤマト側では問題ないと思います。

一部状況で強化できない状態の際に、「強化可能」の表示がされます
船強化に必要な素材のいずれかが 所持数 : 0 の場合のみ表示されています

Comment

#14130 by wada at 2021-10-04T15:52:15; updated 2021-10-04T15:52:15

@iwamoto
あざます。
素材type=0と素材count=0が同じ処理になっていたので、判定修正しました。

Comment

#14131 by iwamoto at 2021-10-04T16:02:27; updated 2021-10-04T16:02:27

@wada
確認しました。
こちらで問題ないと思います。

iwamoto add_label at 2021-10-04T16:02:31: 岩本 LGTM
wada merge at 2021-10-04T17:08:51: 【今日】限定船の強化アイコンの表示
wada close at 2021-10-04T17:08:51: Close

Commit range

basef2b3efae589060843eec66636ba2fc5b2a7a7a9d
head6c8dadd9f5c40dfc3e10b8fac076f2f64a35114e
base branchjolly/jr-web:master
head branchjolly/jr-web:fix_ship_custom

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