Add support for deletion of detached signatures

apps/document_signatures/__init__.py

@@ -17,13 +17,14 @@ from django_gpg.exceptions import GPGDecryptionError
 from django_gpg.runtime import gpg
 from navigation.api import register_links
 
-from .links import (document_signature_upload, document_signature_download,
+from .links import (document_signature_delete, document_signature_download,
-    document_verify)
+    document_signature_upload, document_verify)
 from .models import DocumentVersionSignature
 from .permissions import (
     PERMISSION_DOCUMENT_VERIFY,
+    PERMISSION_SIGNATURE_DELETE,
+    PERMISSION_SIGNATURE_DOWNLOAD,
     PERMISSION_SIGNATURE_UPLOAD,
-    PERMISSION_SIGNATURE_DOWNLOAD
 )
 
 logger = logging.getLogger(__name__)
@@ -76,13 +77,14 @@ def document_post_save_hook(instance):
 
 
 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_DELETE,
+    PERMISSION_SIGNATURE_DOWNLOAD,
     PERMISSION_SIGNATURE_UPLOAD,
-    PERMISSION_SIGNATURE_DOWNLOAD
 ])

apps/document_signatures/links.py

@@ -5,8 +5,9 @@ from django.utils.translation import ugettext_lazy as _
 from .models import DocumentVersionSignature
 from .permissions import (
     PERMISSION_DOCUMENT_VERIFY,
+    PERMISSION_SIGNATURE_DELETE,
+    PERMISSION_SIGNATURE_DOWNLOAD,
     PERMISSION_SIGNATURE_UPLOAD,
-    PERMISSION_SIGNATURE_DOWNLOAD
 )
 
 
@@ -18,6 +19,7 @@ def doesnt_have_detached_signature(context):
     return DocumentVersionSignature.objects.has_detached_signature(context['object']) == False
 
 
-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_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_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_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_verify = {'text': _(u'signatures'), 'view': 'document_verify', 'args': 'object.pk', 'famfam': 'text_signature', 'permissions': [PERMISSION_DOCUMENT_VERIFY]}

apps/document_signatures/permissions.py

@@ -6,5 +6,6 @@ from permissions.models import PermissionNamespace, Permission
 
 document_signatures_namespace = PermissionNamespace('document_signatures', _(u'Document signatures'))
 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_DELETE = Permission.objects.register(document_signatures_namespace, 'signature_delete', _(u'Delete detached signatures'))
 PERMISSION_SIGNATURE_DOWNLOAD = Permission.objects.register(document_signatures_namespace, 'signature_download', _(u'Download detached signatures'))
+PERMISSION_SIGNATURE_UPLOAD = Permission.objects.register(document_signatures_namespace, 'signature_upload', _(u'Upload detached signatures'))

apps/document_signatures/urls.py

@@ -4,4 +4,5 @@ urlpatterns = patterns('document_signatures.views',
     url(r'^verify/(?P<document_pk>\d+)/$', 'document_verify', (), 'document_verify'),
     url(r'^upload/signature/(?P<document_pk>\d+)/$', 'document_signature_upload', (), 'document_signature_upload'),
     url(r'^download/signature/(?P<document_pk>\d+)/$', 'document_signature_download', (), 'document_signature_download'),
+    url(r'^document/(?P<document_pk>\d+)/signature/delete/$', 'document_signature_delete', (), 'document_signature_delete'),
 )

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
 
@@ -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))