diff --git a/islandora.module b/islandora.module index 7ed87813..284fed23 100644 --- a/islandora.module +++ b/islandora.module @@ -43,14 +43,14 @@ function islandora_menu() { $items = array(); $items['admin/islandora'] = array( - 'title' => 'Islandora Configuration', - 'description' => "Configure Islandora's interaction with Fedora", + 'title' => 'Islandora', + 'description' => "Configure settings associated with Islandora.", 'access arguments' => array('administer site configuration'), 'type' => MENU_NORMAL_ITEM, ); $items['admin/islandora/configure'] = array( - 'title' => 'Islandora Configuration', - 'description' => 'Enter the Islandora collection information here.', + 'title' => 'Configuration', + 'description' => 'Configure settings for Islandora.', 'page callback' => 'drupal_get_form', 'page arguments' => array('islandora_repository_admin'), 'file' => 'admin/islandora.admin.inc', @@ -390,32 +390,32 @@ function islandora_theme() { function islandora_permission() { return array( FEDORA_VIEW => array( - 'title' => t('View Fedora Repository objects and datastreams'), - 'description' => t('Users with this permission will be allowed to view Fedora objects and datastreams. Unless Fedora XACML security policies limits this functionality to certain objects.') + 'title' => t('View repository objects and datastreams'), + 'description' => t('View objects in the repository and their associated datastreams. Note: Fedora XACML security policies may override this permission.') ), FEDORA_ADD_DS => array( - 'title' => t('Add Fedora streams to Fedora objects'), - 'description' => t('Users with this permission will be allowed to add datastreams to Fedora objects. Unless Fedora XACML security policies limits this functionality to certain objects.') + 'title' => t('Add datastreams to repository objects'), + 'description' => t('Add datastreams to objects in the repository. Note: Fedora XACML security policies may override this position.') ), FEDORA_METADATA_EDIT => array( - 'title' => t('Edit Fedora Metadata'), - 'description' => t('Users with this permission will be allowed to edit Fedora Metadata streams') + 'title' => t('Edit metadata'), + 'description' => t('Edit metadata for objects in the repository.') ), FEDORA_INGEST => array( - 'title' => t('Create new Fedora objects'), - 'description' => t('Users with this permission will be allowed to create new objects in the Fedora repository') + 'title' => t('Create new repository objects'), + 'description' => t('Create new objects in the repository.') ), FEDORA_PURGE => array( - 'title' => t('Permanently remove objects from the Fedora repository'), - 'description' => t('Users with this permission will be allowed to Permantly remove objects from the Fedora repository.') + 'title' => t('Permanently remove objects from the repository'), + 'description' => t('Permanently remove objects from the repository.') ), FEDORA_MODIFY_STATE => array( - 'title' => t('Change a Fedora objects state'), - 'description' => t('Users with this permission will be allowed to change a Fedora objects state.') + 'title' => t('Change repository object states'), + 'description' => t('Change the state of objects in the repository (e.g. from Active to Inactive).') ), FEDORA_MANAGE => array( - 'title' => t('View Fedora Manage tabs'), - 'description' => t('Users with this permission will be allowed to view fedora manage manage tabs.') + 'title' => t('View object management tabs'), + 'description' => t('View tabs that provide object management functions.') ) ); } diff --git a/islandora_basic_collection/islandora_basic_collection.install b/islandora_basic_collection/islandora_basic_collection.install index 724b76f4..e1163599 100644 --- a/islandora_basic_collection/islandora_basic_collection.install +++ b/islandora_basic_collection/islandora_basic_collection.install @@ -37,10 +37,10 @@ function islandora_basic_collection_install() { drupal_set_message(st('Unable to install content models %e', array('%e' => $e)), 'error'); return; } - drupal_set_message(st('Content models installed!')); + drupal_set_message(st('Content models for the basic collection module installed!')); } else { - drupal_set_message(st('Content models already exist!'), 'warning'); + drupal_set_message(st('Content models for the basic collection module already exist!'), 'warning'); } $collection_query = $rest_connection->api->a->findObjects('query', 'pid=islandora:root'); @@ -56,13 +56,13 @@ function islandora_basic_collection_install() { $datastream->setContentFromUrl($file_path); $fedora_object->ingestDatastream($datastream); } catch (Exception $e) { - drupal_set_message(st('Unable to install collections %e', array('%e' => $e)), 'error'); + drupal_set_message(st('Unable to install collections for the basic collection module %e', array('%e' => $e)), 'error'); return; } - drupal_set_message(st('Collections installed!')); + drupal_set_message(st('Collections installed for the basic collection module!')); } else { - drupal_set_message(st('Collections already exist!'), 'warning'); + drupal_set_message(st('Collections already exist for the basic collection module!'), 'warning'); } } @@ -80,13 +80,13 @@ function islandora_basic_collection_uninstall() { try { $rest_connection->repository->purgeObject('islandora:collectionCModel'); } catch (Exception $e) { - drupal_set_message(st('Unable to purge content models %e', array('%e' => $e)), 'error'); + drupal_set_message(st('Unable to purge content models for the basic collection module %e', array('%e' => $e)), 'error'); return; } - drupal_set_message(st('Content models purged!')); + drupal_set_message(st('Content models for the basic collection module purged!')); } else { - drupal_set_message(st('Content models don\'t exist!'), 'warning'); + drupal_set_message(st('Content models for the basic collection module don\'t exist!'), 'warning'); } $collection_query = $rest_connection->api->a->findObjects('query', 'pid=islandora:root'); @@ -94,12 +94,12 @@ function islandora_basic_collection_uninstall() { try { $rest_connection->repository->purgeObject('islandora:root'); } catch (Exception $e) { - drupal_set_message(st('Unable to purge collections %e', array('%e' => $e)), 'error'); + drupal_set_message(st('Unable to purge collections for the basic collection module %e', array('%e' => $e)), 'error'); return; } - drupal_set_message(st('Collections purged!')); + drupal_set_message(st('Collections for the basic collection module purged!')); } else { - drupal_set_message(st('Collections don\'t exist!'), 'warning'); + drupal_set_message(st('Collections for the basic collection module don\'t exist!'), 'warning'); } } \ No newline at end of file diff --git a/islandora_basic_collection/islandora_basic_collection.module b/islandora_basic_collection/islandora_basic_collection.module index 001e17cd..04ee2ccc 100644 --- a/islandora_basic_collection/islandora_basic_collection.module +++ b/islandora_basic_collection/islandora_basic_collection.module @@ -39,8 +39,8 @@ function islandora_basic_collection_menu() { ); $items['admin/islandora/basic_collection'] = array( - 'title' => 'Islandora basic collection', - 'description' => 'Configure the basic Collection solution pack.', + 'title' => 'Core collection', + 'description' => 'Configure the core Islandora collection functionality.', 'page callback' => 'drupal_get_form', 'access arguments' => array('administer site configuration'), 'page arguments' => array('islandora_basic_collection_admin'), diff --git a/islandora_basic_collection/theme/islandora-basic-collection-wrapper.tpl.php b/islandora_basic_collection/theme/islandora-basic-collection-wrapper.tpl.php index 90f8666f..355da7cf 100644 --- a/islandora_basic_collection/theme/islandora-basic-collection-wrapper.tpl.php +++ b/islandora_basic_collection/theme/islandora-basic-collection-wrapper.tpl.php @@ -25,7 +25,7 @@
- + diff --git a/islandora_basic_collection/theme/islandora_basic_collection.theme.inc b/islandora_basic_collection/theme/islandora_basic_collection.theme.inc index 40feef8d..5f5ade6b 100644 --- a/islandora_basic_collection/theme/islandora_basic_collection.theme.inc +++ b/islandora_basic_collection/theme/islandora_basic_collection.theme.inc @@ -20,23 +20,29 @@ function islandora_basic_collection_preprocess_islandora_basic_collection_wrappe $display = (empty($_GET['display'])) ? variable_get('islandora_basic_collection_default_view', 'grid') : $_GET['display']; $link_text = (empty($_GET['display'])) ? 'grid' : $_GET['display']; $query_params = drupal_get_query_parameters($_GET); - + global $base_url; if ($display == 'grid') { $query_params['display'] = 'list'; - $link_text = 'List View'; - $link_class = 'islandora-view-default'; + $list_link = l('List View', $base_url . '/islandora/object/' . $islandora_object->id, array('attributes' => array('class' => 'islandora-view-list'), + 'query' => $query_params)); + unset($query_params['display']); + $query_params['display'] = 'grid'; + $grid_link = l('Grid View', $base_url . '/islandora/object/' . $islandora_object->id, array('attributes' => array('class' => array('islandora-view-grid', 'active')), + 'query' => $query_params)); $collection_content = theme('islandora_basic_collection_grid', array('islandora_object' => $islandora_object, 'collection_results' => $results)); } else { + $query_params['display'] = 'list'; + $list_link = l('List View', $base_url . '/islandora/object/' . $islandora_object->id, array('attributes' => array('class' => array('islandora-view-list', 'active')), + 'query' => $query_params)); + unset($query_params['display']); $query_params['display'] = 'grid'; - $link_text = 'Grid View'; - $link_class = 'islandora-view-grid'; + $grid_link = l('Grid View', $base_url . '/islandora/object/' . $islandora_object->id, array('attributes' => array('class' => 'islandora-view-grid'), + 'query' => $query_params)); $collection_content = theme('islandora_basic_collection', array('islandora_object' => $islandora_object, 'collection_results' => $results)); } - global $base_url; - $link = l($link_text, $base_url . '/islandora/object/' . $islandora_object->id, array('attributes' => array('class' => $link_class), - 'query' => $query_params)); - $variables['switch_view_link'] = $link; + $variables['list_link'] = $list_link; + $variables['grid_link'] = $grid_link; $variables['collection_content'] = $collection_content; } diff --git a/islandora_basic_image/includes/image.process.inc b/islandora_basic_image/includes/image.process.inc index e6c68aa1..260717ff 100644 --- a/islandora_basic_image/includes/image.process.inc +++ b/islandora_basic_image/includes/image.process.inc @@ -1,11 +1,17 @@ api->m->ingest(array('string' => $xml)); } catch (Exception $e) { - drupal_set_message(t('Unable to install content models %e', array('%e' => $e)), 'error'); + drupal_set_message(t('Unable to install content models for the basic image module %e', array('%e' => $e)), 'error'); return; } - drupal_set_message(t('Content models installed!')); + drupal_set_message(t('Content models for the basic image module installed!')); } else { - drupal_set_message(t('Content models already exist!'), 'warning'); + drupal_set_message(t('Content models for the basic image module already exist!'), 'warning'); } $collection_query = $restConnection->api->a->findObjects('query', 'pid=islandora:sp_basic_image_collection'); @@ -61,13 +61,13 @@ function islandora_basic_image_install() { $fedora_object->ingestDatastream($datastream); } catch (Exception $e) { - drupal_set_message(t('Unable to install collections %e', array('%e' => $e)), 'error'); + drupal_set_message(t('Unable to install collections for the basic image module %e', array('%e' => $e)), 'error'); return; } - drupal_set_message(t('Collections installed!')); + drupal_set_message(t('Collections for the basic image module installed!')); } else { - drupal_set_message(t('Collections already exist!'), 'warning'); + drupal_set_message(t('Collections for the basic image module already exist!'), 'warning'); } } @@ -90,13 +90,13 @@ function islandora_basic_image_uninstall() { try { $restConnection->repository->purgeObject('islandora:sp_basic_image'); } catch (Exception $e) { - drupal_set_message(t('Unable to purge content models %e', array('%e' => $e)), 'error'); + drupal_set_message(t('Unable to purge content models for the basic image module %e', array('%e' => $e)), 'error'); return; } - drupal_set_message(t('Content models purged!')); + drupal_set_message(t('Content models for the basic image module purged!')); } else { - drupal_set_message(t('Content models don\'t exist!'), 'warning'); + drupal_set_message(t('Content models for the basic image module don\'t exist!'), 'warning'); } $collection_query = $restConnection->api->a->findObjects('query', 'pid=islandora:sp_basic_image_collection'); @@ -104,12 +104,12 @@ function islandora_basic_image_uninstall() { try { $restConnection->repository->purgeObject('islandora:sp_basic_image_collection'); } catch (Exception $e) { - drupal_set_message(t('Unable to purge collections %e', array('%e' => $e)), 'error'); + drupal_set_message(t('Unable to purge collections for the basic image module %e', array('%e' => $e)), 'error'); return; } - drupal_set_message(t('Collections purged!')); + drupal_set_message(t('Collections for the basic image module purged!')); } else { - drupal_set_message(t('Collections don\'t exist!'), 'warning'); + drupal_set_message(t('Collections for the basic image module don\'t exist!'), 'warning'); } } \ No newline at end of file diff --git a/islandora_basic_image/islandora_basic_image.module b/islandora_basic_image/islandora_basic_image.module index 9e68cfb6..09e46054 100644 --- a/islandora_basic_image/islandora_basic_image.module +++ b/islandora_basic_image/islandora_basic_image.module @@ -31,8 +31,8 @@ function islandora_basic_image_menu() { $items = array(); $items['admin/islandora/basic_image'] = array( - 'title' => 'Islandora Basic Image', - 'description' => 'Configure the basic image solution pack.', + 'title' => 'Basic Image collection', + 'description' => 'Configure the Basic Image solution pack.', 'page callback' => 'drupal_get_form', 'access arguments' => array('administer site configuration'), 'page arguments' => array('islandora_basic_image_admin'), @@ -124,7 +124,7 @@ function islandora_basic_image_theme($existing, $type, $theme, $path) { 'variables' => array('islandora_objects' => NULL), ), 'islandora_basic_image' => array( - 'template' => 'islandora-basic-image', + 'template' => 'theme/islandora-basic-image', 'pattern' => 'islandora_basic_image__', //we can add pids to the end of this pattern in our preprocess function // and templates will be able to have have a pid appended to the template name to overide a template on a per object basis //an example template would be named islandora-basic-image--islandora-27.tpl.phps diff --git a/islandora_basic_image/islandora-basic-image.tpl.php b/islandora_basic_image/theme/islandora-basic-image.tpl.php similarity index 100% rename from islandora_basic_image/islandora-basic-image.tpl.php rename to islandora_basic_image/theme/islandora-basic-image.tpl.php