Hey all, I was going through some of the code and found a test in StyleNewPageTest that wasn’t being run and had a bug. I’ve made PR #961 to address this. The only thing I am unsure about is that I am calling the correct thing in form.setValue(), but I think it is right.
Cheers,
Travis
Travis Brundage
Software Engineer | Boundless
tbrundage@anonymised.com…
250.888.2820
@boundlessgeo
Looks good, thanks for test-case cleanup.
I will merge, and back port to stable.
On 6 March 2015 at 11:06, Travis Brundage <tbrundage@anonymised.com> wrote:
Hey all, I was going through some of the code and found a test in StyleNewPageTest that wasn’t being run and had a bug. I’ve made PR #961 to address this. The only thing I am unsure about is that I am calling the correct thing in form.setValue(), but I think it is right.
Cheers,
Travis–
Travis Brundage
Software Engineer | Boundless
tbrundage@anonymised.com
250.888.2820
@boundlessgeo
Dive into the World of Parallel Programming The Go Parallel Website, sponsored
by Intel and developed in partnership with Slashdot Media, is your hub for all
things parallel software development, from weekly thought leadership blogs to
news, videos, case studies, tutorials and more. Take a look and join the
conversation now. http://goparallel.sourceforge.net/
Geoserver-devel mailing list
Geoserver-devel@anonymised.comsts.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel
–
Jody Garnett