Skip to content

A
abook_check
Opened by  

一時保存した報告が消えてしまう不具合を修正

一時保存したデータが、サーバとの同期の際に削除されてしまっていた。 そこで、t_task_reportのlocal_saved_flgが立っているレコードについては、t_taskを含め、関連データは削除しないようにした。

Edited by Kazuyuki Hida
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Kazuyuki Hida @hida-k

    added 1 commit

    • 3e188be6 - 同期時に、サーバーにないレコードをすべて削除していたが、local_saved_flgが立っているレコードは削除しないようにした。

    Compare with previous version

    added 1 commit

    • 3e188be6 - 同期時に、サーバーにないレコードをすべて削除していたが、local_saved_flgが立っているレコードは削除しないようにした。

    Compare with previous version

    Toggle commit list
  • Kazuyuki Hida @hida-k

    assigned to @seo

    assigned to @seo

    Toggle commit list
  • Kazuyuki Hida @hida-k

    unmarked as a Work In Progress

    unmarked as a Work In Progress

    Toggle commit list
  • Yujin Seo
    @seo started a discussion on the diff
    Resolved by Yujin Seo
    ABVJE_UI_Android/src/jp/agentec/abook/abv/ui/home/activity/OperationListActivity.java
    1088 1088 }
    1089 1089 // サーバーから取得した作業情報がローカルに存在しないので削除する
    1090 1090 for (TaskDto taskDto : localTaskList) {
    1091 mOperationLogic.deleteTaskFileData(operationId, operationContentDto.contentId, taskDto.taskKey, TaskReportLevel.ReportType);
    1092 mTaskDao.delete(taskDto);
    1091 boolean localSaved = mTaskReportDao.isLocalSaved(taskDto.taskKey, 0, null);
    • Yujin Seo @seo commented
      Developer

      ここより。 984lineあたりの「if (isExistsTaskInList(localTaskList, serverTaskDto)) 」の中で一時保存内容を見て 「localTaskList.remove」をするようにするのが読みやすく明確だと思います。

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

      たしかに

      Edited
    • Kazuyuki Hida @hida-k commented
      Developer

      修正しようと思ったのですが、いろいろ不都合があるので、上記の変更の方がよさそうです。

                  if (isExistsTaskInList(localTaskList, serverTaskDto)) {
                // 作業の報告更新
                mTaskDao.update(serverTaskDto);
                localTaskList.remove(serverTaskDto);
            } else {
                // 作業の報告登録
                mTaskDao.insert(serverTaskDto);
            }

      ここだと、対象となるtaskDtoのlocalSavedFlgが取れないです。 serverTaskDtoはサーバーから来たdtoなので、localSavedFlgがtrueになっていることはないです。 また、isExistsTaskInList内のループで調べるとなると、DBへのアクセスが多くなってしまうので、それはそれでよくないと思います。

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

    resolved all discussions

    resolved all discussions

    Toggle commit list
  • Yujin Seo @seo

    merged

    merged

    Toggle commit list
  • Yujin Seo @seo

    mentioned in commit 382d667f

    mentioned in commit 382d667f

    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