Frank Gasdorf

Results 28 comments of Frank Gasdorf

Any progress here for internationalization and what about suggested `handleCommitRollbackError()` method? I'm asking myself if its allowed to commit/rollback using API from e.g. some background tasks. If the user requested...

Would please update user-documentation as well that describes what happens on any change of this preference value? In addition due to other changes in this area the current changest has...

After a debug-session it looks like that the ShpServiceExtension does't set ShapefileDataStoreFactory.NAMESPACEP.key in parameter map which leads to a Name-Object without a namespace. Not sure why but the `ShpGeoResourceInfo.getSchema(ShpGeoResourceInfo.java:96)` does...

@sschulz92 @ybnkv @olsmn What do you think about this aproach? Throwables are logged as warnings Execptions are logged as errors everything else is just an info

I'm going to add few lines to release-notes and I guess I've to grep through documentations: ``` IStatus status = new Status(IStatus.ERROR, CatalogPlugin.ID, "error message"); CatalogPlugin.getDefault().getLog().log( status ); ``` since...

potential candidats (RST) to update to LoggingSupport: * docs/devel/en/drag_and_drop.rst * docs/devel/en/catalog.rst * docs/devel/en/using_udigapplication.rst * docs/devel/en/export_sld_plugin_tutorial.rst * docs/devel/en/edit_tools.rst

@sschulz92 thanks for providing this pull-request. I totally understand thats easier to maintain the code if you don't have formating clitches which causes merge issues and even more bad-readable code....

@ricfos looks like found a workaround. Nevertheless I’m going to check the difference of Geotools Libraries in udig.libs bundle between mentioned versions.

I had a look at the Implementations of ILayer and isType and hasResource were not implemented the same way: WMSLaser: https://github.com/locationtech/udig-platform/blob/4468dd6df7d6af80e8ef49b9bc7534d17f301452/plugins/org.locationtech.udig.info.tests/src/org/locationtech/udig/info/tests/WMSLayer.java#L229-L235 https://github.com/locationtech/udig-platform/blob/4468dd6df7d6af80e8ef49b9bc7534d17f301452/plugins/org.locationtech.udig.info.tests/src/org/locationtech/udig/info/tests/WMSLayer.java#L274-L277 whereas layerImpl is fine in my option: https://github.com/locationtech/udig-platform/blob/bfd630ea0e8e435b034474263692daf169970847/plugins/org.locationtech.udig.project/src/org/locationtech/udig/project/internal/impl/LayerImpl.java#L992-L1008...