PR

【12/1】自動トレード解禁リリース #1351

wada opened this at 2021-11-30T11:25:44; closed

Description

opened by wada at 2021-11-30T11:25:44; updated 2026-03-06T09:33:29

概要

【11/24】トレード解禁状態の表示 #1313

ブランチ

web: release_1201
db: release_1201

必ず db からデプロイ

トレード登録漏れ確認

select * from master_card as c join master_trade_release_time as t on c.card_no = t.card_no 
where c.trade_flg = 0 and t.card_no is null;

トレード登録確認

select c.trade_flg, t.trade_release_time from master_card as c
left join master_trade_release_time as t on c.card_no = t.card_no
where t.card_no is not null and c.trade_flg = 1;

Comment

#15550 by wada at 2021-11-30T11:26:23; updated 2021-11-30T11:26:23

@iwamoto @kouho
確認をお願いします。

Comment

#15551 by iwamoto at 2021-11-30T15:20:39; updated 2021-11-30T15:20:39

@wada
確認しました。問題ないように思います。

iwamoto add_label at 2021-11-30T15:20:47: 岩本 LGTM
iwamoto delete_label at 2021-11-30T15:20:53: 岩本 LGTM
iwamoto add_label at 2021-11-30T15:20:54: 岩本 LGTM

Comment

#15555 by kouho at 2021-11-30T16:31:19; updated 2021-11-30T16:31:19

@wada
確認しました。ソースとデータの更新内容は問題ないと思いますが、トレード登録漏れ確認のクエリが気になりました。
実際にmaster_trade_release_timeからmaster_cardでtrade_flg=0になっているカードのデータを削除してみましたが検出できませんでした。
例えばこちらのクエリなら登録もれの確認をできるのではと思います。

SELECT * FROM master_card WHERE trade_flg = 0 AND card_no NOT IN ( SELECT card_no FROM master_trade_release_time WHERE 1 )

Comment

#15556 by wada at 2021-11-30T17:30:06; updated 2021-11-30T17:30:06

@kouho
トレード登録漏れ確認のクエリについて、
master_card.trade_flg=0のカードがmaster_trade_release_timeに登録漏れがないかどうかのクエリです。
トレード状態の確認であれば、トレードフラグと解禁日時を比較すると確認しやすいですね。

select c.trade_flg, t.trade_release_time from master_card as c
left join master_trade_release_time as t on c.card_no = t.card_no

Comment

#15557 by kouho at 2021-11-30T17:39:14; updated 2021-11-30T17:39:14

@wada
了解です。

kouho add_label at 2021-11-30T17:39:18: 皇甫 LGTM
wada merge at 2021-12-01T15:02:15: 【12/1】自動トレード解禁リリース
wada close at 2021-12-01T15:02:15: Close

Commit range

base8486cfd2b37bd0d552298d579f228af5f71c737b
head09e82fdf4781c22dbc22d6cfda7859d80d17371c
base branchjolly/jr-web:master
head branchjolly/jr-web:release_1201

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed21 filesView patch
statusfile
modifiedlib/SubFunc.inc
modifiedlib/SubFunc_tournament.inc
modifiedlib/cpanel/m_card_data.inc
modifiedlib/cpanel/m_card_search_result.inc
addedlib/models/Trade_release.php
addedlib/models/dao/master_trade_release_time.php
modifiedlib/public/Card-AlbumDetail.inc
modifiedlib/public/Card-Detail.inc
modifiedlib/public/Card-ListGroup.inc
modifiedlib/public/Friend-Trade.inc
modifiedlib/public/Friend-TradeApplication.inc
modifiedlib/public/Friend-TradeCheck.inc
modifiedpublic_html/css/design.css
addedpublic_html/img/common/btn_trade.png
addedpublic_html/img/common/btn_trade2.png
modifiedpublic_html/js/JOLLYROGER.js
modifiedpublic_html/js/jr_main.min.js
modifiedpublic_html/template/Card-AlbumDetail.tpl
modifiedpublic_html/template/Card-Detail.tpl
modifiedpublic_html/template/Friend-Trade.tpl
modifiedpublic_html/template/header_sp.tpl
Patch body is stored as a separate file to keep this page lightweight.