From 95e401ccb8a6a278f530de00c884b2f70831e748 Mon Sep 17 00:00:00 2001
From: Jason MacWilliams <jason@discoverygarden.ca>
Date: Wed, 6 Jun 2012 11:45:42 -0300
Subject: [PATCH 1/2] islandora now uses a hook to get a list of available
 xslts (for xslt view) so other modules can supply more.  it also means the
 path to the xslt has to be from higher than islandora/, i chose the site root
 for simplicity (it could be the modules folder)

---
 ObjectDetails.inc | 56 ++++++++++++++++++++++++++++++-----------------
 1 file changed, 36 insertions(+), 20 deletions(-)

diff --git a/ObjectDetails.inc b/ObjectDetails.inc
index bc6bf02f..b7b04c8e 100644
--- a/ObjectDetails.inc
+++ b/ObjectDetails.inc
@@ -85,10 +85,10 @@ function fedora_repository_object_details_XSLT($item) {
   }
 
   $proc->setParameter('', 'baseUrl', $base_url);
-  $proc->setParameter('', 'path', $base_url . '/' . $path);
+  $proc->setParameter('', 'path', $path);
   $input = NULL;
 
-  $xsl_file = './'. $path .'/'. variable_get('islandora_object_details_xslt_sheet', 'object_details_xslts/convertQDC.xsl');
+  $xsl_file = variable_get('islandora_object_details_xslt_sheet', 'sites/all/modules/islandora/object_details_xslts/convertQDC.xsl');
   // set an error message in case xslt parsing fails
   $output = t("Failed to parse xslt file at @xsltFile", array('@xsltFile' => $xsl_file));
   if (is_readable($xsl_file)) {
@@ -160,9 +160,13 @@ function fedora_repository_object_details_table($item) {
 }
 
 /**
- * Configuration page for the xslt display mode.  This mode requires two
- * parameters: the datastream to render, and the xslt to apply to it.
- * @return The configuration page.
+ * Configuration page for the xslt display mode.
+ *
+ * This mode requires two parameters: the datastream to render, and the xslt to
+ * apply to it.
+ *
+ * @return
+ *   The configuration page.
  */
 function fedora_repository_object_details_XSLT_config() {
   $form = array();
@@ -171,20 +175,12 @@ function fedora_repository_object_details_XSLT_config() {
     '#title' => t("XSLT display options"),
   );
 
-  // locate the xslts available
-  $xslt_folder = "object_details_xslts/";
-  $folder = drupal_get_path("module", "fedora_repository") ."/". $xslt_folder;
-  // retrieve the filenames from the system
-  $xslts = file_scan_directory($folder, ".xsl");
-  $options = array();
-  foreach ($xslts as $xsl) {
-    $options[$xslt_folder . $xsl->basename] = $xsl->basename;
-  }
+  $options = module_invoke_all("object_details_get_available_xslts");
 
   $form['config']['xslt'] = array(
     '#type' => 'select',
     '#title' => t("XSL transform to use"),
-    '#default_value' => variable_get('islandora_object_details_xslt_sheet', 'object_details_xslts/convertQDC.xsl'),
+    '#default_value' => variable_get('islandora_object_details_xslt_sheet', 'sites/all/modules/islandora/object_details_xslts/convertQDC.xsl'),
     '#options' => $options,
     '#key_type' => 'associative',
     '#required' => TRUE,
@@ -205,18 +201,36 @@ function fedora_repository_object_details_XSLT_config() {
 }
 /**
  * Custom submit handler for the xslt configuration page.
- * @param form
- * @pararm form_state The user supplied values for the form.
+ *
+ * @param $form
+ * @param &$form_state
+ *   The user supplied values for the form.
  */
 function fedora_repository_object_details_XSLT_config_submit($form, &$form_state) {
   variable_set('islandora_object_details_xslt_sheet', $form_state['values']['xslt']);
   variable_set('islandora_object_details_xslt_datastream', $form_state['values']['dsid']);
 }
 
+/**
+ * Base function to supply the available xslt files.
+ */
+function fedora_repository_object_details_get_available_xslts() {
+  $folder = drupal_get_path("module", "fedora_repository") ."/object_details_xslts/";
+  // retrieve the filenames from the system
+  $xslts = file_scan_directory($folder, ".xsl");
+  $options = array();
+  foreach ($xslts as $xsl) {
+    $options[$xsl->filename] = $xsl->basename;
+  }
+  return $options;
+}
+
 /**
  * Configuration page for the table display mode.  This mode requires only one
  * parameter: the datastream to render.
- * @return The configuration page.
+ *
+ * @return
+ *   The configuration page.
  */
 function fedora_repository_object_details_table_config() {
   $form = array();
@@ -242,8 +256,10 @@ function fedora_repository_object_details_table_config() {
 
 /**
  * Custom submit handler for the table configuration page.
- * @param form
- * @pararm form_state The user supplied values for the form.
+ *
+ * @param $form
+ * @param &$form_state
+ *   The user supplied values for the form.
  */
 function fedora_repository_object_details_table_config_submit($form, &$form_state) {
   variable_set('islandora_object_details_table_datastream', $form_state['values']['dsid']);

From 12e80ef0c476bace666cd76a1d71c02d1f24394d Mon Sep 17 00:00:00 2001
From: Jason MacWilliams <jason@discoverygarden.ca>
Date: Wed, 6 Jun 2012 11:52:36 -0300
Subject: [PATCH 2/2] updated docs for hook_object_details_get_available_xslts

---
 ObjectDetails.inc | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/ObjectDetails.inc b/ObjectDetails.inc
index b7b04c8e..9ee2d88b 100644
--- a/ObjectDetails.inc
+++ b/ObjectDetails.inc
@@ -213,6 +213,12 @@ function fedora_repository_object_details_XSLT_config_submit($form, &$form_state
 
 /**
  * Base function to supply the available xslt files.
+ *
+ * Modules implementing this hook need to return an array describing where the
+ * XSLT is.  The array key is the path to the XSLT (paths start with sites/) and
+ * the value in the array is the display name.  In the simplest form you can use
+ * file_scan_directory like we do here - this puts the filename as the display
+ * name and will automatically detect new files as they are added.
  */
 function fedora_repository_object_details_get_available_xslts() {
   $folder = drupal_get_path("module", "fedora_repository") ."/object_details_xslts/";