PR

【7/26】エンブレム設定枠の増設 #2067

wada opened this at 2022-07-11T12:24:12; closed

Description

opened by wada at 2022-07-11T12:24:12; updated 2022-07-27T19:21:29

概要

エンブレム設定枠の増設 #2054

更新内容

団員リストにエンブレム2つ表示(2つ設定時)
戦争系エンブレムをuser_setting_subから取得(type=3)
日常系はuser_info.team_emblemのまま

戦争エンブレムのデータ格納先が変わるため対応が必要
メンテナンスの有無で対応が変化

メンテナンスあり

本番反映後、メンテナンス中にスクリプト実行

メンテナンスなし

本番反映後、チームリーダーがチームページ遷移でメンバーのデータ自動整理
覇戦までに、調整されていないデータのスクリプト対応
整理前に、戦争エンブレムが日常枠に表示されないように対応
自動整理とスクリプトが同時実行されてエラーが不整合が起きないようにチームロック取得

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

user_info.team_emblemに戦争エンブレムを設定
Dual_emblem_preparationクラスの$start_timeを変更して、チームリーダー時に自動整理されるか確認

チームメンバー

user_info.team_emblemに戦争エンブレムが入っている状態で、チームページのエンブレム表示を確認

チームリーダー

ページ遷移後にデータ整理されているか
user_info.team_emblemの戦争エンブレムを0
user_setting_sub.type=3に戦争エンブレムを格納

TODO

メンテナンスせずに対応できるか調査
管理画面は別対応

wada change_title at 2022-07-11T14:04:35: [WIP]エンブレム設定枠の増設 【7/15】エンブレム設定枠の増設
wada add_label at 2022-07-11T14:04:38: レビュー待ち
wada change_title at 2022-07-11T14:06:43: 【7/15】エンブレム設定枠の増設 [WIP]【7/15】エンブレム設定枠の増設
wada delete_label at 2022-07-11T14:10:47: レビュー待ち
wada change_title at 2022-07-11T19:14:36: [WIP]【7/15】エンブレム設定枠の増設 【7/15】エンブレム設定枠の増設
wada add_label at 2022-07-11T19:14:38: レビュー待ち

Comment

#20795 by wada at 2022-07-11T19:14:46; updated 2022-07-11T19:14:46

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

wada add_label at 2022-07-13T10:00:46: 本家マージ
wada change_title at 2022-07-14T12:55:33: 【7/15】エンブレム設定枠の増設 【7/26】エンブレム設定枠の増設
wada merge at 2022-07-27T14:43:35: 【7/26】エンブレム設定枠の増設
wada close at 2022-07-27T14:43:35: Close

Comment

#21143 by wada at 2022-07-27T15:19:26; updated 2022-07-27T15:21:58

@kouho
ファイル名について修正しました。
logファイルはこちらでは出力できていました。

REPLACEについては、
移動済みの場合、戦争系以外を除外しているのでINSERTまで来ない想定です。
(日常系は移動しないため)

//戦争系のエンブレムが対象
$war_emblem_list = array_keys(array_filter($_df['emblem_sort'], function($item) { return $item == 1; }));

万が一INSERTまで処理が走ってしまった場合に、データが上書きされないよう、エラーが出るようにしています。
(エラーが出てS_Query内のS_db_rollback()が実行される)

Comment

#21144 by kouho at 2022-07-27T15:26:38; updated 2022-07-27T15:26:38

@wada
修正確認しました。REPLACEにつきましてもかしこまりました。

kouho add_label at 2022-07-27T15:26:42: 皇甫 LGTM

Comment

#21149 by kouho at 2022-07-27T19:01:04; updated 2022-07-27T19:01:04

@wada
すみません、マージ後に気づいたのですがチームに所属していない時に海賊団ページにアクセスすると以下のエラーが発生していました。確認お願いいたします。

!スクリーンショット 2022-07-27 18

Comment

#21150 by wada at 2022-07-27T19:08:17; updated 2022-07-27T19:08:17

@kouho
ありがとうございます。チーム無所属時の対応を入れました。

Comment

#21151 by kouho at 2022-07-27T19:21:29; updated 2022-07-27T19:21:29

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

commit comment

#132 by kouho at 2022-07-27T14:23:30; lib/exec/tool_team_emblem_data_change.inc / old:None new:100
848edab49dc801368a56a8c92e25247061f1e700

@wada
既に移動済みのエンブレムもある想定だと思いますのでRELACEにしたほうが良いかと思います。

commit comment

#133 by kouho at 2022-07-27T14:24:40; lib/exec/tool_team_emblem_data_change.inc / old:None new:137
848edab49dc801368a56a8c92e25247061f1e700

@wada
こちらではなぜかlogフォルダにテキストが出力されませんでした。
また、タイピングミスをしていますので一応修正しているほうが良いかと思います。
teram → team

Commit range

basea0532c4cdf35a769812df45059cfbcd81fba0e47
head848edab49dc801368a56a8c92e25247061f1e700
base branchjolly/jr-web:develop
head branchjolly/jr-web:dual_emblem

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed28 filesView patch
statusfile
modifiedlib/SubFunc.inc
modifiedlib/SubFunc_mission.inc
modifiedlib/ajax/teamWar_attack.inc
modifiedlib/ajax/teamWar_default.inc
modifiedlib/ajax/teamWar_defense.inc
modifiedlib/ajax/team_emblem.inc
modifiedlib/ajax/team_emblemResult.inc
modifiedlib/ajax/team_leaderSelect.inc
modifiedlib/ajax/team_removeSelect.inc
modifiedlib/ajax/team_subleaderSelect.inc
addedlib/class/const/UserSettingConst.class.php
addedlib/conf/class/TeamEmblemConfig.class.php
modifiedlib/exec/cpanel_daily_war_active_user_minutely.php
modifiedlib/exec/team_battle.php
modifiedlib/exec/team_merge_submit.inc
addedlib/exec/tool_team_emblem_data_change.inc
addedlib/models/Dual_emblem_preparation.php
modifiedlib/public/Team-EmblemRemoveResult.inc
modifiedlib/public/Team-EmblemSelect.inc
modifiedlib/public/Team-EmblemSelectCheck.inc
modifiedlib/public/Team-EmblemSelectResult.inc
modifiedlib/public/Team.inc
modifiedlib/public/TeamWar-Mvp.inc
addedpublic_html/img/info_jrye/1621/info1.jpg
addedpublic_html/img/info_jrye/1621/info2.jpg
addedpublic_html/img/info_jrye/1621/info3.jpg
modifiedpublic_html/js/JOLLYROGER.js
modifiedpublic_html/template/Team.tpl
Patch body is stored as a separate file to keep this page lightweight.

Review comment

#132 by kouho at 2022-07-27T14:23:30; lib/exec/tool_team_emblem_data_change.inc / old:None new:100
848edab49dc801368a56a8c92e25247061f1e700

@wada
既に移動済みのエンブレムもある想定だと思いますのでRELACEにしたほうが良いかと思います。

Review comment

#133 by kouho at 2022-07-27T14:24:40; lib/exec/tool_team_emblem_data_change.inc / old:None new:137
848edab49dc801368a56a8c92e25247061f1e700

@wada
こちらではなぜかlogフォルダにテキストが出力されませんでした。
また、タイピングミスをしていますので一応修正しているほうが良いかと思います。
teram → team