From 89e61faa5ad20a1963d44bf03900b94de7380ba8 Mon Sep 17 00:00:00 2001
From: Neil Blakey-Milner <nbm@knowledgetree.com>
Date: Mon, 7 Nov 2005 14:42:41 +0000
Subject: [PATCH] Group document-specific metadata display by fieldset.

---
 presentation/lookAndFeel/knowledgeTree/documentmanagement/viewUI.inc | 64 ++++++++++++++++++++++++++++++++++++++++++++--------------------
 templates/ktcore/document_specific_metadata.smarty                   | 34 ++++++++++++++++++----------------
 2 files changed, 62 insertions(+), 36 deletions(-)

diff --git a/presentation/lookAndFeel/knowledgeTree/documentmanagement/viewUI.inc b/presentation/lookAndFeel/knowledgeTree/documentmanagement/viewUI.inc
index 176aee8..3b3bfc9 100644
--- a/presentation/lookAndFeel/knowledgeTree/documentmanagement/viewUI.inc
+++ b/presentation/lookAndFeel/knowledgeTree/documentmanagement/viewUI.inc
@@ -169,34 +169,58 @@ function renderTypeSpecificMetaData($oDocument, $bEditable) {
         $aDocuments[] = $iVersionID;
     }
 	
-    /*ok*/ $sQuery = "SELECT DF.name AS name, DFL.value AS value " .
+    $sParam = DBUtil::paramArray($aDocuments);
+    $sQuery = "SELECT D.id AS document_id, DF.id AS field_id, DFL.value AS value, F.id AS fieldset_id " .
               "FROM $default->documents_table AS D INNER JOIN document_fields_link AS DFL ON D.id = DFL.document_id " .
               "INNER JOIN $default->document_fields_table AS DF ON DF.ID = DFL.document_field_id " .
-              "WHERE D.id = ? " .
+              "INNER JOIN $default->fieldsets_table AS F ON F.id = DF.parent_fieldset " .
+              "WHERE D.id IN ($sParam) " .
               "AND DF.name NOT LIKE 'Category' " .
               "AND DF.is_generic = 0";
-
-    $iDocumentsLen = count($aDocuments);
-    $map = array();
-    $iDocumentCount = 0;
-    foreach ($aDocuments as $iDocumentID) {
-        $map['Metadata Version'] = KTUtil::arrayGet($map, 'Metadata Version', array_fill(0, $iDocumentsLen, null));
-        $oThisDocument =& Document::get($iDocumentID);
-        if (PEAR::isError($oThisDocument)) {
-            continue;
+    $aParam = $aDocuments;
+
+    $aResults = DBUtil::getResultArray(array($sQuery, $aParam));
+
+    $aMap = array();
+    foreach ($aResults as $aResult) {
+        $fieldset_id = $aResult['fieldset_id'];
+        $field_id = $aResult['field_id'];
+        $document_id = $aResult['document_id'];
+        $aMyFieldset = KTUtil::arrayGet($aMap, $fieldset_id);
+        if (empty($aMyFieldset)) {
+            $aMap[$fieldset_id]["fieldset"] = KTFieldset::get($fieldset_id);
+            $aMap[$fieldset_id]["fields"] = array();
         }
-        if ($oThisDocument === false) {
-            continue;
+        $aMyField = KTUtil::arrayGet($aMap[$fieldset_id]['fields'], $field_id);
+        if (empty($aMyField)) {
+            $aMap[$fieldset_id]['fields'][$field_id] = array();
+            $aMap[$fieldset_id]['fields'][$field_id]['field'] = DocumentField::get($field_id);
+            $aMap[$fieldset_id]['fields'][$field_id]['values'] = array();
         }
-        $map['Metadata Version'][$iDocumentCount] = $oThisDocument->getMetadataVersion();
-        $aTDRows = DBUtil::getResultArray(array($sQuery, array($iDocumentID)));
-        foreach ($aTDRows as $aRow) {
-            $newarray = KTUtil::arrayGet($map, $aRow['name'], array_fill(0, $iDocumentsLen, null));
-            $newarray[$iDocumentCount] = $aRow['value'];
-            $map[$aRow['name']] = $newarray;
+        $aMap[$fieldset_id]['fields'][$field_id]['values'][$document_id] = $aResult['value'];
+    }
+    $oTemplating =& KTTemplating::getSingleton();
+    $oTemplate = $oTemplating->loadTemplate('ktcore/document_specific_metadata');
+    $sToRender = $oTemplate->render(array(
+        'aDocumentIds' => $aDocuments,
+        'map' => $aMap,
+    ));
+    return $sToRender;
+
+    print "<pre>";
+    foreach ($aMap as $k => $v) {
+        print $v['fieldset']->getName() . "\n";
+        foreach ($v['fields'] as $k => $aField) {
+            print $aField['field']->getName() . ": ";
+            foreach ($aField['values'] as $document_id => $sValue) {
+                print "$document_id => $sValue";
+            }
+            print "\n";
         }
-        $iDocumentCount++;
+        print "\n\n";
     }
+    exit(0);
+
 
     $oTemplating =& KTTemplating::getSingleton();
     $oTemplate = $oTemplating->loadTemplate('ktcore/document_specific_metadata');
diff --git a/templates/ktcore/document_specific_metadata.smarty b/templates/ktcore/document_specific_metadata.smarty
index d81bac1..27b5379 100644
--- a/templates/ktcore/document_specific_metadata.smarty
+++ b/templates/ktcore/document_specific_metadata.smarty
@@ -1,17 +1,19 @@
-<table class="prettysw" cellpadding="0" cellspacing="0">
-<caption colspan="{$map|count}"><b>Type Specific Meta Data</b></caption>
-{ foreach key=k item=values from=$map }
-<tr>
-  <th>{$k}</th>
-  { foreach item=v from=$values }
-    { cycle values="odd,compare" name=$k assign="class" }
-    <td class="{$class}" style="width: 10em">{$v|default:"&nbsp;"}</td>
-  { /foreach }
-</tr>
-{ /foreach }
-</table>
-
-{ if $editable }
-{$editable}
-{ /if }
+{foreach key=fieldset_id item=fieldset_info from=$map}
+  {assign var=fieldset value=$fieldset_info.fieldset}
+  <fieldset>
+    <legend>{$fieldset->getName()}</legend>
 
+    <table class="prettysw" cellpadding="0" cellspacing="0">
+  {foreach key=field_id item=field_info from=$fieldset_info.fields}
+      <tr>
+    {assign var=field value=$field_info.field}
+        <th>{$field->getName()}</th>
+    {assign var=values value=$field_info.values}
+    {foreach item=iDocumentId from=$aDocumentIds}
+        <td>{$values.$iDocumentId}</td>
+    {/foreach}
+      </tr>
+  {/foreach}
+    </table>
+  </fieldset>
+{/foreach}
--
libgit2 0.21.4