From b5a7b6a6078695361d4545172778a1ec15bce57f Mon Sep 17 00:00:00 2001 From: Alan Stanley Date: Thu, 28 Feb 2013 11:00:39 -0400 Subject: [PATCH] removed references to EDIT_TAGS_DATASTREAM, this permission is no longer used --- ObjectHelper.inc | 1 - fedora_repository.module | 1 - fedora_repository.test.inc | 1 - plugins/tagging_form.inc | 6 ++---- tests/fedora_repository.test | 1 - 5 files changed, 2 insertions(+), 8 deletions(-) diff --git a/ObjectHelper.inc b/ObjectHelper.inc index b564ea9f..76a14f37 100644 --- a/ObjectHelper.inc +++ b/ObjectHelper.inc @@ -17,7 +17,6 @@ class ObjectHelper { public static $PURGE_FEDORA_OBJECTSANDSTREAMS = 'purge objects and datastreams'; public static $ADD_FEDORA_STREAMS = 'add fedora datastreams'; public static $INGEST_FEDORA_OBJECTS = 'ingest new fedora objects'; - public static $EDIT_TAGS_DATASTREAM = 'edit tags datastream'; public static $VIEW_DETAILED_CONTENT_LIST = 'view detailed list of content'; public static $VIEW_FEDORA_SOAP_ERROR_MESSAGES = 'view fedora soap error messages'; diff --git a/fedora_repository.module b/fedora_repository.module index 82a43c3e..6d33c27d 100644 --- a/fedora_repository.module +++ b/fedora_repository.module @@ -933,7 +933,6 @@ function fedora_repository_perm() { OBJECTHELPER::$PURGE_FEDORA_OBJECTSANDSTREAMS, OBJECTHELPER::$ADD_FEDORA_STREAMS, OBJECTHELPER::$INGEST_FEDORA_OBJECTS, - OBJECTHELPER::$EDIT_TAGS_DATASTREAM, OBJECTHELPER::$VIEW_DETAILED_CONTENT_LIST, OBJECTHELPER::$VIEW_FEDORA_SOAP_ERROR_MESSAGES, ); diff --git a/fedora_repository.test.inc b/fedora_repository.test.inc index 3e710fb8..070283a0 100644 --- a/fedora_repository.test.inc +++ b/fedora_repository.test.inc @@ -43,7 +43,6 @@ abstract class IslandoraTestCase extends DrupalWebTestCase { 'access content', 'add fedora datastreams', 'edit fedora meta data', - 'edit tags datastream', 'ingest new fedora objects', 'purge objects and datastreams', 'view fedora collection', diff --git a/plugins/tagging_form.inc b/plugins/tagging_form.inc index 0cfbff50..31ee368f 100644 --- a/plugins/tagging_form.inc +++ b/plugins/tagging_form.inc @@ -76,8 +76,7 @@ function fedora_repository_image_tagging_form($form_state, $pid) { 'title' => $tag_title_text ))), ); - if (fedora_repository_check_perm('modify fedora datastreams', $pid) || - fedora_repository_check_perm('edit tags datastream', $pid)) { + if (fedora_repository_check_perm('modify fedora datastreams', $pid)) { // Delete button for each existing tag. $form_tag['delete'] = array( '#type' => 'imagebutton', @@ -87,8 +86,7 @@ function fedora_repository_image_tagging_form($form_state, $pid) { ); } } - if (fedora_repository_check_perm('modify fedora datastreams', $pid) || - fedora_repository_check_perm('edit tags datastream', $pid)) { + if (fedora_repository_check_perm('modify fedora datastreams', $pid)) { $form['tags-wrapper']['addtag'] = array( '#type' => 'textfield', '#title' => t('New Tag'), diff --git a/tests/fedora_repository.test b/tests/fedora_repository.test index bbad2569..17acd24d 100644 --- a/tests/fedora_repository.test +++ b/tests/fedora_repository.test @@ -23,7 +23,6 @@ class FedoraRepositoryTestCase extends DrupalWebTestCase { // Create and login user. $repository_user = $this->drupalCreateFedoraUser(array('add fedora datastreams', 'edit fedora meta data', - 'edit tags datastream', 'ingest new fedora objects', 'purge objects and datastreams', 'view fedora collection',