Move open branch state storage in ObjectTree to static scope (refactor) by hehoon · Pull Request #3297 · AliceO2Group/WebUi

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@hehoon

Copy link Copy Markdown

Collaborator

@hehoon hehoon commented

Jan 21, 2026

I DON'T have JIRA ticket

  • explain what this PR does
  • if it is new feature explain how plan to use it
  • test are added
  • documentation is changed or added
  • FLP integration tests were ran successful

Commit changes:

  • Extracted BrowserStorage initialization from the ObjectTree constructor and replaced it with a static, shared instance.

Why was this change made:

  • Reduced per-node (ObjectTree is created for each node in the tree) instantiation overhead.

No functional behaviour change intended.

@hehoon hehoon self-assigned this

Jan 21, 2026

@hehoon hehoon marked this pull request as ready for review

January 21, 2026 11:22

@hehoon hehoon requested a review from graduta as a code owner

January 21, 2026 11:22

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@graduta graduta Awaiting requested review from graduta graduta is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@hehoon hehoon

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@hehoon