Revision: 1189
http://mrbs.svn.sourceforge.net/mrbs/?rev=1189&view=rev
Author: cimorrison
Date: 2009-09-03 13:49:45 +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/day.php
mrbs/trunk/web/month.php
Modified: mrbs/trunk/web/day.php
===================================================================
--- mrbs/trunk/web/day.php 2009-09-03 13:31:50 UTC (rev 1188)
+++ mrbs/trunk/web/day.php 2009-09-03 13:49:45 UTC (rev 1189)
@@ -238,7 +238,7 @@
// pull the data from the db and store it. Convienently we can print the room
// headings and capacities at the same time
-$sql = "select room_name, capacity, id, description from $tbl_room where
area_id=$area order by 1";
+$sql = "SELECT room_name, capacity, id, description FROM $tbl_room WHERE
area_id=$area ORDER BY room_name";
$res = sql_query($sql);
Modified: mrbs/trunk/web/month.php
===================================================================
--- mrbs/trunk/web/month.php 2009-09-03 13:31:50 UTC (rev 1188)
+++ mrbs/trunk/web/month.php 2009-09-03 13:49:45 UTC (rev 1189)
@@ -300,7 +300,7 @@
FROM $tbl_entry
WHERE room_id=$room
AND start_time <= $midnight_tonight[$day_num] AND end_time >
$midnight[$day_num]
- ORDER by 1";
+ ORDER by start_time";
// Build an array of information about each day in the month.
// The information is stored as:
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
