Menu ▾ ▴

#7 Concat aggregator parsing

v1.0 (example)
closed-fixed
None
7
2014-11-14
2014-09-13
ben.12
No

This example does not work (separator parsed as empty):
Aggregator agg = Aggregator.getAggregator("Concat(property, \",\")")

The constructor:
public ConcatAggregator(String property)
{
this(property, "");
}

Must be:
public ConcatAggregator(String property)
{
setProperty(property);
}

because setProperty(property) initialize also the separator !

Discussion

  • Randy Gettman

    Randy Gettman - 2014-10-13

    The separator wasn't being processed properly from the aggregator specification string. Your analysis is correct. The constructor should call setProperty.

    I'll make your change and include it in the next version of jAgg.

    Thanks,
    Randy Gettman

     
  • Randy Gettman

    Randy Gettman - 2014-11-14

    This is now resolved. It has been released as part of jAgg 0.9.0, on November 13, 2014.

     
  • Randy Gettman

    Randy Gettman - 2014-11-14
    • status: open --> closed-fixed
     

Anonymous
Anonymous

Add attachments
Cancel