2013/6/20 Justin Deoliveira <jdeol...@opengeo.org>

> Ok, took a closer look, and it is indeed a bit of pain this issue. So,
> here are my thoughts.
>
> My two issues with your patch are as follows.
>
> 1. It duplicates functionality provided by ResolvingProxy, and requires
> maintaing a "shadow" interface in parallel to LayerGroupInfo
> 2. The "late binding" is already done by
> CatalogImpl.resolve(LayerGroupInfo)
>
> It is a tricky problem to be sure, i couldn't come up with a great
> solution, but here is a patch.
>
>   https://gist.github.com/jdeolive/5825868
>

Many thanks for your effort. I just needed to add one more null check to
make it work with my test case.
I added a unit test and created a pull request:
https://github.com/geoserver/geoserver/pull/255

Mauro

-- 
==
Our support, Your Success! Visit http://opensdi.geo-solutions.it for more
information.
==

Dott. Mauro Bartolomeoli
@mauro_bart
Senior 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

-------------------------------------------------------
------------------------------------------------------------------------------
This SF.net email is sponsored by Windows:

Build for Windows Store.

http://p.sf.net/sfu/windows-dev2dev
_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

Reply via email to