[API] ol.Object inconsistency: #getProperties() vs. #setValues()

[ahocevar]

I think ol.Object#setValues() should be renamed to ol.Object#setProperties(). Opinions?

[bartvde]

or getProperties should be renamed to getValues?

[bartvde]

as an aside, searching for ol.Object in the API docs is kind of painful now since there is this long list of classes descending from ol.Object that show up first, but maybe this is ticketed already somewhere

[ahocevar]

or getProperties should be renamed to getValues?

That would be confusing with getKeys being in place, in my opinion at least.

[bartvde]

for reference the Google Maps MVCObject https://developers.google.com/maps/documentation/javascript/reference?csw=1#MVCObject

there is no getProperties method, just get.
setValues is there.

[fredj]

That would be confusing with getKeys being in place, in my opinion at least.

Yes. getKeys returns an array of string but getProperties an object

[elemoine]

+1 for getProperties and setProperties.

[marcjansen]

+1 for getProperties and setProperties.

[elemoine]

I'm on it.