added 1 commit

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

Compare with previous version

assigned to @seo

unmarked as a Work In Progress

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

たしかに

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

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

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

resolved all discussions

merged

mentioned in commit 382d667f