Select throws NullPointerException

[Paulo Ricardo Ribeiro]

Hello,

I'm using tapestry 5.3.1.

Using a select with the zone parameter, if the session timeout has been
reached and the user changed the selected value, I was getting a
NullPointerException, so, I've downloaded the source, and looked at the
Select.java (tapestry-core) and noticed that the "this.encoder" on the
following method, was null:
  protected Object toValue(String submittedValue)
    {
        return InternalUtils.isBlank(submittedValue) ? null :
this.encoder.toValue(submittedValue);
    }


so, I've changed the source code to:

 protected Object toValue(String submittedValue)
    {
        return (this.encoder == null ||
InternalUtils.isBlank(submittedValue)) ? null :
this.encoder.toValue(submittedValue);
    }

and it worked.


I'm relatively new to Tapestry, so I'd like to ask if this is a feature
(and I'm doing something wrong), or is it a bug?

If this is a bug, should I create an issue here (
https://issues.apache.org/jira/browse/TAP5) and also upload the patch?


Cheers
Paulo Ricardo Ribeiro