Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #26227 from edx/ork/MICROBA-905_add-history-to-certificate-invalidation

Browse Source 
MICROBA-905 Add django-simple-history  to CertificateInvalidation model
This commit is contained in:
Olivia Ruiz-Knott
2021-02-04 10:11:05 -05:00
committed by GitHub
parent d81215c1e9 d33d9a66c8
commit 016b1ce3f7
2 changed files with 49 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

42
lms/djangoapps/certificates/migrations/0018_historicalcertificateinvalidation.py Normal file
View File

@@ -0,0 +1,42 @@
# Generated by Django 2.2.17 on 2021-01-28 20:33
from django.conf import settings
from django.db import migrations, models
import django.db.models.deletion
import django.utils.timezone
import model_utils.fields
import simple_history.models
class Migration(migrations.Migration):
dependencies = [
migrations.swappable_dependency(settings.AUTH_USER_MODEL),
('certificates', '0017_add_mode_20201118_1725'),
]
operations = [
migrations.CreateModel(
name='HistoricalCertificateInvalidation',
fields=[
('id', models.IntegerField(auto_created=True, blank=True, db_index=True, verbose_name='ID')),
('created', model_utils.fields.AutoCreatedField(default=django.utils.timezone.now, editable=False, verbose_name='created')),
('modified', model_utils.fields.AutoLastModifiedField(default=django.utils.timezone.now, editable=False, verbose_name='modified')),
('notes', models.TextField(default=None, null=True)),
('active', models.BooleanField(default=True)),
('history_id', models.AutoField(primary_key=True, serialize=False)),
('history_date', models.DateTimeField()),
('history_change_reason', models.CharField(max_length=100, null=True)),
('history_type', models.CharField(choices=[('+', 'Created'), ('~', 'Changed'), ('-', 'Deleted')], max_length=1)),
('generated_certificate', models.ForeignKey(blank=True, db_constraint=False, null=True, on_delete=django.db.models.deletion.DO_NOTHING, related_name='+', to='certificates.GeneratedCertificate')),
('history_user', models.ForeignKey(null=True, on_delete=django.db.models.deletion.SET_NULL, related_name='+', to=settings.AUTH_USER_MODEL)),
('invalidated_by', models.ForeignKey(blank=True, db_constraint=False, null=True, on_delete=django.db.models.deletion.DO_NOTHING, related_name='+', to=settings.AUTH_USER_MODEL)),
],
options={
'verbose_name': 'historical certificate invalidation',
'ordering': ('-history_date', '-history_id'),
'get_latest_by': 'history_date',
},
bases=(simple_history.models.HistoricalChanges, models.Model),
),
]

7
lms/djangoapps/certificates/models.py
View File

@@ -514,6 +514,13 @@ class CertificateInvalidation(TimeStampedModel):
notes = models.TextField(default=None, null=True)
active = models.BooleanField(default=True)
# This is necessary because CMS does not install the certificates app, but
# this code is run when other models in this file are imported there (or in
# common code). Simple History will attempt to connect to the installed
# model in the certificates app, which will fail.
if 'certificates' in apps.app_configs:
history = HistoricalRecords()
class Meta(object):
app_label = "certificates"