groovy

ProxyGenerator.instantiateDelegate(Map, Class) behavior should align with ConvertedMap behavior

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Critical Critical
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 1.6-rc-1, 1.5.8, 1.7-beta-1
  • Component/s: None
  • Labels:
    None
  • Number of attachments :
    0

Description

We recently changed ConvertedMap to throw an UnsupportedOperationException for unused methods. The behavior should be the same in other places where maps are added to proxy classes, e.g. ProxyGenerator.

Issue Links

is related to

Bug - A problem which impairs or prevents the functions of the product. GROOVY-3200 Map coercion should not throw an NPE for non implemented methods from the map

  • Minor - Minor loss of function, or other problem where easy workaround is present.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Paul King made changes -
Field Original Value New Value
Link This issue is related to GROOVY-3200 [ GROOVY-3200 ]
Hide
Permalink
Paul King added a comment -

Other issue dealing with Map Coercion

Show
Paul King added a comment - Other issue dealing with Map Coercion
Hide
Permalink
Paul King added a comment -

Hmmm ... trivial fix of making ProxyGenerator use a ConvertedMap broke 10 tests.
Need to think about this further. I wonder if throwing UOE was really what we wanted to do?

Show
Paul King added a comment - Hmmm ... trivial fix of making ProxyGenerator use a ConvertedMap broke 10 tests. Need to think about this further. I wonder if throwing UOE was really what we wanted to do?
Hide
Permalink
Paul King added a comment -

Found a better fix. Added to trunk and 1.6 at the moment. Plus it has a flag to revert to using empty methods instead of the Exception.

Show
Paul King added a comment - Found a better fix. Added to trunk and 1.6 at the moment. Plus it has a flag to revert to using empty methods instead of the Exception.
Hide
Permalink
Paul King added a comment -

ported also to 1.5 branch

Show
Paul King added a comment - ported also to 1.5 branch
Paul King made changes -
Status Open [ 1 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]
Paul King made changes -
Status Resolved [ 5 ] Closed [ 6 ]

People

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

Dates

  • Created:
    Updated:
    Resolved: