X-Git-Url: https://delta.frontaccounting.com/gitweb/?a=blobdiff_plain;f=includes%2Fdb_pager.inc;h=f9089e9ee5a0f2ba89f152762f0645f5dab020ad;hb=5e63c6ace55729bbb5ee3b060035a25a4426eb0a;hp=64a79eda8e633cd4316147599bc1be0062fa0346;hpb=46c5f7a65a7659a44ae8254c63152074363d3987;p=fa-stable.git diff --git a/includes/db_pager.inc b/includes/db_pager.inc index 64a79eda..f9089e9e 100644 --- a/includes/db_pager.inc +++ b/includes/db_pager.inc @@ -57,7 +57,12 @@ class db_pager { var $key; // key field name // db_pager constructor - // accepts $sql like 'SELECT ...[FROM ...][WHERE ...][GROUP ...][ORDER ...]' + // accepts $sql query either as: + // a. string in form 'SELECT field_list FROM table_joins [WHERE conditions [GROUP group_list [ORDER order_list]]]' + // - if WHERE keyword is used in table_joins, WHERE conditions is obligatory + // b. associative array using select, where, group and order keys ex : + // array('select' => 'SUM(quantity)', 'from' => TB_PREF."stock_moves", 'group' => 'location') + // // $name is base name for pager controls function db_pager($sql, $name, $table = null, $page_len=0) { @@ -68,35 +73,57 @@ class db_pager { $this->set_sql($sql); } // - // Parse base sql select query. + // Parse base sql select query or use an associative array. // function set_sql($sql) { + global $SysPrefs; + if ($sql != $this->sql) { $this->sql = $sql; $this->ready = false; - $parts = preg_split('/\sORDER\s*BY\s/si', $sql, 2); - if(count($parts) == 2) { - $sql = $parts[0]; - $this->order = $parts[1]; - } - $parts = preg_split('/\sGROUP\s*BY\s/si', $sql, 2); - if(count($parts) == 2) { - $sql = $parts[0]; - $this->group = $parts[1]; - } - $parts = preg_split('/\sWHERE\s/si', $sql, 2); - if(count($parts) == 2) { - $sql = $parts[0]; - $this->where = $parts[1]; - } - $parts = preg_split('/\sFROM\s/si', $sql, 2); - if(count($parts) == 2) { - $sql = $parts[0]; - $this->from = $parts[1]; + + if(is_array($sql)) { + foreach(explode(' ', 'select from where group order') as $section) { + $this->$section = @$sql[$section]; + } + $this->select = "SELECT ".$this->select; + } + else { + // parse the query + $parts = preg_split('/\sFROM\s/si', $sql, 2); + if (count($parts) == 2) { + $this->select = $parts[0]; + $sql = $parts[1]; + } else { + if ($SysPrefs->go_debug) + display_error("Invalid sql input for db_pager"); + } + + $parts = preg_split('/\sWHERE(?!.*WHERE.*)\s/si', $sql, 2); // last occurence + if(count($parts) == 2) { + $this->from = $parts[0]; + $sql = $parts[1]; + + $parts = preg_split('/\sORDER\s*BY\s/si', $sql, 2); + if(count($parts) == 2) { + $sql = $parts[0]; + $this->order = $parts[1]; + } + $parts = preg_split('/\sGROUP\s*BY\s/si', $sql, 2); + if(count($parts) == 2) { + $sql = $parts[0]; + $this->group = $parts[1]; + } + $this->where = $sql; + } } - $this->select = $sql; } + //_vd($this->select); + //_vd($this->from); + //_vd($this->where); + //_vd($this->group); + //_vd($this->order); } // // Set additional constraint on record set @@ -129,7 +156,16 @@ class db_pager { // function sort_table($col) { + + $max_priority = 0; + foreach($this->columns as $id => $_col) { + if(!isset($_col['ord_priority'])) continue; + $max_priority = max($max_priority, $_col['ord_priority']); + }; + + $ord = $this->columns[$col]['ord']; + $this->columns[$col]['ord_priority'] = $max_priority+1; // set priority , higher than anything else $ord = ($ord == '') ? 'asc' : (($ord == 'asc') ? 'desc' : ''); $this->columns[$col]['ord'] = $ord; $this->set_page(1); @@ -174,7 +210,7 @@ class db_pager { } else return false; return true; - } + } // // Calculates page numbers for html controls. // @@ -274,7 +310,16 @@ class db_pager { if ($group) $sql.= " GROUP BY $group"; $ord = array(); + // sort order column by priority instead of table order. + $columns = array(); foreach ($this->columns as $col) { + if(isset($col['ord_priority'])) { + $columns[$col['ord_priority']] = $col; + } + } + krsort($columns); + + foreach ($columns as $col) { if (isset($col['ord'])) { if ( $col['ord'] != '' && isset($col['name'])) { $ord[] = $col['name'] .' '. $col['ord']; @@ -283,7 +328,8 @@ class db_pager { } if (count($ord)) { - $sql .= " ORDER BY " . implode($ord, ','); + $ord = array_map('db_escape_function', $ord); + $sql .= " ORDER BY " . implode(',', $ord); } else { if($order) $sql .= " ORDER BY $order"; // original base query order @@ -302,7 +348,7 @@ class db_pager { // function _init() { - global $go_debug; + global $SysPrefs; if ($this->ready == false ) { $sql = $this->_sql_gen(true); @@ -314,14 +360,14 @@ class db_pager { $this->max_page = $this->page_len ? ceil($this->rec_count/$this->page_len) : 0; - if ($go_debug) { // FIX - need column name parsing, but for now: + if ($SysPrefs->go_debug) { // FIX - need column name parsing, but for now: // check if field names are set explicite in col def // for all initially ordered columns foreach ($this->columns as $col) { if (isset($col['ord']) && $col['ord'] != '' && !isset($col['name'])) { display_warning("Result field names must be set - for all intially ordered db_pager columns."); + for all initially ordered db_pager columns."); } } } @@ -411,7 +457,7 @@ class db_pager { update_record_status($id, !$value, $table, $key); $value = !$value; } - echo ''. checkbox(null, $name, $value, true, '', "align='center'") + echo ''. checkbox(null, $name, $value, true, '') . hidden("LInact[$id]", $value, false) . ''; } } else @@ -470,4 +516,3 @@ function refresh_pager($name) if (isset($_SESSION[$name])) $_SESSION[$name]->ready = false; } -?> \ No newline at end of file