2016-04-05 7:29 GMT+02:00 Jody Garnett <[email protected]>:
> That sounds like a pretty serious glitch, I wonder how the test cases are
> passing?
>
>
There are of course other FeatureTypeFactory implementations that are on
the classpath during testing; and many testcases use FeatureTypeFactoryImpl
directly.
fix looks good to me; I prepped an issue:
https://osgeo-org.atlassian.net/browse/GEOT-5400
> Checking the default constructor ... avoids SPI:
>
> public AttributeTypeBuilder() {
> this( new FeatureTypeFactoryImpl() );
> init();
> }
>
> And I see a duplicate call to init(), and nothing in the codebase calls
> CommonFactoryFinder.getFeatureTypeFactory(null), and ...
> as much as I like design - we have far too much unrealized flexibility in
> the GeoTools codebase.
>
> Here is a small fix - https://github.com/geotools/geotools/pull/1163
>
> --
> Jody Garnett
>
> On 4 April 2016 at 10:33, Mark Prins <[email protected]> wrote:
>
>> I'm trying to use the AttributeTypeBuilder, but getting the error
>> org.geotools.factory.FactoryNotFoundException: No factory of kind
>> "FeatureTypeFactory" found.
>>
>> Looking int the FeatureTypeFactory SPI [1] in gt-main.jar (14.3) it lists
>> `org.geotools.feature.DefaultFeatureTypeFactory` as a factory, but no
>> such file exists in the repository nowadays (and not in that jar).
>>
>> Shouldn't the SPI file point
>> to org.geotools.feature.type.FeatureTypeFactoryImpl ?
>>
>> Any other gt-<>.jar I could add to my pom to get this running? (i
>> currently have just gt-xml and gt-epsg-hsql)
>>
>>
>> thanx, Mark
>>
>>
>> [1]
>> https://github.com/geotools/geotools/blob/master/modules/library/main/src/main/resources/META-INF/services/org.geotools.feature.FeatureTypeFactory
>>
>> --
>> Disclaimer;
>> This message is just a reflection of what I thought at the time of
>> sending. The message may contain information that is not intended for you
>> or that you don't understand.
>>
>>
>> ------------------------------------------------------------------------------
>>
>> _______________________________________________
>> GeoTools-GT2-Users mailing list
>> [email protected]
>> https://lists.sourceforge.net/lists/listinfo/geotools-gt2-users
>>
>>
>
--
Disclaimer;
This message is just a reflection of what I thought at the time of sending.
The message may contain information that is not intended for you or that
you don't understand.
------------------------------------------------------------------------------
_______________________________________________
GeoTools-GT2-Users mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-gt2-users
