jbpm icon indicating copy to clipboard operation
jbpm copied to clipboard

Published 20 hours ago • verified publisher icon kiegroup

JBPM-10112: Expand class name of fundamental java classes if not defi…

Open martinweiler opened this issue 2 years ago • 5 comments

…ned as FQCN to address org.xmlpull.v1.XmlPullParserException in VariableScope.validateVariable

Thank you for submitting this pull request

JIRA: JBPM-10112

Due to the changes from JBPM-9924, a variable defined as "Object" (without package name), holding a String value fails to be validated. The proposed fix improves the class resolution in ObjectDataType.verifyDataType by adding the 'java.lang.' package to the className in an additional check.

martinweiler avatar Aug 09 '22 23:08 martinweiler

Jenkins retest this please

elguardian avatar Aug 10 '22 06:08 elguardian

@elguardian please review the updated fix which has been extended to handle validation of Date variables.

martinweiler avatar Aug 10 '22 20:08 martinweiler

@martinweiler failing one test. seems related.

elguardian avatar Aug 11 '22 06:08 elguardian

@elguardian thanks, this should be addressed now

martinweiler avatar Aug 11 '22 16:08 martinweiler

Jenkins retest this please

martinweiler avatar Aug 12 '22 15:08 martinweiler

SonarCloud Quality Gate failed.    Quality Gate failed

Bug D 2 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

70.0% 70.0% Coverage
0.0% 0.0% Duplication

sonarqubecloud[bot] avatar Aug 12 '22 17:08 sonarqubecloud[bot]