Skip to content

Grails 7280#47

Closed
micfra wants to merge 2 commits into
apache:masterfrom
micfra:GRAILS-7280
Closed

Grails 7280#47
micfra wants to merge 2 commits into
apache:masterfrom
micfra:GRAILS-7280

Conversation

@micfra
Copy link
Copy Markdown

@micfra micfra commented Feb 16, 2011

Just fixed the issue behind the mentioned behavior in the ticket http://jira.codehaus.org/browse/GRAILS-7280

Check on empty string was missing. Would be nice to have it resolved to show minor bugs in Grails are fixed as well.

Tnx & Regards
MicFra

Michael Frankfurter added 2 commits February 17, 2011 00:20
…ounds is avoided in general which is the root cause of the error mentioned in the ticket.
…ounds is avoided in general which is the root cause of the error mentioned in the ticket.
@codescape
Copy link
Copy Markdown
Contributor

Oh, sorry. I didn't realize there is a second commit. That one would do the fix, yes!

@micfra
Copy link
Copy Markdown
Author

micfra commented Mar 8, 2011

Doesn't matter. Better to have somebody taking a look ;-)

@micfra
Copy link
Copy Markdown
Author

micfra commented Apr 27, 2011

Is there anything I may or should do to get this trivial but bug pulled in?

@jeffscottbrown
Copy link
Copy Markdown
Contributor

This has been addressed with 0ade6fd which includes a slightly more general solution and test coverage.

jdaugherty pushed a commit to jdaugherty/grails-core that referenced this pull request Jan 7, 2025
jdaugherty pushed a commit to jdaugherty/grails-core that referenced this pull request Feb 9, 2025
jdaugherty pushed a commit to jdaugherty/grails-core that referenced this pull request Apr 14, 2025
Configure DuplicatesStrategy.INCLUDE for processResources task
jdaugherty pushed a commit to jdaugherty/grails-core that referenced this pull request Apr 19, 2025
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants