Skip to content

Add root project folder to each key in projectTreeItemMap#1617

Merged
jdneo merged 1 commit intomicrosoft:developfrom
jjohannes:1614-project-hierarchy-fix
Oct 16, 2024
Merged

Add root project folder to each key in projectTreeItemMap#1617
jdneo merged 1 commit intomicrosoft:developfrom
jjohannes:1614-project-hierarchy-fix

Conversation

@jjohannes
Copy link
Contributor

@jjohannes jjohannes commented Oct 14, 2024

In #1612, the keys of this map were changed to the Gradle project path. This path is unique for each project within one Gradle build, but not if there are multiple Gradle builds loaded in one workspace by using: 'gradle.nestedProjects': true

This change combined the old and new behavior by creating the Map key from two parts:

  1. The root project folder which uniquely identifies the build (accessible via definition.projectFolder)
  2. The Gradle project path that uniquely identifies a project inside the build

Fixes #1614 by adjusting a change made in #1612

In #1612, the keys of this map were changed to the Gradle project path.
This path is unique for each project within one Gradle build, but not
if there are multiple Gradle builds loaded in one workspace by using:
'gradle.nestedProjects': true

This change combined the old and new behavior by creating the Map key
from two parts:
(1) The root project folder which uniquely identifies the build
    (accessible via definition.projectFolder)
(2) The Gradle project path that uniquely identifies a project inside
    the build

Signed-off-by: Jendrik Johannes <jendrik.johannes@gmail.com>
@jdneo jdneo added this to the 3.16.5 milestone Oct 16, 2024
@jdneo jdneo added the bug Something isn't working label Oct 16, 2024
Copy link
Member

@jdneo jdneo left a comment

Choose a reason for hiding this comment

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

LGTM

@jdneo jdneo merged commit 9542e91 into microsoft:develop Oct 16, 2024
@jdneo
Copy link
Member

jdneo commented Oct 16, 2024

Thank you @jjohannes !

@jjohannes jjohannes deleted the 1614-project-hierarchy-fix branch October 16, 2024 07:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

sub projects belong to wrong folder in explorer

2 participants