PR

【8/24】ニャッカCP交換所 #2222

wada opened this at 2022-08-23T16:02:03; closed

Description

opened by wada at 2022-08-23T16:02:03; updated 2022-08-26T11:37:11

概要

【8/26】ニャッカ育成キャンペーン #2133
https://docs.google.com/spreadsheets/d/15rYveAZFsTT6PC6XQGmf9rdbLqnyXyX1bu7k1W4ufC4/edit#gid=512488926
交換ページ作成
専用アイテムのアイテム一覧非表示
クエスト一覧から交換ページへの導線追加

DB更新内容

【8/24】ニャッカCP交換所 jolly/jrye-db#150

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

確認用ブランチにcampaign_quest_0826、image_campaign_quest_0826をマージ(web,db)
・クエスト一覧やアイテム画像のために必要
チェックシートの交換所、限定クエスト一覧のチェック内容

TODO

交換の期限対応は後日対応

Comment

#21788 by wada at 2022-08-23T16:06:35; updated 2022-08-23T16:06:35

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

Comment

#21789 by kouho at 2022-08-23T19:24:35; updated 2022-08-23T19:24:35

@wada
以下確認お願いいたします。
1. プレゼントから猫缶と勲章を受け取っても加算されず交換所ページでも増えていませんでした。

2. 交換のページで表記揺れ
マグロ缶詰となっているがマグロ猫缶が正式名称かどうか
!スクリーンショット 2022-08-23 18

Comment

#21790 by wada at 2022-08-23T20:11:42; updated 2022-08-23T20:11:42

@kouho
確認ありがとうございます。
> プレゼントから猫缶と勲章を受け取っても加算されず交換所ページでも増えていませんでした。

管理画面から付与して受け取れるか確認お願いします。
また、log_del_present_boxの受け取りレコードの確認もお願いします。

受け取れたプレゼントログ

<img src="https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/166125302607039BStoiOmQ" width="600">

> 交換のページで表記揺れ

修正しました。

Comment

#21800 by kouho at 2022-08-24T13:53:38; updated 2022-08-24T13:53:38

@wada
対応ありがとうございます。表記揺れ修正確認しました。

プレゼントから猫缶と勲章を受け取っても加算されないと言う話でしたが、正確にはuser_itemのでデータは入りますが交換所でのデータは更新されず交換アイテムの所持数が0のままになるという状態です。

Comment

#21807 by wada at 2022-08-24T14:37:36; updated 2022-08-24T14:37:36

@kouho
了解しました。
こちらでは取得できるので、一度リセットして再度確認してみてください。

#環境設定
cd ./jr-web
git checkout 確認ブランチ or campaign_quest_0826
git merge origin/campaign_quest_0826
git merge origin/image_campaign_quest_0826
git merge origin/exchange_store_0826

cd ./jrye-db
git checkout campaign_quest_0826
php bin/csv2mysql.php jrye csv/master_campaign_quest.csv csv/master_campaign_quest_border.csv csv/master_campaign_quest_reward.csv csv/master_card.csv csv/master_card_text.csv csv/master_composition_bonus.csv csv/master_deluxe_bar_card.csv csv/master_deluxe_bar_card_exec_list.csv csv/master_item.csv
git checkout exchange_store_0826
php bin/csv2mysql.php jrye csv/master_card.csv csv/master_card_text.csv csv/master_composition_bonus.csv csv/master_exchange_store_item.csv csv/master_exchange_store_item_rate.csv

#データリセット
DELETE FROM user_item WHERE item_no IN(92,93);
DELETE FROM user_campaign_quest WHERE campaign_no = 2;
TRUNCATE TABLE log_user_exchange_store_item;

#確認状態に変更
UPDATE master_campaign_quest SET start_date = NOW() WHERE campaign_no = 2;
UPDATE user_campaign_quest SET clear_count = border_count WHERE campaign_no = 2;

Comment

#21808 by kouho at 2022-08-24T14:41:04; updated 2022-08-24T14:43:21

@wada
交換所について以下確認お願いいたします。
1. 指摘していた交換所での所持数表示はExchange_store.phpの以下の修正がいるようです。

