merge fix for KEP-70 from preview.

git-svn-id: https://kt-dms.svn.sourceforge.net/svnroot/kt-dms/trunk@5625 c91229c3-7414-0410-bfa2-8a42b809f60b

merge fix for KEP-70 from preview.
git-svn-id: https://kt-dms.svn.sourceforge.net/svnroot/kt-dms/trunk@5625 c91229c3-7414-0410-bfa2-8a42b809f60b
Brad Shuttleworth
1 parent 3e936041
Showing 5 changed files with 71 additions and 25 deletions
lib/metadata/metadatautil.inc.php
plugins/ktcore/admin/ajaxComplexConditionals.php
plugins/ktcore/admin/manageConditionals.php
templates/ktcore/metadata/conditional/editsimple.smarty
templates/ktcore/metadata/editFieldset.smarty
@@ -533,7 +533,7 @@ class KTMetadataUtil {
         $sTable = KTUtil::getTableName('field_value_instances');
         $sLookupTable = KTUtil::getTableName('metadata_lookup');
         $aQuery = array(
-            "SELECT COUNT(FVI.id) AS cnt FROM $sTable AS FVI LEFT JOIN $sLookupTable AS ML ON (FVI.field_value_id = ML.id) WHERE FVI.field_id = ?",
+            "SELECT COUNT(FVI.id) AS cnt FROM $sTable AS FVI LEFT JOIN $sLookupTable AS ML ON (FVI.field_value_id = ML.id) WHERE FVI.field_id = ? AND ML.disabled = 0",
             array($iMasterFieldId),
         );
         $iCount = DBUtil::getOneResultKey($aQuery, 'cnt');
@@ -551,10 +551,12 @@ class KTMetadataUtil {
         // check that each master-field value has a valueinstance assigned.
         $sTable = KTUtil::getTableName('metadata_lookup');
         $aQuery = array(
-            "SELECT COUNT(id) AS cnt FROM $sTable WHERE document_field_id = ? AND disabled <> 0 ",
+            "SELECT COUNT(id) AS cnt FROM $sTable WHERE document_field_id = ? AND disabled = 0 ",
             array($iMasterFieldId),
         );
         $iValCount = DBUtil::getOneResultKey($aQuery, 'cnt');
+        
+        // assumes that there cannot be more than 1 value instance for each master-field-value.
         if ($iValCount != $iCount) {
             return PEAR::raiseError(sprintf(_kt('%d values for the Master Field are not assigned to behaviours.'), ($iValCount - $iCount)));
         }
@@ -64,7 +64,7 @@ class AjaxConditionalAdminDispatcher extends KTAdminDispatcher {
         $oTemplate =& $oTemplating->loadTemplate('ktcore/metadata/conditional/ajax_complex_get_item_list');
         $aValues = array();
-        foreach ($oField->getValues() as $oValue) {
+        foreach ($oField->getEnabledValues() as $oValue) {
             if (empty($parent_behaviour)) {
                 $oInstance = KTValueInstance::getByLookupSingle($oValue);
                 if (empty($oInstance)) {
@@ -132,7 +132,7 @@ class AjaxConditionalAdminDispatcher extends KTAdminDispatcher {
         $aValues = array();
         $aBehaviours = array();
-        foreach ($oField->getValues() as $oValue) {
+        foreach ($oField->getEnabledValues() as $oValue) {
             if (empty($parent_behaviour)) {
                 $oInstance = KTValueInstance::getByLookupSingle($oValue);
                 if (!empty($oInstance)) {
@@ -80,7 +80,7 @@ class ManageConditionalDispatcher extends KTAdminDispatcher {
         $oFieldset =& KTFieldset::get($fieldset_id);
         $aFields =& $oFieldset->getFields();
-        
+
         $this->aBreadcrumbs[] = array(
             'url' => KTUtil::ktLink('admin.php','documents/fieldmanagement','action=edit&fFieldsetId=' . $oFieldset->getId()),
             'name' => $oFieldset->getName()
@@ -102,11 +102,37 @@ class ManageConditionalDispatcher extends KTAdminDispatcher {
             $aChildren[] = $row['child_field_id'];
             $aOrders[$row['parent_field_id']] = $aChildren;
         } 
+        
+        // for useability, they can go in any order
+        // but master field should be first.  beyond that 
+        // it can get odd anyway. 
+        
+        $aKeyedFields = array();
+        $aOrderedFields = array();
+        $aStack = array($oFieldset->getMasterFieldId());
+        
+        // first, key
+        foreach ($aFields as $oField) {
+            $aKeyedFields[$oField->getId()] = $oField;
+        }
+        
+        while (!empty($aStack)) {
+            $iKey = array_shift($aStack);
+            // this shouldn't happen, but avoid it anyway.
+            if (!is_null($aKeyedFields[$iKey])) {
+                $aOrderedFields[] = $aKeyedFields[$iKey];
+                unset($aKeyedFields[$iKey]);
+            }
+            // add children to stack
+            $aStack = kt_array_merge($aStack, $aOrders[$iKey]);
+        }
+        
+        
         $aTemplateData = array(
             "context" => &$this,
             "fieldset_id" => $fieldset_id,
             "ordering" => $aOrders,
-            "aFields" => $aFields,
+            "aFields" => $aOrderedFields,
             "iMasterFieldId" => $oFieldset->getMasterFieldId(),
         );
         return $oTemplate->render($aTemplateData);
@@ -126,7 +152,8 @@ class ManageConditionalDispatcher extends KTAdminDispatcher {
          *  FIXME we fake it here with nested arrays...
          */
         $oFieldset =& KTFieldset::get($fieldset_id);
-        $aFields =& $oFieldset->getFields();
+        $aFields =& $oFieldset->getFields();        
+        
         $this->aBreadcrumbs[] = array(
             'url' => KTUtil::ktLink('admin.php','documents/fieldmanagement','action=edit&fFieldsetId=' . $oFieldset->getId()),
             'name' => $oFieldset->getName()
@@ -149,12 +176,33 @@ class ManageConditionalDispatcher extends KTAdminDispatcher {
             $aOrders[$row['parent_field_id']] = $aChildren;
         } 
+
+        $aKeyedFields = array();
+        $aOrderedFields = array();
+        $aStack = array($oFieldset->getMasterFieldId());
+        
+        // first, key
+        foreach ($aFields as $oField) {
+            $aKeyedFields[$oField->getId()] = $oField;
+        }
+        
+        while (!empty($aStack)) {
+            $iKey = array_shift($aStack);
+            // this shouldn't happen, but avoid it anyway.
+            if (!is_null($aKeyedFields[$iKey])) {
+                $aOrderedFields[] = $aKeyedFields[$iKey];
+                unset($aKeyedFields[$iKey]);
+            }
+            // add children to stack
+            $aStack = kt_array_merge($aStack, $aOrders[$iKey]);
+        }        
+        
         $this->oPage->setBreadcrumbDetails(_kt('Manage complex conditional'));
         $aTemplateData = array(
             "context" => &$this,
             "fieldset_id" => $fieldset_id,
             "ordering" => $aOrders,
-            "aFields" => $aFields,
+            "aFields" => $aOrderedFields,
             "iMasterFieldId" => $oFieldset->getMasterFieldId(),
         );
         return $oTemplate->render($aTemplateData);
@@ -95,7 +95,7 @@ refresh the page.{/i18n}&lt;/p&gt;
      <p class="inactivity_message">{i18n}This field is not controlled by the currently active group.{/i18n}</p>
      <div class="lookup_items">
          <select class="item_list">
-  {foreach from=$oField->getValues() item=oMetaData}
+  {foreach from=$oField->getEnabledValues() item=oMetaData}
 				<option value="{$oMetaData->getId()}">{$oMetaData->getName()|escape}</option>
   {/foreach}
            </select>
@@ -25,21 +25,6 @@
   {$oWidget->render()}
 {/foreach}
-{if ($oFieldset->getIsConditional() || $context->haveConditional())}
-<div class="field">
-<label>{i18n}Conditional{/i18n}</label>
-<p class="descriptiveText">
-{i18n}A <strong>conditional</strong> fieldset contains only lookup fields.  The values for each
-field can depend on the user's selections for the others.{/i18n}
-</p>
-{if $oFieldset->getIsConditional() }
-    {i18n}Yes{/i18n} 
-{else}
-    {i18n}No{/i18n} 
-{/if}
-</div>
-{/if}
-
 <div class="form_actions">
 <input type="submit" name="submit" value="{i18n}Change{/i18n}" />
 </div>
@@ -65,7 +50,16 @@ field can depend on the user&#39;s selections for the others.{/i18n}
 field can depend on the user's selections for the others.{/i18n}
 </p>
-
+<div class="field">
+<label>{i18n}Conditional{/i18n}</label>
+<p class="descriptiveText">
+</p>
+{if $oFieldset->getIsConditional() }
+    {i18n}Yes{/i18n} 
+{else}
+    {i18n}No{/i18n} 
+{/if}
+</div>
 {if $oFieldset->getIsConditional() }
@@ -75,6 +69,8 @@ field can depend on the user&#39;s selections for the others.{/i18n}
 <input type="submit" name="submit" value="{i18n}Manage conditional{/i18n}" />
 </form>
+<br />
+
 <form action="{$smarty.server.PHP_SELF}" method="POST">
 <input type="hidden" name="fFieldsetId" value="{$oFieldset->getId()}" />
 <input type="hidden" name="action" value="removeConditional" />