Fix for GPRESOURCES-127 #11

Merged
merged 0 commits into from Apr 27, 2012

Conversation

Projects
None yet
1 participant

As the first ever request to an ad-hoc resource may arrive through a /static/ URL, there is no sure way of telling that it is not an ad-hoc resource. Removing the early return with null fixes the issue, however I'm not fully aware of what the implications of this are...?

Sounds good to me.

I'm not at all sure about this - the reason is that this changes the behaviour for resources that don't exist at startup I think. > It needs more thought

Actually, looking at the codebase, I can't find any other references that would call ResourceProcessor.getResourceMetaForURI() with createAdHocResourceIfNeeded = false besides ResourceProcessor.processModernResource(). All the code that runs upon startup to generate the modules has this parameter set to true...

@lauripiispanen lauripiispanen merged commit 46092ae into grails-plugins:master Apr 27, 2012

Updated pull request to latest version of resources: #16

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment