Opened 2 years ago
Last modified 2 years ago
#23157 closed defect
Fix jrenderpgsql build errors — at Version 1
| Reported by: | oobayly | Owned by: | malcolmh |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | Plugin seachart | Version: | |
| Keywords: | Cc: |
Description (last modified by )
A previous commit - https://github.com/JOSM/josm-plugins/commit/edd4b35b37f324f49f2a39a15cb1f5a3aff441e7 - changed the signature of S57osm.OSMmap to only accept a File object. This has made it pretty inflexible as we now have to pass a physical file.
The jrenderpgsql package generates OSM XML from a Postgres Query in-memory, and therefore is no longer able use pass the data to the OSMmap method.
I suggest overloading the S57osm.OSMmap method to make it more flexible by allowing it to accept InputStream and Document objects as well.
The reason for the Document overload is that at the moment jrenderpgsql generates hand written XML, which isn't ideal. It would be better that it instead creates an Document directly and pass it to S57osm.OSMmap without the overhead of it having to be parsed. The extra overload prepares the method for that future change which I'd also like to implement.
I've attached a diff of my changes and would welcome any comments.
Change History (2)
by , 2 years ago
| Attachment: | seachart.diff added |
|---|
comment:1 by , 2 years ago
| Description: | modified (diff) |
|---|


