PR

【2/】覇戦リーグ予選 #1526

kouho opened this at 2022-02-02T17:49:35; closed

Description

opened by kouho at 2022-02-02T17:49:35; updated 2022-02-04T17:27:08

概要

【2/8】覇戦リーグ予選 #1478

  • 今回報酬が大幅に変更されています
  • 前回総試合数についての表示不具合があったので自動計算する形に修正しています

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

  • 各リーグ毎に海賊団の覇戦リーグのページでの表示がおかしくないか
  • メインページ看板がおかしくないか

TODO

  • 告知の作成が必要

Comment

#17050 by kouho at 2022-02-02T17:52:11; updated 2022-02-02T17:52:11

@wada @iwamoto
確認お願いいたします

Comment

#17075 by iwamoto at 2022-02-04T11:42:50; updated 2022-02-04T11:42:50

@kouho
確認しました。問題ないと思います。

ただ、仕様書側の本戦終了日時が正しくないので、合わせておいた方がいいと思います。

iwamoto add_label at 2022-02-04T11:42:54: 岩本 LGTM

Comment

#17081 by kouho at 2022-02-04T13:36:45; updated 2022-02-04T13:39:15

@iwamoto
確認ありがとうございます。仕様書側修正しておきました。

@wada
確認ありがとうございます。確かにループ回数が多すぎましたので何日間かをDateTime::diffを使って取得してその回数分ループする形に修正しました。

Comment

#17086 by wada at 2022-02-04T14:49:24; updated 2022-02-04T14:49:24

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

wada add_label at 2022-02-04T14:49:27: 和田 LGTM
kouho merge at 2022-02-04T17:27:08: 【2/】覇戦リーグ予選
kouho close at 2022-02-04T17:27:08: Close

commit comment

#111 by wada at 2022-02-04T11:50:46; lib/conf/class/JackpotWarConfig.class.php / old:None new:244
5e805b26c9c821a38a0a00794cd19ff062311c09

@kouho
数値の算出は問題ないのですが、条件が年月日('Ymd')だとループ数が膨大になる可能性があります。
呼び出すdefineは各ページで毎回読み込まれるため、他の数値を条件にして欲しいです。($weekやday_countなど)

例:start=20211215, end=20220103の場合、約9000回ループする

Commit range

base6bf5cf91de12d4d86b6c080e3a182f0930c8c33b
head5e805b26c9c821a38a0a00794cd19ff062311c09
base branchjolly/jr-web:develop
head branchjolly/jr-web:jackpot_1478

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed18 filesView patch
statusfile
modifiedlib/conf/class/JackpotWarConfig.class.php
modifiedlib/conf/define_y.inc
addedpublic_html/img/info_jrye/1252/item_1.jpg
addedpublic_html/img/info_jrye/1252/item_1_1.jpg
addedpublic_html/img/info_jrye/1252/item_1_2.jpg
addedpublic_html/img/info_jrye/1252/item_1_3.jpg
addedpublic_html/img/info_jrye/1252/item_2.jpg
addedpublic_html/img/info_jrye/1252/item_2_1.jpg
addedpublic_html/img/info_jrye/1252/item_2_2.jpg
addedpublic_html/img/info_jrye/1252/item_2_3.jpg
addedpublic_html/img/info_jrye/1252/item_3.jpg
addedpublic_html/img/info_jrye/1252/item_3_1.jpg
addedpublic_html/img/info_jrye/1252/item_3_2.jpg
addedpublic_html/img/info_jrye/1252/item_3_3.jpg
addedpublic_html/img/info_jrye/1252/item_4.jpg
addedpublic_html/img/info_jrye/1252/item_4_1.jpg
addedpublic_html/img/info_jrye/1252/item_4_2.jpg
addedpublic_html/img/info_jrye/1252/item_4_3.jpg
Patch body is stored as a separate file to keep this page lightweight.

Review comment

#111 by wada at 2022-02-04T11:50:46; lib/conf/class/JackpotWarConfig.class.php / old:None new:244
5e805b26c9c821a38a0a00794cd19ff062311c09

@kouho
数値の算出は問題ないのですが、条件が年月日('Ymd')だとループ数が膨大になる可能性があります。
呼び出すdefineは各ページで毎回読み込まれるため、他の数値を条件にして欲しいです。($weekやday_countなど)

例:start=20211215, end=20220103の場合、約9000回ループする