PR

【1/22】新人ランキングシーズン修正 #4473

wada opened this at 2026-01-19T14:22:33; closed

Description

opened by wada at 2026-01-19T14:22:33; updated 2026-01-27T14:56:05

概要

シーズン決定処理で+1 monthがうまく機能していなかったため修正
・毎月29日以降は2月がスキップされる

修正手順

  • 1: デプロイ
  • 2: user_newbie_rankingをバックアップ
  • 3: user_newbie_rankingから202602移行のレコードを削除
DELETE FROM user_newbie_ranking WHERE season_id >= 202602;
  • 4: tool_fix_newbie_ranking_season.php実行で202602移行を再設定
ENVIRONMENT=production PF=yamada php tool_fix_newbie_ranking_season.php

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

本番からuser_settingとuser_newbie_rankingテーブルをローカルにインポート
user_newbie_rankingを複製して、202602移行のレコードを削除
tool_fix_newbie_ranking_season.phpを実行
新旧user_newbie_rankingを比較して更新を確認

Comment

#43948 by wada at 2026-01-19T14:22:44; updated 2026-01-19T14:22:44

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

Comment

#44050 by kouho at 2026-01-26T10:55:47; updated 2026-01-26T10:55:47

@wada
手順に沿ってローカル環境でスクリプト実行をしましたが、
user_newbie_rankingから202602移行のレコードを削除した後に、
user_newbie_rankingをSELECTするので、season_id 202602以降が更新されないようです。

kouho add_label at 2026-01-26T10:56:38: 皇甫 LGTM
kouho delete_label at 2026-01-26T10:56:40: 皇甫 LGTM
kouho add_label at 2026-01-26T10:56:42: コメントあり

Comment

#44059 by kouho at 2026-01-26T11:29:10; updated 2026-01-26T11:29:10

@wada
ログを仕込んだところsqlクエリは発行されますが、実行時にLock wait timeout exceeded; try restarting transactionのエラーが起きていました。僕の環境の問題の可能性があります。

kouho add_label at 2026-01-26T14:01:21: 皇甫 LGTM

Comment

#44063 by kouho at 2026-01-26T14:01:33; updated 2026-01-26T14:01:33

@wada
実行確認しました。確認OKです。

wada merge at 2026-01-27T14:56:00: 【1/22】新人ランキングシーズン修正
wada close at 2026-01-27T14:56:00: Close
wada delete_branch at 2026-01-27T14:56:05: fix_newbie_month

commit comment

#322 by kouho at 2026-01-26T10:35:51; lib/exec/tool_fix_newbie_ranking_season.php / old:None new:63
9d592a7fb757b4b2795a41bc6b4e6d8d937c05e2

@wada
テスト時のuser_newbie_ranking_copy のままになっていました

Commit range

basec200212cfc99c4eadcc5992fc1f6c9a0b7c714fa
head90e7e080b80de3f20b2b702f78e06f72427cc8e9
base branchjolly/jr-web:master
head branchjolly/jr-web:fix_newbie_month

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed2 filesView patch
statusfile
modifiedlib/conf/class/NewbieRankingConfig.class.php
addedlib/exec/tool_fix_newbie_ranking_season.php
Patch body is stored as a separate file to keep this page lightweight.

Review comment

#322 by kouho at 2026-01-26T10:35:51; lib/exec/tool_fix_newbie_ranking_season.php / old:None new:63
9d592a7fb757b4b2795a41bc6b4e6d8d937c05e2

@wada
テスト時のuser_newbie_ranking_copy のままになっていました