OCSInventory-NG / OCSInventory-ocsreports

Webconsole for OCS Inventory NG
https://www.ocsinventory-ng.org
GNU General Public License v2.0
231 stars 151 forks source link

Unsorted show/hide columns in All computers #171

Closed pchamo closed 7 years ago

pchamo commented 7 years ago

The "All computers, Show/hide, Select columns to show/hide" drop-down menu list is unsorted :( I think this is due to new ajax code in ms_all_computers.php, but I can't realize how to fix that. It seems, the line to fix is the one that contains ajaxtab_entete_fixe:

if (is_defined($_SESSION['OCS']["mesmachines"])) { $queryDetails .= "AND " . $_SESSION['OCS']["mesmachines"]; } $queryDetails .= " group by h.id"; $tab_options['LBL_POPUP']['SUP'] = 'name'; $tab_options['LBL']['SUP'] = $l->g(122); $tab_options['TRI']['DATE']['e.bdate'] = "%m/%d/%Y"; // BIOS date format in database (varchar)

$entete = ajaxtab_entete_fixe($list_fields, $default_fields, $tab_options, $list_col_cant_del);

Previous non-ajax ocs reports versions, included this code:

if (isset($_SESSION['OCS']["mesmachines"]) and $_SESSION['OCS']["mesmachines"] != '') $queryDetails .= "AND ".$_SESSION['OCS']["mesmachines"]; $queryDetails .=" group by h.id"; $tab_options['LBL_POPUP']['SUP']='name'; $tab_options['LBL']['SUP']=$l->g(122); $tab_options['TRI']['DATE']['e.bdate']="%m/%d/%Y";

$result_exist=tab_req($table_name,$list_fields,$default_fields,$list_col_cant_del,$queryDetails,$form_name,95,$tab_options);

Here the same question plus an image about this issue: http://ask.ocsinventory-ng.org/5568/unsorted-show-hide-columns-list-in-ajax-ocs-reports-versions

pchamo commented 7 years ago

dropdown

vdeville commented 7 years ago

Hello, Thanks for your report Will be fixed soon

gillesdubois commented 7 years ago

Hi, A fix will be available soon, see the screen below :)

showhide

Thanks for your report. Regards, Gilles Dubois.