refs #16433 [fid_bbi] bugfixes in detail view for SolrDefault records

* remove seemingly duplicate holdings tab
* fix bug in publicationDetails template

module/fid_bbi/config/module.config.php

@@ -70,6 +70,7 @@ $config = [
                 'tabs'       => [
                     'Description'   => 'Description',
                     'Worldcat'      => 'Worldcat',
+                    'Holdings'      => null,
                     'TOC'           => 'TOC',
                     'UserComments'  => 'UserComments',
                     'Reviews'       => 'Reviews',

themes/fid_bbi/templates/RecordDriver/DefaultRecord/data-publicationDetails.phtml

 <!-- fid_bbi: RecordDriver - solrDefault - data-publicationDetails -->
 <? if (!empty($data)): ?>
 <div itemscope itemtype="http://schema.org/publisher">
-    <? foreach ($data as $field): ?>
+    <? foreach ((array)$data as $field): ?>
+    <?php if(is_string($data)): ?>
+    <?=$this->escapeHtml($data)?>
+    <?php else: ?>
     <span property="publisher" typeof="Organization">
         <? $pubPlace = $field->getPlace(); if (!empty($pubPlace)): ?>
           <span property="location" typeof="Place">
@@ -20,7 +23,7 @@
     <? /*else: ?>
         <?=$this->escapeHtml($pubDateSort)*/?></span>
     <? endif; ?>
-    </span><br/>
+    </span><?php endif; ?><br/>
     <? endforeach; ?>
 </div>
 <? endif ?>