Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
  • This project
    • Loading...
  • Sign in / Register
A
abook_check
  • Overview
    • Overview
    • Details
    • Activity
    • Cycle Analytics
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Charts
  • Issues 0
    • Issues 0
    • List
    • Board
    • Labels
    • Milestones
  • Merge Requests 0
    • Merge Requests 0
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
    • Charts
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • abook_android
  • abook_check
  • Merge Requests
  • !278

Merged
Opened Feb 14, 2023 by Kazuyuki Hida@hida-k 
  • Report abuse
Report abuse

一時保存時した報告のロック処理を修正

一時保存した報告は、サーバ上にはないので、ロック要求が失敗する。 そのため、それに該当する場合は、ロック要求をスルーすることにした。

Edited Feb 14, 2023 by Kazuyuki Hida
  • Discussion 2
  • Commits 2
  • Changes 2
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Kazuyuki Hida
    @hida-k started a discussion on the diff Feb 14, 2023
    Resolved by Yujin Seo Feb 14, 2023
    ABVJE_UI_Android/src/jp/agentec/abook/abv/ui/common/activity/ABVContentViewActivity.java
    1106 1108 String taskKey = abookCheckParam.get(TaskKey);
    1107 1109 Date startDate = dateOrNull(abookCheckParam.get("reportStartDate"));
    1108 // ロック
    1109 LockReportLogic.Result r = LockReportLogic.newInstance().lock(taskKey, taskReportId, startDate);
    1110 // JSコールバック
    1111 afterABookCheckApi(mCmd, mTaskKey, r.getResult(), r.getMessage(), r.getExtParam().json());
    1110 TaskReportDao dao = AbstractDao.getDao(TaskReportDao.class);
    1111 if (dao.isLocalSaved(mTaskKey, taskReportId, DateTimeUtil.toString(startDate, DateTimeFormat.yyyyMMddHHmmss_hyphen))) {
    1112 // ローカルにしかない場合はロックしない
    1113 afterABookCheckApi(mCmd, mTaskKey, 0, "", "{'reportStatus':0}");
    1114 } else {
    1115 // ロック
    1116 LockReportLogic.Result r = LockReportLogic.newInstance().lock(taskKey, taskReportId, startDate);
    1117 // JSコールバック
    1118 afterABookCheckApi(mCmd, mTaskKey, r.getResult(), r.getMessage(), r.getExtParam().json());
    1119 }
    • Kazuyuki Hida @hida-k commented Feb 14, 2023
      Developer

      一時保存はロックをしないようにした。

      Edited Feb 14, 2023
      一時保存はロックをしないようにした。
    Please register or sign in to reply
  • Kazuyuki Hida
    @hida-k started a discussion on the diff Feb 14, 2023
    Resolved by Yujin Seo Feb 14, 2023
    ABVJE_UI_Android/src/jp/agentec/abook/abv/ui/common/activity/ABVContentViewActivity.java
    1117 // JSコールバック
    1118 afterABookCheckApi(mCmd, mTaskKey, r.getResult(), r.getMessage(), r.getExtParam().json());
    1119 }
    1112 1120 } else if (mCmd.equals(ABookKeys.CMD_UNLOCK_REPORT)) {
    1113 1121 String taskKey = abookCheckParam.get(TaskKey);
    1114 1122 Date startDate = dateOrNull(abookCheckParam.get("reportStartDate"));
    1115 1123 // アンロック
    1116 1124 UnlockReportLogic.Result r = UnlockReportLogic.newInstance().unlock(taskKey, taskReportId, startDate);
    1117 1125 // JSコールバック
    1118 afterABookCheckApi(mCmd, mTaskKey, r.getResult(), r.getMessage(), r.getExtParam().json());
    1126 if (r.getMessage().length() > 0) {
    1127 // アンロックの失敗は無視
    1128 afterABookCheckApi(mCmd, mTaskKey, 0, "", "{'reportStatus':0}");
    1129 } else {
    1130 afterABookCheckApi(mCmd, mTaskKey, r.getResult(), r.getMessage(), r.getExtParam().json());
    1131 }
    • Kazuyuki Hida @hida-k commented Feb 14, 2023
      Developer

      その場合、ロックしていないので、アンロックが失敗するが、それを無視するようにした。 実際、アンロックが失敗した場合、なにをすることもできないので、アンロック処理は行ったうえで、単純にエラーを無視するようにした。

      Edited Feb 14, 2023
      その場合、ロックしていないので、アンロックが失敗するが、それを無視するようにした。 実際、アンロックが失敗した場合、なにをすることもできないので、アンロック処理は行ったうえで、単純にエラーを無視するようにした。
    Please register or sign in to reply
  • Kazuyuki Hida @hida-k

    assigned to @seo

    Feb 14, 2023

    assigned to @seo

    assigned to @seo
    Toggle commit list
  • Kazuyuki Hida @hida-k

    unmarked as a Work In Progress

    Feb 14, 2023

    unmarked as a Work In Progress

    unmarked as a **Work In Progress**
    Toggle commit list
  • Yujin Seo @seo

    resolved all discussions

    Feb 14, 2023

    resolved all discussions

    resolved all discussions
    Toggle commit list
  • Yujin Seo @seo

    merged

    Feb 14, 2023

    merged

    merged
    Toggle commit list
  • Yujin Seo @seo

    mentioned in commit 3224b543

    Feb 14, 2023

    mentioned in commit 3224b543

    mentioned in commit 3224b543aaeb18e121ec6b44b4705034e4591528
    Toggle commit list
  • Write
  • Preview
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or sign in to comment
Yujin Seo
Assignee
Yujin Seo @seo
Assign to
None
Milestone
None
Assign milestone
Time tracking
2
2 participants
Reference: abook_android/abook_check!278
×

Revert this merge request

Switch branch
Cancel
A new branch will be created in your fork and a new merge request will be started.
×

Cherry-pick this merge request

Switch branch
Cancel
A new branch will be created in your fork and a new merge request will be started.