← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1567047] Re: Panels in local/enabled always appear last in nav

 

Reviewed:  https://review.openstack.org/302417
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=ea92e735829ae4271fcbae932f69ffdbda268546
Submitter: Jenkins
Branch:    master

commit ea92e735829ae4271fcbae932f69ffdbda268546
Author: Tyr Johanson <tyr@xxxxxx>
Date:   Wed Apr 6 13:36:05 2016 -0600

    Allow local/enabled panels to order relative to enabled panels
    
    A dashboard enabled file in local/enabled is not able to appear before,
    or inbetween any core panels.
    
    The list of panels appears to be intended to be sorted by file name,
    but all files in /enabled and always presented in the nav ahead of
    any files from local/enabled, no matter the file name.
    
    This appears to be a bug in util/settings.py that does an rsplit to
    separate file name from path, but accidentally uses the full list of
    split items, instead of just the file name.
    
    For example, a file with __name__ of
    'openstack_dashboard.enabled._1040_project_volumes_panel' splits into
    ['openstack_dashboard.enabled', '_1040_project_volumes_panel']. When
    this list is fed to cmp(), it will always come before a panel in
    local/enabled such as
    ['openstack_dashboard.local.enabled', '_0001_my_new_panel']
    
    Change-Id: Ic169ccf0db1e04ec42fe999df6648117ce9efe84
    Closes-Bug: 1567047


** Changed in: horizon
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1567047

Title:
  Panels in local/enabled always appear last in nav

Status in OpenStack Dashboard (Horizon):
  Fix Released

Bug description:
  The list of panels appears to be intended to be sorted by file name,
  but all files in /enabled and always presented in the nav ahead of any
  files from local/enabled, no matter the file name.

  This appears to be a bug in util/settings.py that does an rsplit to
  separate file name from path, but accidentally uses the full list of
  split items, instead of just the file name.

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1567047/+subscriptions


References