Skip to content

Conversation

JonasDov
Copy link
Contributor

@JonasDov JonasDov commented Aug 11, 2025

Prerequisite for #1362
Changed categories tree visibility handler methods.

  1. Renamed all visibility change methods to: change<SomeType>VisibilityStatus.
  2. Renamed all visibility get methods to: get<SomeType>VisibilityStatus.
  3. Used Id64Arg instead of Id64Array | Id64Set.
  4. Hierarchy config no longer changes how visibility is determined. In all cases we check elements and subCategories.
  5. Added getModels, getCategories, getSubModels, getSubCategories, getElementsCount methods which will be needed when models, category and classifications visibility handlers are merged.
  6. Updated get and changed methods, they can now be used in other visibility handlers.*

*All updated methods will be almost the same when models, category and classifications visibility handlers are merged. Only minor changes will be needed from this point.

@JonasDov JonasDov requested review from a team as code owners August 11, 2025 14:08
@JonasDov JonasDov linked an issue Aug 11, 2025 that may be closed by this pull request
@JonasDov JonasDov enabled auto-merge (squash) August 12, 2025 09:19
@JonasDov JonasDov merged commit 9defc89 into tree-widget/next Aug 13, 2025
10 checks passed
@JonasDov JonasDov deleted the JonasD/use-Id64Arg-in-tree-cache branch August 13, 2025 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tree widget: Unify visibility handlers
2 participants