-
Notifications
You must be signed in to change notification settings - Fork 352
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Built-In Functions to be customized #378
Comments
* CustomUriFunctions - public API to add/remove or override builtIn functions. * CustomUriFunctions and BuiltInFunctions tests * Changed class name from 'BuiltInFunctions' to 'BuiltInUriFunctions'. * The new class 'UriFunctionsHelper' contains general usage for both BuiltIn and Custom functions.
- Usage of 'ExceptionUtils' - Typo fixes - lock 'TryGetCustomFunction' method
* CustomUriFunctions - public API to add/remove or override builtIn functions. * CustomUriFunctions and BuiltInFunctions tests * Changed class name from 'BuiltInFunctions' to 'BuiltInUriFunctions'. * The new class 'UriFunctionsHelper' contains general usage for both BuiltIn and Custom functions.
- Usage of 'ExceptionUtils' - Typo fixes - lock 'TryGetCustomFunction' method
* CustomUriFunctions - public API to add/remove or override builtIn functions. * CustomUriFunctions and BuiltInFunctions tests * Changed class name from 'BuiltInFunctions' to 'BuiltInUriFunctions'. * The new class 'UriFunctionsHelper' contains general usage for both BuiltIn and Custom functions. * Changed name of some methods : 'MatchSignatureToBuiltInFunction' to 'MatchSignatureToUriFunction'. 'BindAsBuiltInFunction' to 'BindAsUriFunction'.
* CustomUriFunctions - public API to add/remove or override builtIn functions. * CustomUriFunctions and BuiltInFunctions tests * Changed class name from 'BuiltInFunctions' to 'BuiltInUriFunctions'. * The new class 'UriFunctionsHelper' contains general usage for both BuiltIn and Custom functions. * Changed name of some methods : 'MatchSignatureToBuiltInFunction' to 'MatchSignatureToUriFunction'. 'BindAsBuiltInFunction' to 'BindAsUriFunction'.
* CustomUriFunctions - public API to add/remove or override builtIn functions. * CustomUriFunctions and BuiltInFunctions tests * Changed class name from 'BuiltInFunctions' to 'BuiltInUriFunctions'. * The new class 'UriFunctionsHelper' contains general usage for both BuiltIn and Custom functions. * Changed name of some methods : 'MatchSignatureToBuiltInFunction' to 'MatchSignatureToUriFunction'. 'BindAsBuiltInFunction' to 'BindAsUriFunction'.
@LaylaLiu @YogiBear52 Why aren't custom functions determined based on the EDM model? Having a global table of functions is unfortunate since in some OData services, there are multiple models in the same AppDomain and not all of them have the same set of custom functions. The global table breaks the isolation between EDM models. |
Per comment in #430, this issue has been resolved. Closing issue, but please feel free to continue the discussion here for any questions. |
OData.Core has Built-In Functions like 'contains' 'endswith' 'geo.intersects' and so on. These functions are hard coded and cannot be customized or be modify.
OData lib has to be open and easy to be customized and modified by the users needs.
I suggest to add Custom Functions mechanism so the user could add,modify or override the existing pre defined built-in functions.
The text was updated successfully, but these errors were encountered: