[
https://issues.apache.org/jira/browse/CAMEL-24967?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Claus Ibsen resolved CAMEL-24967.
---------------------------------
Resolution: Fixed
Fixed by https://github.com/apache/camel/pull/26806 (merged as cbe03f3fee0d).
_Claude Code on behalf of davsclaus_
> simple - function argument parsing fixes (quotes, commas, parentheses,
> prefixes)
> --------------------------------------------------------------------------------
>
> Key: CAMEL-24967
> URL: https://issues.apache.org/jira/browse/CAMEL-24967
> Project: Camel
> Issue Type: Sub-task
> Components: camel-core
> Reporter: Claus Ibsen
> Assignee: Claus Ibsen
> Priority: Major
> Fix For: 4.23.0
>
>
> Bugs in how function names are matched and arguments are split
> (functions/*FunctionFactory, SimpleFunctionHelper). A depth-aware splitter
> (tracking quotes, parentheses and ${}) would fix most of these at once.
> * (verified) removeQuotes strips every quote, not just the outer ones:
> {{${size(${body.split(',')})}}} becomes {{split(,)}}. Affects trim,
> capitalize, quote, uppercase, lowercase, length, size, normalizeWhitespace,
> isEmpty etc. Use removeLeadingAndEndingQuotes.
> * (verified) {{${convertTo(${header.foo.trim()},Integer)}}} is cut at the
> first ) (ClassNotFoundException: ${header.foo.trim( ). Same for
> headerAs/variableAs/bodyAs/messageAs.
> * (verified) A comma inside a quoted argument gives
> ArrayIndexOutOfBoundsException: {{${throwException('Order failed, retry
> later')}}}, {{${concat('Hello, ')}}}. hash splits on a raw comma too.
> * (verified) Too few arguments give ArrayIndexOutOfBoundsException instead of
> a syntax error: {{${iif(${header.foo} > 0,'yes')}}}, {{${replace(a)}}}. With
> a nested ${} it escapes at runtime per exchange.
> * (verified) {{${bean:type:java.lang.System.lineSeparator}}} is split at the
> first dot (ClassNotFoundException: java). The documented type: form only
> works with ?method= or ::.
> * (verified) {{${header['a.b']}}} is still treated as OGNL: with header a
> present it calls method b on it. Same for variable and exchangeProperty.
> * (verified) Prefix-glued names are silently accepted: {{${headerfoo}}}
> returns header foo, {{${uuidv7}}} returns a default UUID. Only body and
> exchange got the "must be followed by . ? [" guard. Custom functions named
> e.g. uuidv7 or headerCount can be registered but are unreachable.
> * {{${in.headers.size}}} returns header "size" (only headers.size is
> special-cased); propertiesExist:key:junk and empty(...)junk silently ignore
> the rest.
> * SimpleTest ~867 asserts {{${variableAA('bar', 'int')}}} returns null, which
> pins the prefix bug (looks like a typo test).
> _Claude Code on behalf of davsclaus (found by an AI review of the simple
> language source; each item marked (verified) was reproduced on main at
> c3ef718de2e8)_
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
