[uima-uimaj] 01/01: [UIMA-6473] CasToComparableText is broken

[rec]

This is an automated email from the ASF dual-hosted git repository.

rec pushed a commit to branch bugfix/UIMA-6473-CasToComparableText-is-broken
in repository https://gitbox.apache.org/repos/asf/uima-uimaj.git

commit d88e91e1db6702c7ab52486ff2743e5c0e32d727
Author: Richard Eckart de Castilho <r...@apache.org>
AuthorDate: Wed Jun 22 09:24:07 2022 +0200

    [UIMA-6473] CasToComparableText is broken
    
    - Add missing else
    - Fix use of wrong variable in if statement
---
 .../src/test/java/org/apache/uima/cas/serdes/CasToComparableText.java | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/uimaj-core/src/test/java/org/apache/uima/cas/serdes/CasToComparableText.java 
b/uimaj-core/src/test/java/org/apache/uima/cas/serdes/CasToComparableText.java
index 3a79a98d4..6d8a0475f 100644
--- 
a/uimaj-core/src/test/java/org/apache/uima/cas/serdes/CasToComparableText.java
+++ 
b/uimaj-core/src/test/java/org/apache/uima/cas/serdes/CasToComparableText.java
@@ -717,7 +717,7 @@ public class CasToComparableText {
           values.stream().filter(v -> v instanceof FeatureStructure).filter(v 
-> !seen.contains(v))
                   .forEach(v -> toProcess.add((FeatureStructure) v));
         }
-
+      } else {
         for (Feature feature : fs.getType().getFeatures()) {
           if (feature.getRange().isPrimitive()) {
             continue;
@@ -736,7 +736,7 @@ public class CasToComparableText {
             List<Object> featureValues = multiValuedFeatureStructureToList(
                     fs.getFeatureValue(feature));
 
-            if (values != null) {
+            if (featureValues != null) {
               for (Object value : featureValues) {
                 if (value instanceof FeatureStructure && 
!seen.contains(value)) {
                   toProcess.add((FeatureStructure) value);