mirror of
https://github.com/pretix/pretix.git
synced 2026-05-18 17:24:03 +00:00
Improve error messages
This commit is contained in:
@@ -223,7 +223,10 @@ class OutboundSyncProvider:
|
|||||||
|
|
||||||
def get_field_value(self, inputs, mapping_entry):
|
def get_field_value(self, inputs, mapping_entry):
|
||||||
key = mapping_entry["pretix_field"]
|
key = mapping_entry["pretix_field"]
|
||||||
required_input, label, ptype, enum_opts, getter = self.data_fields.get(key)
|
try:
|
||||||
|
required_input, label, ptype, enum_opts, getter = self.data_fields[key]
|
||||||
|
except KeyError:
|
||||||
|
raise SyncConfigError(['Field "%s" is not valid for %s. Please check your %s settings.' % (key, "/".join(inputs.keys()), self.display_name)])
|
||||||
input = inputs[required_input]
|
input = inputs[required_input]
|
||||||
val = getter(input)
|
val = getter(input)
|
||||||
if isinstance(val, list):
|
if isinstance(val, list):
|
||||||
@@ -271,7 +274,7 @@ class OutboundSyncProvider:
|
|||||||
|
|
||||||
def sync_order(self, order):
|
def sync_order(self, order):
|
||||||
if not self.order_valid_for_sync(order):
|
if not self.order_valid_for_sync(order):
|
||||||
logger.debug("Skipping order (not valid for sync)", order)
|
logger.debug("Skipping order %r (not valid for sync)", order)
|
||||||
return
|
return
|
||||||
|
|
||||||
logger.debug("Syncing order %r", order)
|
logger.debug("Syncing order %r", order)
|
||||||
|
|||||||
@@ -438,7 +438,7 @@ class OrderDataSyncLogentrytype(OrderLogEntryType):
|
|||||||
if obj and 'external_link_href' in obj and 'external_link_display_name' in obj
|
if obj and 'external_link_href' in obj and 'external_link_display_name' in obj
|
||||||
))
|
))
|
||||||
|
|
||||||
return mark_safe(super().display(logentry, data) + "".join("<p>" + link + "</p>" for link in links))
|
return mark_safe(escape(super().display(logentry, data)) + "".join("<p>" + link + "</p>" for link in links))
|
||||||
|
|
||||||
|
|
||||||
@log_entry_types.new_from_dict({
|
@log_entry_types.new_from_dict({
|
||||||
@@ -449,7 +449,7 @@ class OrderDataSyncErrorLogentrytype(OrderLogEntryType):
|
|||||||
errmes = data["error"]
|
errmes = data["error"]
|
||||||
if not isinstance(errmes, list):
|
if not isinstance(errmes, list):
|
||||||
errmes = [errmes]
|
errmes = [errmes]
|
||||||
return mark_safe(escape(self.plain) + "".join("<p>" + escape(msg) + "</p>" for msg in errmes))
|
return mark_safe(escape(super().display(logentry, data)) + "".join("<p>" + escape(msg) + "</p>" for msg in errmes))
|
||||||
|
|
||||||
|
|
||||||
@receiver(signal=logentry_display, dispatch_uid="pretixcontrol_logentry_display")
|
@receiver(signal=logentry_display, dispatch_uid="pretixcontrol_logentry_display")
|
||||||
|
|||||||
Reference in New Issue
Block a user