Display errors while adding a metadata to a document

This commit is contained in:
Roberto Rosario
2014-10-30 15:16:58 -04:00
parent 8824b62be4
commit 199a07ef8f

View File

@@ -147,13 +147,18 @@ def metadata_add(request, document_id=None, document_id_list=None):
if form.is_valid(): if form.is_valid():
metadata_type = form.cleaned_data['metadata_type'] metadata_type = form.cleaned_data['metadata_type']
for document in documents: for document in documents:
document_metadata, created = DocumentMetadata.objects.get_or_create(document=document, metadata_type=metadata_type, defaults={'value': u''}) try:
if created: document_metadata, created = DocumentMetadata.objects.get_or_create(document=document, metadata_type=metadata_type, defaults={'value': u''})
messages.success(request, _(u'Metadata type: %(metadata_type)s successfully added to document %(document)s.') % { except Exception as exception:
'metadata_type': metadata_type, 'document': document}) messages.error(request, _(u'Error adding metadata type "%(metadata_type)s" to document: %(document)s; %(exception)s') % {
'metadata_type': metadata_type, 'document': document, 'exception': ', '.join(exception.messages)})
else: else:
messages.warning(request, _(u'Metadata type: %(metadata_type)s already present in document %(document)s.') % { if created:
'metadata_type': metadata_type, 'document': document}) messages.success(request, _(u'Metadata type: %(metadata_type)s successfully added to document %(document)s.') % {
'metadata_type': metadata_type, 'document': document})
else:
messages.warning(request, _(u'Metadata type: %(metadata_type)s already present in document %(document)s.') % {
'metadata_type': metadata_type, 'document': document})
if len(documents) == 1: if len(documents) == 1:
return HttpResponseRedirect(u'%s?%s' % ( return HttpResponseRedirect(u'%s?%s' % (