PR

ガチャ実行処理の修正 #353

wada opened this at 2020-10-17T22:30:22; closed

Description

opened by wada at 2020-10-17T22:30:22; updated 2023-05-15T10:14:43

API送信後にアイテム付与

  • [x] ガチャ
  • [x] 酒場雇用
wada change_title at 2020-10-18T18:22:27: [WIP]ガチャ実行処理の修正 ガチャ実行処理の修正
wada add_label at 2020-10-18T18:22:32: レビュー待ち

Comment

#7819 by wada at 2020-10-19T10:19:43; updated 2020-10-19T10:19:43

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

Comment

#7826 by wada at 2020-10-19T14:39:27; updated 2020-10-19T14:39:27

@iwamoto
修正しました。

Comment

#7829 by iwamoto at 2020-10-19T14:49:32; updated 2020-10-19T14:49:32

@wada
修正箇所大丈夫ですー

所持カードがいっぱいの際にガチャを回すとエラーになってしまいます。flashは流れません

2020-10-19 14:42:49.733737: {"ip":"172.18.0.1","uri":"/?M=Bar&A=GachaExec&gacha_id=110&type=10&sk=ZchDXeyYIR1PlWKy","ua":"Mozilla/5.0 (Linux; Android 6.0.1; Moto G (4)) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/86.0.4240.75 Mobile Safari/537.36","req":{"M":"Bar","A":"GachaExec","gacha_id":"110","type":"10","sk":"ZchDXeyYIR1PlWKy"},"code":0,"func":"{closure}","trace":"Bar-GachaExec(4)/Bar-GachaExec(4)/Bar-GachaExec(1182)/index(533)","msg":"count(): Parameter must be an array or an object that implements Countable"}

所持カードがいっぱいの際、Bar-GachaExec.tplに渡した
$_temp['card_list']がnull みたいです

Comment

#7830 by wada at 2020-10-19T15:45:47; updated 2020-10-19T15:45:47

@iwamoto
修正しました。
クラス変数にデータを入れてなかったため表示されていませんでした。

Comment

#7833 by iwamoto at 2020-10-19T16:08:37; updated 2020-10-19T16:08:37

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

wada merge at 2020-10-19T16:08:40: ガチャ実行処理の修正
wada close at 2020-10-19T16:08:40: Close

Commit range

basead6346001e326fa1fd61bb51992cb76cb67355bd
head4360d05480f3abd74ebee9c19a27ba92d31b7a9b
base branchjolly/jr-web:develop
head branchjolly/jr-web:fix_payment_333

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed4 filesView patch
statusfile
addedlib/models/Gacha_payment.php
modifiedlib/models/shop/Base_item.php
modifiedlib/public/Bar-GachaExec.inc
modifiedlib/public/Bar-SelectCoinResult.inc
Patch body is stored as a separate file to keep this page lightweight.