Bug #5405
closed
Many pages - sorting missing
Added by Jim Pingle over 8 years ago.
Updated over 8 years ago.
Description
The following pages/widgets had sortable output on 2.2.x but are not sortable on 2.3.x
carp_status.php - List of VIPs/status
diag_dump_states.php - State table output
diag_ipsec_leases.php - IPsec mobile user leases
diag_ipsec.php - IPsec tunnel status
diag_ipsec_spd.php - IPsec SPD Entries
diag_ndp.php - Neighbor Discovery Protocol hosts (similar to ARP table)
diag_routes.php - List of IPv4 and IPv6 routes should be separately sortable
diag_sockets.php - List of IPv4 and IPv6 sockets should be separately sortable
services_dnsmasq.php - Host overrides and Domain overrides should be separately sortable
services_unbound_acls.php - ACL list
services_unbound.php - Host overrides and Domain overrides should be separately sortable
status_gateways.php - Gateway list
status_lb_pool.php - LB Pool entries
status_lb_vs.php - LB Virtual Server entries
status_ntpd.php - NTP status server entries
status_openvpn.php - Status entries inside each type
status_services.php - Service list
status_wireless.php - Connected wireless clients
system_authservers.php - Auth server list
system_groupmanager.php - Group list
system_usermanager.php - User list
vpn_openvpn_client.php - OpenVPN Client list
vpn_openvpn_server.php - OpenVPN Server list
widgets/widgets/captive_portal_status.widget.php - Connected Captive Portal clients
widgets/widgets/openvpn.widget.php - Status entries inside each type
I had already opened a couple other tickets for these individually before noticing there were so many.
diag_arp.php - #5400
status_dhcp_leases.php - #5399
status_dhcpv6_leases.php - #5399
- Status changed from Confirmed to Feedback
- Assignee changed from Anonymous to Jim Pingle
All three tickets completed.
Note that we still need to add IP address sorting (which is now alphanumeric). Making a new ticket for that one.
- Status changed from Feedback to Assigned
- Assignee changed from Jim Pingle to Anonymous
- % Done changed from 0 to 70
Most of them appear to be working OK, but a few issues yet:
diag_routes.php - Column headers appear to be clickable but no sorting happens and no sort order indicators show up
status_lb_vs.php - No sorting at all
status_wireless.php - Nearby AP table is sortable but not "Associated or ad-hoc peers"
system_usermanager.php - Column headers are clickable, sort order arrows show up, but table contents do not sort
widgets/widgets/openvpn.widget.php - Some parts are sortable but not "Peer to Peer Service Instance Statistics" and "Client Instance Statistics"
diag_ipsec_leases.php - Still needs testing, unrelated issue with the page yet in master at the moment
All static tables are working correctly, but tables that are populated dynamically via Javascript are not. Investigating . .
- Status changed from Assigned to Feedback
- Assignee changed from Anonymous to Jim Pingle
Interestingly this sorting library only works correctly if the table is perfectly formatted with all of the proper tags. In some of the pages that did not work there were very old bugs in the way the table was dynamically created which introduce bonus tags. Browsers don't care but the sorting library does.
diag_routes.php no longer sorts.
- % Done changed from 70 to 100
- Status changed from Feedback to Assigned
- Assignee changed from Jim Pingle to Anonymous
- % Done changed from 100 to 90
All fixed but one now:
status_lb_vs.php - Column headers appear to be clickable but no sorting happens
I've got one of my VMs setup to test if you need data, it's the one on 34444
- Status changed from Assigned to Feedback
- Assignee changed from Anonymous to Jim Pingle
status_lb_vs.php was missing thead/tbody tags. Now sorts correctly.
- % Done changed from 90 to 100
- Status changed from Feedback to Resolved
That one works now, too, thanks!
Also available in: Atom
PDF
Updated by Anonymous 
Updated by