olingo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bruno Haller (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (OLINGO-1313) Add support for "toupper()" function
Date Mon, 29 Oct 2018 08:37:00 GMT

     [ https://issues.apache.org/jira/browse/OLINGO-1313?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Bruno Haller updated OLINGO-1313:
---------------------------------
    Description: 
When using the "toupper" function in a $filter expression, the JPA processor fails with a ODataNotImplementedException.

The same logic is already present for "tolower" in class ODataExpressionParser where it
says
{code:java}
case TOLOWER:
return String.format("LOWER(%s)", first);
{code}
The attached .patch file contains an implementation for toupper.

 

Reasoning: SAPUI5 allows a "case-insensitive" filter, which uses the toupper() function internally.

  was:
When using the "toupper" function in a $filter expression, the JPA processor fails with a ODataNotImplementedException.

The same logic is already present for "tolower" in class ODataExpressionParser where it
says
{code:java}
case TOLOWER:
return String.format("LOWER(%s)", first);
{code}
Adding the TOUPPER support there should be easy. 

 

Reasoning: SAPUI5 allows a "case-insensitive" filter, which uses the toupper() function internally.


> Add support for "toupper()" function
> ------------------------------------
>
>                 Key: OLINGO-1313
>                 URL: https://issues.apache.org/jira/browse/OLINGO-1313
>             Project: Olingo
>          Issue Type: Wish
>          Components: odata2-jpa
>    Affects Versions: V2 2.0.10
>            Reporter: Bruno Haller
>            Priority: Major
>         Attachments: 0001-OLINGO-1313-Add-support-for-toUpper-function.patch
>
>
> When using the "toupper" function in a $filter expression, the JPA processor fails with
a ODataNotImplementedException.
> The same logic is already present for "tolower" in class ODataExpressionParser where
it says
> {code:java}
> case TOLOWER:
> return String.format("LOWER(%s)", first);
> {code}
> The attached .patch file contains an implementation for toupper.
>  
> Reasoning: SAPUI5 allows a "case-insensitive" filter, which uses the toupper() function
internally.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Mime
View raw message