fix: Populate HTML Page URL from identifier when field has defaultValue #34309
+94
−0
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Proposed Changes
defaultValueduring workflow actions (e.g., Publish)ESContentletAPIImpl.find()to populate the URL fromidentifier.asset_namewhen the URL field has adefaultValuedefined in the Content Typetest_find_HTMLPage_URL_ShouldBePopulatedFromIdentifierto verify the fixChecklist
Additional Info
Issue: Fixes #33893
Root Cause: For HTML Pages, the URL is stored in
identifier.asset_name, not in thecontentlettable (text3column is empty). When loading an HTML Page contentlet viafind(), if the URL field in the Content Type has adefaultValuedefined (e.g., "test"), that default value was being applied instead of the actual URL from the identifier. This caused the Publish button on the Page Edit screen to overwrite the user-typed URL with the default value.Fix Location:
ESContentletAPIImpl.find()(lines 441-454)Behavior:
defaultValuedefined in the Content Typeidentifier.asset_nameand sets it on the contentletScreenshots
Screen.Recording.2026-01-15.at.11.28.15.AM.mov