added category name and document type list sorting

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

added category name and document type list sorting
git-svn-id: https://kt-dms.svn.sourceforge.net/svnroot/kt-dms/trunk@1746 c91229c3-7414-0410-bfa2-8a42b809f60b
Michael Joseph
1 parent f8bee8f2
Showing 3 changed files with 91 additions and 40 deletions
lib/documentmanagement/DocumentBrowser.inc
presentation/lookAndFeel/knowledgeTree/documentmanagement/browseBL.php
presentation/lookAndFeel/knowledgeTree/documentmanagement/browseUI.inc
@@ -92,8 +92,18 @@ class DocumentBrowser {
             // FIXME: in the same unit?
             // if we're sorting by name then sort folders in the appropriate direction
-            $folderQuery = "SELECT id FROM $default->owl_folders_table WHERE parent_id=" . $folderID . " ORDER BY name " . ($sSortField == "name" ? $sSortDirection : "ASC");
-            $default->log->debug("DocumentBrowser::browseByFolder child folder query=$folderQuery");
+            $folderQuery = "SELECT f.id FROM $default->owl_folders_table f ";
+                     
+            if ( $sSortField == "creator_id" ) {
+            	$folderQuery .= "INNER JOIN " .  $this->aSortCriteria["creator_id"]["lookup"]["lookupTable"] . " lt ON f.$sSortField=lt.id ";
+            }            
+            $folderQuery .= "WHERE f.parent_id=$folderID ";            
+            if ( $sSortField == "creator_id" ) {
+            	$folderQuery .= "ORDER BY lt." . $this->aSortCriteria["creator_id"]["lookup"]["lookupField"] . " $sSortDirection";
+            } else {
+            	$folderQuery .= "ORDER BY name " . ($sSortField == "name" ? $sSortDirection : "ASC");
+            }
+            
             if ($sql->query($folderQuery)) {
                 while ($sql->next_record()) {
                     // add the child folders to the array
@@ -161,8 +171,10 @@ class DocumentBrowser {
             $results["categories"][] = "Categories";
             // get a list of category values
-            $query = "SELECT DISTINCT value FROM $default->owl_document_fields_table WHERE document_field_id=$categoryFieldID ORDER BY value ASC";
-            $default->log->debug("DocumentBrowser::browseByCategory category listing query=$query");
+            $query = "SELECT DISTINCT value FROM $default->owl_document_fields_table " .
+                     "WHERE document_field_id=$categoryFieldID " .
+                     "ORDER BY value " . ($sSortField == "name" ? $sSortDirection : "ASC");
+ 
             $sql->query($query);
             // loop through resultset, build array and return
             while ($sql->next_record()) {
@@ -210,7 +222,7 @@ class DocumentBrowser {
      */    
     function browseByDocumentType($documentTypeID = -1, $sSortField = "name", $sSortDirection = "asc") {
         global $default;
-        
+
         $results = array();
         $sql = $default->db;
@@ -220,7 +232,8 @@ class DocumentBrowser {
             $results["documentTypes"][] = array("name" => "Document Types");
             // return a list of document types
-            $query = "SELECT * FROM $default->owl_document_types_table";
+            $query = "SELECT * FROM $default->owl_document_types_table ORDER BY name " . ($sSortField == "name" ? $sSortDirection : "ASC");
+
             $sql->query($query);
             while ($sql->next_record()) {
                 $results["documentTypes"][] = array ("id" => $sql->f("id"), "name" => $sql->f("name"));
@@ -61,7 +61,7 @@ if (checkSession()) {
     $oDocBrowser = new DocumentBrowser();
     // instantiate my content pattern
     $oContent = new PatternCustom();
-    
+
     switch ($fBrowseType) {
         case "folder" : // retrieve folderID if present
                         if (!$fFolderID) {
@@ -75,7 +75,7 @@ if (checkSession()) {
         case "category" :
                         $sectionName = "Manage Categories";
                         if (!$fCategoryName) {
-                            $aResults = $oDocBrowser->browseByCategory();
+                            $aResults = $oDocBrowser->browseByCategory("", $fSortBy, $fSortDirection);
                         } else {
                             $aResults = $oDocBrowser->browseByCategory($fCategoryName, $fSortBy, $fSortDirection);
                         }
@@ -84,7 +84,7 @@ if (checkSession()) {
         case "documentType" :
                         $sectionName = "Manage Document Types";
                         if (!$fDocumentTypeID) {
-                            $aResults = $oDocBrowser->browseByDocumentType();
+                            $aResults = $oDocBrowser->browseByDocumentType(-1, $fSortBy, $fSortDirection);
                         } else {
                             $aResults = $oDocBrowser->browseByDocumentType($fDocumentTypeID, $fSortBy, $fSortDirection);
                         }
@@ -21,7 +21,7 @@ function renderHeading($sHeading) {
     global $default;
     $sSectionName = $default->siteMap->getSectionName(substr($_SERVER["PHP_SELF"], strlen($default->rootUrl), strlen($_SERVER["PHP_SELF"])));
     $sColor = $default->siteMap->getSectionColour($sSectionName, "th");
-    $sToRender  = "<table border=\"0\" width=\"100%\">\n";
+    $sToRender  = "<table border=\"0\" width=\"600\">\n";
     $sToRender .= "<tr align=\"left\"><th class=\"sectionHeading\" bgcolor=\"$sColor\"><font color=\"ffffff\">$sHeading</font></th></tr>\n";
     $sToRender .= "<tr/>\n";
     $sToRender .= "<tr/>\n";
@@ -46,7 +46,7 @@ function renderBrowseTypeSelect($sBrowseType) {
                                (($sBrowseType=="documentType") ? " checked=\"checked\"" : "") . "/> Document Type\n</span>";
 }
-// <category display helper methods>
+// <category browse helper methods>
 /**
  * Displays the passed category path as a link
@@ -92,7 +92,20 @@ function renderCategoryResults($aResults) {
     if ($aResults["categories"][0] == "Categories") {
         // loop through categories and display them
         for ($i=1; $i<count($aResults["categories"]); $i++) {
-            $sToRender .= "<tr><td bgcolor=\"" . getColour($i) . "\">" . displayCategoryLink($aResults["categories"][$i]) . "</td></tr>\n";
+            $sToRender .= "<tr bgcolor=\"" . getColour($i) . "\">";
+                     
+            // category name
+            $sToRender .= "<td>" . displayCategoryLink($aResults["categories"][$i]) . "</td>";
+            // blank filename
+            $sToRender .= "<td></td>";
+            // creator name
+            $sToRender .= "<td></td>";
+            // modified date
+            $sToRender .= "<td></td>";
+            // document type
+            $sToRender .= "<td></td>";                      
+
+            $sToRender .= "</tr>\n";
         }
     } else {
         // else the first entry is the category name, so display the documents in the category
@@ -102,7 +115,7 @@ function renderCategoryResults($aResults) {
     return $sToRender;
 }
-// </category display helper methods>
+// </category browse helper methods>
 // <document type display helper methods>
 /**
@@ -145,7 +158,18 @@ function renderDocumentTypeResults($aResults) {
     if ($aResults["documentTypes"][0]["name"] == "Document Types") {
         // loop through document types and display them
         for ($i=1; $i<count($aResults["documentTypes"]); $i++) {
-            $sToRender .= "<tr><td bgcolor=\"" . getColour($i) . "\">" . displayDocumentTypeLink($aResults["documentTypes"][$i]) . "</td></tr>";
+            $sToRender .= "<tr bgcolor=\"" . getColour($i) . "\">";			
+            // document type name
+            $sToRender .= "<td>" . displayDocumentTypeLink($aResults["documentTypes"][$i]) . "</td>";
+            // blank filename
+            $sToRender .= "<td></td>";
+            // creator name
+            $sToRender .= "<td></td>";
+            // modified date
+            $sToRender .= "<td></td>";
+            // document type
+            $sToRender .= "<td></td>";                      
+            $sToRender .= "</tr>\n";                   	
         }
     } else {
         // else the first entry is the document type name, so display the documents in the document type
@@ -154,9 +178,48 @@ function renderDocumentTypeResults($aResults) {
     }
     return $sToRender;
 }
-// </document type display helper methods>
+// </document type browse helper methods>
+// <folder browse helper methods>
+/**
+ * Displays the folders in the browse results
+ *
+ * @param array the browse result objects
+ */
+function renderFolderResults($aResults) {
+    global $default;
+    $sToRender = "";
+	    
+    // now loop through the rest of the folders and display links
+    if (count($aResults["folders"]) > 1) {
+        for ($i=1; $i<count($aResults["folders"]); $i++) {
+            $sFolderLink = displayFolderLink($aResults["folders"][$i]);
+            $oCreator = User::get($aResults["folders"][$i]->getCreatorID());
+            
+            $sToRender .= "<tr bgcolor=\"" . getColour($i) . "\">";            
+            // folder name
+            $sToRender .= "<td>" . $sFolderLink . "</td>";
+            // blank filename (??: folder description?)
+            $sToRender .= "<td></td>";
+            // creator name
+            $sToRender .= "<td>" . $oCreator->getName() . "</td>";
+            // modified date (TODO: add to db)
+            $sToRender .= "<td></td>";
+            // document type (??: display one of the mapped document types?)
+            $sToRender .= "<td></td>";                        
+            $sToRender .= "</tr>\n";
+        }
+    } else {
+        $sToRender .= "<tr><td colspan=\"5\">This folder contains no sub folders</td></tr>";
+    }
+        
+    $sToRender .= "<tr><td>" . renderDocumentList($aResults, "This folder contains no documents", "You don't have access to the documents in this folder") . "</td></tr>\n";
+    
+    return $sToRender;
+}
+// </folder browse helper methods>
+
 /**
  * Displays the headings for the displayed document details and enables
  * resorting the contents
@@ -201,31 +264,6 @@ function renderSortHeadings($sSortBy, $sSortDirection) {
 }
 /**
- * Displays the folders in the browse results
- *
- * @param array the browse result objects
- */
-function renderFolderResults($aResults) {
-    global $default;
-    $sToRender = "";
-	    
-    // now loop through the rest of the folders and display links
-    if (count($aResults["folders"]) > 1) {
-        for ($i=1; $i<count($aResults["folders"]); $i++) {
-            $sFolderLink = displayFolderLink($aResults["folders"][$i]);
-            $oCreator = User::get($aResults["folders"][$i]->getCreatorID());
-            $sToRender .= "<tr bgcolor=\"" . getColour($i) . "\"><td colspan=\"2\" nowrap width=\"100%\">" . $sFolderLink . "</td><td>" . $oCreator->getName() . "</td><td colspan=\"2\"></td></tr>\n";
-        }
-    } else {
-        $sToRender .= "<tr><td>This folder contains no sub folders</td></tr>";
-    }
-        
-    $sToRender .= "<tr><td>" . renderDocumentList($aResults, "This folder contains no documents", "You don't have access to the documents in this folder") . "</td></tr>\n";
-    
-    return $sToRender;
-}
-
-/**
  * Displays the documents in the browse results
  *
  * @param array the browse result objects