DataNucleus JIRA is now in read-only mode. Raise any new issues in GitHub against the plugin that it applies to. DataNucleus JIRA will remain for the foreseeable future but will eventually be discontinued
Issue Details (XML | Word | Printable)

Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Unassigned
Reporter: Andy Jefferson
Votes: 0
Watchers: 0

If you were logged in you would be able to see more operations.
DataNucleus Types : Geospatial

SpatialHelper is badly designed and user should pass in PM(/EM)

Created: 26/Dec/12 03:31 PM   Updated: 22/Nov/13 11:14 AM   Resolved: 22/Nov/13 11:14 AM
Component/s: None
Affects Version/s: None
Fix Version/s: 3.2.4

 Description  « Hide
The current SpatialHelper (and derivatives) takes in a PMF. On each method call it creates a PM but doesn't close it afterwards. Would be much better to pass in the PM/EM (or maybe better the ExecutionContext so it doesn't impose deps?).

Andy Jefferson added a comment - 22/Nov/13 11:14 AM
SVN trunk removes all methods that create a new PM without closing it. If someone was using them they should swap to the method passing in the PM

Andy Jefferson made changes - 22/Nov/13 11:14 AM
Field Original Value New Value
Status Open [ 1 ] Resolved [ 5 ]
Fix Version/s 3.2.4 [ 12053 ]
Resolution Fixed [ 1 ]
Andy Jefferson made changes - 22/Nov/13 11:14 AM
Status Resolved [ 5 ] Closed [ 6 ]