Merge pull request #143 from adam-vessey/6.x

Little more checking if there is any tab content...

ObjectHelper.inc

@@ -137,9 +137,6 @@ class ObjectHelper {
       $fedoraPass = $user->pass;
     }
     
-    $dataStreamInfo = $item->get_datastream_info($dsID);
-    $contentSize = $dataStreamInfo->datastream->size;
-
     if (function_exists("curl_init")) {
       $url = variable_get('fedora_base_url', 'http://localhost:8080/fedora') . '/objects/' . $pid . '/datastreams/' . $dsID . '/content';
       $query_options = array();

plugins/FedoraObjectDetailedContent.inc

@@ -76,7 +76,7 @@ class FedoraObjectDetailedContent {
       $dc_array = $dc_returned;
       $dc_array['#weight'] = $i++;
     }
-    else {
+    elseif (!empty($dc_returned)) {
       $dc_array = array(
         '#type' => 'markup',
         '#value' => $dc_returned, //XXX:  This could easily be done in Drupal, instead of using an XSL
@@ -87,8 +87,12 @@ class FedoraObjectDetailedContent {
     $tabset['fedora_object_details']['tabset']['view'] = array(
       '#type' => 'tabpage',
       '#title' => t('View'),
-      'dc' => $dc_array,
     );
+
+    if (!empty($dc_array)) {
+      $tabset['fedora_object_details']['tabset']['view']['dc'] = $dc_array;
+    }
+
     if (fedora_repository_access(ObjectHelper :: $VIEW_DETAILED_CONTENT_LIST, $this->pid, $user)) {
       $tabset['fedora_object_details']['tabset']['view'] += array(
         'list' => array(
@@ -135,7 +139,8 @@ class FedoraObjectDetailedContent {
     }
 
     $ts = $tabset['fedora_object_details']['tabset'];
-    if (array_key_exists('view', $ts) || array_key_exists('edit', $ts)) {
+    if ((array_key_exists('view', $ts) && (count(element_children($ts['view'])) > 0)) || 
+      (array_key_exists('edit', $ts) && (count(element_children($ts['edit'])) > 0 || array_key_exists('#content', $ts['edit'])))) {
       return $tabset;
     }
     else {