Github user afs commented on a diff in the pull request:

    https://github.com/apache/jena/pull/436#discussion_r195400731
  
    --- Diff: 
jena-text/src/main/java/org/apache/jena/query/text/assembler/DefineAnalyzersAssembler.java
 ---
    @@ -84,6 +121,37 @@ public static boolean open(Assembler a, Resource list) {
                             throw new TextIndexException("addAnalyzers 
text:defineAnalyzer property must be a non-blank resource: " + adding);
                         }
                     }
    +                
    +                String langCode = null;
    +                
    +                if (adding.hasProperty(TextVocab.pAddLang)) {
    +                    Statement langStmt = 
adding.getProperty(TextVocab.pAddLang);
    +                    langCode = langStmt.getString();
    +                    Util.addAnalyzer(langCode, analyzer);
    +                    isMultilingualSupport = true;
    +                }
    +                
    +                if (langCode != null && 
adding.hasProperty(TextVocab.pSearchFor)) {
    +                    Statement searchForStmt = 
adding.getProperty(TextVocab.pSearchFor);
    +                    List<String> tags = getStringList(searchForStmt, 
"text:searchFor");
    +                    Util.addSearchForTags(langCode, tags);
    +                }
    +                
    +                if (langCode != null && 
adding.hasProperty(TextVocab.pAuxIndex)) {
    +                    Statement searchForStmt = 
adding.getProperty(TextVocab.pAuxIndex);
    +                    List<String> tags = getStringList(searchForStmt, 
"text:auxIndex");
    +                    Util.addAuxIndexes(langCode, tags);
    +                    log.trace("addAuxIndexes for {} with tags: {}", 
langCode, tags);
    +                }
    +                
    +                
    --- End diff --
    
    Minor:Two blank lines.
    
    It is one-style higher up at L139.


---

Reply via email to