PR

【11/27】新人ランキングシーズン切り替え #4384

wada opened this at 2025-11-21T14:45:19; closed

Description

opened by wada at 2025-11-21T14:45:19; updated 2025-12-01T11:34:55

概要

仕様書

12/1 12時報酬スクリプト実行予定

対応内容

個人、チーム報酬配布
チームチャット成績通知
過去ランキングページの作成(1ヶ月のみ)

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

本番から202511のユーザデータとuser_infoを取得
報酬とチャット通知、ランキングページを確認

TODO

  • [x] webデプロイ

Comment

#43216 by wada at 2025-11-21T14:45:28; updated 2025-11-21T14:45:28

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

wada change_priority at 2025-11-21T14:45:32: No priority:highest

Comment

#43236 by kouho at 2025-11-21T17:02:11; updated 2025-11-21T17:02:11

@wada
①/?M=Ranking&A=NewbieRanking&sid
sidが検証されていないため、URL書き換えで意図しない動作になるようです。

1. sid=10等出たらめな値を入れるとエラー
2. sid=202509等1ヶ月以上前の値を入れても取れてしまう

Comment

#43242 by kouho at 2025-11-25T10:03:12; updated 2025-11-25T10:03:12

@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);
		}
kouho add_label at 2025-11-25T10:03:20: コメントあり

Comment

#43280 by kouho at 2025-11-27T11:03:30; updated 2025-11-27T11:03:30

@wada
上記指摘以外は問題ないと思います

Comment

#43304 by wada at 2025-12-01T10:18:39; updated 2025-12-01T10:18:39

@kouho

現状大きな問題はないため別PRで対応します。

> ①/?M=Ranking&A=NewbieRanking&sid
sidが検証されていないため、URL書き換えで意図しない動作になるようです。

スクリプト内でトランザクション設定しました。

> tool_newbie_ranking_reward.inc
報酬送信スクリプトreward_flgの更新タイミングをプレゼント送信後にしている方が安全だと思います。

wada merge at 2025-12-01T11:34:42: 【11/27】新人ランキングシーズン切り替え
wada close at 2025-12-01T11:34:42: Close
wada delete_branch at 2025-12-01T11:34:47: past_newbie_ranking

Commit range

basea8f150e9cedb876e95d4e885d501b9a7f110ad5b
head312c1b71f513e93ccba030960d554077d679fbfa
base branchjolly/jr-web:master
head branchjolly/jr-web:past_newbie_ranking

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed10 filesView patch
statusfile
modifiedlib/SubFunc.inc
modifiedlib/conf/class/NewbieRankingConfig.class.php
addedlib/exec/tool_newbie_ranking_reward.inc
modifiedlib/models/Newbie_ranking.php
modifiedlib/models/dao/user_newbie_ranking.php
modifiedlib/public/Ranking-NewbieRanking.inc
addedpublic_html/img/info_event/info_event_1615_1.jpg
addedpublic_html/img/info_event/info_event_1615_2.jpg
modifiedpublic_html/template/Ranking-NewbieRanking.tpl
addedpublic_html/template/Ranking-NewbieRankingPast.tpl
Patch body is stored as a separate file to keep this page lightweight.