From a359a171d3384d024f30f455674f686ede10087c Mon Sep 17 00:00:00 2001 From: Roberto Rosario Date: Wed, 14 Mar 2012 22:17:45 -0400 Subject: [PATCH] Add detached signature delete view --- apps/document_signatures/__init__.py | 13 +++--- apps/document_signatures/permissions.py | 1 + .../static/images/icons/pencil_delete.png | Bin 0 -> 1513 bytes apps/document_signatures/urls.py | 7 +-- apps/document_signatures/views.py | 41 ++++++++++++++++-- 5 files changed, 48 insertions(+), 14 deletions(-) create mode 100644 apps/document_signatures/static/images/icons/pencil_delete.png diff --git a/apps/document_signatures/__init__.py b/apps/document_signatures/__init__.py index ac4eb1413e..d413453dc2 100644 --- a/apps/document_signatures/__init__.py +++ b/apps/document_signatures/__init__.py @@ -19,9 +19,8 @@ from acls.api import class_permissions from .models import DocumentVersionSignature from .permissions import ( - PERMISSION_DOCUMENT_VERIFY, - PERMISSION_SIGNATURE_UPLOAD, - PERMISSION_SIGNATURE_DOWNLOAD + PERMISSION_DOCUMENT_VERIFY, PERMISSION_SIGNATURE_UPLOAD, + PERMISSION_SIGNATURE_DOWNLOAD, PERMISSION_SIGNATURE_DELETE ) logger = logging.getLogger(__name__) @@ -82,16 +81,16 @@ def document_post_save_hook(instance): document_signature_upload = {'text': _(u'upload signature'), 'view': 'document_signature_upload', 'args': 'object.pk', 'famfam': 'pencil_add', 'permissions': [PERMISSION_SIGNATURE_UPLOAD], 'conditional_disable': has_embedded_signature} document_signature_download = {'text': _(u'download signature'), 'view': 'document_signature_download', 'args': 'object.pk', 'famfam': 'disk', 'permissions': [PERMISSION_SIGNATURE_DOWNLOAD], 'conditional_disable': doesnt_have_detached_signature} +document_signature_delete = {'text': _(u'delete signature'), 'view': 'document_signature_delete', 'args': 'object.pk', 'famfam': 'pencil_delete', 'permissions': [PERMISSION_SIGNATURE_DELETE], 'conditional_disable': doesnt_have_detached_signature} document_verify = {'text': _(u'signatures'), 'view': 'document_verify', 'args': 'object.pk', 'famfam': 'text_signature', 'permissions': [PERMISSION_DOCUMENT_VERIFY]} register_links(Document, [document_verify], menu_name='form_header') -register_links(['document_verify', 'document_signature_upload', 'document_signature_download'], [document_signature_upload, document_signature_download], menu_name='sidebar') +register_links(['document_verify', 'document_signature_upload', 'document_signature_download', 'document_signature_delete'], [document_signature_upload, document_signature_download, document_signature_delete], menu_name='sidebar') DocumentVersion.register_pre_open_hook(1, document_pre_open_hook) DocumentVersion.register_post_save_hook(1, document_post_save_hook) class_permissions(Document, [ - PERMISSION_DOCUMENT_VERIFY, - PERMISSION_SIGNATURE_UPLOAD, - PERMISSION_SIGNATURE_DOWNLOAD + PERMISSION_DOCUMENT_VERIFY, PERMISSION_SIGNATURE_UPLOAD, + PERMISSION_SIGNATURE_DOWNLOAD, PERMISSION_SIGNATURE_DELETE ]) diff --git a/apps/document_signatures/permissions.py b/apps/document_signatures/permissions.py index e4f05bc1b7..8b5653cd60 100644 --- a/apps/document_signatures/permissions.py +++ b/apps/document_signatures/permissions.py @@ -8,3 +8,4 @@ document_signatures_namespace = PermissionNamespace('document_signatures', _(u'D PERMISSION_DOCUMENT_VERIFY = Permission.objects.register(document_signatures_namespace, 'document_verify', _(u'Verify document signatures')) PERMISSION_SIGNATURE_UPLOAD = Permission.objects.register(document_signatures_namespace, 'signature_upload', _(u'Upload detached signatures')) PERMISSION_SIGNATURE_DOWNLOAD = Permission.objects.register(document_signatures_namespace, 'signature_download', _(u'Download detached signatures')) +PERMISSION_SIGNATURE_DELETE = Permission.objects.register(document_signatures_namespace, 'signature_delete', _(u'Delete detached signatures')) diff --git a/apps/document_signatures/static/images/icons/pencil_delete.png b/apps/document_signatures/static/images/icons/pencil_delete.png new file mode 100644 index 0000000000000000000000000000000000000000..37b9ba6efb1c7df923a091739595c528c1e09837 GIT binary patch literal 1513 zcmVTiaBJYEoljn%dY^tv0P|QmqTMni_4J))gCDEl`_Q zTmcnCpw$sY))|L&mN&2G&O9c9EQ;bwP8jaH_x{d3=R5b_$Kl~T>f$esIXoT-klYx* z^qBx^bFvrw!+@7A&vR%^Cg>)NhGdAYJ^jH?4%YrK98hoHwa0NJycFu8Dnw0+M6f<^ z*e$K0u5i(IuM^^6!gF>XbR?N#5oa8ZHb*P2fZ{K6cxd4 zwV}jw38}R*QZ||&r@r;#V8Byf&;vpeV4R1Mr_K2GWI9Bb9o3d<1Vs!*^2WJOP~Yoh zt=yo%MS{m=1E-b)75m8KXV8CK{(J49N1bEkGD4*?-9-(?!Mq368w1vA470+ zSGbzN6EJT>L9cN3i1z~@H#wMANpNEi@Z$dA19d=whhRv|1cG;}o#31==UYnRVP4-q zd>|U|ue>Wf*9TW~*nBYo=5==mzt;%(1jlmj4|^w^))ByA6T#=N&cAE;y*SWS?f#s1 zgMXPnZPCbRO=d*cP7(5S8M06590!Tx@>@Jul2uq%NKNv+R6*~hri6V{sTV%|$MK1; zjfm8~H9TV`bfc0W3<>m3Ahorlsq6x(|2T?fyM5cNf6B61^W8YmcbzLa&t9FMvG+04 z!kFQPtfYAh;3_|lj=}=C>uY>rj);P0(ln?{V{l>1*Ra*r<;*%!W@b(Z1BZjuPVlt_ z)82_47ChM){n=Rae01#J1?RaFfZGi|Km%S)w=ahS?#fcgH!O&LWe%+8OOmIC1|Hj2 z(`4ZX3TJh$sJ%LCnD%|+Q<-q)9fDMQ9lVBtRp3R@XY4n%v3IPG@ySd?sfEwkeLx)8 z_S$8W>ClMW6s;}{ZF%M3B#BNv=j}T4%Z(ERNLA&~j!#7Z&rhbt!0ryHPkxl*NO?}z zJG?l)`wkZ-lghbkRjdcXa*O&D)vc@v;4D?0?sd5n@@wagdQ(4fG@wl*n+}{@22>vI z2LH2ropat&v{}KcH2s#8w-5F8j`1DV{wE+xr}-o{DIFN`5?m#lpeo--BLbA1_Xt;_ z%fVx_K?n^)%j^ug_qizp-BQ4!M0#L-SYP)4?Hdud{0&oT+Ke;<-_ZzX)h}?-EtZRW z8e+{+Y%5Oe9Zo2_TRLlN??~1UrO8S8GK1eK0q=&y3nCh9x4_1*v7HX6rcO*)IKmVU z$=*t3bA^QD!39eT&Xpx#ZtrmRIDf0Nt&1F@*%_(Vlj>D;>-O{%0Pw!B<7!Pk*cktn z?B(>B2vx+RTMrc6#LN@}DoQJ`fA3-E?%yx1S@HAb_t=;|_ICM{wb_~;9y-$1+G2_^ z7`#sKq3=d4W5^c8^yCf+^{7B)u+nen^i#FZvpNM?LoEcAL-#!7%m6AcXwsJvo9c~f)l zoU@mgD2r-lgM(Lo{BCnld*!dIa*l1h(df9zJX4^Q53o-tPe0=JRv=l->#>njo{iFF zhl|2k`8E$5-BQVcnx=1dTI&8%uH\d+)/$', 'document_verify', (), 'document_verify'), - url(r'^upload/signature/(?P\d+)/$', 'document_signature_upload', (), 'document_signature_upload'), - url(r'^download/signature/(?P\d+)/$', 'document_signature_download', (), 'document_signature_download'), + url(r'^document/(?P\d+)/signature/verify/$', 'document_verify', (), 'document_verify'), + url(r'^document/(?P\d+)/signature/upload/$', 'document_signature_upload', (), 'document_signature_upload'), + url(r'^document/(?P\d+)/signature/download/$', 'document_signature_download', (), 'document_signature_download'), + url(r'^document/(?P\d+)/signature/delete/$', 'document_signature_delete', (), 'document_signature_delete'), ) diff --git a/apps/document_signatures/views.py b/apps/document_signatures/views.py index 024f59ffb4..99b3fd1ec2 100644 --- a/apps/document_signatures/views.py +++ b/apps/document_signatures/views.py @@ -21,7 +21,7 @@ from acls.models import AccessEntry from django_gpg.api import SIGNATURE_STATES from . import (PERMISSION_DOCUMENT_VERIFY, PERMISSION_SIGNATURE_UPLOAD, - PERMISSION_SIGNATURE_DOWNLOAD) + PERMISSION_SIGNATURE_DOWNLOAD, PERMISSION_SIGNATURE_DELETE) from .forms import DetachedSignatureForm from .models import DocumentVersionSignature @@ -95,15 +95,15 @@ def document_signature_upload(request, document_pk): DocumentVersionSignature.objects.add_detached_signature(document, request.FILES['file']) messages.success(request, _(u'Detached signature uploaded successfully.')) return HttpResponseRedirect(next) - except Exception, msg: - messages.error(request, msg) + except Exception, exc: + messages.error(request, exc) return HttpResponseRedirect(previous) else: form = DetachedSignatureForm() return render_to_response('generic_form.html', { 'title': _(u'Upload detached signature for: %s') % document, - 'form_icon': 'key_delete.png', + 'submit_label': _(u'Submit'), 'next': next, 'form': form, 'previous': previous, @@ -133,3 +133,36 @@ def document_signature_download(request, document_pk): return HttpResponseRedirect(request.META['HTTP_REFERER']) return HttpResponseRedirect(request.META['HTTP_REFERER']) + + +def document_signature_delete(request, document_pk): + document = get_object_or_404(Document, pk=document_pk) + + try: + Permission.objects.check_permissions(request.user, [PERMISSION_SIGNATURE_DELETE]) + except PermissionDenied: + AccessEntry.objects.check_access(PERMISSION_SIGNATURE_DELETE, request.user, document) + + RecentDocument.objects.add_document_for_user(request.user, document) + + post_action_redirect = None + previous = request.POST.get('previous', request.GET.get('previous', request.META.get('HTTP_REFERER', '/'))) + next = request.POST.get('next', request.GET.get('next', post_action_redirect if post_action_redirect else request.META.get('HTTP_REFERER', '/'))) + + if request.method == 'POST': + try: + DocumentVersionSignature.objects.clear_detached_signature(document) + messages.success(request, _(u'Detached signature deleted successfully.')) + return HttpResponseRedirect(next) + except Exception, exc: + messages.error(request, _(u'Error while deleting the detached signature; %s') % exc) + return HttpResponseRedirect(previous) + + return render_to_response('generic_confirm.html', { + 'title': _(u'Are you sure you wish to delete the detached signature from document: %s') % document, + 'form_icon': 'pencil_delete.png', + 'next': next, + 'previous': previous, + 'object': document, + 'delete_view': True, + }, context_instance=RequestContext(request))