-
Notifications
You must be signed in to change notification settings - Fork 0
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
LPS-147234 Update the ContentImagesUpgradeProcess could identify the fileEntry in the dynamic element data #7306
Conversation
…TTP_PORT and WEB_SERVER_HTTPS_PORT non-final to avoid failure caused by JIT runtime inlining
CI is automatically triggering the following test suites:
The pull request will automatically be forwarded to the user
|
Skipping previously passed test suites: |
❌ ci:test:stable - 23 out of 27 jobs passed❌ ci:test:relevant - 23 out of 28 jobs passed in 4 hours 7 minutesClick here for more details.This pull is eligible for reevaluation. When this upstream build has completed, using the following CI command will compare this pull request result against a more recent upstream result: ci:reevaluate:1297401_1829 Base Branch:Branch Name: master Upstream Comparison:Branch GIT ID: b19ef8e4ee5f4f63722a74e2040ec9abb5ce43ea ci:test:stable - 23 out of 27 jobs PASSED4 Failed Jobs:
23 Successful Jobs:
ci:test:relevant - 23 out of 28 jobs PASSED5 Failed Jobs:
23 Successful Jobs:
For more details click here.Failures unique to this pull:
Failures in common with acceptance upstream results at b19ef8e:
|
Jenkins Build:test-portal-acceptance-pullrequest(master)#1829 Jenkins Report:jenkins-report.html Jenkins Suite:relevant Pull Request:liferay-echo#7306 Testray Routine:EE Pull Request Testray Importer:publish-testray-report#5517 |
ci:test:relevant |
ci:forward |
CI is automatically triggering the following test suites:
The pull request will automatically be forwarded to the user
|
Skipping previously passed test suites: |
All required test suite(s) passed. |
Pull request has been successfully forwarded to brianchandotcom#113794 |
Jenkins Build:test-portal-acceptance-pullrequest(master)#2292 Jenkins Report:jenkins-report.html Jenkins Suite:relevant Pull Request:liferay-echo#7306 Testray Routine:EE Pull Request Testray Importer:publish-testray-report#6951 |
Hi,
This is an specific case only for 7.1GA1. In that version we generate journal articles with the following content for an image field:
And during the upgrade ContentImagesUpgradeProcess we are not able to process it because the fileEntryId is not provided in the dynamic element fields, only in the JSON field.
The final format after the upgrade is:
More info in:
https://github.com/liferay-upgrades/liferay-portal/pull/258#issue-1139449413
Thank you.
cc/ @Binhtran92 @hongvo2308