Merge remote-tracking branch 'origin/master'
This commit is contained in:
commit
37d739f156
@ -46,6 +46,7 @@ phpMyAdmin - ChangeLog
|
||||
- bug #3375325 [interface] Page list in navigation frame looks odd
|
||||
- bug #3313235 [interface] Error div misplaced
|
||||
- bug #3374802 [interface] Comment on a column breaks inline editing
|
||||
- bug #3383711 [display] Order by a column in a view doesn't work in some cases
|
||||
|
||||
3.4.4.0 (not yet released)
|
||||
- bug #3323060 [parser] SQL parser breaks AJAX requests if query has unclosed quotes
|
||||
|
||||
@ -842,8 +842,10 @@ function PMA_displayTableHeaders(&$is_display, &$fields_meta, $fields_cnt = 0, $
|
||||
// the orgname member does not exist for all MySQL versions
|
||||
// but if found, it's the one on which to sort
|
||||
$name_to_use_in_sort = $fields_meta[$i]->name;
|
||||
$is_orgname = false;
|
||||
if (isset($fields_meta[$i]->orgname) && strlen($fields_meta[$i]->orgname)) {
|
||||
$name_to_use_in_sort = $fields_meta[$i]->orgname;
|
||||
$is_orgname = true;
|
||||
}
|
||||
// $name_to_use_in_sort might contain a space due to
|
||||
// formatting of function expressions like "COUNT(name )"
|
||||
@ -870,12 +872,15 @@ function PMA_displayTableHeaders(&$is_display, &$fields_meta, $fields_cnt = 0, $
|
||||
// 2.1.3 Check the field name for a bracket.
|
||||
// If it contains one, it's probably a function column
|
||||
// like 'COUNT(`field`)'
|
||||
if (strpos($name_to_use_in_sort, '(') !== false) {
|
||||
// It still might be a column name of a view. See bug #3383711
|
||||
// Check is_orgname.
|
||||
if (strpos($name_to_use_in_sort, '(') !== false && ! $is_orgname) {
|
||||
$sort_order = ' ORDER BY ' . $name_to_use_in_sort . ' ';
|
||||
} else {
|
||||
$sort_order = ' ORDER BY ' . $sort_tbl . PMA_backquote($name_to_use_in_sort) . ' ';
|
||||
}
|
||||
unset($name_to_use_in_sort);
|
||||
unset($is_orgname);
|
||||
|
||||
// 2.1.4 Do define the sorting URL
|
||||
if (! $is_in_sort) {
|
||||
|
||||
Loading…
Reference in New Issue
Block a user