create provenance for finished visits (only if is_finished changes)

smash/web/redcap_connector.py

@@ -206,8 +206,17 @@ class RedcapConnector(object):
                                                                          status=Appointment.APPOINTMENT_STATUS_SCHEDULED)
                         for smasch_appointment in smasch_appointments:
                             smasch_appointment.mark_as_finished()
-                            smasch_appointment.visit.is_finished = True
-                            smasch_appointment.visit.save()
+                            if smasch_appointment.visit.is_finished != True:
+                                description = u'{} changed from "{}" to "{}"'.format('is_finished', smasch_appointment.visit.is_finished, True)
+                                p = Provenance(modified_table=Visit._meta.db_table,
+                                            modified_table_id=smasch_appointment.visit.id,
+                                            modification_author=self.importer_user,
+                                            previous_value=smasch_appointment.visit.is_finished,
+                                            new_value=True,
+                                            modification_description=description,
+                                            modified_field='is_finished')
+                                smasch_appointment.visit.is_finished = True
+                                smasch_appointment.visit.save()
                         if visit.virus is not None or visit.virus_inconclusive:
                             if visit.visit_number == 1 and subject.virus_test_1 != visit.virus:
                                 subject.virus_test_1 = visit.virus