> > So if I understand this correctly this means two additional changes. > > 1. Remove the ElementSet argument from CatalogStore.getRecords() method. > 2. Remove CatalogStore.translateProperty() > > (1) is trivial, more or less all calls to it passed in null. Made that > change and tests are happy. > > (2) is not so clear to me. At the moment there is code calling this > method, and multiple implementations of of it, one from > AbstractCatalogStore and one from InternalCatalogStore which appear to > be doing different things. So it's not quite obvious to me what should > be done here.
Oh no, only 1 needs to be changed, the second is indeed necessary as I explained further in the email. I was just apologising for the missing javadoc for the method, making it harder to understand. Don't worry - the changes are in fact trivial. Kind Regards Niels ------------------------------------------------------------------------------ Master Visual Studio, SharePoint, SQL, ASP.NET, C# 2012, HTML5, CSS, MVC, Windows 8 Apps, JavaScript and much more. Keep your skills current with LearnDevNow - 3,200 step-by-step video tutorials by Microsoft MVPs and experts. ON SALE this month only -- learn more at: http://p.sf.net/sfu/learnnow-d2d _______________________________________________ Geoserver-devel mailing list Geoserver-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geoserver-devel