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
  • !293

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

報告書ロック機能の改善

内部的には、正しく動作するようにした。 ただ、ロックアイコンの表示や非表示が不安定(時々、正しく表示する)なので、それも含めて確認したいので、Jenkinsでびるどするためにマージしたいです。

  • Discussion 4
  • Commits 4
  • Changes 5
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Kazuyuki Hida
    @hida-k started a discussion on the diff Feb 24, 2023
    Resolved by Yujin Seo Feb 24, 2023
    ABVJE_BL/src/jp/agentec/abook/abv/bl/data/dao/TaskReportDao.java
    250 250 } else {
    251 251 sql.append(" WHERE task_key=? ");
    252 252 }
    253 if (loginId != null) {
    254 sql.append(" AND report_lock_user_id=? "); //自分以外のロックを変更しないため
    255 }
    • Kazuyuki Hida @hida-k commented Feb 24, 2023
      Developer

      この条件が余計なお世話だったので、loginIdが不要になった。

      Edited Feb 24, 2023
      この条件が余計なお世話だったので、loginIdが不要になった。
    Please register or sign in to reply
  • Kazuyuki Hida
    @hida-k started a discussion on the diff Feb 24, 2023
    Resolved by Yujin Seo Feb 24, 2023
    ABVJE_BL/src/jp/agentec/abook/abv/bl/data/dao/TaskReportDao.java
    251 251 sql.append(" WHERE task_key=? ");
    252 252 }
    253 if (loginId != null) {
    254 sql.append(" AND report_lock_user_id=? "); //自分以外のロックを変更しないため
    255 }
    256 253 args.add(reportStatus);
    257 254 args.add(reportLockUserId);
    258 255 args.add(reportLockUserName);
    259 256 args.add(reportLockTime);
    260 257 if (reportStartDate != null) {
    261 258 args.add(taskReportId);
    262 args.add(DateTimeUtil.toStringInTimeZone(reportStartDate, DateTimeFormat.yyyyMMddHHmmss_hyphen, "UTC"));
    259 int offset = TimeZone.getDefault().getOffset(reportStartDate.getTime());
    260 Date date = new Date(reportStartDate.getTime() - offset);
    261 String dateStr = DateTimeUtil.toStringInTimeZone(date, DateTimeFormat.yyyyMMddHHmmss_hyphen, "UTC");
    262 args.add(dateStr);
    • Kazuyuki Hida @hida-k commented Feb 24, 2023
      Developer

      UTCではない時刻が入ってくるので、TimeZoneのオフセットを引き算している。 toStringInTimeZone()に、TimeZone.getDefault.getId()を使ったら、逆方向に補正されたので、コードで明示的に補正した。 あまり、よくないことは自覚しています。

      Edited Feb 24, 2023
      UTCではない時刻が入ってくるので、TimeZoneのオフセットを引き算している。 `toStringInTimeZone()`に、TimeZone.getDefault.getId()を使ったら、逆方向に補正されたので、コードで明示的に補正した。 あまり、よくないことは自覚しています。
    Please register or sign in to reply
  • Kazuyuki Hida
    @hida-k started a discussion on the diff Feb 24, 2023
    Resolved by Yujin Seo Feb 24, 2023
    ABVJE_UI_Android/src/jp/agentec/abook/abv/ui/viewer/activity/HTMLWebViewActivity.java
    337 337 // AndroidOSが5以下のPANO_SERVER処理のため、置き換える必要がある。
    338 338 url = "/" + url;
    339 339 if (url.contains(ABookKeys.ABOOK_CHECK_API)) {
    340 String cmd = uri.getQueryParameter("cmd");
    341 if (cmd.contains("lockReport")) {
    342 String sd = uri.getQueryParameter("reportStartDate");
    343 Logger.d(TAG, "mCmd:" + cmd + ",reportStartDate: "+ sd+", CheckApiUrl: " + url);
    344 } else {
    345 Logger.d(TAG, "mCmd:" + cmd + ", CheckApiUrl: " + url);
    346 }
    • Kazuyuki Hida @hida-k commented Feb 24, 2023
      Developer

      デバッグのためのログ出力

      Edited Feb 24, 2023
      デバッグのためのログ出力
    Please register or sign in to reply
  • Kazuyuki Hida @hida-k

    assigned to @seo

    Feb 24, 2023

    assigned to @seo

    assigned to @seo
    Toggle commit list
  • Yujin Seo
    @seo started a discussion on an old version of the diff Feb 24, 2023
    Last updated by Kazuyuki Hida Feb 24, 2023
    ABVJE_UI_Android/src/jp/agentec/abook/abv/ui/viewer/activity/HTMLWebViewActivity.java
    337 337 // AndroidOSが5以下のPANO_SERVER処理のため、置き換える必要がある。
    338 338 url = "/" + url;
    339 339 if (url.contains(ABookKeys.ABOOK_CHECK_API)) {
    340 String cmd = uri.getQueryParameter("cmd");
    • Yujin Seo @seo commented Feb 24, 2023
      Developer

      AbookKeys.CMD AbookKeys.CMD_LOCK_REPORT AbookKeys.REPORT_START_DATE などを使うようにしてください。

      AbookKeys.CMD AbookKeys.CMD_LOCK_REPORT AbookKeys.REPORT_START_DATE などを使うようにしてください。
    • Kazuyuki Hida @hida-k

      changed this line in version 2 of the diff

      Feb 24, 2023

      changed this line in version 2 of the diff

      changed this line in [version 2 of the diff](https://gitlab.agentec.jp/abook_android/abook_check/merge_requests/293/diffs?diff_id=10963&start_sha=0586c611e29f25352c3d4c701ef88295cff74d1f#e939676582abd8862331550c9bb73fea7a0d4011_340_341)
      Toggle commit list
    Please register or sign in to reply
  • Kazuyuki Hida @hida-k

    added 1 commit

    • 88cc2571 - 文字列リテラルを用意された定数に変更

    Compare with previous version

    Feb 24, 2023

    added 1 commit

    • 88cc2571 - 文字列リテラルを用意された定数に変更

    Compare with previous version

    added 1 commit * 88cc2571 - 文字列リテラルを用意された定数に変更 [Compare with previous version](https://gitlab.agentec.jp/abook_android/abook_check/merge_requests/293/diffs?diff_id=10963&start_sha=0586c611e29f25352c3d4c701ef88295cff74d1f)
    Toggle commit list
  • Yujin Seo @seo

    merged

    Feb 24, 2023

    merged

    merged
    Toggle commit list
  • Yujin Seo @seo

    mentioned in commit 50646853

    Feb 24, 2023

    mentioned in commit 50646853

    mentioned in commit 50646853e3c66156263e01771cc7b050fc934103
    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!293
×

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.