Skip to content

A
abook_check
Opened by  

定期点検で点検中、点検済みの点検を削除しようとすると、処理中で止まる不具合の修正

deleteRoutineTaskReportメソッドで、reportStartDateの書式変換する際に、変換前の書式が想定しているものと異なっていたため、変換後の文字列がnullになっていて、それを使った際に、NullPointerExceptionが発生していた。そのため、終了時のコールバックが実行されずに、処理中のダイアログが消えない状態になっていた。

そのため、Dateの書式を事前に変更した。

Edited by Yujin Seo
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Kazuyuki Hida
    @hida-k started a discussion on the diff
    Resolved by Yujin Seo
    ABVJE_UI_Android/src/jp/agentec/abook/abv/ui/home/helper/ABookCheckWebViewHelper.java
    139 139
    140 140 if (reportType == Constant.ReportType.RoutineTask) {
    141 141 taskReportId = Integer.valueOf(param.get(ABookKeys.TASK_REPORT_ID));
    142 reportStartDate = param.get(ABookKeys.REPORT_START_DATE);
    142 reportStartDate = param.get(ABookKeys.REPORT_START_DATE).replace("T", " ").replace("Z", "");
    143 143 mOperationLogic.deleteRoutineTaskReport(operationId, contentId, taskKey, taskReportId, reportStartDate);
    • Kazuyuki Hida @hida-k commented
      Developer

      「yyyy-MM-ddTHH:㎜:ss」という書式で来ていたので、「yyyy-MM-ddTHH:㎜:ssZ」が来た場合も対応できるように書式を変更している。

      あまりきれいではない方法ではあるが、内部で使用している書式変換メソッドは、他でも使用している可能性があり影響を局所化する意味も含めて、このような手法を採用した。

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

    assigned to @seo

    assigned to @seo

    Toggle commit list
  • Yujin Seo @seo

    resolved all discussions

    resolved all discussions

    Toggle commit list
  • Yujin Seo @seo

    unmarked as a Work In Progress

    unmarked as a Work In Progress

    Toggle commit list
  • Yujin Seo @seo

    merged

    merged

    Toggle commit list
  • Yujin Seo @seo

    mentioned in commit a026403b

    mentioned in commit a026403b

    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