-
Notifications
You must be signed in to change notification settings - Fork 2
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-149618 Add condition when ddmFormFieldType is fieldset #504
Conversation
ci:test:sf |
Hi, you don’t need to run the CI commands anymore, I’ll do it for you and let you know if any issue happens. |
CI is automatically triggering the following test suites:
|
✔️ ci:test:sf - 1 out of 1 jobs passed in 3 minutesClick here for more details.Base Branch:Branch Name: master Sender Branch:Branch Name: LPS-149618 1 Successful Jobs:For more details click here. |
ci:test:relevant |
✔️ ci:test:sf - 1 out of 1 jobs passed in 3 minutesClick here for more details.Base Branch:Branch Name: master Sender Branch:Branch Name: LPS-149618 1 Successful Jobs:For more details click here. |
ci:test:relevant |
Jenkins Build:test-portal-source-format#1569 Jenkins Report:jenkins-report.html Jenkins Suite:sf Pull Request:liferay-objects#504 Testray Routine:EE Pull Request Testray Importer:publish-testray-report#5660 |
Jenkins Build:test-portal-source-format#1873 Jenkins Report:jenkins-report.html Jenkins Suite:sf Pull Request:liferay-objects#504 Testray Routine:EE Pull Request Testray Importer:publish-testray-report#6889 |
@@ -315,7 +315,9 @@ private long _getObjectDefinitionId( | |||
DDMFormField ddmFormField = ddmFormFieldsMap.get( | |||
ddmFormFieldValue.getName()); | |||
|
|||
if (ddmFormField.isTransient()) { | |||
if (ddmFormField.isTransient() && | |||
!StringUtil.equals(ddmFormField.getType(), "fieldset")) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@danvb112 can you use DDMFormFieldTypeConstants.FIELDSET here?
ci:test:sf |
✔️ ci:test:sf - 1 out of 1 jobs passed in 3 minutesClick here for more details.Base Branch:Branch Name: master Sender Branch:Branch Name: LPS-149618 1 Successful Jobs:For more details click here. |
❌ ci:test:relevant - 0 out of 1 jobs passed in 4 minutesClick here for more details.Base Branch:Branch Name: master Upstream Comparison:Branch GIT ID: 3b7d86f190ce40d1b041500ab66ece2273f291f0 ci:test:relevant - 0 out of 1 jobs PASSED1 Failed Jobs:For more details click here.Failures unique to this pull:
For upstream results, click here. |
Hey @danvb112, can you please check if these failures are related to your changes? Thanks |
❌ ci:test:stable - 24 out of 29 jobs passed❌ ci:test:relevant - 24 out of 30 jobs passed in 6 hours 25 minutesClick here for more details.Base Branch:Branch Name: master Upstream Comparison:Branch GIT ID: 55f13bee30428986f588a75e14319d3697d7fa87 ci:test:stable - 24 out of 29 jobs PASSED5 Failed Jobs:
24 Successful Jobs:
ci:test:relevant - 24 out of 30 jobs PASSED6 Failed Jobs:
24 Successful Jobs:
For more details click here.Failures unique to this pull:
Failures in common with acceptance upstream results at 55f13be:
|
Hey @danvb112, can you please check if these failures are related to your changes? Thanks |
Jenkins Build:test-portal-acceptance-pullrequest(master)#1048 Jenkins Report:jenkins-report.html Jenkins Suite:relevant Pull Request:liferay-objects#504 Testray Routine:EE Pull Request Testray Build:[master] ci:test:relevant - danvb112 > liferay-objects - PR#504 - 2022-03-18[14:52:27] Testray Importer:publish-testray-report#2635 |
ci:test:relevant |
Jenkins Build:test-portal-acceptance-pullrequest(master)#722 Jenkins Report:jenkins-report.html Jenkins Suite:relevant Pull Request:liferay-objects#504 Testray Routine:EE Pull Request Testray Build:[master] ci:test:relevant - danvb112 > liferay-objects - PR#504 - 2022-03-21[06:17:24] Testray Importer:publish-testray-report#1768 |
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#114956 |
Related Issue: https://issues.liferay.com/browse/LPS-149618