[IMP] runbot: use batch date for first seen date

The log date can be useful to identify a pattern in apparition
(alway arround midnight, ...) but it is usually less relevant for
apparition when when usually focus on the "commit" so the batch date

Moreover the log date can be misleading when the build ran in faketime.
Even if it is not always the case, it is possible to have log_dates in
the future.

Author: Xavier-Do

runbot/models/build_error.py

@@ -39,6 +39,8 @@ class BuildErrorLink(models.Model):
     build_id = fields.Many2one('runbot.build', required=True, index=True)
     error_content_id = fields.Many2one('runbot.build.error.content', required=True, index=True, ondelete='cascade')
     log_date = fields.Datetime(string='Log date')
+    batch_id = fields.Many2one('runbot.batch', related='build_id.create_batch_id', string='Batch', store=True, readonly=True)
+    batch_date = fields.Datetime(related='batch_id.create_date', string='Batch date', store=True, readonly=True)
     host = fields.Char(related='build_id.host')
     dest = fields.Char(related='build_id.dest')
     version_id = fields.Many2one(related='build_id.version_id')
@@ -75,8 +77,8 @@ def _compute_seen(self):
             if error_link_ids:
                 first_error_link = error_link_ids[0]
                 last_error_link = error_link_ids[-1]
-                record.first_seen_date = first_error_link.log_date
-                record.last_seen_date = last_error_link.log_date
+                record.first_seen_date = first_error_link.batch_date
+                record.last_seen_date = last_error_link.batch_date
                 record.first_seen_build_id = first_error_link.build_id
                 record.last_seen_build_id = last_error_link.build_id
                 record.build_count = len(error_link_ids.build_id)

runbot/views/build_error_views.xml

@@ -40,7 +40,7 @@
                   <field name="team_id"/>
                   <field name="fixing_pr_id"/>
                   <field name="fixing_pr_url" widget="pull_request_url" invisible="not fixing_pr_id"/>
-                  <field name="fixing_bundle_url" invisible="not fixing_pr_id"/>
+                  <field name="fixing_bundle_url" invisible="not fixing_pr_id" widget="url"/>
                   <field name="active"/>
                 </group>
                 <group name="appearance_info" string="Appearance" col="2">