PR

新大陸イベント_ブラバ対策 #387

iwamoto opened this at 2020-11-02T11:18:04; closed

Description

opened by iwamoto at 2020-11-02T11:18:04; updated 2023-05-15T10:14:43

ブラウザバック対策(新大陸イベント) #284

レアモンスターモード
メモキャッシュ登録からDB登録へ変更

DROP TABLE IF EXISTS `team_event_new_island_mode_limit`;
CREATE TABLE `team_event_new_island_mode_limit` (
  `id` int(11) NOT NULL AUTO_INCREMENT,
  `team_no` int(11) NOT NULL,
  `mode_start` int(11) NOT NULL DEFAULT '0' COMMENT 'レアモンスター出現モード開始',
  `mode_end` int(11) NOT NULL DEFAULT '0' COMMENT 'レアモンスター出現モード終了',
  `user_no` int(11) NOT NULL COMMENT 'フックモンスター討伐ユーザー',
  `nickname` char(50) NOT NULL DEFAULT 'no_name' COMMENT 'フックモンスター討伐ユーザーのニックネーム',
  PRIMARY KEY (`id`),
  KEY `team_no` (`team_no`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4;
iwamoto add_label at 2020-11-02T11:19:41: レビュー待ち

Comment

#8052 by iwamoto at 2020-11-02T11:20:22; updated 2020-11-02T11:20:22

@wada
こちらレビューをお願い致します。

Comment

#8079 by iwamoto at 2020-11-04T13:39:35; updated 2020-11-04T13:39:35

@wada
一応レアモードにlock 追加してみました
レビューお願いします。

Comment

#8081 by wada at 2020-11-04T14:35:43; updated 2020-11-04T14:35:43

今あるget_mode_limitにロック機能つけたメソッドを追加しても良いかも。

SELECT
	*
FROM
	`team_event_new_island_mode_limit`
WHERE
	`team_no` = %d AND `mode_start` <= %d AND `mode_end` >= %d
FOR UPDATE

Comment

#8088 by iwamoto at 2020-11-04T15:05:17; updated 2020-11-04T15:05:17

@wada
上記にて指摘頂きました箇所を修正しました。

#@get_lock を修正し
 -> @get_mode_limitにロック機能つけたメソッドへ変更

Comment

#8091 by wada at 2020-11-04T15:38:47; updated 2020-11-04T15:38:47

@iwamoto
レビューOKです。

iwamoto delete_label at 2020-11-04T15:44:39: レビュー待ち
iwamoto merge at 2020-11-04T23:19:46: 新大陸イベント_ブラバ対策
iwamoto close at 2020-11-04T23:19:46: Close

commit comment

#22 by wada at 2020-11-02T19:50:32; lib/models/dao/team_event_new_island_mode_limit.php / old:None new:64
65bf4bd47fa6d0371b6f070fa86da0f1bd1a50b0

mode_endだけだと過去のデータを全てとってきそう

commit comment

#24 by wada at 2020-11-04T14:33:33; lib/models/dao/team_event_new_island_mode_limit.php / old:None new:82
2e61df674a19921966da208d0021420b61ac0e46

これだとテーブルロックにならないかな?

Commit range

base8c9831a4f68dd7b36dbe7f1a9cec38b4c785d2f8
headd00d98c10f40711a819042d0cae1ad338e2b7061
base branchjolly/jr-web:develop
head branchjolly/jr-web:fix_new_island_284

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed11 filesView patch
statusfile
modifieddatabase/jrye.sql
addeddatabase/migrate/2020102901_team_event_new_island_mode_limit.sql
modifiedlib/SubFunc.inc
modifiedlib/SubFunc_mission.inc
modifiedlib/ajax/mission_default.inc
addedlib/models/Newisland_event_model.php
addedlib/models/dao/team_data.php
addedlib/models/dao/team_event_new_island_mode_limit.php
addedlib/models/dao/user_event_new_island.php
addedlib/models/newisland/Newisland_user.php
modifiedlib/public/Main.inc
Patch body is stored as a separate file to keep this page lightweight.

Review comment

#22 by wada at 2020-11-02T19:50:32; lib/models/dao/team_event_new_island_mode_limit.php / old:None new:64
65bf4bd47fa6d0371b6f070fa86da0f1bd1a50b0

mode_endだけだと過去のデータを全てとってきそう

Review comment

#24 by wada at 2020-11-04T14:33:33; lib/models/dao/team_event_new_island_mode_limit.php / old:None new:82
2e61df674a19921966da208d0021420b61ac0e46

これだとテーブルロックにならないかな?