Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

--replay option to make system export workbasket as it was at publish time. #597

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
68 changes: 63 additions & 5 deletions exporter/management/commands/dump_transactions.py
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
import logging
import os
import sys

Expand All @@ -6,6 +7,8 @@
from django.db.transaction import atomic
from lxml import etree

from common.models import Transaction
from common.models.utils import override_current_transaction
from common.serializers import validate_envelope
from exporter.serializers import MultiFileEnvelopeTransactionSerializer
from exporter.util import dit_file_generator
Expand All @@ -17,6 +20,8 @@
# VARIATION_SELECTOR enables emoji presentation
WARNING_SIGN_EMOJI = "\N{WARNING SIGN}\N{VARIATION SELECTOR-16}"

logger = logging.getLogger(__name__)


class Command(BaseCommand):
"""
Expand Down Expand Up @@ -48,7 +53,7 @@ def add_arguments(self, parser):
parser.add_argument(
"workbasket_ids",
help=(
"Override the default selection of APPROVED workbaskets "
"Override the default selection of 'all workbaskets with status APPROVED'"
"with a comma-separated list of workbasket ids."
),
nargs="*",
Expand All @@ -72,6 +77,13 @@ def add_arguments(self, parser):
action="store_true",
)

parser.add_argument(
"--replay",
help="Output transactions as at the time the workbasket was published, by setting current_transaction to "
"one before the specified workbasket.",
action="store_true",
)

@atomic
def handle(self, *args, **options):
workbasket_ids = options.get("workbasket_ids")
Expand All @@ -84,7 +96,6 @@ def handle(self, *args, **options):
if not workbaskets:
sys.exit("Nothing to upload: No workbaskets with status APPROVED.")

# transactions: will be serialized, then added to an envelope for uploaded.
transactions = workbaskets.ordered_transactions()

if not transactions:
Expand All @@ -106,6 +117,54 @@ def handle(self, *args, **options):

directory = options.get("directory", ".")

# 'replay' sets the current transaction to the one 'before' the specified workbasket
# to allow the system to export the data as it was when the workbasket was first published.
replay_from_tx = None
if options.get("replay"):
replay_from_tx = Transaction.objects.filter(
pk=transactions[0].pk - 1,
).last()

if replay_from_tx is None:
# No replay, just serialize the transactions.
if not self.do_serialize_envelopes(
directory,
envelope_id,
max_envelope_size,
transactions,
):
sys.exit(1)
else:
# Simulate latest_transaction being set to replay_from_tx
if not set(
workbaskets.values_list("status", flat=True).distinct(),
).issubset(WorkflowStatus.approved_statuses()):
sys.exit(
"Replay only applies to approved workbaskets.",
)

with override_current_transaction(replay_from_tx):
logging.debug("Replay from transaction: %s", replay_from_tx)
if not self.do_serialize_envelopes(
directory,
envelope_id,
max_envelope_size,
transactions,
):
sys.exit(1)

def do_serialize_envelopes(
self,
directory,
envelope_id,
max_envelope_size,
transactions,
):
"""
Serialize transactions to a series of files.
:return: True if no errors occurred.
"""
output_file_constructor = dit_file_generator(directory, envelope_id)
serializer = MultiFileEnvelopeTransactionSerializer(
output_file_constructor,
Expand All @@ -128,12 +187,11 @@ def handle(self, *args, **options):
validate_envelope(envelope_file)
except etree.DocumentInvalid:
self.stdout.write(
f"{envelope_file.name} {WARNING_SIGN_EMOJI}️ Envelope invalid:",
f"{envelope_file.name} {WARNING_SIGN_EMOJI} ️ XML invalid.",
)
else:
total_transactions = len(rendered_envelope.transactions)
self.stdout.write(
f"{envelope_file.name} \N{WHITE HEAVY CHECK MARK} XML valid. {total_transactions} transactions, serialized in {time_to_render:.2f} seconds using {envelope_file.tell()} bytes.",
)
if errors:
sys.exit(1)
return not errors
1 change: 0 additions & 1 deletion exporter/serializers.py
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,6 @@ def __init__(
self,
output_constructor: callable,
envelope_id=1,
benchmark=False,
*args,
**kwargs,
) -> None:
Expand Down