Hi!

In wicket 1.4.17 it seems like there is deouble invocation of
substitutePropertyExpressions, is this intentional?

    public String getString(final String key, final Component component,
final IModel<?> model,
        final String defaultValue) throws MissingResourceException
    {
       ...
        String value = getStringIgnoreSettings(key, component, model, null);
---> calls substitutePropertyExpressions(component, value, model);
       ...
          if (value != null)
        {
            return substitutePropertyExpressions(component, value, model);
        }
       ...
    {

and inside getStringIgnoreSettings:

    public String getStringIgnoreSettings(final String key, final Component
component,
        final IModel<?> model, final String defaultValue)
    {
       ...
          if (value != null)
        {
            return substitutePropertyExpressions(component, value, model);
        }
       ...
    {


Seems redundant and might have some performance considerations?

**
Martin

Reply via email to