Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GROOVY-6158 ObjectGraphBuilder.NewInstanceResolver disregards node value #358

Open
wants to merge 1 commit into
base: GROOVY_2_2_X
Choose a base branch
from

Conversation

aalmiray
Copy link
Contributor

This fix introduces a binary incompatibility as a method signature
was changed from

Object newInstance(Class klass, Map attributes)

to

Object newInstance(Class klass, Object value, Map attributes)

Merge with caution (into 2.2 or greater) or wait until 3.0

This fix introduces a binary incompatibility as a method signature
was changed from

    Object newInstance(Class klass, Map attributes)

to

    Object newInstance(Class klass, Object value, Map attributes)

Merge with caution (into 2.2 or greater) or wait until 3.0
@melix melix added this to the 3.0 milestone Mar 21, 2014
traneHead pushed a commit to traneHead/groovy-core that referenced this pull request Jul 29, 2017
…curity Manager (closes groovy#358)

When running under a Security Manager an exception would be thrown because
the empty codebase value was added as a Permission name which can not be
empty.  So use the same codebase value that is used by GStringTemplateEngine
and StreamingTemplateEngine for unnamed code sources.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants