Fall back to create_date when updated_at is null

For records that existed before the updated_at migration,
the field is null. Show create_date instead in detail views,
filter responses and update responses for both TestCase and
TestPlan.

Author: veenone

tcms/rpc/api/testcase.py

@@ -306,7 +306,11 @@ def filter(query=None):  # pylint: disable=redefined-builtin
         .distinct()
     )
 
-    return list(qs)
+    result = list(qs)
+    for item in result:
+        if item["updated_at"] is None:
+            item["updated_at"] = item["create_date"]
+    return result
 
 
 @permissions_required("testcases.view_testcase")
@@ -383,7 +387,7 @@ def update(case_id, values):
         result = model_to_dict(test_case, exclude=["component", "plan", "tag"])
         # b/c date may be None and model_to_dict() doesn't return it
         result["create_date"] = test_case.create_date
-        result["updated_at"] = test_case.updated_at
+        result["updated_at"] = test_case.updated_at or test_case.create_date
 
         # additional information
         result["case_status__name"] = test_case.case_status.name

tcms/rpc/api/testplan.py

@@ -87,7 +87,7 @@ def filter(query=None):  # pylint: disable=redefined-builtin
     if query is None:
         query = {}
 
-    return list(
+    result = list(
         TestPlan.objects.filter(**query)
         .values(
             "id",
@@ -111,6 +111,10 @@ def filter(query=None):  # pylint: disable=redefined-builtin
         .order_by("product", "id")
         .distinct()
     )
+    for item in result:
+        if item["updated_at"] is None:
+            item["updated_at"] = item["create_date"]
+    return result
 
 
 @permissions_required("testplans.add_testplantag")
@@ -183,7 +187,7 @@ def update(plan_id, values):
         # b/c value is set in the DB directly and if None
         # model_to_dict() will not return it
         result["create_date"] = test_plan.create_date
-        result["updated_at"] = test_plan.updated_at
+        result["updated_at"] = test_plan.updated_at or test_plan.create_date
         return result
 
     raise ValueError(list(form.errors.items()))

tcms/testcases/templates/testcases/get.html

@@ -71,7 +71,7 @@ <h2 class="card-pf-title kiwi-text-align-left">
 
                 <h2 class="card-pf-title kiwi-text-align-left">
                     <span class="fa fa-pencil"></span>{% trans 'Last modified' %}:
-                    {{ object.updated_at|default:"-" }}
+                    {{ object.updated_at|default:object.create_date }}
                 </h2>
 
                 <h2 class="card-pf-title kiwi-text-align-left">

tcms/testplans/templates/testplans/get.html

@@ -61,7 +61,7 @@ <h2 class="card-pf-title kiwi-text-align-left">
 
                 <h2 class="card-pf-title kiwi-text-align-left">
                     <span class="fa fa-pencil"></span>{% trans 'Last modified' %}:
-                    {{ object.updated_at|default:"-" }}
+                    {{ object.updated_at|default:object.create_date }}
                 </h2>
 
                 <h2 class="card-pf-title kiwi-text-align-left">