Skip to content
This repository has been archived by the owner on Apr 14, 2023. It is now read-only.

Commit

Permalink
Merge pull request #1270 from finos/1220-duplicate-constraint-error
Browse files Browse the repository at this point in the history
1220 duplicate constraint error
  • Loading branch information
hashbyhayter authored Sep 2, 2019
2 parents 4396398 + b4c685e commit fe43491
Show file tree
Hide file tree
Showing 2 changed files with 129 additions and 153 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -100,12 +100,7 @@ public MergeResult<StringRestrictions> intersect(StringRestrictions other){
* @return Whether this restrictions type is contradictory
*/
private boolean isContradictory() {
if (matchingRegex.isEmpty() && containingRegex.isEmpty()){
return false; //no regular expressions exist that can contradict
}

StringGenerator generator = createGenerator();
return generator instanceof NoStringsStringGenerator;
return createGenerator() instanceof NoStringsStringGenerator;
}

/**
Expand Down
Loading

0 comments on commit fe43491

Please sign in to comment.