【6/3】朝ログインボーナス #4099

概要

https://docs.google.com/spreadsheets/d/1XIouoGN28e5YgBYAevg10YeqMrBstxYRAWJAfLZPMyg/edit?gid=1284699318#gid=1284699318

TODO

• [x] 期間中メインページアイコン表示対応
• [x] バナー画像

備考

multiple_login_bonusを複数設定できる形で対応しているため、DBテーブルの構造は変更なしです。

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

@kouho
EventScheduleConfig.class.phpの対応もお願いします。

@kouho
S_CheckMultipleLoginBonusCheck関数で引数に
user_infoがありますが、globalにもuser_infoが設定してあります。

function S_CheckMultipleLoginBonusCheck( $user_info ){
	global $_df , $_dbc, $user_info;

@kouho
S_CheckMultipleLoginBonusCheck関数のforeach内で期間チェックが重複しています。

	foreach( $_df['multiple_login_bonus'] as $event_no => $c ){
		// ログボが何回あるのか取得
		$slots = max(array_keys($c['present_end_time']));

		// イベント開催中かチェック
		if( $c['present_start_time'][1] <= $time && $c['present_end_time'][$slots] >= $time ){
			$cfg = $c;
			$multiple_login_bonus = $c;

			// ログボが何回あるのか取得
			$max_count = max( array_keys($multiple_login_bonus['present_end_time']) );

			// イベント開催中かチェック
			if( $multiple_login_bonus['present_start_time'][1] > $time || $multiple_login_bonus['present_end_time'][$max_count] < $time ){
				// イベント中ではない
				return;
			}

@kouho
仕様書では全国の秘宝がランダムで1つもらえるがとありますが、現在地の秘宝からランダムで1つという意味であっていますか？

@kouho
S_CheckMultipleLoginBonusCheck関数でプレゼント配布をforeachを抜けた後に一括でできますか？

@kouho
S_CheckMultipleLoginBonusCheck関数で、最後のreturnが現状意味のないコードになっています。

	// どのイベントも開催中でなければ何もしない
	if( ! $cfg ){
	    return;
	}

①：+ が入っています。
②：EventScheduleConfig.class.phpの対応もお願いします。
③：user_infoがありますが、globalにもuser_infoが設定してあります。
④：foreach内で期間チェックが重複しています。
⑤：現在地の秘宝からランダムで1つという意味であっていますか？
⑥：プレゼント配布をforeachを抜けた後に一括でできますか？
⑦：最後のreturnが現状意味のないコードになっています。

@wada
確認ありがとうございます。以下修正いたしました。現在地の秘宝ではなく、全国の秘宝を一つとプレゼント文言をお出かけ気分に合わせています。
> ①：+ が入っています。
> ③：user_infoがありますが、globalにもuser_infoが設定してあります。
> ④：foreach内で期間チェックが重複しています。
> ⑤：現在地の秘宝からランダムで1つという意味であっていますか？
> ⑥：プレゼント配布をforeachを抜けた後に一括でできますか？
> ⑦：最後のreturnが現状意味のないコードになっています。

@kouho
②以外確認しました。

トップページのバナー表示が1種類だけになっています。
カウントダウンログインボーナスのバナーがなくなっています。

@wada
以下も修正いたしました。
> ②：EventScheduleConfig.class.phpの対応もお願いします。
> トップページのバナー表示が1種類だけになっています。
カウントダウンログインボーナスのバナーがなくなっています。

@kouho
確認OKです。
EventScheduleConfigのお試しセット対応抜けていたため、対応してcommitしました。

@wada
対応ありがとうございます。確認OKです。
> お試しセット対応抜けていたため、対応してcommitしました。

@kouho
+ が入っています。