Fixed loadonstartup ordering bug

[Jason Rowland]

There is a loadonstartup ordering problem with the plugin.  load-on-startup should come after the init-param.  This causes a problem in strict containers like glassfish.

See:

http://stackoverflow.com/questions/5175034/strange-web-xml-error

diff --git a/JaxrsGrailsPlugin.groovy b/JaxrsGrailsPlugin.groovy

index f9ad5ed..3154c9b 100644

--- a/JaxrsGrailsPlugin.groovy

+++ b/JaxrsGrailsPlugin.groovy

@@ -101,11 +101,11 @@ Apache Wink are likely to be added in upcoming versions of the plugin.

                        'servlet' {

                                'servlet-name'('grails')

                                'servlet-class'(GrailsDispatcherServlet.name)

-                               'load-on-startup'('1')

                                'init-param'{

                                        'param-name'('dispatchOptionsRequest')

                                        'param-value'('true')

                                }

+                               'load-on-startup'('1')

                        }

                }

     }

[Noam Tenne]

Again, thanks; Can you please open a pull request for this too? 

Cheers,

Noam

--
You received this message because you are subscribed to the Google Groups "grails-jaxrs-discuss" group.
To unsubscribe from this group and stop receiving emails from it, send an email to grails-jaxrs-dis...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.
 
 

[Noam Tenne]

No pull request was opened, so I've pushed this fix directly to the master