From 9f9b2d38546138ae3fea06bb49e8a25fcd042cb0 Mon Sep 17 00:00:00 2001 From: Roberto Rosario Date: Mon, 20 Jun 2011 00:04:54 -0400 Subject: [PATCH] Added views to setup the default metadata types and sets for a given document type --- apps/documents/__init__.py | 8 ++--- apps/metadata/__init__.py | 7 ++++- apps/metadata/urls.py | 2 ++ apps/metadata/views.py | 62 ++++++++++++++++++++++++++++++++++++-- 4 files changed, 72 insertions(+), 7 deletions(-) diff --git a/apps/documents/__init__.py b/apps/documents/__init__.py index b2cbc6789f..41b388f4f4 100644 --- a/apps/documents/__init__.py +++ b/apps/documents/__init__.py @@ -95,24 +95,24 @@ staging_file_delete = {'text': _(u'delete'), 'view': 'staging_file_delete', 'arg # Document type related links document_type_list = {'text': _(u'document type list'), 'view': 'document_type_list', 'famfam': 'layout', 'permissions': [PERMISSION_DOCUMENT_VIEW]} -document_type_document_list = {'text': _(u'list of documents of this type'), 'view': 'document_type_document_list', 'args': 'object.id', 'famfam': 'page_go', 'permissions': [PERMISSION_DOCUMENT_VIEW]} +document_type_document_list = {'text': _(u'documents of this type'), 'view': 'document_type_document_list', 'args': 'object.id', 'famfam': 'page_go', 'permissions': [PERMISSION_DOCUMENT_VIEW]} document_type_edit = {'text': _(u'edit'), 'view': 'document_type_edit', 'args': 'object.id', 'famfam': 'layout_edit', 'permissions': [PERMISSION_DOCUMENT_TYPE_EDIT]} document_type_delete = {'text': _(u'delete'), 'view': 'document_type_delete', 'args': 'object.id', 'famfam': 'layout_delete', 'permissions': [PERMISSION_DOCUMENT_TYPE_DELETE]} document_type_create = {'text': _(u'create document type'), 'view': 'document_type_create', 'famfam': 'layout_add', 'permissions': [PERMISSION_DOCUMENT_TYPE_CREATE]} -document_type_filename_list = {'text': _(u'document type filenames'), 'view': 'document_type_filename_list', 'args': 'object.id', 'famfam': 'database', 'permissions': [PERMISSION_DOCUMENT_VIEW]} +document_type_filename_list = {'text': _(u'filenames'), 'view': 'document_type_filename_list', 'args': 'object.id', 'famfam': 'database', 'permissions': [PERMISSION_DOCUMENT_VIEW]} document_type_filename_create = {'text': _(u'add filename to document type'), 'view': 'document_type_filename_create', 'args': 'document_type.id', 'famfam': 'database_add', 'permissions': [PERMISSION_DOCUMENT_TYPE_EDIT]} document_type_filename_edit = {'text': _(u'edit'), 'view': 'document_type_filename_edit', 'args': 'object.id', 'famfam': 'database_edit', 'permissions': [PERMISSION_DOCUMENT_TYPE_EDIT]} document_type_filename_delete = {'text': _(u'delete'), 'view': 'document_type_filename_delete', 'args': 'object.id', 'famfam': 'database_delete', 'permissions': [PERMISSION_DOCUMENT_TYPE_EDIT]} document_type_filename_return_to_document_type = {'text': _(u'return to document type filenames'), 'view': 'document_type_filename_list', 'args': 'object.document_type.id', 'famfam': 'database', 'permissions': [PERMISSION_DOCUMENT_TYPE_EDIT]} -document_type_views = ['document_type_list', 'document_type_document_list', 'document_type_edit', 'document_type_delete', 'document_type_create', 'document_type_filename_list', 'document_type_filename_create', 'document_type_filename_edit', 'document_type_filename_delete'] +document_type_views = ['setup_document_type_metadata', 'document_type_list', 'document_type_document_list', 'document_type_edit', 'document_type_delete', 'document_type_create', 'document_type_filename_list', 'document_type_filename_create', 'document_type_filename_edit', 'document_type_filename_delete'] # Register document type links register_links(DocumentType, [document_type_document_list, document_type_filename_list, document_type_edit, document_type_delete]) register_links(DocumentTypeFilename, [document_type_filename_edit, document_type_filename_delete]) -register_links(['document_type_filename_delete', 'document_type_create', 'document_type_filename_create', 'document_type_filename_edit', 'document_type_filename_list', 'document_type_list', 'document_type_document_list', 'document_type_edit', 'document_type_delete'], [document_type_create], menu_name='sidebar') +register_links(['setup_document_type_metadata', 'document_type_filename_delete', 'document_type_create', 'document_type_filename_create', 'document_type_filename_edit', 'document_type_filename_list', 'document_type_list', 'document_type_document_list', 'document_type_edit', 'document_type_delete'], [document_type_create], menu_name='sidebar') register_links(['document_type_filename_create', 'document_type_filename_list', 'document_type_filename_edit', 'document_type_filename_delete'], [document_type_filename_create], menu_name='sidebar') register_links(['document_type_filename_edit', 'document_type_filename_delete'], [document_type_filename_return_to_document_type], menu_name='sidebar') diff --git a/apps/metadata/__init__.py b/apps/metadata/__init__.py index ae7f71ee3a..c14a36de0e 100644 --- a/apps/metadata/__init__.py +++ b/apps/metadata/__init__.py @@ -3,7 +3,8 @@ from django.utils.translation import ugettext_lazy as _ from navigation.api import register_links, register_multi_item_links, \ register_sidebar_template from permissions.api import register_permission, set_namespace_title -from documents.models import Document +from documents.models import Document, DocumentType +from documents.literals import PERMISSION_DOCUMENT_TYPE_EDIT from metadata.models import MetadataType, MetadataSet PERMISSION_METADATA_DOCUMENT_EDIT = {'namespace': 'metadata', 'name': u'metadata_document_edit', 'label': _(u'Edit a document\'s metadata')} @@ -53,6 +54,8 @@ setup_metadata_set_edit = {'text': _(u'edit'), 'view': 'setup_metadata_set_edit' setup_metadata_set_delete = {'text': _(u'delete'), 'view': 'setup_metadata_set_delete', 'args': 'object.id', 'famfam': 'application_form_delete', 'permissions': [PERMISSION_METADATA_SET_DELETE]} setup_metadata_set_create = {'text': _(u'create new'), 'view': 'setup_metadata_set_create', 'famfam': 'application_form_add', 'permissions': [PERMISSION_METADATA_SET_CREATE]} +setup_document_type_metadata = {'text': _(u'default metadata'), 'view': 'setup_document_type_metadata', 'args': 'object.id', 'famfam': 'xhtml', 'permissions': [PERMISSION_DOCUMENT_TYPE_EDIT]} + register_links(Document, [metadata_add, metadata_edit, metadata_remove]) register_multi_item_links(['document_type_document_list', 'search', 'results', 'document_group_view', 'document_list', 'document_list_recent'], [metadata_multiple_add, metadata_multiple_edit, metadata_multiple_remove]) @@ -62,6 +65,8 @@ register_links(['setup_metadata_type_delete', 'setup_metadata_type_edit', 'setup register_links(MetadataSet, [setup_metadata_set_edit, setup_metadata_set_delete]) register_links(['setup_metadata_set_delete', 'setup_metadata_set_edit', 'setup_metadata_set_list', 'setup_metadata_set_create'], [setup_metadata_set_create], menu_name='sidebar') +register_links(DocumentType, [setup_document_type_metadata]) + metadata_type_setup_views = ['setup_metadata_type_list', 'setup_metadata_type_edit', 'setup_metadata_type_delete', 'setup_metadata_type_create'] metadata_set_setup_views = ['setup_metadata_set_list', 'setup_metadata_set_edit', 'setup_metadata_set_delete', 'setup_metadata_set_create'] diff --git a/apps/metadata/urls.py b/apps/metadata/urls.py index 2dcdbb4fa3..60a2c23bf6 100644 --- a/apps/metadata/urls.py +++ b/apps/metadata/urls.py @@ -17,4 +17,6 @@ urlpatterns = patterns('metadata.views', url(r'^setup/set/create/$', 'setup_metadata_set_create', (), 'setup_metadata_set_create'), url(r'^setup/set/(?P\d+)/edit/$', 'setup_metadata_set_edit', (), 'setup_metadata_set_edit'), url(r'^setup/set/(?P\d+)/delete/$', 'setup_metadata_set_delete', (), 'setup_metadata_set_delete'), + + url(r'^setup/document/type/(?P\d+)/metadata/default/edit/$', 'setup_document_type_metadata', (), 'setup_document_type_metadata'), ) diff --git a/apps/metadata/views.py b/apps/metadata/views.py index e1ce3f4b55..e6a86cb279 100644 --- a/apps/metadata/views.py +++ b/apps/metadata/views.py @@ -7,7 +7,8 @@ from django.http import HttpResponseRedirect from django.core.urlresolvers import reverse from django.utils.http import urlencode -from documents.models import Document, RecentDocument +from documents.literals import PERMISSION_DOCUMENT_TYPE_EDIT +from documents.models import Document, RecentDocument, DocumentType from permissions.api import check_permissions from document_indexing.api import update_indexes, delete_indexes @@ -24,7 +25,7 @@ from metadata.forms import MetadataFormSet, AddMetadataForm, \ MetadataRemoveFormSet, MetadataTypeForm, MetadataSetForm from metadata.api import save_metadata_list from metadata.models import DocumentMetadata, MetadataType, MetadataSet, \ - MetadataSetItem + MetadataSetItem, DocumentTypeDefaults def metadata_edit(request, document_id=None, document_id_list=None): @@ -482,3 +483,60 @@ def setup_metadata_set_delete(request, metadata_set_id): return render_to_response('generic_confirm.html', context, context_instance=RequestContext(request)) + + +def get_document_type_metadata_members(document_type): + metadata_types = set(document_type.documenttypedefaults_set.get().default_metadata.all()) + metadata_sets = set(document_type.documenttypedefaults_set.get().default_metadata_sets.all()) + return list(metadata_types | metadata_sets) + + +def get_document_type_metadata_non_members(document_type): + members = set(get_document_type_metadata_members(document_type)) + all_metadata_objects = set(MetadataType.objects.all()) | set(MetadataSet.objects.all()) + return list(all_metadata_objects - members) + + +def add_document_type_metadata(document_type, selection): + model, pk = selection.split(u',') + if model == 'metadata type': + metadata_type = get_object_or_404(MetadataType, pk=pk) + document_type.documenttypedefaults_set.get().default_metadata.add(metadata_type) + elif model == 'metadata set': + metadata_set = get_object_or_404(MetadataSet, pk=pk) + document_type.documenttypedefaults_set.get().default_metadata_sets.add(metadata_set) + else: + raise Exception + + +def remove_document_type_metadata(document_type, selection): + model, pk = selection.split(u',') + if model == 'metadata type': + metadata_type = get_object_or_404(MetadataType, pk=pk) + document_type.documenttypedefaults_set.get().default_metadata.remove(metadata_type) + elif model == 'metadata set': + metadata_set = get_object_or_404(MetadataSet, pk=pk) + document_type.documenttypedefaults_set.get().default_metadata_sets.remove(metadata_set) + else: + raise Exception + + +def setup_document_type_metadata(request, document_type_id): + check_permissions(request.user, [PERMISSION_DOCUMENT_TYPE_EDIT]) + + document_type = get_object_or_404(DocumentType, pk=document_type_id) + + # Initialize defaults + DocumentTypeDefaults.objects.get_or_create(document_type=document_type) + + return assign_remove( + request, + left_list=lambda: generate_choices_w_labels(get_document_type_metadata_non_members(document_type)), + right_list=lambda: generate_choices_w_labels(get_document_type_metadata_members(document_type)), + add_method=lambda x: add_document_type_metadata(document_type, x), + remove_method=lambda x: remove_document_type_metadata(document_type, x), + left_list_title=_(u'non members of document type: %s') % document_type, + right_list_title=_(u'members of document type: %s') % document_type, + obj=document_type, + object_name=_(u'document type'), + )