Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

Rename method #45

Open
ridoo opened this issue Mar 18, 2014 · 2 comments
Open

Rename method #45

ridoo opened this issue Mar 18, 2014 · 2 comments
Labels

Comments

@ridoo
Copy link
Member

ridoo commented Mar 18, 2014

I think the setObject method could be better renamed to createProperty to better reflect what it does.

@ridoo ridoo added the question label Mar 18, 2014
@ridoo
Copy link
Member Author

ridoo commented Mar 18, 2014

To be honest it is unclear to me why an object created/set on the resource, prepared afterwards and its reference is returned. In GDALConnector the reference is being set explicitly afterwards as normal property ...

@MartinSchouwenburg
Copy link
Contributor

It might create a property, or not. But no it can not be set directly. The function is at GdalConnector level and at that level you can not know if the caller wants to set the property (in this case) or wants to do something else withit. The purpose of the function is to create an IlwisObject from the info in the resource. What you want to do with the created object is upto the caller. In the two cases were it is used it sets idd a property.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants