Added doc tags to the api

api/dublin_core.inc

@@ -1,13 +1,19 @@
 <?php
+
 // $Id$
 
-/* 
+/**
+ * @file
  * Implements a simple class for working with Dublin Core data and exporting it
  * back to XML. Inspiration and design shamelessly stolen from the pyfedora
  * project at http://pypi.python.org/pypi/pyfedora/0.1.0
  */
 
+/**
+ * Dublin Core Class
+ */
 class Dublin_Core {
+
   public $dc = array(
     'dc:title' => array(),
     'dc:creator' => array(),
@@ -41,17 +47,17 @@ class Dublin_Core {
   }
 
   /**
-   *
+   * Add Elements
    * @param <type> $element_name
    * @param <type> $value
    */
-  function add_element( $element_name, $value ) {
+  function add_element($element_name, $value) {
     if (is_string($value) && is_array($this->dc[$element_name])) {
       $this->dc[$element_name][] = $value;
     }
   }
 
-/**
+  /**
    * Replace the given DC element with the values in $values
    * @param string $elemnt_name
    * @param array $values 
@@ -67,8 +73,10 @@ class Dublin_Core {
 
   /**
    * Serialize this object to XML and return it.
+   * @param type $with_preamble
+   * @return type 
    */
-  function as_xml( $with_preamble = FALSE ) {
+  function as_xml($with_preamble = FALSE) {
     $dc_xml = new DomDocument();
     $oai_dc = $dc_xml->createElementNS('http://www.openarchives.org/OAI/2.0/oai_dc/', 'oai_dc:dc');
     $oai_dc->setAttribute('xmlns:dc', 'http://purl.org/dc/elements/1.1/');
@@ -88,10 +96,17 @@ class Dublin_Core {
     return $dc_xml->saveXML();
   }
 
+  /**
+   * Create dc from dict ( does nothing )
+   */
   static function create_dc_from_dict() {
     
   }
 
+  /**
+   * Save ??
+   * @param type $alt_owner 
+   */
   function save($alt_owner = NULL) {
     $item_to_update = (!empty($alt_owner) ? $alt_owner : $this->owner);
     // My Java roots showing, trying to do polymorphism in PHP.

api/fedora_collection.inc

@@ -1,10 +1,11 @@
 <?php
+
 // $Id$
 
-/* 
+/**
+ * @file
  * Operations that affect a Fedora repository at a collection level.
  */
-
 module_load_include('inc', 'fedora_repository', 'CollectionClass');
 module_load_include('inc', 'fedora_repository', 'api/fedora_item');
 module_load_include('inc', 'fedora_repository', 'api/fedora_utils');
@@ -13,9 +14,12 @@ module_load_include('module', 'fedora_repository');
 /**
  * Exports a fedora collection object and all of its children in a format
  * that will let you import them into another repository.
- * @param <type> $format
+ * @param type $collection_pid
+ * @param type $relationship
+ * @param type $format
+ * @return type 
  */
-function export_collection($collection_pid, $relationship = 'isMemberOfCollection', $format = 'info:fedora/fedora-system:FOXML-1.1' ) {
+function export_collection($collection_pid, $relationship = 'isMemberOfCollection', $format = 'info:fedora/fedora-system:FOXML-1.1') {
   $collection_item = new Fedora_Item($collection_pid);
   $foxml = $collection_item->export_as_foxml();
 
@@ -25,15 +29,15 @@ function export_collection($collection_pid, $relationship = 'isMemberOfCollectio
   $container = tempnam($file_dir, 'export_');
   file_delete($container);
   print $container;
-  if (mkdir($container) && mkdir($container . '/'. $collection_pid)) {
+  if (mkdir($container) && mkdir($container . '/' . $collection_pid)) {
-    $foxml_dir = $container . '/'. $collection_pid;
+    $foxml_dir = $container . '/' . $collection_pid;
-    $file = fopen($foxml_dir . '/'. $collection_pid . '.xml', 'w');
+    $file = fopen($foxml_dir . '/' . $collection_pid . '.xml', 'w');
     fwrite($file, $foxml);
     fclose($file);
 
     $member_pids = get_related_items_as_array($collection_pid, $relationship);
     foreach ($member_pids as $member) {
-      $file = fopen($foxml_dir . '/'. $member . '.xml', 'w');
+      $file = fopen($foxml_dir . '/' . $member . '.xml', 'w');
       $item = new Fedora_Item($member);
       $item_foxml = $item->export_as_foxml();
       fwrite($file, $item_foxml);
@@ -41,13 +45,13 @@ function export_collection($collection_pid, $relationship = 'isMemberOfCollectio
     }
     if (system("cd $container;zip -r $collection_pid.zip $collection_pid/* >/dev/NULL") == 0) {
       header("Content-type: application/zip");
-      header('Content-Disposition: attachment; filename="' . $collection_pid . '.zip'. '"');
+      header('Content-Disposition: attachment; filename="' . $collection_pid . '.zip' . '"');
-      $fh = fopen($container . '/'. $collection_pid . '.zip', 'r');
+      $fh = fopen($container . '/' . $collection_pid . '.zip', 'r');
-      $the_data = fread($fh, filesize($container . '/'. $collection_pid . '.zip'));
+      $the_data = fread($fh, filesize($container . '/' . $collection_pid . '.zip'));
       fclose($fh);
       echo $the_data;
     }
-    if (file_exists($container . '/'. $collection_pid)) {
+    if (file_exists($container . '/' . $collection_pid)) {
       system("rm -rf $container"); // I'm sorry.
     }
   }
@@ -82,14 +86,14 @@ function get_related_items_as_xml($collection_pid, $relationship = array('isMemb
 
   if (is_array($relationship)) {
     foreach ($relationship as $rel) {
-      $query_string .= '$object <fedora-rels-ext:'. $rel . '> <info:fedora/'. $collection_pid . '>';
+      $query_string .= '$object <fedora-rels-ext:' . $rel . '> <info:fedora/' . $collection_pid . '>';
       if (next($relationship)) {
         $query_string .= ' OR ';
       }
     }
   }
   elseif (is_string($relationship)) {
-    $query_string .= '$object <fedora-rels-ext:'. $relationship . '> <info:fedora/'. $collection_pid . '>';
+    $query_string .= '$object <fedora-rels-ext:' . $relationship . '> <info:fedora/' . $collection_pid . '>';
   }
   else {
     return '';
@@ -104,19 +108,30 @@ function get_related_items_as_xml($collection_pid, $relationship = array('isMemb
 
   $query_string .= ')
                     minus $content <mulgara:is> <info:fedora/fedora-system:FedoraObject-3.0>
-                    order by '.$orderby;
+                    order by ' . $orderby;
 
   $query_string = htmlentities(urlencode($query_string));
 
 
   $content = '';
   $url = variable_get('fedora_repository_url', 'http://localhost:8080/fedora/risearch');
-  $url .= "?type=tuples&flush=TRUE&format=Sparql&limit=$limit&offset=$offset&lang=itql&stream=on&query=". $query_string;
+  $url .= "?type=tuples&flush=TRUE&format=Sparql&limit=$limit&offset=$offset&lang=itql&stream=on&query=" . $query_string;
   $content .= do_curl($url);
 
   return $content;
 }
 
+/**
+ * Get Related Items as Arrays
+ * @param type $collection_pid
+ * @param type $relationship
+ * @param type $limit
+ * @param type $offset
+ * @param type $active_objects_only
+ * @param type $cmodel
+ * @param type $orderby
+ * @return type 
+ */
 function get_related_items_as_array($collection_pid, $relationship = 'isMemberOfCollection', $limit = 10000, $offset = 0, $active_objects_only = TRUE, $cmodel = NULL, $orderby = '$title') {
   $content = get_related_items_as_xml($collection_pid, $relationship, $limit, $offset, $active_objects_only, $cmodel, $orderby);
   if (empty($content)) {

api/fedora_export.inc

@@ -2,6 +2,10 @@
 
 // $Id$
 
+/**
+ * @file
+ * Fedora Export
+ */
 define('FOXML_10', 'info:fedora/fedora-system:FOXML-1.0');
 define('FOXML_11', 'info:fedora/fedora-system:FOXML-1.1');
 define('METS_10', 'info:fedora/fedora-system:METSFedoraExt-1.0');
@@ -11,6 +15,11 @@ define('ATOMZip_11', 'info:fedora/fedora-system:ATOMZip-1.1');
 
 /**
  * Function to to export all objects assocoiated with a given pid to the export area
+ * @param type $pid
+ * @param type $foxml_dir
+ * @param type $ob_dir
+ * @param type $log
+ * @return type 
  */
 function export_to_export_area($pid, $foxml_dir, $ob_dir, &$log = array()) {
   if (!$paths = export_objects_for_pid($pid, $ob_dir, $log)) {
@@ -24,6 +33,13 @@ function export_to_export_area($pid, $foxml_dir, $ob_dir, &$log = array()) {
   return TRUE;
 }
 
+/**
+ * Export objects for pids ??
+ * @param type $pid
+ * @param type $dir
+ * @param type $log
+ * @return string 
+ */
 function export_objects_for_pid($pid, $dir, &$log) {
   module_load_include('inc', 'fedora_repository', 'api/fedora_item');
   $item = new Fedora_Item($pid);
@@ -38,7 +54,7 @@ function export_objects_for_pid($pid, $dir, &$log) {
   $paths = array();
   foreach ($object->datastreamDef as $ds) {
     if (!in_array($ds->ID, $ignore_dsids)) {
-      $file = $dir .'/'. $ds->label .'.'. get_file_extension($ds->MIMEType);
+      $file = $dir . '/' . $ds->label . '.' . get_file_extension($ds->MIMEType);
       $paths[$ds->ID] = $file;
 
       //$content = $ob_helper->getDatastreamDissemination($pid, $ds->ID);
@@ -58,6 +74,16 @@ function export_objects_for_pid($pid, $dir, &$log) {
   return $paths;
 }
 
+/**
+ * Export foxml for pid
+ * @param type $pid
+ * @param type $dir
+ * @param type $paths
+ * @param type $log
+ * @param type $format
+ * @param type $remove_islandora
+ * @return type 
+ */
 function export_foxml_for_pid($pid, $dir, $paths, &$log, $format = FOXML_11, $remove_islandora = FALSE) {
   module_load_include('inc', 'fedora_repository', 'ObjectHelper');
   $ob_helper = new ObjectHelper();
@@ -136,7 +162,7 @@ function export_foxml_for_pid($pid, $dir, $paths, &$log, $format = FOXML_11, $re
           foreach ($xpath->query('METS:file[@OWNERID!="E"][@OWNERID!="R"]/METS:FLocat[@xlink:href]', $dsNode) as $Floc) {
             $Floc->setAttribute('xlink:href', url($paths[$dsId], array('absolute' => TRUE)));
           }
-/*  
+          /*
             foreach ($dsNode->getElementsByTagName('METS:file') as $contentNode) {
             // Don't update datastreams having external uris
             if (in_array($dsNode->getAttribute('OWNERID'), $disallowed_groups)) {
@@ -147,18 +173,18 @@ function export_foxml_for_pid($pid, $dir, $paths, &$log, $format = FOXML_11, $re
             $Floc->setAttribute('xlink:href', url($paths[$dsId], array('absolute' => true)));
             }
             `}
-*/  
+           */
         }
 
         break;
 
       default:
-        $log[] = log_line(t("Unknown or invalid format: ". $format), 'error');
+        $log[] = log_line(t("Unknown or invalid format: " . $format), 'error');
         return FALSE;
     }
   } //if $remove_islandora
 
-  $file = $dir .'/'. $pid .'.xml';
+  $file = $dir . '/' . $pid . '.xml';
   if (!$foxml->save($file)) {
     $log[] = log_line(t("Failed to write datastream %dsid for pid %pid to %file", array('%dsid' => $ds->ID, '%pid' => $pid, '%file' => $file)), 'error');
     return FALSE;
@@ -170,10 +196,22 @@ function export_foxml_for_pid($pid, $dir, $paths, &$log, $format = FOXML_11, $re
   return TRUE;
 }
 
+/**
+ * Get file extension
+ * @param type $mimeType
+ * @return type 
+ */
 function get_file_extension($mimeType) {
   return substr(strstr($mimeType, '/'), 1);
 }
 
+/**
+ * Log line
+ * @param type $msg
+ * @param type $severity
+ * @param type $sep
+ * @return type 
+ */
 function log_line($msg, $severity = 'info', $sep = "\t") {
   return date("Y-m-d H:i:s") . $sep . ucfirst($severity) . $sep . $msg;
 }

api/fedora_item.inc

@@ -2,9 +2,16 @@
 
 // $Id$
 
+/**
+ * @file
+ * Fedora Item
+ */
 define('RELS_EXT_URI', 'info:fedora/fedora-system:def/relations-external#');
 define("FEDORA_MODEL_URI", 'info:fedora/fedora-system:def/model#');
 
+/**
+ * Fedora Item Class
+ */
 class Fedora_Item {
 
   public $pid = NULL; // The $pid of the fedora object represented by an instance of this class.
@@ -51,10 +58,24 @@ class Fedora_Item {
     }
   }
 
+  /**
+   * Exists 
+   * @return type 
+   */
   function exists() {
     return (!empty($this->objectProfile));
   }
 
+  /**
+   * Add datastream from file
+   * @param type $datastream_file
+   * @param type $datastream_id
+   * @param type $datastream_label
+   * @param type $datastream_mimetype
+   * @param type $controlGroup
+   * @param type $logMessage
+   * @return type 
+   */
   function add_datastream_from_file($datastream_file, $datastream_id, $datastream_label = NULL, $datastream_mimetype = '', $controlGroup = 'M', $logMessage = null) {
     module_load_include('inc', 'fedora_repository', 'MimeClass');
     if (empty($datastream_mimetype)) {
@@ -76,6 +97,16 @@ class Fedora_Item {
     return $return_value;
   }
 
+  /**
+   * Add datastream from url
+   * @param type $datastream_url
+   * @param type $datastream_id
+   * @param type $datastream_label
+   * @param type $datastream_mimetype
+   * @param type $controlGroup
+   * @param type $logMessage
+   * @return type 
+   */
   function add_datastream_from_url($datastream_url, $datastream_id, $datastream_label = NULL, $datastream_mimetype = '', $controlGroup = 'M', $logMessage = null) {
     if (empty($datastream_label)) {
       $datastream_label = $datastream_id;
@@ -98,10 +129,19 @@ class Fedora_Item {
     );
 
 
-    return $this->soap_call( 'addDataStream', $params );
+    return $this->soap_call('addDataStream', $params);
-
   }
 
+  /**
+   * Add datastream from string
+   * @param type $str
+   * @param type $datastream_id
+   * @param type $datastream_label
+   * @param type $datastream_mimetype
+   * @param type $controlGroup
+   * @param type $logMessage
+   * @return type 
+   */
   function add_datastream_from_string($str, $datastream_id, $datastream_label = NULL, $datastream_mimetype = 'text/xml', $controlGroup = 'M', $logMessage = null) {
     $dir = sys_get_temp_dir();
     $tmpfilename = tempnam($dir, 'fedoratmp');
@@ -116,8 +156,9 @@ class Fedora_Item {
   /**
    * Add a relationship string to this object's RELS-EXT.
    * does not support rels-int yet.
-   * @param string $relationship
+   * @param type $relationship
-   * @param <type> $object
+   * @param type $object
+   * @param type $namespaceURI 
    */
   function add_relationship($relationship, $object, $namespaceURI = RELS_EXT_URI) {
     $ds_list = $this->get_datastreams_list_as_array();
@@ -214,6 +255,10 @@ class Fedora_Item {
     //print ($description->dump_node());
   }
 
+  /**
+   * Export as foxml
+   * @return type 
+   */
   function export_as_foxml() {
     $params = array(
       'pid' => $this->pid,
@@ -290,6 +335,12 @@ class Fedora_Item {
     return $results;
   }
 
+  /**
+   * Get datastream dissemination
+   * @param type $dsid
+   * @param type $as_of_date_time
+   * @return string 
+   */
   function get_datastream_dissemination($dsid, $as_of_date_time = "") {
     $params = array(
       'pid' => $this->pid,
@@ -307,6 +358,12 @@ class Fedora_Item {
     return $content;
   }
 
+  /**
+   * Get datastream
+   * @param type $dsid
+   * @param type $as_of_date_time
+   * @return type 
+   */
   function get_datastream($dsid, $as_of_date_time = "") {
     $params = array(
       'pid' => $this->pid,
@@ -318,6 +375,11 @@ class Fedora_Item {
     return $object->datastream;
   }
 
+  /**
+   * Get datastream history
+   * @param type $dsid
+   * @return type 
+   */
   function get_datastream_history($dsid) {
     $params = array(
       'pid' => $this->pid,
@@ -332,6 +394,14 @@ class Fedora_Item {
     return $ret;
   }
 
+  /**
+   * Get dissemination
+   * @param type $service_definition_pid
+   * @param type $method_name
+   * @param type $parameters
+   * @param type $as_of_date_time
+   * @return string 
+   */
   function get_dissemination($service_definition_pid, $method_name, $parameters = array(), $as_of_date_time = null) {
     $params = array(
       'pid' => $this->pid,
@@ -530,6 +600,9 @@ class Fedora_Item {
 
   /**
    * Removes this object form the repository.
+   * @param type $log_message
+   * @param type $force
+   * @return type 
    */
   function purge($log_message = 'Purged using Islandora API.', $force = FALSE) {
     $params = array(
@@ -541,6 +614,15 @@ class Fedora_Item {
     return $this->soap_call('purgeObject', $params);
   }
 
+  /**
+   * Purge datastream
+   * @param type $dsID
+   * @param type $start_date
+   * @param type $end_date
+   * @param type $log_message
+   * @param type $force
+   * @return type 
+   */
   function purge_datastream($dsID, $start_date = NULL, $end_date = NULL, $log_message = 'Purged datastream using Islandora API', $force = FALSE) {
     $params = array(
       'pid' => $this->pid,
@@ -553,11 +635,21 @@ class Fedora_Item {
     return $this->soap_call('purgeDatastream', $params);
   }
 
+  /**
+   * URL
+   * @global type $base_url
+   * @return type 
+   */
   function url() {
     global $base_url;
     return $base_url . '/fedora/repository/' . $this->pid . (!empty($this->objectProfile) ? '/-/' . drupal_urlencode($this->objectProfile->objLabel) : '');
   }
 
+  /**
+   * Get Next PID in Namespace
+   * @param type $pid_namespace
+   * @return type 
+   */
   static function get_next_PID_in_namespace($pid_namespace = '') {
 
     if (empty($pid_namespace)) {
@@ -581,18 +673,32 @@ class Fedora_Item {
     return $result->pid;
   }
 
+  /**
+   * ingest from FOXML
+   * @param type $foxml
+   * @return Fedora_Item 
+   */
   static function ingest_from_FOXML($foxml) {
     $params = array('objectXML' => $foxml->saveXML(), 'format' => "info:fedora/fedora-system:FOXML-1.1", 'logMessage' => "Fedora Object Ingested");
     $object = self::soap_call('ingest', $params);
     return new Fedora_Item($object->objectPID);
   }
 
+  /**
+   * ingest from FOXML file
+   * @param type $foxml_file
+   * @return type 
+   */
   static function ingest_from_FOXML_file($foxml_file) {
     $foxml = new DOMDocument();
     $foxml->load($foxml_file);
     return self::ingest_from_FOXML($foxml);
   }
 
+  /**
+   * ingest from FOXML files in directory
+   * @param type $path 
+   */
   static function ingest_from_FOXML_files_in_directory($path) {
     // Open the directory
     $dir_handle = @opendir($path);
@@ -612,6 +718,15 @@ class Fedora_Item {
     closedir($dir_handle);
   }
 
+  /**
+   * Modify Object
+   * @param type $label
+   * @param type $state
+   * @param type $ownerId
+   * @param type $logMessage
+   * @param type $quiet
+   * @return type 
+   */
   function modify_object($label = '', $state = null, $ownerId = null, $logMessage = 'Modified by Islandora API', $quiet=TRUE) {
 
     $params = array(
@@ -625,6 +740,17 @@ class Fedora_Item {
     return self::soap_call('modifyObject', $params, $quiet);
   }
 
+  /**
+   * Modify datastream by reference
+   * @param type $external_url
+   * @param type $dsid
+   * @param type $label
+   * @param type $mime_type
+   * @param type $force
+   * @param type $logMessage
+   * @param type $quiet
+   * @return type 
+   */
   function modify_datastream_by_reference($external_url, $dsid, $label, $mime_type, $force = FALSE, $logMessage = 'Modified by Islandora API', $quiet=FALSE) {
     $params = array(
       'pid' => $this->pid,
@@ -642,6 +768,17 @@ class Fedora_Item {
     return self::soap_call('modifyDatastreamByReference', $params, $quiet);
   }
 
+  /**
+   * Modify datastream by value
+   * @param type $content
+   * @param type $dsid
+   * @param type $label
+   * @param type $mime_type
+   * @param type $force
+   * @param type $logMessage
+   * @param type $quiet
+   * @return type 
+   */
   function modify_datastream_by_value($content, $dsid, $label, $mime_type, $force = FALSE, $logMessage = 'Modified by Islandora API', $quiet=FALSE) {
     $params = array(
       'pid' => $this->pid,
@@ -659,6 +796,13 @@ class Fedora_Item {
     return self::soap_call('modifyDatastreamByValue', $params, $quiet);
   }
 
+  /**
+   * Soap call
+   * @param type $function_name
+   * @param type $params_array
+   * @param type $quiet
+   * @return type 
+   */
   static function soap_call($function_name, $params_array, $quiet = FALSE) {
     if (!self::$connection_helper) {
       module_load_include('inc', 'fedora_repository', 'ConnectionHelper');
@@ -730,6 +874,9 @@ class Fedora_Item {
    *
    * @param string $pid if none given, getnextpid will be called.
    * @param string $state The initial state, A - Active, I - Inactive, D - Deleted
+   * @param type $label
+   * @param type $owner
+   * @return DOMDocument 
    */
   static function create_object_FOXML($pid = '', $state = 'A', $label = 'Untitled', $owner = '') {
     $foxml = new DOMDocument("1.0", "UTF-8");
@@ -778,10 +925,23 @@ class Fedora_Item {
     return $foxml;
   }
 
+  /**
+   * ingest new item
+   * @param type $pid
+   * @param type $state
+   * @param type $label
+   * @param type $owner
+   * @return type 
+   */
   static function ingest_new_item($pid = '', $state = 'A', $label = '', $owner = '') {
     return self::ingest_from_FOXML(self::create_object_FOXML($pid, $state, $label, $owner));
   }
 
+  /**
+   * fedora item exists 
+   * @param type $pid
+   * @return type 
+   */
   static function fedora_item_exists($pid) {
     $item = new Fedora_Item($pid);
     return $item->exists();

api/fedora_utils.inc

@@ -1,35 +1,48 @@
 <?php
-// $Id$
 
-// @file fedora_utils.inc
+// $Id$
-// Base utilities used by the Islansora fedora module.
 
+/**
+ * @file
+ * Base utilities used by the Islandora fedora module.
+ */
 /*
-* Functions that emulate php5.3 functionality for backwards compatiablity
+ * Functions that emulate php5.3 functionality for backwards compatiablity
-*/
+ */
 if (!function_exists('str_getcsv')) {
+
   function str_getcsv($input, $delimiter=',', $enclosure='"', $escape=null, $eol=null) {
-    $temp=fopen("php://memory", "rw");
+    $temp = fopen("php://memory", "rw");
     fwrite($temp, $input);
     fseek($temp, 0);
-    $r=fgetcsv($temp, 4096, $delimiter, $enclosure);
+    $r = fgetcsv($temp, 4096, $delimiter, $enclosure);
     fclose($temp);
     return $r;
   }
+
 }
+
 /*
-* Functions that emulate php5.3 functionality for backwards compatiablity
+ * Functions that emulate php5.3 functionality for backwards compatiablity
-*/
+ */
 
 /*
  * Static functions used by the Fedora PHP API.
  */
 
-
+/**
+ * do curl
+ * @global type $user
+ * @param type $url
+ * @param type $return_to_variable
+ * @param type $number_of_post_vars
+ * @param type $post
+ * @return type 
+ */
 function do_curl($url, $return_to_variable = 1, $number_of_post_vars = 0, $post = NULL) {
   global $user;
   // Check if we are inside Drupal and there is a valid user.
-  if ((!isset ($user)) || $user->uid == 0) {
+  if ((!isset($user)) || $user->uid == 0) {
     $fedora_user = 'anonymous';
     $fedora_pass = 'anonymous';
   }
@@ -51,7 +64,7 @@ function do_curl($url, $return_to_variable = 1, $number_of_post_vars = 0, $post
     curl_setopt($ch, CURLOPT_URL, $url);
     curl_setopt($ch, CURLOPT_USERPWD, "$fedora_user:$fedora_pass");
     //curl_setopt($ch, CURLOPT_HTTPAUTH, CURLAUTH_ANY);
-    if ($number_of_post_vars>0&&$post) {
+    if ($number_of_post_vars > 0 && $post) {
       curl_setopt($ch, CURLOPT_POST, $number_of_post_vars);
       curl_setopt($ch, CURLOPT_POSTFIELDS, "$post");
     }
@@ -65,21 +78,23 @@ function do_curl($url, $return_to_variable = 1, $number_of_post_vars = 0, $post
   }
 }
 
+/**
+ * Fedora available
+ * @return type 
+ */
 function fedora_available() {
 
-  $response = do_curl(variable_get('fedora_base_url', 'http://localhost:8080/fedora').'/describe');
+  $response = do_curl(variable_get('fedora_base_url', 'http://localhost:8080/fedora') . '/describe');
   return strstr($response, 'Repository Information HTML Presentation') !== FALSE;
-
 }
 
-
 /**
  * Returns a UTF-8-encoded transcripiton of the string given in $in_str.
  * @param string $in_str
  * @return string A UTF-8 encoded string.
  */
 function fix_encoding($in_str) {
-  $cur_encoding = mb_detect_encoding($in_str) ;
+  $cur_encoding = mb_detect_encoding($in_str);
   if ($cur_encoding == "UTF-8" && mb_check_encoding($in_str, "UTF-8")) {
     return $in_str;
   }
@@ -88,6 +103,11 @@ function fix_encoding($in_str) {
   }
 }
 
+/**
+ * valid pid ??
+ * @param type $pid
+ * @return boolean 
+ */
 function validPid($pid) {
   $valid = FALSE;
   if (strlen(trim($pid)) <= 64 && preg_match('/^([A-Za-z0-9]|-|\.)+:(([A-Za-z0-9])|-|\.|~|_|(%[0-9A-F]{2}))+$/', trim($pid))) {
@@ -97,6 +117,11 @@ function validPid($pid) {
   return $valid;
 }
 
+/**
+ * Valid Dsid ??
+ * @param type $dsid
+ * @return boolean 
+ */
 function validDsid($dsid) {
   $valid = FALSE;
   if (strlen(trim($dsid)) <= 64 && preg_match('/^[a-zA-Z0-9\_\-\.]+$/', trim($dsid))) {
@@ -106,6 +131,11 @@ function validDsid($dsid) {
   return $valid;
 }
 
+/**
+ * fixDsid ??
+ * @param type $dsid
+ * @return string 
+ */
 function fixDsid($dsid) {
   $new_dsid = trim($dsid);
 
@@ -113,16 +143,15 @@ function fixDsid($dsid) {
   $replace = '';
   $new_dsid = preg_replace($find, $replace, $new_dsid);
 
-  if( strlen($new_dsid) > 63 )
+  if (strlen($new_dsid) > 63)
     $new_dsid = substr($new_dsid, -63);
 
-  if( preg_match('/^[^a-zA-Z]/', $dsid ) )
+  if (preg_match('/^[^a-zA-Z]/', $dsid))
     $new_dsid = 'x' . $new_dsid;
 
-  if( strlen($new_dsid) == 0 )
+  if (strlen($new_dsid) == 0)
     $new_dsid = 'item' . rand(1, 100);
 
   return $new_dsid;
-
 }
 

api/rels-ext.inc

@@ -1,34 +1,37 @@
 <?php
+
 // $Id$
 
-/* 
+/**
- * To change this template, choose Tools | Templates
+ * @file
- * and open the template in the editor.
+ * RelsExt class
  */
 
 /**
- * Description of relsext
+ * RelsExt class
- *
- * @author aoneill
  */
 class RelsExt {
+
   // Instance variables
   public $relsExtArray = array();
   private $originalRelsExtArray = array(); // Used to determine the result of modified() funciton.
-    // Member functions
 
   /**
    * Constructor that builds itself by retrieving the RELS-EXT stream from
    * the repository for the given Fedora_Item.
    * @param Fedora_Item $item
    */
-    function RelsExt( $item ) {
-      $relsextxml = $item->get_datastream_dissemination('RELS-EXT');
 
+  function RelsExt($item) {
+    $relsextxml = $item->get_datastream_dissemination('RELS-EXT');
   }
 
+  /**
+   * modified 
+   * @return type 
+   */
   function modified() {
-      return !(empty(array_diff($this->relsExtArray, $this->originalRelsExtArray)) &&
+    return!(empty(array_diff($this->relsExtArray, $this->originalRelsExtArray)) &&
     empty(array_diff($this->originalRelsExtArray, $this->relsExtArray)));
   }
 
@@ -39,5 +42,6 @@ class RelsExt {
   function save() {
     
   }
+
 }
 

api/tagging.inc

@@ -1,20 +1,25 @@
 <?php
+
 // $Id$
 
-/*
+/**
- * @file tagging.inc
+ * @file
+ * TagSet Class
  */
 
 /**
- * Description of tagging
+ * TagSet Class
- *
- * @author aoneill
  */
 class TagSet {
+
   public $tags = array();
   public $item = NULL;
   public $tagsDSID = 'TAGS';
 
+  /**
+   * Constructor
+   * @param type $item 
+   */
   function TagSet($item = NULL) {
     if (!empty($item) && get_class($item) == 'Fedora_Item') {
       $this->item = $item;
@@ -22,8 +27,12 @@ class TagSet {
     }
   }
 
+  /**
+   * Load ??
+   * @return type 
+   */
   function load() {
-    $tagsxml = isset($this->item->datastreams[$this->tagsDSID])? $this->item->get_datastream_dissemination($this->tagsDSID) : NULL;
+    $tagsxml = isset($this->item->datastreams[$this->tagsDSID]) ? $this->item->get_datastream_dissemination($this->tagsDSID) : NULL;
     if (empty($tagsxml)) {
       $this->tags = array();
       return FALSE;
@@ -59,11 +68,11 @@ class TagSet {
       else {
         $this->item->modify_datastream_by_value($tagdoc->saveXML(), $this->tagsDSID, 'Tags', 'text/xml', 'X');
       }
-    } 
+    } catch (exception $e) {
-    catch (exception $e) {
       drupal_set_message('There was an error saving the tags datastream: !e', array('!e' => $e), 'error');
       return FALSE;
     }
     return TRUE;
   }
+
 }