groovy

Stateful metaclass mixins are completely undocumented

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.6.7
  • Fix Version/s: 1.7.2, 1.8-beta-1
  • Component/s: None
  • Labels:
    None
  • Number of attachments :
    0

Description

http://groovyland.wordpress.com/2008/07/09/groovy-dynamic-stateful-mixins/ is the only documentation... and it's wrong.

It's "owner", not "mixinOwner".

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Paul King added a comment -

corrected URL to point to article mentioning erroneous 'mixinOwner'

Show
Paul King added a comment - corrected URL to point to article mentioning erroneous 'mixinOwner'
Paul King made changes -
Field Original Value New Value
Description http://groovyland.wordpress.com/2008/06/02/groovy-per-instance-meta-classes-and-expandometaclass-dsl/ is the only documentation... and it's wrong.

It's "owner", not "mixinOwner". 		http://groovyland.wordpress.com/2008/07/09/groovy-dynamic-stateful-mixins/ is the only documentation... and it's wrong.

It's "owner", not "mixinOwner".
Hide
Permalink
Paul King added a comment -

currently the best "doco" is the unit test: groovy.lang.MixinTest

Show
Paul King added a comment - currently the best "doco" is the unit test: groovy.lang.MixinTest
Hide
Permalink
Paul King added a comment -

Here is the currently supported version of that example:

class NoDuplicateCollection {
    void put (def obj) {
        def clone = find { it == obj }
        if (!clone) add obj
    }
}

def list = new Object()
list.metaClass.mixin NoDuplicateCollection, LinkedList
[1, 1, 2, 3, 2, 3].each{ list.put it }
assert list.size() == 3
assert list == [1, 2, 3]

Also, if you are in a position to look at trunk (or a CI server)
can you check the latest JavaDoc for EMC and see if that looks like
it covers everything for mixins. Thanks.

Show
Paul King added a comment - Here is the currently supported version of that example: class NoDuplicateCollection { void put (def obj) { def clone = find { it == obj } if (!clone) add obj } } def list = new Object () list.metaClass.mixin NoDuplicateCollection, LinkedList [1, 1, 2, 3, 2, 3].each{ list.put it } assert list.size() == 3 assert list == [1, 2, 3] Also, if you are in a position to look at trunk (or a CI server) can you check the latest JavaDoc for EMC and see if that looks like it covers everything for mixins. Thanks.
Hide
Permalink
Paul King added a comment -

Any chance to look at the javadoc? Ok to close this issue? Thanks.

Show
Paul King added a comment - Any chance to look at the javadoc? Ok to close this issue? Thanks.
Hide
Permalink
Paul King added a comment -

No further comments - resolving.

Show
Paul King added a comment - No further comments - resolving.
Paul King made changes -
Status Open [ 1 ] Resolved [ 5 ]
Assignee Paul King [ paulk ]
Fix Version/s 1.8-beta-1 [ 16013 ]
Fix Version/s 1.7.2 [ 16237 ]
Resolution Fixed [ 1 ]
Paul King made changes -
Status Resolved [ 5 ] Closed [ 6 ]

People

  • Assignee:
    Paul King
    Reporter:
    Luke Daley
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: