-
Notifications
You must be signed in to change notification settings - Fork 9
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
feat(#405): Remove LDname limitation #406
base: develop
Are you sure you want to change the base?
Conversation
@@ -169,8 +170,8 @@ public String getXPath() { | |||
*/ | |||
public void updateLDName() throws ScdException { | |||
String newLdName = parentAdapter.getCurrentElem().getName() + currentElem.getInst(); | |||
if (newLdName.length() > 33) { | |||
throw new ScdException(newLdName + "(IED.name + LDevice.inst) has more than 33 characters"); | |||
if (newLdName.length() > MAXIMUM_LENGHT_LD_NAME_BY_XSD) { |
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.
Knowing that the maxLength is an XSD constraint I think we shouldn't even care about verifying this.
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.
the problem is this occurs during the SCD creation
log.info(STEP_3);
sclEditor.importSTDElementsInSCD(scd, stds);
and the SCD XSD validation occurs far away, in this method :
com.rte_france.rspace.rconf.service.ScdService#createSCD
So maybe the users would like to be warned as soon as the error occurs ?
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.
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.
After discussing together, I will completely remove the check
sct-commons/src/test/java/org/lfenergy/compas/sct/commons/scl/ldevice/LDeviceAdapterTest.java
Outdated
Show resolved
Hide resolved
Signed-off-by: gleizesDor <115622893+gleizesDor@users.noreply.github.com>
61a0d23
to
5e7ce2c
Compare
No description provided.