-
Notifications
You must be signed in to change notification settings - Fork 32
LPD-74358 putSiteStructuredContentByExternalReferenceCode does not save the updated values #169139
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
To conserve resources, the PR Tester does not automatically run for forwarded pull requests. |
✔️ ci:test:sf - 1 out of 1 jobs passed in 3 minutesRan com.liferay.source.formatter at released version 1.0.1556. Click here for more details.Base Branch:Branch Name: master Sender Branch:Branch Name: LPD-74358 1 Successful Jobs:For more details click here. |
✔️ ci:test:stable - 12 out of 12 jobs passed❌ ci:test:relevant - 16 out of 20 jobs passed in 1 hour 39 minutesClick here for more details.Base Branch:Branch Name: master Upstream Comparison:Branch GIT ID: 3caaae541983f488a2604020ca9a51b57f821b2a ci:test:stable - 12 out of 12 jobs PASSED12 Successful Jobs:ci:test:relevant - 16 out of 20 jobs PASSED4 Failed Jobs:
16 Successful Jobs:For more details click here.Failures unique to this pull:For upstream results, click here. |
|
@ak-ragnor you use the word "complete" and "complex". What's the difference? Please resend on top of ak-ragnor#11 |
Forwarded from: liferay-content-management#7588 (Took 4
ci:forwardattempts in 8 days 2 hours 34 minutes)Console
@ak-ragnor
@liferay-content-management
Original pull request comment:
What is this trying to solve?
https://liferay.atlassian.net/browse/LPD-74358
putSiteStructuredContentByExternalReferenceCode does not work correctly for the following field types:
How am I fixing it?
By using DDMValueUtil to properly extract and handle the field values.
How can you verify that it works?
An existing test already covers this scenario: testPutStructuredContent.