adobe / adobe/aem-core-wcm-components

Dialog link V1

Open
#2,265 8 comments 2 reactions 0 assignees View on GitHub
enhancement
Dominant language
Java
Stars
795
Forks
775
Avg merge
3d 22h
Merged PRs (30d)
8

Description

## Bug Report

**Current Behavior**
Every Component that is using the Link Dialog ("/apps/core/wcm/components/commons/editor/dialog/link/v1/link/edit/.content.xml") like Button V2 or Image V3 has the Problem that you can't choose Files from Assets in the link pathfield. It was changed from Pathfield ("granite/ui/components/coral/foundation/form/pathfield") to Pagefield ("cq/gui/components/coral/common/form/pagefield"), and the exceptions for the files in property "nodetypes" are not working with the "Pagefield"

**Expected behavior/code**
In URL field, selection of Assets should be possible. As there are already definition in Property "nodetypes" to allow Assets.

**Environment**
- AEM 6.5.13
- Core Components 2.19.0
- JRE 11

**Possible Solution**
Just go back to "Pathfield", that the Property for "nodetypes" will work. Or make the notetypes Property work with "Pagefield"

**Additional context / Screenshots**
Add any other context about the problem here. If applicable, add screenshots to help explain.

![image](https://user-images.githubusercontent.com/70140719/182636764-0f692563-f192-41f2-8dba-1924a4cadf5b.png)

https://github.com/adobe/aem-core-wcm-components/blob/main/content/src/content/jcr_root/apps/core/wcm/components/commons/editor/dialog/link/v1/link/edit/.content.xml

Contributor guide

Open the contributing guide

Research direction

Start with /apps/core/wcm/components/commons/editor/dialog/link/v1/link/edit/.content.xml and inspect the current Pagefield configuration, including its nodetypes property. Compare the Pathfield and Pagefield behavior in AEM 6.5.13, then verify that the URL field allows selecting Assets while preserving the intended link selection behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
frontend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.