[Geoserver-devel] CSSDemoPage may create the sld file before a CatalogAddEvent is fired

GeoServer GeoServer Developer
1

Hi,
I’m working with a plugin that implements a CatalogListener to handle the incoming CatalogAddEvent.

This implementation works fine with the event generated from:

org.geoserver.wms.web.data.StyleNewPage.onStyleFormSubmit()

which writes the SLD to the FileSystem then fire the event.

Using the CssDemoPage when I try to create the css the event is fired but the file does not exists on the FileSystem.

Looking at the CssDemoPage.createCssTemplate() the patch seems quite simple so I propose the following pull req:

https://github.com/geoserver/geoserver/pull/576

Let me know if it is acceptable for you.
Cheers,
Carlo

–

==
Meet us at GEO Business 2014! in London! Visit http://goo.gl/fES3aK for more information.

Dott. Carlo Cancellieri
@cancellieric
Software Engineer

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272

http://www.geo-solutions.it
http://twitter.com/geosolutions_it

2

Looks like it has already been accepted :slight_smile:

···

Jody Garnett

On Thu, May 1, 2014 at 2:22 AM, carlo cancellieri <carlo.cancellieri@anonymised.com> wrote:

Hi,
I’m working with a plugin that implements a CatalogListener to handle the incoming CatalogAddEvent.

This implementation works fine with the event generated from:

org.geoserver.wms.web.data.StyleNewPage.onStyleFormSubmit()

which writes the SLD to the FileSystem then fire the event.

Using the CssDemoPage when I try to create the css the event is fired but the file does not exists on the FileSystem.

Looking at the CssDemoPage.createCssTemplate() the patch seems quite simple so I propose the following pull req:

https://github.com/geoserver/geoserver/pull/576

Let me know if it is acceptable for you.
Cheers,
Carlo

–

==
Meet us at GEO Business 2014! in London! Visit http://goo.gl/fES3aK for more information.

Dott. Carlo Cancellieri
@cancellieric
Software Engineer

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272

http://www.geo-solutions.it
http://twitter.com/geosolutions_it

“Accelerate Dev Cycles with Automated Cross-Browser Testing - For FREE
Instantly run your Selenium tests across 300+ browser/OS combos. Get
unparalleled scalability from the best Selenium testing platform available.
Simple to use. Nothing to install. Get started now for free.”
http://p.sf.net/sfu/SauceLabs

Geoserver-devel mailing list
Geoserver-devel@anonymised.comsts.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel