[GitHub] jena pull request #500: JENA-1642: Check has value for header instead of is ...

2018-11-27 Thread jsarjeant
GitHub user jsarjeant opened a pull request:

https://github.com/apache/jena/pull/500

JENA-1642: Check has value for header instead of is true

When using schemagen from the maven plugin the `header` option does not 
work. `header` is intended to be a template string and when `isTrue` is checked 
the string is cast to a boolean and the `BadBooleanException` is thrown. I 
believe we should check for existence here instead of "truthiness"


Also, it appears that the maven tools are no being built as part of the 
main project. Is that the case?

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/jsarjeant/jena master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/jena/pull/500.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #500


commit 471d1f8511def353f1b64776c2a986d1e6d8ab45
Author: Joseph Sarjeant 
Date:   2018-11-27T18:08:43Z

Check has value for header instead of is true




---


[GitHub] jena pull request #500: JENA-1642: Check has value for header instead of is ...

2018-11-29 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/jena/pull/500


---