[GROOVY-967] remove PropertyValue from Object.allProperties() and eachProperty{}, refactor methods to use a Map

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.0-JSR-2
• Fix Version/s: 1.0-JSR-4
• Assignee: Dierk König
• Reporter: Dierk König
• Created: 19/Jul/05 02:28
• Updated: 05/Sep/05 13:15
• Resolved: 05/Sep/05 13:15

Description

> Object.allProperties() and eachProperty

{pv ->}

> deal with PropertyValue objects that merely encapsulate a name/value
> pair.
> This is not very consistent with Maps and the general handling of
> key/value pairs throughout the GDK.
>
> a) name/value in PV vs. key/value in Maps
> b) eachProperty doesn't allow

{key,value -> }

like Maps do
>
> Is there a reason for calling it "all"Properties instead of "get" ?
>
> Can we let it return a Map instead of a list of PropertyValue objects?
> We can even remove eachProperty and eachPropertyName then because
> obj.properties.each{}
> does the job.
>
> any opinions?

Sounds like a wise and consistent change.
You can go for it.

–
Guillaume Laforge