Revert "Merge pull request #8370 from vespa-engine/revert-8308-bratseth/disallow-dash-rebased"

This reverts commit 43cf4fd1ec196cb0543af73db8e3bd1f6774f0a8, reversing
changes made to 14172aa5cd890445980202909d1277429e4c5a3a.

1 files changed, 32 insertions, 22 deletions

diff --git a/config-model/src/main/javacc/SDParser.jj b/config-model/src/main/javacc/SDParser.jj
index 883dfc7ac72..f2eb45205df 100644
--- a/config-model/src/main/javacc/SDParser.jj
+++ b/config-model/src/main/javacc/SDParser.jj
@@ -357,7 +357,8 @@ TOKEN :
 | < CONSTANTS: "constants" >
 | < FILE: "file" >
 | < URI: "uri" >
-| < IDENTIFIER: ["a"-"z","A"-"Z", "_"] (["a"-"z","A"-"Z","0"-"9","_","-"])* >
+| < IDENTIFIER:           ["a"-"z","A"-"Z", "_"] (["a"-"z","A"-"Z","0"-"9","_"])* >
+| < IDENTIFIER_WITH_DASH: ["a"-"z","A"-"Z", "_"] (["a"-"z","A"-"Z","0"-"9","_","-"])* >
 | < QUOTEDSTRING: "\"" ( ~["\""] )* "\"" >
 | < CONTEXT: ["a"-"z","A"-"Z"] (["a"-"z", "A"-"Z", "0"-"9"])* >
 | < DOUBLE: ("-")? (["0"-"9"])+ "." (["0"-"9"])+ >
@@ -415,10 +416,7 @@ Search rootSearch(String dir) :
     Search search;
 }
 {
-    ( <SEARCH> name = identifier() { if (!name.matches("[a-zA-Z_][a-zA-Z_:0-9]*")) {
-                                         deployLogger.log(Level.WARNING, name + " can not be used in YQL+ expressions.");
-                                     }
-                                     search = new Search(name, app);
+    ( <SEARCH> name = identifier() { search = new Search(name, app);
                                      rankProfileRegistry.add(new DefaultRankProfile(search, rankProfileRegistry));
                                      rankProfileRegistry.add(new UnrankedRankProfile(search, rankProfileRegistry));}
       lbrace() (rootSearchItem(search) (<NL>)*)* <RBRACE> (<NL>)* <EOF>)
@@ -669,14 +667,15 @@ void fieldSetItem(String setName, Search search) :
   ( <FIELDS><COLON> field=identifier() { search.fieldSets().addUserFieldSetItem(setName, field); }
     ( <COMMA> field=identifier() { search.fieldSets().addUserFieldSetItem(setName, field); } )* )
   |
-    ( <QUERYCOMMAND> <COLON> (queryCommand = identifier() | queryCommand = quotedString())) { search.fieldSets().userFieldSets().get(setName).queryCommands().add(queryCommand);}
+    ( <QUERYCOMMAND> <COLON> (queryCommand = identifierWithDash() | queryCommand = quotedString())) { search.fieldSets().userFieldSets().get(setName).queryCommands().add(queryCommand);}
   |
     ( match(matchSettings) ) { matchSettings.applyOperations(); search.fieldSets().userFieldSets().get(setName).setMatching(matchSettings.getMatching());}
 }
 
 /**
  * This rule consumes a annotation block from within either a document element or a search element.
- * @param search The search object to add content to.
+
+ * @param search the search object to add content to.
  */
 void annotationOutside(Search search) :
 {
@@ -698,6 +697,7 @@ void annotationOutside(Search search) :
 
 /**
  * This rule consumes a annotation block from within either a document element.
+ *
  * @param document The document object to add content to.
  */
 void annotation(Search search, SDDocumentType document) :
@@ -1183,7 +1183,7 @@ Object sortingSetting(SortingOperation sorting, String attributeName) :
                              | <QUATERNARY> { sorting.setStrength(Sorting.Strength.QUATERNARY); }
                              | <IDENTICAL>  { sorting.setStrength(Sorting.Strength.IDENTICAL); }
                            )
-      | <LOCALE> <COLON> locale = identifier() { sorting.setLocale(locale); }
+      | <LOCALE> <COLON> locale = identifierWithDash() { sorting.setLocale(locale); }
     )
     { return null; }
 }
