Fix link resolution in the document signature list view. This view exposes 3 navigable objects, use 'resolved_object' for correct link resolution.

This commit is contained in:
Roberto Rosario
2016-03-26 16:27:47 -04:00
parent 5de63c4477
commit 0ffe20befd
17 changed files with 33 additions and 46 deletions

View File

@@ -45,5 +45,6 @@ link_checkout_info = Link(
icon='fa fa-shopping-cart', permissions=( icon='fa fa-shopping-cart', permissions=(
permission_document_checkin, permission_document_checkin_override, permission_document_checkin, permission_document_checkin_override,
permission_document_checkout permission_document_checkout
), text=_('Check in/out'), view='checkouts:checkout_info', args='object.pk' ), text=_('Check in/out'), view='checkouts:checkout_info',
args='resolved_object.pk'
) )

View File

@@ -19,5 +19,6 @@ link_comment_delete = Link(
) )
link_comments_for_document = Link( link_comments_for_document = Link(
icon='fa fa-comment', permissions=(permission_comment_view,), icon='fa fa-comment', permissions=(permission_comment_view,),
text=_('Comments'), view='comments:comments_for_document', args='object.pk' text=_('Comments'), view='comments:comments_for_document',
args='resolved_object.pk'
) )

View File

@@ -16,7 +16,7 @@ def is_not_root_node(context):
link_document_index_list = Link( link_document_index_list = Link(
icon='fa fa-list-ul', text=_('Indexes'), icon='fa fa-list-ul', text=_('Indexes'),
view='indexing:document_index_list', args='object.pk' view='indexing:document_index_list', args='resolved_object.pk'
) )
link_index_main_menu = Link( link_index_main_menu = Link(
icon='fa fa-list-ul', text=_('Indexes'), view='indexing:index_list' icon='fa fa-list-ul', text=_('Indexes'), view='indexing:index_list'

View File

@@ -23,13 +23,11 @@ from .links import (
link_document_version_signature_download, link_document_version_signature_download,
link_document_version_signature_list, link_document_version_signature_list,
link_document_version_signature_upload, link_document_version_signature_upload,
link_document_version_signature_verify
) )
from .permissions import ( from .permissions import (
permission_document_version_signature_delete, permission_document_version_signature_delete,
permission_document_version_signature_download, permission_document_version_signature_download,
permission_document_version_signature_upload, permission_document_version_signature_upload,
permission_document_version_signature_verify,
permission_document_version_signature_view, permission_document_version_signature_view,
) )
@@ -75,7 +73,6 @@ class DocumentSignaturesApp(MayanAppConfig):
model=Document, permissions=( model=Document, permissions=(
permission_document_version_signature_delete, permission_document_version_signature_delete,
permission_document_version_signature_download, permission_document_version_signature_download,
permission_document_version_signature_verify,
permission_document_version_signature_view, permission_document_version_signature_view,
permission_document_version_signature_upload, permission_document_version_signature_upload,
) )
@@ -143,7 +140,6 @@ class DocumentSignaturesApp(MayanAppConfig):
menu_sidebar.bind_links( menu_sidebar.bind_links(
links=( links=(
link_document_version_signature_upload, link_document_version_signature_upload,
link_document_version_signature_verify,
), sources=(DocumentVersion,) ), sources=(DocumentVersion,)
) )
post_delete.connect( post_delete.connect(

View File

@@ -9,8 +9,6 @@ from .permissions import (
permission_document_version_signature_delete, permission_document_version_signature_delete,
permission_document_version_signature_download, permission_document_version_signature_download,
permission_document_version_signature_upload, permission_document_version_signature_upload,
permission_document_version_signature_verify,
permission_document_version_signature_view
) )
@@ -51,12 +49,7 @@ link_document_version_signature_download = Link(
) )
link_document_version_signature_upload = Link( link_document_version_signature_upload = Link(
#permissions=(permission_document_version_signature_upload,), #permissions=(permission_document_version_signature_upload,),
text=_('Upload signature'), view='signatures:document_version_signature_upload', text=_('Upload signature'),
args='resolved_object.pk' view='signatures:document_version_signature_upload',
)
link_document_version_signature_verify = Link(
icon='fa fa-certificate',
#permissions=(permission_document_version_signature_verify,),
text=_('Verify signatures'), view='signatures:document_verify',
args='resolved_object.pk' args='resolved_object.pk'
) )

View File

@@ -12,10 +12,6 @@ permission_document_version_signature_view = namespace.add_permission(
name='document_version_signature_view', name='document_version_signature_view',
label=_('View details of document signatures') label=_('View details of document signatures')
) )
permission_document_version_signature_verify = namespace.add_permission(
name='document_version_signature_verify',
label=_('Verify document signatures')
)
permission_document_version_signature_delete = namespace.add_permission( permission_document_version_signature_delete = namespace.add_permission(
name='document_version_signature_delete', name='document_version_signature_delete',
label=_('Delete detached signatures') label=_('Delete detached signatures')

View File

@@ -24,10 +24,6 @@ urlpatterns = patterns(
DocumentVersionSignatureListView.as_view(), DocumentVersionSignatureListView.as_view(),
name='document_version_signature_list' name='document_version_signature_list'
), ),
url(
r'^documents/version/(?P<document_pk>\d+)/signature/verify/$',
'document_verify', name='document_version_signature_verify'
),
url( url(
r'^documents/version/(?P<pk>\d+)/signature/upload/$', r'^documents/version/(?P<pk>\d+)/signature/upload/$',
'document_version_signature_upload', 'document_version_signature_upload',

View File

@@ -26,7 +26,6 @@ from .forms import DetachedSignatureForm, DocumentVersionSignatureDetailForm
from .models import DetachedSignature, SignatureBaseModel from .models import DetachedSignature, SignatureBaseModel
from .permissions import ( from .permissions import (
permission_document_version_signature_view, permission_document_version_signature_view,
permission_document_version_signature_verify,
permission_document_version_signature_upload, permission_document_version_signature_upload,
permission_document_version_signature_download, permission_document_version_signature_download,
permission_document_version_signature_delete permission_document_version_signature_delete

View File

@@ -13,7 +13,8 @@ from .permissions import (
link_document_workflow_instance_list = Link( link_document_workflow_instance_list = Link(
icon='fa fa-sitemap', permissions=(permission_workflow_view,), icon='fa fa-sitemap', permissions=(permission_workflow_view,),
text=_('Workflows'), text=_('Workflows'),
view='document_states:document_workflow_instance_list', args='object.pk' view='document_states:document_workflow_instance_list',
args='resolved_object.pk'
) )
link_setup_workflow_create = Link( link_setup_workflow_create = Link(
permissions=(permission_workflow_create,), text=_('Create workflow'), permissions=(permission_workflow_create,), text=_('Create workflow'),

View File

@@ -18,7 +18,7 @@ from .settings import setting_zoom_max_level, setting_zoom_min_level
def is_not_current_version(context): def is_not_current_version(context):
return context['resolved_object'].document.latest_version.timestamp != context['resolved_object'].timestamp return context['object'].document.latest_version.timestamp != context['object'].timestamp
def is_first_page(context): def is_first_page(context):
@@ -40,7 +40,8 @@ def is_min_zoom(context):
# Facet # Facet
link_document_preview = Link( link_document_preview = Link(
icon='fa fa-eye', permissions=(permission_document_view,), icon='fa fa-eye', permissions=(permission_document_view,),
text=_('Preview'), view='documents:document_preview', args='resolved_object.id' text=_('Preview'), view='documents:document_preview',
args='resolved_object.id'
) )
link_document_properties = Link( link_document_properties = Link(
icon='fa fa-info', permissions=(permission_document_view,), icon='fa fa-info', permissions=(permission_document_view,),
@@ -50,7 +51,7 @@ link_document_properties = Link(
link_document_version_list = Link( link_document_version_list = Link(
icon='fa fa-code-fork', permissions=(permission_document_view,), icon='fa fa-code-fork', permissions=(permission_document_view,),
text=_('Versions'), view='documents:document_version_list', text=_('Versions'), view='documents:document_version_list',
args='object.pk' args='resolved_object.pk'
) )
link_document_pages = Link( link_document_pages = Link(
icon='fa fa-files-o', permissions=(permission_document_view,), icon='fa fa-files-o', permissions=(permission_document_view,),
@@ -65,11 +66,13 @@ link_document_clear_transformations = Link(
) )
link_document_delete = Link( link_document_delete = Link(
permissions=(permission_document_delete,), tags='dangerous', permissions=(permission_document_delete,), tags='dangerous',
text=_('Delete'), view='documents:document_delete', args='resolved_object.id' text=_('Delete'), view='documents:document_delete',
args='resolved_object.id'
) )
link_document_trash = Link( link_document_trash = Link(
permissions=(permission_document_trash,), tags='dangerous', permissions=(permission_document_trash,), tags='dangerous',
text=_('Move to trash'), view='documents:document_trash', args='resolved_object.id' text=_('Move to trash'), view='documents:document_trash',
args='resolved_object.id'
) )
link_document_edit = Link( link_document_edit = Link(
permissions=(permission_document_properties_edit,), permissions=(permission_document_properties_edit,),
@@ -89,8 +92,9 @@ link_document_print = Link(
view='documents:document_print', args='resolved_object.id' view='documents:document_print', args='resolved_object.id'
) )
link_document_update_page_count = Link( link_document_update_page_count = Link(
permissions=(permission_document_tools,), text=_('Recalculate page count'), args='resolved_object.pk', permissions=(permission_document_tools,),
view='documents:document_update_page_count', args='object.pk' text=_('Recalculate page count'),
view='documents:document_update_page_count'
) )
link_document_restore = Link( link_document_restore = Link(
permissions=(permission_document_restore,), text=_('Restore'), permissions=(permission_document_restore,), text=_('Restore'),
@@ -124,7 +128,7 @@ link_document_multiple_restore = Link(
text=_('Restore'), view='documents:document_multiple_restore' text=_('Restore'), view='documents:document_multiple_restore'
) )
link_document_version_download = Link( link_document_version_download = Link(
args='object.pk', permissions=(permission_document_download,), args='resolved_object.pk', permissions=(permission_document_download,),
text=_('Download version'), view='documents:document_version_download' text=_('Download version'), view='documents:document_version_download'
) )

View File

@@ -13,7 +13,8 @@ from .permissions import (
link_document_folder_list = Link( link_document_folder_list = Link(
icon='fa fa-folder', permissions=(permission_document_view,), icon='fa fa-folder', permissions=(permission_document_view,),
text=_('Folders'), view='folders:document_folder_list', args='object.pk' text=_('Folders'), view='folders:document_folder_list',
args='resolved_object.pk'
) )
link_folder_add_document = Link( link_folder_add_document = Link(
permissions=(permission_folder_add_document,), text=_('Add to a folder'), permissions=(permission_folder_add_document,), text=_('Add to a folder'),

View File

@@ -52,7 +52,7 @@ link_smart_link_instance_view = Link(
link_smart_link_instances_for_document = Link( link_smart_link_instances_for_document = Link(
icon='fa fa-link', permissions=(permission_document_view,), icon='fa fa-link', permissions=(permission_document_view,),
text=_('Smart links'), view='linking:smart_link_instances_for_document', text=_('Smart links'), view='linking:smart_link_instances_for_document',
args='object.pk' args='resolved_object.pk'
) )
link_smart_link_list = Link( link_smart_link_list = Link(
permissions=(permission_smart_link_create,), text=_('Smart links'), permissions=(permission_smart_link_create,), text=_('Smart links'),

View File

@@ -10,12 +10,12 @@ from .permissions import (
) )
link_send_document = Link( link_send_document = Link(
permissions=(permission_mailing_send_document,), text=_('Email document'), args='resolved_object.pk', permissions=(permission_mailing_send_document,),
view='mailer:send_document', args='object.pk' text=_('Email document'), view='mailer:send_document'
) )
link_send_document_link = Link( link_send_document_link = Link(
permissions=(permission_mailing_link,), text=_('Email link'), args='resolved_object.pk', permissions=(permission_mailing_link,),
view='mailer:send_document_link', args='object.pk' text=_('Email link'), view='mailer:send_document_link'
) )
link_document_mailing_error_log = Link( link_document_mailing_error_log = Link(
icon='fa fa-envelope', permissions=(permission_view_error_log,), icon='fa fa-envelope', permissions=(permission_view_error_log,),

View File

@@ -36,7 +36,7 @@ link_metadata_remove = Link(
) )
link_metadata_view = Link( link_metadata_view = Link(
icon='fa fa-pencil', permissions=(permission_metadata_document_view,), icon='fa fa-pencil', permissions=(permission_metadata_document_view,),
text=_('Metadata'), view='metadata:metadata_view', args='object.pk' text=_('Metadata'), view='metadata:metadata_view', args='resolved_object.pk'
) )
link_setup_document_type_metadata = Link( link_setup_document_type_metadata = Link(
permissions=(permission_document_type_edit,), text=_('Optional metadata'), permissions=(permission_document_type_edit,), text=_('Optional metadata'),

View File

@@ -14,8 +14,8 @@ link_document_content = Link(
text=_('OCR'), view='ocr:document_content', args='resolved_object.id' text=_('OCR'), view='ocr:document_content', args='resolved_object.id'
) )
link_document_submit = Link( link_document_submit = Link(
permissions=(permission_ocr_document,), text=_('Submit for OCR'), args='resolved_object.id', permissions=(permission_ocr_document,),
view='ocr:document_submit', args='object.id' text=_('Submit for OCR'), view='ocr:document_submit'
) )
link_document_submit_all = Link( link_document_submit_all = Link(
icon='fa fa-font', permissions=(permission_ocr_document,), icon='fa fa-font', permissions=(permission_ocr_document,),

View File

@@ -75,10 +75,9 @@ link_staging_file_delete = Link(
args=('source.pk', 'object.encoded_filename',) args=('source.pk', 'object.encoded_filename',)
) )
link_upload_version = Link( link_upload_version = Link(
condition=document_new_version_not_blocked, args='resolved_object.pk', condition=document_new_version_not_blocked,
permissions=(permission_document_new_version,), permissions=(permission_document_new_version,),
text=_('Upload new version'), view='sources:upload_version', text=_('Upload new version'), view='sources:upload_version',
args='object.pk'
) )
link_setup_source_logs = Link( link_setup_source_logs = Link(
text=_('Logs'), view='sources:setup_source_logs', text=_('Logs'), view='sources:setup_source_logs',

View File

@@ -38,7 +38,7 @@ link_tag_edit = Link(
) )
link_tag_document_list = Link( link_tag_document_list = Link(
icon='fa fa-tag', permissions=(permission_tag_view,), text=_('Tags'), icon='fa fa-tag', permissions=(permission_tag_view,), text=_('Tags'),
view='tags:document_tags', args='object.pk' view='tags:document_tags', args='resolved_object.pk'
) )
link_tag_list = Link(icon='fa fa-tag', text=_('Tags'), view='tags:tag_list') link_tag_list = Link(icon='fa fa-tag', text=_('Tags'), view='tags:tag_list')
link_tag_multiple_delete = Link( link_tag_multiple_delete = Link(