Org unit selective hiding and sorting in tpac library selector

Bug #954310 reported by Bill Erickson
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Undecided
Unassigned

Bug Description

References :

http://masslnc.cwmars.org/node/2516
http://www.esilibrary.com/esi/docs/?p=841

This bug addresses two tightly linked features. Feature one supports non-inherited org unit hiding. The goal is to allow hiding nodes in the org unit tree without hiding the child nodes. Among other things, this allows systems to insert org units into the org unit tree for grouping/routing/policy/etc. without exposing that information to the OPAC. This is managed by a new config.global_flag "opac.org_unit.non_inheritied_visibility" which, when enabled, will cause all opac_visible org units to display in the library selector even if their parent org units are hidden.

Feature two aims to support org unit sibling sort order within the tpac library selector. This allows staff to define the display order of each branch as compared to its sibling branches. For example, you could configure a stock Evergreen instance to look like:

CONS
 - SYS1
  -- BR2
  -- BR1
...

Depending on permissions, any level of the tree (that has a parent) can be manually sorted. Sorting is managed via a new admin interface (Admin -> Local Admin -> Library Sort Order).

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/berick/org-unit-opac-visibility-and-sorting

Tags: pullrequest
Bill Erickson (berick)
description: updated
Revision history for this message
Bill Erickson (berick) wrote :

Removing pullrequest tag. Revisiting the implementation to consider more flexible sorting options.

tags: removed: pullrequest
Revision history for this message
Bill Erickson (berick) wrote :

New plan: avoid altering actor.org_uint directly. Create a new table that allows us to build newly shaped trees from the full set of org units, which do not enforce existing sibling relationships. In other words, it will be possible to hand-craft an entirely new org unit tree for use within the tpac for display purposes only.

Revision history for this message
Bill Erickson (berick) wrote :

Updated branch pushed to:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/collab/berick/ou-visibility-and-sorting

Feature one described above has not changed. Feature two now supports free-form org unit hierarchies for display instead of being limited to intra-parent (sibling) sorting. Instead of adding a column to actor.org_unit, the new implementation creates 2 new tables, a tree table and a tree node table, which implements a generic org unit re-tree-ification structure.

tags: added: pullrequest
Revision history for this message
Bill Erickson (berick) wrote :

Rebased branch force-pushed to collab/berick/ou-visibility-and-sorting

Revision history for this message
Bill Erickson (berick) wrote :

Pushed a small UI tweak to the admin UI:

Previously, activating a new tree would work, but the UI did not update
to reflect it. Now the activate link correctly turns into a de-activate
link and vice versa.

Revision history for this message
Mike Rylander (mrylander) wrote :

Merged to master for beta pounding!

Changed in evergreen:
status: New → Fix Committed
Revision history for this message
Mike Rylander (mrylander) wrote :

Remerged ... I merged the original branch the first time around, which Bill then reverted so I could merge the correct (collab) branch. Sorry for the git churn, folks.

Ben Shum (bshum)
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.