PR

自分の順位ページへのジャンプ修正 #710

iwamoto opened this at 2021-04-02T15:32:29; closed

Description

opened by iwamoto at 2021-04-02T15:32:29; updated 2024-04-15T09:43:05

ランキングページで自分の順位へ飛んだ際に
同じポイントのユーザーが多い状態で「自分の順位ページへ」のリンクを押すと、自分がいないページに遷移する

同じポイントのユーザーが多い状態で表示される場合、user_no が大きい順からの表示になる

Comment

#10152 by iwamoto at 2021-04-02T15:38:17; updated 2021-04-02T15:38:17

@wada @kouho
お手すきの際にレビューをお願い致します。
また、同率の際のランキング表示は現在のままでいいか、意見を頂ければと思います。

iwamoto add_label at 2021-04-02T15:52:38: レビュー待ち

Comment

#10165 by wada at 2021-04-05T15:03:02; updated 2021-04-05T15:08:29

@iwamoto
確認しました。
コメントはランキング順位のズレ対応についてです。

同ポイントの場合は、以下のパラメータあたりを参照して順位付けしたら良いかなと思います。

イベント中の戦闘勝利数(?)
イベント中の戦闘敗北数(?)
通算の戦闘勝利数(user_info.win)
デッキ攻撃力(user_info.battle_deck_power)
ユーザ登録日時(user_info.reg_date)

と思ったけど、ランキング報酬配布時と同じ方法にした方が良さそう。

Comment

#10173 by kouho at 2021-04-06T11:34:09; updated 2021-04-06T11:34:09

@iwamoto @wada
確認しました。入れた処理については問題ないと思いますが、
イベント内の同ポイントの順位について確認を失念していました。

SELECT `user_no`, `event_pt` FROM `user_event_pt` WHERE `event_pt` > 0 ORDER BY `event_pt` desc

event_ptが同一ポイントの順位はuser_noが大きい順にソートされる
これについてイベント内に記載している箇所がありませんでした。

他のイベントも確認してみたのですが、少なくともレイド・新大陸も同様の処理になっているように見えますので、
同一ポイントの順位の扱いについて改めて決めた方が良いのではと思いました。

Comment

#10174 by wada at 2021-04-06T12:03:14; updated 2021-04-06T12:03:14

今回の修正では同一ptの場合、ランキング報酬配布時の処理と同様に、
同じptなら同じ順位として扱う対応で良いと思います。
その場合、ランキングページの順位は同じ、並びはそのままになります。

同一ポイントの順位の扱いについては、
天下無双イベントはptが同じになりやすい(獲得ptが一定)ので、とりあえず天下無双だけ対応を入れて、他のイベントは開催時に(報酬配布処理が同じなら)同じ対応で良いと思います。

Comment

#10176 by iwamoto at 2021-04-06T15:25:47; updated 2021-04-06T15:25:47

@wada @kouho

Comment

#10184 by wada at 2021-04-06T17:28:39; updated 2021-04-06T17:28:39

@iwamoto
順位確認OKです。

wada add_label at 2021-04-06T17:28:45: 和田 LGTM

Comment

#10198 by kouho at 2021-04-07T10:14:54; updated 2021-04-07T10:14:54

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

kouho add_label at 2021-04-07T10:14:58: 皇甫 LGTM
iwamoto merge at 2021-04-19T18:16:32: 自分の順位ページへのジャンプ修正
iwamoto close at 2021-04-19T18:16:32: Close
iwamoto refer at 2021-04-19T18:22:15: 745:【4/20】リリース

commit comment

#44 by wada at 2021-04-05T14:33:13; lib/public/Ranking-Event.inc / old:27 new:None
38d9e6a5d72457476c1593526bb798f91c586587

$_temp['my_position']で、自分の(ランキング上での)正確な順位が取れているので、表示($_temp['my_rank'])も同じ順位にすることで表示順位のズレがなくなると思います。

$_temp['my_rank'] = $_temp['my_position'];

commit comment

#45 by wada at 2021-04-05T14:36:23; lib/public/Ranking-Event.inc / old:56 new:None
38d9e6a5d72457476c1593526bb798f91c586587

正確な順位位置が取れているので、ページトップは$num * n + 1にしたい。

$position = (ceil($position / $num) - 1) * $num + 1;

Commit range

base9e1a38e3357381711686f8afea98ad7d94451a5f
head38d9e6a5d72457476c1593526bb798f91c586587
base branchjolly/jr-web:develop
head branchjolly/jr-web:fix_rankingevent

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
modifiedlib/public/Ranking-Event.inc
modifiedpublic_html/template/Ranking-Event.tpl
Patch body is stored as a separate file to keep this page lightweight.

Review comment

#44 by wada at 2021-04-05T14:33:13; lib/public/Ranking-Event.inc / old:27 new:None
38d9e6a5d72457476c1593526bb798f91c586587

$_temp['my_position']で、自分の(ランキング上での)正確な順位が取れているので、表示($_temp['my_rank'])も同じ順位にすることで表示順位のズレがなくなると思います。

$_temp['my_rank'] = $_temp['my_position'];

Review comment

#45 by wada at 2021-04-05T14:36:23; lib/public/Ranking-Event.inc / old:56 new:None
38d9e6a5d72457476c1593526bb798f91c586587

正確な順位位置が取れているので、ページトップは$num * n + 1にしたい。

$position = (ceil($position / $num) - 1) * $num + 1;