Cleaned up code.

airtime_mvc/application/models/Datatables.php

@@ -57,7 +57,9 @@ class Application_Model_Datatables
     public static function findEntries($con, $displayColumns, $fromTable,
         $data, $dataProp = "aaData")
     {
-        $librarySetting = Application_Model_Preference::getCurrentLibraryTableColumnMap();
+        //$librarySetting = Application_Model_Preference::getCurrentLibraryTableColumnMap();
+        $librarySetting = 
+            Application_Model_Preference::getCurrentLibraryTableColumnMap();
         //$displayColumns[] = 'owner';
 
         // map that maps original column position to db name
@@ -95,9 +97,8 @@ class Application_Model_Datatables
         }
 
         $where = array();
-        /* Holds the parameters for binding after the
+        /* Holds the parameters for binding after the statement has been
-         * statement has been prepared
+            prepared */
-         */
         $params = array();
 
         $advancedWhere = self::buildWhereClauseForAdvancedSearch($dbname2searchTerm);

airtime_mvc/application/models/PlayoutHistory.php

@@ -60,7 +60,7 @@ class Application_Model_PlayoutHistory
         );
 
         $start = $this->startDT->format("Y-m-d H:i:s");
-        $end = $this->endDT->format("Y-m-d H:i:s");
+        $end   = $this->endDT->format("Y-m-d H:i:s");
 
         $historyTable = "(
             select count(schedule.file_id) as played, schedule.file_id as file_id

airtime_mvc/application/models/Preference.php

@@ -1092,26 +1092,41 @@ class Application_Model_Preference
     }
 
 
+
+    public static function getOrderingMap($pref_param) 
+    {
+        $v = self::getValue($pref_param, true);
+
+        $id = function ($x) { return $x; };
+
+        if ($v === '') {
+            return $id;
+        }
+
+        $ds = unserialize($v);
+
+        if (!array_key_exists('ColReorder', $ds)) {
+            return $id;
+        }
+
+        return function ($x) use ($ds) {
+            if (array_key_exists($x, $ds['ColReorder'])) {
+                return $ds['ColReorder'][$x];
+            } else {
+                /*For now we just have this hack for debugging. We should not
+                    rely on this crappy behaviour in case of failure*/
+                Logging::info("Pref: $pref_param");
+                Logging::warn("Index $x does not exist preferences");
+                Logging::warn("Defaulting to identity and printing preferences");
+                Logging::warn($ds);
+                return $x;
+            }
+        };
+    }
+
     public static function getCurrentLibraryTableColumnMap()
     {
-        $v = self::getValue("library_datatable", true);
+        return self::getOrderingMap("library_datatable");
-
-        if ( $v === '' ) {
-            return function ($x) { return $x; };
-        } else {
-            $ds = unserialize($v);
-            return function ($x) use ($ds) { 
-                if ( array_key_exists($x, $ds['ColReorder'] ) ) {
-                    return $ds['ColReorder'][$x]; 
-                } else {
-                    Logging::warn("Index $x does not exist preferences");
-                    Logging::warn("Defaulting to identity and printing 
-                        preferences");
-                    Logging::warn($ds);
-                    return $x;
-                }
-            } ;
-        }
     }
 
     public static function setCurrentLibraryTableSetting($settings)