Skip to content

dbeaver/pro#9638 [CB] refresh is needed to see replaced connectionfolder in another instance#4464

Open
SychevAndrey wants to merge 4 commits into
develfrom
9638-cb-refresh-is-needed-to-see-replaced-connectionfolder-in-another-instance
Open

dbeaver/pro#9638 [CB] refresh is needed to see replaced connectionfolder in another instance#4464
SychevAndrey wants to merge 4 commits into
develfrom
9638-cb-refresh-is-needed-to-see-replaced-connectionfolder-in-another-instance

Conversation

@SychevAndrey

Copy link
Copy Markdown
Contributor

There are two parts. The first commit is just fixing stale reference and the second one is an improvement that we might want to have or not (need to discuss)

…Ds for active projects

Changed all four spots from reference equality to id comparison, matching the rest of the codebase, see #4430
@SychevAndrey SychevAndrey self-assigned this Jul 15, 2026
@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity

Metric Results
Complexity 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

const folderId = node?.parentId; // current parent

if (folderId && folderId !== parentId && this.navTreeResource.has(parentId)) {
this.navTreeResource.moveToNode(connectionInfo.nodePath, parentId);

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should handle this on the CbDatasourceUpdated level as it affects only this branch of logic. One session case is already handled

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.

2 participants