Description
概要
12/1 12時報酬スクリプト実行予定
対応内容
個人、チーム報酬配布
チームチャット成績通知
過去ランキングページの作成(1ヶ月のみ)
テスト方法、確認した内容
本番から202511のユーザデータとuser_infoを取得
報酬とチャット通知、ランキングページを確認
TODO
- [x] webデプロイ
12/1 12時報酬スクリプト実行予定
個人、チーム報酬配布
チームチャット成績通知
過去ランキングページの作成(1ヶ月のみ)
本番から202511のユーザデータとuser_infoを取得
報酬とチャット通知、ランキングページを確認
@kouho
レビューお願いします。
@wada
①/?M=Ranking&A=NewbieRanking&sid
sidが検証されていないため、URL書き換えで意図しない動作になるようです。
1. sid=10等出たらめな値を入れるとエラー
2. sid=202509等1ヶ月以上前の値を入れても取れてしまう
@wada
tool_newbie_ranking_reward.inc
報酬送信スクリプトreward_flgの更新タイミングをプレゼント送信後にしている方が安全だと思います。
if (!empty($update_newbie_data)) {
S_LoadModelClass('dao/user_newbie_ranking')->update_season_data($season_id, $update_newbie_data);
}@wada
上記指摘以外は問題ないと思います
@kouho
現状大きな問題はないため別PRで対応します。
> ①/?M=Ranking&A=NewbieRanking&sid
sidが検証されていないため、URL書き換えで意図しない動作になるようです。
スクリプト内でトランザクション設定しました。
> tool_newbie_ranking_reward.inc
報酬送信スクリプトreward_flgの更新タイミングをプレゼント送信後にしている方が安全だと思います。
| base | a8f150e9cedb876e95d4e885d501b9a7f110ad5b |
|---|---|
| head | 312c1b71f513e93ccba030960d554077d679fbfa |
| base branch | jolly/jr-web:master |
| head branch | jolly/jr-web:past_newbie_ranking |
| status | file |
|---|---|
| modified | lib/SubFunc.inc |
| modified | lib/conf/class/NewbieRankingConfig.class.php |
| added | lib/exec/tool_newbie_ranking_reward.inc |
| modified | lib/models/Newbie_ranking.php |
| modified | lib/models/dao/user_newbie_ranking.php |
| modified | lib/public/Ranking-NewbieRanking.inc |
| added | public_html/img/info_event/info_event_1615_1.jpg |
| added | public_html/img/info_event/info_event_1615_2.jpg |
| modified | public_html/template/Ranking-NewbieRanking.tpl |
| added | public_html/template/Ranking-NewbieRankingPast.tpl |