Revision: 1188
          http://mrbs.svn.sourceforge.net/mrbs/?rev=1188&view=rev
Author:   cimorrison
Date:     2009-09-03 13:31:50 +0000 (Thu, 03 Sep 2009)

Log Message:
-----------
Changed SQL ORDER BY clauses to use column names rather than column positions 
in order to improve maintainability.   No functional change to MRBS.

Modified Paths:
--------------
    mrbs/trunk/web/report.php

Modified: mrbs/trunk/web/report.php
===================================================================
--- mrbs/trunk/web/report.php   2009-09-03 13:17:03 UTC (rev 1187)
+++ mrbs/trunk/web/report.php   2009-09-03 13:31:50 UTC (rev 1188)
@@ -673,12 +673,12 @@
   if ( $sortby == "r" )
   {
     // Order by Area, Room, Start date/time
-    $sql .= " ORDER BY 9,10,2";
+    $sql .= " ORDER BY area_name, room_name, start_time";
   }
   else
   {
     // Order by Start date/time, Area, Room
-    $sql .= " ORDER BY 2,9,10";
+    $sql .= " ORDER BY start_time, area_name, room_name";
   }
 
   // echo "<p>DEBUG: SQL: <tt> $sql </tt></p>\n";


This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.

------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with 
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
_______________________________________________
Mrbs-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/mrbs-commits

Reply via email to