【1/16】必ずもらえるキャンペーン #3873

概要

https://docs.google.com/spreadsheets/d/19pVBwoWVaJdaTS-rzvplpWOpszDg57nR963u4JHXET0/edit?gid=2082506508#gid=2082506508

以下修正いたしました。
スクリプト内でsqlを保存したり特殊対応がされていますので、PresentBoxManagerを使わず個別で対応をしております。
global $user_infoを定義しているのはSeasonPass内で参照している箇所があるため、定義しています。
> 技玉メダルは直接プレゼントBOXに配布しないアイテムなので、直接配布の対応をお願いします。

備考

バナー画像は管理画面告知バナー設定で設定

@wada
レビューお願いいたします

@kouho
1,000コイン以上/技玉メダルのみの報酬が配布されません。
3,000コイン以上の報酬配布時に最初のユーザにしか技玉メダルが配布されません。

@wada
確認ありがとうございます。以下修正いたしました。
> 1,000コイン以上/技玉メダルのみの報酬が配布されません。
3,000コイン以上の報酬配布時に最初のユーザにしか技玉メダルが配布されません。

@kouho
$present_rewardsにループ回数分のアイテムが設定されています。

1人目：アイテム1個配布
2人目：アイテム2個配布
3人目：アイテム3個配布
...
n人目：アイテムn個配布

@wada
> $user_info['user_no']を上書きする理由はなんですか？

globalの$user_infoがS_SetUserSkillBallMedalChange関数のSeasonPass::getInstance()->track(SeasonPass::ACTION_GET_SKILL_BALL_MEDAL, 0, $medal);で使われていたので、上書きするようにしました。

> $present_rewardsにループ回数分のアイテムが設定されています。

こちら修正しました。$present_rewardsの初期化位置を修正しています。

@kouho
SeasonPass::getInstance()->track()を呼び出すということは、このスクリプトを使用するヤマダCPでの技玉メダル配布もシーズンパスのクエスト達成条件に含むということですね。

@wada
> SeasonPass::getInstance()->track()を呼び出すということは、このスクリプトを使用するヤマダCPでの技玉メダル配布もシーズンパスのクエスト達成条件に含むということですね。

はい。シーズンミッションの達成条件が技玉メダルを⚪︎枚入手するというものなので、キャンペーンで配布するものも条件に含めるものとして対応していました。

@wada
反映できておりませんでしのでプッシュしました
> こちら修正しました。$present_rewardsの初期化位置を修正しています。

@kouho
変数が増えることで管理が難しくなっているので、以下のように変更することは可能ですか？
対応の時間がなければそのままで大丈夫です。

//$present_rewards = [];

...

//変数を使用せず、直接modelクラスのメソッドを呼び出し
$present_rewards[] = $reward_item;
↓
$pre_model->get_presents_by_item($user_no, $item, $message, &$return);
or //処理内容は同じなのでどちらでも良い
$pre_model->get_presents($user_no, $item_type, $item_id, $count, $message, &$return);

...

//foreach内で実行済み
//if(!empty($present_rewards)){
//	$pre_model->get_presents_by_items($row['user_no'], $present_rewards, $reward['message'], $presents);
//}

@wada
度々すみません、以下も対応いたしました。

> 変数が増えることで管理が難しくなっているので、以下のように変更することは可能ですか？
対応の時間がなければそのままで大丈夫です。

@kouho
確認OKです。

@kouho
$user_info['user_no']を上書きする理由はなんですか？