forked from CGM_Public/pretix_original
Export: Gracefully handle ExportEmptyError
This commit is contained in:
@@ -63,7 +63,7 @@ class ExportEmptyError(ExportError):
|
|||||||
pass
|
pass
|
||||||
|
|
||||||
|
|
||||||
@app.task(base=ProfiledEventTask, throws=(ExportError,), bind=True)
|
@app.task(base=ProfiledEventTask, throws=(ExportError, ExportEmptyError), bind=True)
|
||||||
def export(self, event: Event, fileid: str, provider: str, form_data: Dict[str, Any]) -> None:
|
def export(self, event: Event, fileid: str, provider: str, form_data: Dict[str, Any]) -> None:
|
||||||
def set_progress(val):
|
def set_progress(val):
|
||||||
if not self.request.called_directly:
|
if not self.request.called_directly:
|
||||||
@@ -94,7 +94,7 @@ def export(self, event: Event, fileid: str, provider: str, form_data: Dict[str,
|
|||||||
return str(file.pk)
|
return str(file.pk)
|
||||||
|
|
||||||
|
|
||||||
@app.task(base=ProfiledOrganizerUserTask, throws=(ExportError,), bind=True)
|
@app.task(base=ProfiledOrganizerUserTask, throws=(ExportError, ExportEmptyError), bind=True)
|
||||||
def multiexport(self, organizer: Organizer, user: User, device: int, token: int, fileid: str, provider: str,
|
def multiexport(self, organizer: Organizer, user: User, device: int, token: int, fileid: str, provider: str,
|
||||||
form_data: Dict[str, Any], staff_session=False) -> None:
|
form_data: Dict[str, Any], staff_session=False) -> None:
|
||||||
if device:
|
if device:
|
||||||
|
|||||||
@@ -2503,7 +2503,7 @@ class ExportMixin:
|
|||||||
|
|
||||||
class ExportDoView(EventPermissionRequiredMixin, ExportMixin, AsyncAction, TemplateView):
|
class ExportDoView(EventPermissionRequiredMixin, ExportMixin, AsyncAction, TemplateView):
|
||||||
permission = 'can_view_orders'
|
permission = 'can_view_orders'
|
||||||
known_errortypes = ['ExportError']
|
known_errortypes = ['ExportError', 'ExportEmptyError']
|
||||||
task = export
|
task = export
|
||||||
template_name = 'pretixcontrol/orders/export_form.html'
|
template_name = 'pretixcontrol/orders/export_form.html'
|
||||||
|
|
||||||
|
|||||||
@@ -1726,7 +1726,7 @@ class ExportMixin:
|
|||||||
|
|
||||||
|
|
||||||
class ExportDoView(OrganizerPermissionRequiredMixin, ExportMixin, AsyncAction, TemplateView):
|
class ExportDoView(OrganizerPermissionRequiredMixin, ExportMixin, AsyncAction, TemplateView):
|
||||||
known_errortypes = ['ExportError']
|
known_errortypes = ['ExportError', 'ExportEmptyError']
|
||||||
task = multiexport
|
task = multiexport
|
||||||
template_name = 'pretixcontrol/organizers/export_form.html'
|
template_name = 'pretixcontrol/organizers/export_form.html'
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user