Skip to content

ESB-1299 Resource attribute REST mapping (batch 1/4) - #382

Open
MEM2677 wants to merge 7 commits into
developfrom
ESB-1299
Open

MEM2677 wants to merge 7 commits into
developfrom
ESB-1299

Conversation

@MEM2677

@MEM2677 MEM2677 commented Oct 21, 2026

Copy link
Copy Markdown
Member

• the content REST layer stores and reads resource metadata with key and language swapped, so the Admin Console and templates do not see metadata written through REST, REST GET does not see Admin Console metadata, and every REST save deletes the alt and title of Image attributes.
• a null or non-string metadata value returns 500 instead of a validation error, and a null value stored by a future fix would fail later when the content XML is generated.
• a resource value sent without name stores the text representation of the map ({id=…}) as the attribute text, which is published as alt text or caption.
• a null language value in a resource attribute throws a NullPointerException and returns 500.
• REST accepts a resource only in a non-default language, a content the Admin Console cannot produce, and the default language then renders no image.
• an Image attribute accepts a resource of another type (for example a PDF), which breaks image renditions on the portal.

@github-actions

Copy link
Copy Markdown

PUBLICATION for COMMIT ID ``

IMG/TAG: entando/entando-de-app-tomcat:7.5.4-ESB-1299-PR382
IMG/SHA: entando/entando-de-app-tomcat@sha256:c0bb91c404a8e6862c0a1e38600e2714ba9bfe3d799d3726e9fa0005861952f3

…ributes sharing a resource keep their own metadata.
@sonarqubecloud

Copy link
Copy Markdown

@github-actions

Copy link
Copy Markdown

PUBLICATION for COMMIT ID ``

IMG/TAG: entando/entando-de-app-tomcat:7.5.4-ESB-1299-PR382
IMG/SHA: entando/entando-de-app-tomcat@sha256:8364de4f6194c786ce7d0eedd65db2dcad53e5ebcbe520327d615567bd66064b

@MEM2677
MEM2677 marked this pull request as ready for review October 23, 2026 15:50
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