$user_items = S_GetUserItem($rate_item_no);
→$user_items = S_GetUserItem($user_no, $rate_item_no);

2. 交換所で交換後のアイテム名が異なるアイテムが表示されました。
ニャッカを交換した場合とくぱんを交換と表示
とくぱんを交換した場合エメティアを交換と表示

Comment

#21809 by kouho at 2022-08-24T14:42:19; updated 2022-08-24T14:42:32

@wada
すみません、上で書いた2の受け取りアイテム名間違いについては対応済みでした。

Comment

#21810 by wada at 2022-08-24T16:23:13; updated 2022-08-24T16:23:13

@kouho
ありがとうございます。以下修正しました。
> 指摘していた交換所での所持数表示はExchange_store.phpの以下の修正がいるようです。
> $store_idと$user_noが逆になっていました。

Comment

#21811 by kouho at 2022-08-24T16:26:50; updated 2022-08-24T16:26:50

@wada
対応ありがとうございます。修正確認しました。

Comment

#21812 by kouho at 2022-08-24T16:35:43; updated 2022-08-24T16:35:43

@wada
確認して動作とデータは問題ないと思いますが以下は要望です。

・交換済み表示
制限回数分交換済みのアイテムはリストの一番下に並び替えると嬉しい。

・複数回交換するのが手間
例えばGOLDを60回交換するのにたくさんポチらないといけない。
ルシエ紹介でも同様なので改善できれば程度の希望。

Comment

#21813 by kouho at 2022-08-24T17:30:16; updated 2022-08-24T17:30:16

要望への対応は必須ではないです。その他は確認OKです。

kouho add_label at 2022-08-24T17:30:20: 皇甫 LGTM

Comment

#21818 by wada at 2022-08-24T18:30:09; updated 2022-08-24T18:30:09

@kouho
ありがとうございます。

wada merge at 2022-08-25T09:41:38: 【8/24】ニャッカCP交換所
wada close at 2022-08-25T09:41:38: Close
wada refer at 2022-08-25T15:51:01: 2226:【8/26 12:00】ニャッカ育成キャンペーン
wada delete_branch at 2022-08-26T11:37:11: exchange_store_0826

commit comment

#135 by kouho at 2022-08-24T14:57:29; lib/models/dao/log_user_exchange_store_item.php / old:None new:29
c679b4abbac56083a6c83b269b154af3a580f87b

@wada
交換したアイテムの交換可能数が変動せず

lib/models/dao/log_user_exchange_store_item.phpで
get_user_store_log関数で
$store_idと$user_noが逆になっていました。

Commit range

base970c53a9be8c308f58c5d4432131177350f6ab45
headc679b4abbac56083a6c83b269b154af3a580f87b
base branchjolly/jr-web:develop
head branchjolly/jr-web:exchange_store_0826

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed17 filesView patch
statusfile
modifiedlib/ajax/item_default.inc
modifiedlib/ajax/item_result.inc
modifiedlib/ajax/item_select.inc
modifiedlib/class/const/ItemConst.class.php
modifiedlib/models/Campaign_quest.php
addedlib/models/Exchange_store.php
addedlib/models/dao/log_user_exchange_store_item.php
modifiedlib/models/dao/master_campaign_quest.php
addedlib/models/dao/master_exchange_store_item.php
addedlib/models/dao/master_exchange_store_item_rate.php
addedlib/public/ExchangeStore-ExecExchange.inc
addedlib/public/ExchangeStore-ItemList.inc
modifiedpublic_html/index.php
modifiedpublic_html/js/JOLLYROGER.js
modifiedpublic_html/template/CardPieceShop-Default.tpl
addedpublic_html/template/ExchangeStore-ItemList.tpl
modifiedpublic_html/template/Quest-Campaign.tpl
Patch body is stored as a separate file to keep this page lightweight.

Review comment

#135 by kouho at 2022-08-24T14:57:29; lib/models/dao/log_user_exchange_store_item.php / old:None new:29
c679b4abbac56083a6c83b269b154af3a580f87b

@wada
交換したアイテムの交換可能数が変動せず

lib/models/dao/log_user_exchange_store_item.phpで
get_user_store_log関数で
$store_idと$user_noが逆になっていました。