Uploaded image for project: 'Apache Guacamole - Contributions'
  1. Apache Guacamole - Contributions
  2. GUAC-1670

Filter affects only the first level of the connection permission editor tree

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Done
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: guacamole
    • Labels:
    • Sprint:
      DEV 2019-03-09

      Description

      When a tree of connections and connection groups within the Guacamole interface is filtered, the filter is supposed to flatten the hierarchy of the tree such that all matching items are visible at the root level. This functionality no longer works for the connection tree within the connection permission editor (part of the user and group edit screens): the hierarchy is not flattened, and only the items which are already at the root level are filtered.

      This is a regression due to the changes which introduced a new, tabbed interface for connection permissions ("Current Connections" and "All Connections"):

      1. The guacGroupListFilter directive had to be modified to filter both GroupListItem (the internal representation of a connection tree node) and ConnectionGroup (the raw REST API representation of a connection group).
      2. Copypasta within these changes continues to call the flatten function specific to ConnectionGroup when the function specific to GroupListItem should be used.
      3. This silently breaks flattening when GroupListItem is involved, effectively restricting filtering to only those items which are already at the root level.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              mike.jumper Michael Jumper
              Reporter:
              mike.jumper Michael Jumper
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: