Archive / jolly / jr-web
PR

【10/6】覇戦リーグリセットクエリをスクリプト化 #2268

kouho opened this at 2022-09-08T18:40:42; closed

Description

opened by kouho at 2022-09-08T18:40:42; updated 2022-10-07T11:51:32

概要

覇戦リーグリセットクエリをスクリプト化
開催期間前でなければ実行できず、
実行しようとしたスクリプトがjackpot_flgと齟齬があれば実行できない
ファイル名に予選と本戦のローマ字を入れて間違いをしないようにする

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

JackPotWarConfigのtrial_start trial_endとflgを修正して実行
ENVIRONMENT=development PF=yamada IS_DOCKER=true php jackpot_reset_yosen.inc

JackPotWarConfigのfinal_start final_endとflgを修正して実行
ENVIRONMENT=development PF=yamada IS_DOCKER=true php jackpot_reset_honsen.inc

TODO

次回本戦開催時にjackpot_reset_honsen.incを利用

Comment

#22244 by kouho at 2022-09-08T18:41:06; updated 2022-09-08T18:41:06

@wada
リセットクエリをスクリプト化してみました。確認お願いいたします。

Comment

#22618 by wada at 2022-09-27T11:00:11; updated 2022-09-27T11:00:11

@kouho
トランザクションが開始されていないので、S_db_rollbackは機能していないようです。

以下の内容も出力して欲しいです。
・失敗した箇所
・成功
・実行確認用クエリの結果

Comment

#22716 by wada at 2022-09-29T16:44:24; updated 2022-09-29T16:45:00

@kouho
ロールバックのタイミングでエラー内容の出力をお願いします。

例：どこそこでしっぱいしました

スクリプトが成功した際に出力をお願いします。

例：せいこうしました

現状だと、スクリプトが最後まで実行されたのか、途中で失敗したのかわからない状態です。

kouho change_title at 2022-10-05T16:30:24: 覇戦リーグリセットクエリをスクリプト化 【10/6】覇戦リーグリセットクエリをスクリプト化

Comment

#22805 by kouho at 2022-10-05T18:29:09; updated 2022-10-05T18:29:09

@wada
確認ありがとうございます。以下修正対応しました。
> define_exec.incの中で、define.inc, SubFunc.incを読み込んでいるので、ここで記載しなくても問題ありません。

> dirname(__LINE__)はどの階層から実行しても.なので、__LINE__より__FILE__の方が正確です。

> 頭のUが余分に記載されています。

> トランザクションが開始されていないので、S_db_rollbackは機能していないようです。

> 以下の内容も出力して欲しいです。
> ・失敗した箇所の
> ・成功
> ・実行確認用クエリの結果

> ロールバックのタイミングでエラー内容の出力をお願いします。
> スクリプトが成功した際に出力をお願いします。
> 現状だと、スクリプトが最後まで実行されたのか、途中で失敗したのかわからない状態です。

Comment

#22825 by wada at 2022-10-06T12:58:20; updated 2022-10-06T12:58:20

@kouho
ありがとうございます。確認OKです。

wada add_label at 2022-10-06T12:58:23: 和田 LGTM
kouho merge at 2022-10-07T11:51:32: 【10/6】覇戦リーグリセットクエリをスクリプト化
kouho close at 2022-10-07T11:51:32: Close

commit comment

#138 by wada at 2022-09-27T10:39:53; lib/exec/jackpot_reset_honsen.inc / old:None new:7
492aedf6f666eebf686e3821178cb0b61addd045

@kouho
dirname(__LINE__)はどの階層から実行しても.なので、__LINE__より__FILE__の方が正確です。

commit comment

#139 by wada at 2022-09-27T10:43:12; lib/exec/jackpot_reset_honsen.inc / old:None new:8
492aedf6f666eebf686e3821178cb0b61addd045

@kouho
define_exec.incの中で、define.inc, SubFunc.incを読み込んでいるので、ここで記載しなくても問題ありません。
（require_onceなのでこのままでも良いです）

commit comment

#140 by wada at 2022-09-27T10:54:37; lib/exec/jackpot_reset_honsen.inc / old:None new:56
492aedf6f666eebf686e3821178cb0b61addd045

@kouho
頭のUが余分に記載されています。

Commit range

base0c6170ca1e7994e2b8a9303a2a81e2ab81c61163
head1f7fbb4b6ed5fcef51915e7b2b59f2645d53bd26
base branchjolly/jr-web:develop
head branchjolly/jr-web:scprit_jackpot

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
addedlib/exec/jackpot_reset_honsen.inc
addedlib/exec/jackpot_reset_yosen.inc
Patch body is stored as a separate file to keep this page lightweight.

Review comment

#138 by wada at 2022-09-27T10:39:53; lib/exec/jackpot_reset_honsen.inc / old:None new:7
492aedf6f666eebf686e3821178cb0b61addd045

@kouho
dirname(__LINE__)はどの階層から実行しても.なので、__LINE__より__FILE__の方が正確です。

Review comment

#139 by wada at 2022-09-27T10:43:12; lib/exec/jackpot_reset_honsen.inc / old:None new:8
492aedf6f666eebf686e3821178cb0b61addd045

@kouho
define_exec.incの中で、define.inc, SubFunc.incを読み込んでいるので、ここで記載しなくても問題ありません。
（require_onceなのでこのままでも良いです）

Review comment

#140 by wada at 2022-09-27T10:54:37; lib/exec/jackpot_reset_honsen.inc / old:None new:56
492aedf6f666eebf686e3821178cb0b61addd045

@kouho
頭のUが余分に記載されています。