PR

【5/14】集金レポート #788

wada opened this at 2021-05-07T11:11:10; closed

Description

opened by wada at 2021-05-07T11:11:10; updated 2024-07-11T11:12:01

jolly/jr-rzst#23
集金ページ下部にレポート表示
表示するデータは#786で用意

wada change_title at 2021-05-07T11:29:29: [WIP]【5/14】集金レポート 【5/14】集金レポート
wada add_label at 2021-05-07T11:29:49: レビュー待ち

Comment

#10745 by wada at 2021-05-07T11:31:51; updated 2021-05-07T11:31:51

@iwamoto @kouho
レビューお願いします。
確認に必要なデータは準備pr(#786)で作成してください。
正しく表示されるかエラーないか確認お願いします。

Comment

#10756 by kouho at 2021-05-07T15:50:38; updated 2021-05-07T15:50:38

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

kouho add_label at 2021-05-07T15:50:44: 皇甫 LGTM

Comment

#10758 by kouho at 2021-05-07T15:55:18; updated 2021-05-07T15:55:18

@wada
集金レポート自体には問題ないですが、既存のエラーで集金できる建物がない状態で集金ページに行くと以下のエラーが出るようですので、S_GetUserBuildGold_MIN関数内で$return=0の初期化を入れている方が良いかと思いました。

"/ajax.php?M=main&A=listGold&_=1620370342983","ua":"Mozilla/5.0 (Linux; Android 6.0; Nexus 5 Build/MRA58N) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/90.0.4430.85 Mobile Safari/537.36","req":{"M":"main","A":"listGold","_":"1620370342983"},"code":0,"func":"{closure}","trace":"SubFunc(7921)","msg":"Undefined variable: return"}

Comment

#10768 by iwamoto at 2021-05-07T17:52:21; updated 2021-05-07T17:52:21

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

iwamoto add_label at 2021-05-07T17:52:27: 岩本 LGTM

Comment

#10772 by wada at 2021-05-07T18:37:32; updated 2021-05-07T18:37:32

@iwamoto @kouho
集金レポート下に表示されるキャラがまだ未実装なので非表示にしました。
また、集金可能な建物を所持していない場合に、次の集金時間が1970-01-01 09:00:00で表示されていた件も対応入れました。

Comment

#10773 by kouho at 2021-05-07T19:04:13; updated 2021-05-07T19:04:13

@wada
確認しました。OKかと思います。

Comment

#10775 by iwamoto at 2021-05-07T19:37:52; updated 2021-05-07T19:37:52

@wada
確認しました、大丈夫です。

wada merge at 2021-05-16T15:58:55: 【5/14】集金レポート
wada close at 2021-05-16T15:58:55: Close
wada refer at 2021-05-18T17:12:42: 819:【5/19】リリース

Commit range

base718b87b0da968696903ff4b59140f2b58aa6b053
head2faca0ea23bbcd4b7b67beab17e2978da0ba2216
base branchjolly/jr-web:develop
head branchjolly/jr-web:gold_collect_report

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed24 filesView patch
statusfile
modifiedlib/SubFunc.inc
modifiedlib/ajax/main_listGold.inc
modifiedlib/ajax/main_listGoldExec.inc
addedlib/exec/daily_user_collect_gold.inc
addedlib/exec/daily_user_data.sh
modifiedlib/exec/del_log_gold_collect.inc
addedlib/exec/tool_get_all_player_best_gold_collect.inc
addedlib/exec/tool_get_all_player_collect.inc
modifiedlib/mobile/Main__ListGold.inc
modifiedlib/mobile/Main__ListGoldComplete.inc
modifiedlib/mobile/Main__ListGoldCompleteResult.inc
modifiedlib/public/Main-ListGoldComplete.inc
addedpublic_html/chart_js/Chart.min.js
modifiedpublic_html/css/design.css
addedpublic_html/img/gold_collect_report/icon_calendar.png
addedpublic_html/img/gold_collect_report/report_back_middle.png
addedpublic_html/img/gold_collect_report/report_back_top.png
addedpublic_html/img/gold_collect_report/report_back_under.png
addedpublic_html/img/gold_collect_report/report_best.png
addedpublic_html/img/gold_collect_report/report_today.png
addedpublic_html/img/gold_collect_report/report_yesterday.png
addedpublic_html/img/gold_collect_report/secretary/1.png
modifiedpublic_html/js/JOLLYROGER.js
modifiedpublic_html/template/Main.tpl
Patch body is stored as a separate file to keep this page lightweight.