Skip to content

A
abook_check
Opened by  

ダッシュボードの実装

ダッシュボードのために、DB上のテーブルを追加し、DTOやDAOなどを作成しました。 また、ロック、アンロックで不備があったところを修正しました。

Edited by Kazuyuki Hida
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Kazuyuki Hida
    @hida-k started a discussion on the diff
    Resolved by Yujin Seo
    .gitmodules 0 → 100644
    1 [submodule "ABVJE_Launcher_Android/assets/dashboard"]
    2 path = ABVJE_Launcher_Android/assets/dashboard
    3 url = git@gitlab.agentec.jp:haccp_atform/app_dashboard.git
    Please register or sign in to reply
  • Kazuyuki Hida
    @hida-k started a discussion on the diff
    Resolved by Yujin Seo
    ABVJE_BL/src/jp/agentec/abook/abv/bl/acms/client/AcmsClient.java
    568 575
    569 576 AcmsMessageJSON json = new AcmsMessageJSON(result.httpResponseBody);
    570 577
    578 // JSのコールバックに渡すために必要な情報をストックしておく（暫定的な実装 fixme ）
    579 if (apiUrl.contains(AcmsApis.ApiSendTaskData)) {
    580 JSONObject res = new JSONObject(result.httpResponseBody);
    581 int reportStatus = res.has(ReportStatus) ? res.getInt(ReportStatus) : null;
    582 String message = res.has(Message) ? res.getString(Message) : null;
    583 ReportStatusLogic.TaskReportExtParam.stock(String.valueOf(reportStatus), message);
    584 }
    585
    • Kazuyuki Hida @hida-k commented
      Developer

      APIのレスポンスをJavaScript側に渡す経路がないので、大域変数で渡すように実装しました。 あまりよくない作り方だとは思うのですが、他に方法が見つかりませんでした。

      Edited
    • Yujin Seo @seo commented
      Developer

      詳しい実装までは見てないのであくまで意見ですけど「Callback」を使うのはどうかと思います。

      Edited
    Please register or sign in to reply
  • Kazuyuki Hida
    @hida-k started a discussion on the diff
    ABVJE_BL/src/jp/agentec/abook/abv/bl/acms/client/AcmsClient.java
    735 750 return new UnlockReportJSON(json);
    736 751 }
    737 752
    753 public DashboardStatusJSON getDashboardStatus(AcmsParameters sid) throws NetworkDisconnectedException, AcmsException {
    754 HttpResponse response = send(AcmsApis.ApiGetDashboardStatus, sid);
    755 String json = response.httpResponseBody;
    756 return new DashboardStatusJSON(json);
    757 }
    758
    • Kazuyuki Hida @hida-k commented
      Developer

      ダッシュボードで使用するAPI（getDashboardStatus()）です。

    Please register or sign in to reply
  • Kazuyuki Hida
    @hida-k started a discussion on the diff
    Resolved by Yujin Seo
    ABVJE_BL/src/jp/agentec/abook/abv/bl/acms/client/json/AcmsJSONParser.java
    133 133 }
    134 134 }
    135 135
    • Kazuyuki Hida @hida-k commented
      Developer

      JSONから値を取り出す処理は、AcmsJSONParserに集約しました。

      Edited
    Please register or sign in to reply
  • Kazuyuki Hida
    @hida-k started a discussion on the diff
    Resolved by Yujin Seo
    ABVJE_BL/src/jp/agentec/abook/abv/bl/acms/client/json/DashboardStatusJSON.java 0 → 100644
    1 package jp.agentec.abook.abv.bl.acms.client.json;
    Please register or sign in to reply
  • Kazuyuki Hida
    @hida-k started a discussion on the diff
    Resolved by Yujin Seo
    ABVJE_BL/src/jp/agentec/abook/abv/bl/acms/client/parameters/LockReportParameters.java
    10 13 private String sid;
    11 14 private String taskKey;
    12 15 private Long taskReportId;
    13 private Date reportStartDate;
    16 private String reportStartDate; // 書式の問題があるので、あえて文字列
    14 17 private String userId;
    • Kazuyuki Hida @hida-k commented
      Developer

      APIに渡すDateは決まった書式でなければならないうえ、事実上、文字列としてしか使用しないので、reportStartDateは、あえて文字列にしました。

      Edited
    Please register or sign in to reply
  • Kazuyuki Hida
    @hida-k started a discussion on the diff
    Resolved by Yujin Seo
    ABVJE_BL/src/jp/agentec/abook/abv/bl/acms/client/parameters/UnlockReportParameters.java
    9 12
    10 13 private String taskKey;
    11 14 private Long taskReportId;
    12 private Date reportStartDate;
    15 private String reportStartDate; // 書式の問題があるので、あえて文字列
    • Kazuyuki Hida @hida-k commented
      Developer

      APIに渡すDateは決まった書式でなければならないうえ、事実上、文字列としてしか使用しないので、reportStartDateは、あえて文字列にしました。

      Edited
    Please register or sign in to reply
  • Kazuyuki Hida
    @hida-k started a discussion on the diff
    Resolved by Yujin Seo
    ABVJE_BL/src/jp/agentec/abook/abv/bl/acms/client/parameters/UnlockReportParameters.java
    26 29
    27 String getTaskKey() {
    30 public String getTaskKey() {
    28 31 return taskKey;
    29 32 }
    30 33
    31 Long getTaskReportId() {
    34 public Long getTaskReportId() {
    32 35 return taskReportId;
    33 36 }
    34 37
    35 Date getReportStartDate() {
    38 public String getReportStartDate() {
    36 39 return reportStartDate;
    37 40 }
    38 41 }
    • Kazuyuki Hida @hida-k commented
      Developer

      CMSのフレームワークでは、publicなgetterが使われるのでpublicにしました。

      Edited
    Please register or sign in to reply
  • Kazuyuki Hida
    @hida-k started a discussion on the diff
    Resolved by Yujin Seo
    ABVJE_BL/src/jp/agentec/abook/abv/bl/acms/type/AcmsApis.java
    211 213 methodName.equals(ApiSendPushMessage) || methodName.equals(ApiGetPushMessages) || methodName.equals(ApiSendRoutineTaskData) ||
    212 214 methodName.equals(ApiGetMasterData) ||
    213 215 // カテゴリ選択機能、IO帳票で３つ追加
    214 methodName.equals(ApiOperationGroupMaster) || methodName.equals(ApiQuickReportSearch) || methodName.equals(ApiQuickReportRevision)) {
    216 methodName.equals(ApiOperationGroupMaster) || methodName.equals(ApiQuickReportSearch) || methodName.equals(ApiQuickReportRevision) ||
    217 // ダッシュボード、ロック追加
    218 methodName.equals(ApiGetDashboardStatus) || methodName.equals(ApiLockReport) || methodName.equals(ApiUnlockReport)) {
    • Kazuyuki Hida @hida-k commented
      Developer

      WebViewClientが反応するコマンドを追加しました。

      Edited
    Please register or sign in to reply
  • Kazuyuki Hida @hida-k

    changed the description

    changed the description

    Toggle commit list
  • Kazuyuki Hida @hida-k

    unmarked as a Work In Progress

    unmarked as a Work In Progress

    Toggle commit list
  • Kazuyuki Hida @hida-k

    assigned to @seo

    assigned to @seo

    Toggle commit list
  • Kazuyuki Hida @hida-k

    added 1 commit

    • eb63cad7 - サブモジュールの更新

    Compare with previous version

    added 1 commit

    • eb63cad7 - サブモジュールの更新

    Compare with previous version

    Toggle commit list
  • Yujin Seo
    @seo started a discussion on the diff
    Resolved by Yujin Seo
    ABVJE_BL/src/jp/agentec/abook/abv/bl/logic/ReportStatusLogic.java 0 → 100644
    68
    69 private static TaskReportExtParam instance = null;
    70
    71 public static void stock(String reportStatus, String message) {
    72 if (instance == null) {
    73 instance = new TaskReportExtParam();
    74 }
    75 instance.reportStatus = reportStatus;
    76 instance.message = message;
    77 }
    78
    79 public static TaskReportExtParam pick() {
    80 if (instance == null || instance.reportStatus == null) {
    81 return null;
    82 } else {
    83 return instance;
    • Yujin Seo @seo commented
      Developer

      pick()と同時に値を初期化するのはどうでしょうか？

      Edited
    • Kazuyuki Hida @hida-k commented
      Developer

      stock()で値を確保するのが、API呼び出し後（APIのレスポンスから値を取得）で、それを使うのが、いくつかスレッドをまたがったのちの、JavaScriptコールバック呼び出し時なので、よいタイミングで初期化できないのです。 JavaScript側では、渡されたパラメータがnullやundifinedなら無視するようになっているので、値が設定される前のpick()ではnullを返すのが妥当かなと考えました。

      Edited
    Please register or sign in to reply
  • Yujin Seo
    @seo started a discussion on the diff
    Resolved by Yujin Seo
    ABVJE_BL/src/jp/agentec/abook/abv/bl/data/tables/TTaskReportStatus.java 0 → 100644
    18
    19 sql.append("CREATE TABLE " + tableName +" ( ");
    20 sql.append(" task_key TEXT NOT NULL DEFAULT '',");
    21 sql.append(" task_code TEXT NOT NULL DEFAULT '',");
    22 sql.append(" task_report_id INTEGER,");
    23 sql.append(" report_start_date DATETIME,");
    24 sql.append(" operation_id BIGINT NOT NULL DEFAULT 0,");
    25 sql.append(" operation_name TEXT NOT NULL DEFAULT '',");
    26 sql.append(" report_status INTEGER NOT NULL DEFAULT 0,");
    27 sql.append(" untouched_flg INTEGER NOT NULL DEFAULT 0,"); // 未実施（0x01）
    28 sql.append(" working_flg INTEGER NOT NULL DEFAULT 0,"); // 作業中（0x02)
    29 sql.append(" complete_ok_flg INTEGER NOT NULL DEFAULT 0,"); // 作業完了 異常なし(0x04)
    30 sql.append(" complete_ng_flg INTEGER NOT NULL DEFAULT 0,"); // 作業完了 異常あり(0x08)
    31 sql.append(" incomplete_flg INTEGER NOT NULL DEFAULT 0,"); // 期限切れ(0x10)
    32 sql.append(" alert_flg INTEGER NOT NULL DEFAULT 0,"); // アラート(0x20)
    33 sql.append(" send_backed_flg INTEGER NOT NULL DEFAULT 0)"); // 差し戻し(0x40)
    • Yujin Seo @seo commented
      Developer

      他のテーブルとの関係やSelectを考慮してIndexを追加してください。

      Edited by Yujin Seo
    • Kazuyuki Hida @hida-k commented
      Developer

      了解しました。

      Edited
    • Kazuyuki Hida @hida-k commented
      Developer

      インデックスを付けました。 ついでに、開発の都合でDatabaseVersions.Ver1_0_7まで分けていたのをver1_0_4にまとめました。

      Edited
    Please register or sign in to reply
  • Yujin Seo @seo

    assigned to @hida-k

    assigned to @hida-k

    Toggle commit list
  • Kazuyuki Hida @hida-k

    added 1 commit

    • 67bed2d6 - サブモジュールの更新

    Compare with previous version

    added 1 commit

    • 67bed2d6 - サブモジュールの更新

    Compare with previous version

    Toggle commit list
  • Kazuyuki Hida @hida-k

    merged

    merged

    Toggle commit list
  • Kazuyuki Hida @hida-k

    mentioned in commit 658b9ed9

    mentioned in commit 658b9ed9

    Toggle commit list
  • Yujin Seo @seo

    assigned to @seo

    assigned to @seo

    Toggle commit list
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 to comment