Hi,

Just found out that FeatureTypeSchemaBuilder$GML3.addApplicationTypes is 
creating schemas for disabled types because it's checking on 
FeatureTypeInfo.isEnabled() instead of the derived property 
FeatureTypeInfo.enabled() which also checks if the parent StoreInfo is 
enabled.

This may cause a number of problems, mine being I've a disabled 
DirectoryDataStore because there's no way to tell it not to build a 
spatial index for shapefiles, leading to OOM.

Now, I suppose you'll agree to change the call to enabled() instead. 
Remining question being I also found other uses of isEnabled() where it 
seems enabled() should be rather used:

- <org.geoserver.wcs.response.WCSCapsTransformer.handleContents()>
- 
<org.geoserver.wfs.CapabilitiesTransformer$WFS1_0$CapabilitiesTranslator1_0.handleFeatureTypes()>
- 
<org.geoserver.wfs.CapabilitiesTransformer$WFS1_1$CapabilitiesTranslator1_1.featureTypeList()>
- <org.geoserver.wfs.DescribeFeatureType.run()>
- <org.geoserver.wfs.xml.v1_0_0.WFSConfiguration.configureContext()>
- <org.geoserver.wfs.xml.v1_1_0.WFSConfiguration.configureContext()>
- 
<org.geoserver.wcs.response.WCSCapsTransformer.WCSCapsTranslator.handleMetadataLink()>

Could someone verify and tell if I should go and change those by .enabled()?

Cheers,
Gabriel
-- 
Gabriel Roldan
OpenGeo - http://opengeo.org
Expert service straight from the developers.

------------------------------------------------------------------------------
Come build with us! The BlackBerry&reg; Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay 
ahead of the curve. Join us from November 9&#45;12, 2009. Register now&#33;
http://p.sf.net/sfu/devconf
_______________________________________________
Geoserver-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

Reply via email to