@@ -1426,7 +1426,7 @@ void summaryProperty(SummaryInFieldLongOperation field) :
     String name, value;
 }
 {
-    name = identifier() <COLON> (value = identifier() | value = quotedString())
+    name = identifierWithDash() <COLON> (value = identifierWithDash() | value = quotedString())
     { field.addProperty(new SummaryField.Property(name, value)); }
 }
 
@@ -1441,7 +1441,7 @@ void fieldStemming(FieldOperationContainer field) :
     StemmingOperation op = new StemmingOperation();
 }
 {
-    <STEMMING> <COLON> setting = identifier()
+    <STEMMING> <COLON> setting = identifierWithDash()
     {
         op.setSetting(setting);
         field.addOperation(op);
@@ -1458,7 +1458,7 @@ void searchStemming(Search search) :
     String setting;
 }
 {
-    <STEMMING> <COLON> setting = identifier()
+    <STEMMING> <COLON> setting = identifierWithDash()
     { search.setStemming(Stemming.get(setting)); }
 }
 
@@ -1473,7 +1473,7 @@ void normalizing(FieldOperationContainer field) :
     String setting;
 }
 {
-    <NORMALIZING> <COLON> setting = identifier()
+    <NORMALIZING> <COLON> setting = identifierWithDash()
     {
         field.addOperation(new NormalizingOperation(setting));
     }
@@ -1527,7 +1527,7 @@ void queryCommand(FieldOperationContainer container) :
     QueryCommandOperation field = new QueryCommandOperation();
 }
 {
-    <QUERYCOMMAND> <COLON> command = identifier()
+    <QUERYCOMMAND> <COLON> command = identifierWithDash()
     {
         field.addQueryCommand(command);
         container.addOperation(field);
@@ -1540,7 +1540,7 @@ void alias(FieldOperationContainer container) :
     String alias;
 }
 {
-    <ALIAS> [aliasedName = identifier()] <COLON> alias = identifier()
+    <ALIAS> [aliasedName = identifier()] <COLON> alias = identifierWithDash()
     {
        AliasOperation op = new AliasOperation(aliasedName, alias);
        container.addOperation(op);
@@ -1773,9 +1773,9 @@ Object indexBody(IndexOperation index) :
     double threshold;
 }
 {
-    ( <PREFIX>                                   { index.setPrefix(true); }
-      | <ALIAS> <COLON> str = identifier()       { index.addAlias(str); }
-      | <STEMMING> <COLON> str = identifier()    { index.setStemming(str); }
+    ( <PREFIX>                                           { index.setPrefix(true); }
+      | <ALIAS> <COLON> str = identifierWithDash()       { index.addAlias(str); }
+      | <STEMMING> <COLON> str = identifierWithDash()    { index.setStemming(str); }
       | <ARITY> <COLON> arity = integer()                              { index.setArity(arity); }
       | <LOWERBOUND> <COLON> num = consumeLong()                       { index.setLowerBound(num); }
       | <UPPERBOUND> <COLON> num = consumeLong()                       { index.setUpperBound(num); }
@@ -1855,7 +1855,7 @@ void rankProfile(Search search) :
     RankProfile profile;
 }
 {
-    ( <RANKPROFILE> name = identifier()
+    ( <RANKPROFILE> name = identifierWithDash()
         {
             if (documentsOnly) {
                 profile = new DocumentsOnlyRankProfile(name, search, rankProfileRegistry);
@@ -1912,7 +1912,7 @@ void inheritsRankProfile(RankProfile profile) :
     String str;
 }
 {
-    <INHERITS> str = identifier()
+    <INHERITS> str = identifierWithDash()
     { profile.setInherited(str); }
 }
 
@@ -2211,9 +2211,9 @@ String rankPropertyItem() :
     String image, ret = "";
 }
 {
-    ( ( image = identifier()              { ret += image; }
-        | image = quotedString()          { ret += image; }
-        | ( "(" | ")" | <DOT> | <COMMA> ) { ret += token.image; } )+ )
+    ( ( image = identifierWithDash()              { ret += image; }
+        | image = quotedString()                  { ret += image; }
+        | ( "(" | ")" | <DOT> | <COMMA> )         { ret += token.image; } )+ )
     { return ret; }
 }
 
@@ -2454,6 +2454,16 @@ String expression() :
     { return exp; }
 }
 
+String identifierWithDash() :
+{
+    String identifier;
+}
+{
+    ( identifier = identifier() { return identifier; } )
+    |
+    ( <IDENTIFIER_WITH_DASH> { return token.image; } )
+}
+
 /**
  * This rule consumes an identifier. This must be kept in sync with all word tokens that should be parseable as
  * identifiers.