Archive / jolly / jr-web
PR

【8/25】本来プレゼントしないアイテムの受け取り処理 #3027

wada opened this at 2023-08-18T18:47:04; closed

Description

opened by wada at 2023-08-18T18:47:04; updated 2023-08-29T15:44:05

概要

プレゼント配布はすべてPresentBoxManagerで実装 #1933

ホルダ受け取り時に、ページ内のホルダ最大値が更新されないが、
そもそもプレゼントされないはずなので対応しない。

対応アイテム

  • カード複数枚

一括で受け取れるホルダの空きが必要（1枚ずつ受け取れない）

  • 技玉メダル

直接配布

  • カードホルダー
  • カードストック
  • ぱんだま
  • コインガチャチケット

タイプがアイテムのプレゼント（下4種）は、受け取り時にPresentBoxManagerで直接付与

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

https://docs.google.com/spreadsheets/d/19En7kGzovKVW7lJ7UtXmNlO8wV8ePvBfRRziLo1ak94/edit#gid=1706561250

wada change_title at 2023-08-21T16:41:22: [WIP]本来プレゼントしないアイテムの受け取り処理 【8/25】本来プレゼントしないアイテムの受け取り処理
wada add_label at 2023-08-21T16:41:24: レビュー待ち
wada add_label at 2023-08-21T16:41:27: to皇甫

Comment

#29813 by wada at 2023-08-21T17:03:00; updated 2023-08-21T17:03:00

@kouho
確認お願いします。

Comment

#29899 by kouho at 2023-08-24T16:25:40; updated 2023-08-24T16:25:40

@wada
その他は確認OKです。
一応以下の問題がありますが、基本的には起きない問題に対してのものなので対応する必要はないと思います。メモだけしておきます。

  • 複数枚設定されるカードの成長型が全て一緒
  • ぱんだまとコインガチャチケットを隠した際のスタンプシートの表示がない

Comment

#29900 by wada at 2023-08-24T17:00:13; updated 2023-08-24T17:00:13

@kouho
以下、対応しました。

> Presentのidに存在しないデータになっている場合実行時エラーになっているようです。

Comment

#29901 by wada at 2023-08-24T17:02:00; updated 2023-08-24T17:02:10

@kouho
> 一応以下の問題がありますが、基本的には起きない問題に対してのものなので対応する必

ゲームプレイの障害にならない限り対応はせず放置しておきます。
誤配布に気付きやすい処理を入れるべきかもしれません。

kouho add_label at 2023-08-24T17:25:12: 皇甫 LGTM

Comment

#29903 by kouho at 2023-08-24T17:25:26; updated 2023-08-24T17:25:26

@wada
対応ありがとうございます。確認OKです。

wada merge at 2023-08-24T17:50:13: 【8/25】本来プレゼントしないアイテムの受け取り処理
wada close at 2023-08-24T17:50:13: Close
wada refer at 2023-08-29T15:44:05: 3048:本来プレゼントしないアイテムの受け取り処理

commit comment

#196 by kouho at 2023-08-24T16:04:45; lib/models/Present_receive.php / old:24 new:None
f28b6290df49471ed9799aff4ed5bce2e9e79fc4

@wada
Presentのidに存在しないデータになっている場合実行時エラーになっているようです。
元々は特にエラーは出ておりませんでした。

例以下のようなURLにアクセス
http://127.0.0.1:20080/?M=Present&id=2415&p=0

validate_item_receive関数の第一引数に初期値を加えたら解決できそうです$item = null

Commit range

baseb9fde34e707ef62c69cd8aa72c3443b8a131001d
headf28b6290df49471ed9799aff4ed5bce2e9e79fc4
base branchjolly/jr-web:develop
head branchjolly/jr-web:update_present

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
addedlib/models/Present_receive.php
modifiedlib/public/Present-Default.inc
modifiedpublic_html/template/Present-Default.tpl
Patch body is stored as a separate file to keep this page lightweight.

Review comment

#196 by kouho at 2023-08-24T16:04:45; lib/models/Present_receive.php / old:24 new:None
f28b6290df49471ed9799aff4ed5bce2e9e79fc4

@wada
Presentのidに存在しないデータになっている場合実行時エラーになっているようです。
元々は特にエラーは出ておりませんでした。

例以下のようなURLにアクセス
http://127.0.0.1:20080/?M=Present&id=2415&p=0

validate_item_receive関数の第一引数に初期値を加えたら解決できそうです$item = null