Implement validator support for function objects

fixes #8291
This commit is contained in:
Gunnar Beutner 2015-01-27 10:55:14 +01:00
parent a11cdbdc39
commit 13917e07b0
5 changed files with 93 additions and 86 deletions

View File

@ -174,6 +174,7 @@ do { \
%string { yylval->type = TypeString; return T_TYPE_STRING; }
%scalar { yylval->type = TypeScalar; return T_TYPE_SCALAR; }
%any { yylval->type = TypeAny; return T_TYPE_ANY; }
%function { yylval->type = TypeFunction; return T_TYPE_FUNCTION; }
%name { yylval->type = TypeName; return T_TYPE_NAME; }
%validator { return T_VALIDATOR; }
%require { return T_REQUIRE; }

View File

@ -152,6 +152,7 @@ static void MakeRBinaryOp(Expression** result, Expression *left, Expression *rig
%token <type> T_TYPE_STRING "string (T_TYPE_STRING)"
%token <type> T_TYPE_SCALAR "scalar (T_TYPE_SCALAR)"
%token <type> T_TYPE_ANY "any (T_TYPE_ANY)"
%token <type> T_TYPE_FUNCTION "function (T_TYPE_FUNCTION)"
%token <type> T_TYPE_NAME "name (T_TYPE_NAME)"
%token T_VALIDATOR "%validator (T_VALIDATOR)"
%token T_REQUIRE "%require (T_REQUIRE)"
@ -435,6 +436,7 @@ type: T_TYPE_DICTIONARY
| T_TYPE_STRING
| T_TYPE_SCALAR
| T_TYPE_ANY
| T_TYPE_FUNCTION
| T_TYPE_NAME
;

View File

@ -73,6 +73,9 @@ bool TypeRule::MatchValue(const Value& value, String *hint, const TypeRuleUtilit
case TypeArray:
return value.IsObjectType<Array>();
case TypeFunction:
return value.IsObjectType<Function>();
case TypeName:
if (!value.IsScalar())
return false;

View File

@ -51,6 +51,7 @@ enum TypeSpecifier
TypeString,
TypeDictionary,
TypeArray,
TypeFunction,
TypeName
};

View File

@ -174,7 +174,7 @@
%attribute %string "display_name",
%require "update",
%attribute %any "update",
%attribute %function "update",
/* %if (methods.update == "LegacyTimePeriod") { */
// %require "ranges",
@ -188,7 +188,7 @@
%validator "ValidateCommandAttributes",
%require "execute",
%attribute %any "execute",
%attribute %function "execute",
/* %if (methods.execute == "PluginNotification" || methods.execute == "PluginCheck" || methods.execute == "PluginEvent") { */
// %require "command",