FeatureService.UpdateFeatures doesn't log errors and reports them as succesful

I am bumping this to critical, only because this is a potential dataloss bug

Most code (including samples which many users follow) will assume if an update/insert completes without throwing an error, it was successful, which is not true

Please add step-by-step instructions to recreate the problem; and it seems that a package and sample PHP script would also be required. Thanks, Tom

Ok, test case attached, it uses the a feature source from the Sheboygan Samples. It will fail to make any changes, due to the SDF being marked as READ ONLY and the FeatureClass? name is wrong ( should be Rail, is Rail3)

Simply run the script, tweaking the user/pass if required.

Current Behaviour: An error is returned in the result which must be manually inspected and the access.log reports the UpdateFeatures? operation as being Successful.

UpdateFeatures.1.0.0:3(Library://Samples/Sheboygan/Data/Rail.FeatureSource,MgFeatureCommandCollection,false,) Success

Expected Behaviour: An error should be thrown (to be caught), the UpdateFeatures? accesslog entry should be marked as Failure

UpdateFeatures.1.0.0:3(Library://Samples/Sheboygan/Data/Rail.FeatureSource,MgFeatureCommandCollection,false,) Failure

In addition the error message should be logged to error.log

An exception occurred in FDO component. Item 'Rail3' not found in collection
Exception occurred in method MgServerUpdateFeatures.UpdateFeatures at line 97 in 
file d:\buildforgeprojects\mapguide_open_source_v2.0\build_27.13\mgdev\server
\src\services\feature\ServerUpdateFeatures.cpp 

Hello,

i wolud like to report the same behaviour on deleting features. In fact, I don't get it, the code looks and works well, class name is just fine, filter condition also but no errors or exceptions has been thrown.

        string TIMgisUser = System.Configuration.ConfigurationManager.AppSettings["TIMgisUser"];
        MapGuideApi.MgInitializeWebTier(@"C:\program files\MapguideOpenSource2.0\WebServerExtensions\www\webconfig.ini");
        MgUserInformation userInfo = new MgUserInformation("Administrator", "" + System.Configuration.ConfigurationManager.AppSettings["mgP"] + "");
        MgSite site = new MgSite();
        site.Open(userInfo);
        String mgSessionId = site.CreateSession();

        MgUserInformation userInfoNew = new MgUserInformation(mgSessionId);
        MgSiteConnection siteConnection = new MgSiteConnection();
        siteConnection.Open(userInfoNew);

        MgResourceService resourceService = siteConnection.CreateService(MgServiceType.ResourceService) as MgResourceService;
        MgFeatureService featureService = siteConnection.CreateService(MgServiceType.FeatureService) as MgFeatureService;

        MgResourceIdentifier resId = new MgResourceIdentifier("Library://PROJEKTI/ZLZ/PODACI/DIONICE.FeatureSource");
        MgFeatureCommandCollection commands = new MgFeatureCommandCollection();

        string layerClassName = "DIONICE";
        
        MgDeleteFeatures deleteCmd = new MgDeleteFeatures(layerClassName , "(FeatId=1)");
        commands.Add(deleteCmd);

        featureService.UpdateFeatures(resId, commands, false);

Replying to milicevic:

Hello, i wolud like to report the same behaviour on deleting features. In fact, I don't get it, the code looks and works well, class name is just fine, filter condition also but no errors or exceptions has been thrown. {{{ string TIMgisUser = System.Configuration.ConfigurationManager?.AppSettings?TIMgisUser?; MapGuideApi?.MgInitializeWebTier?(@"C:\program files\MapguideOpenSource?2.0\WebServerExtensions?\www\webconfig.ini"); MgUserInformation? userInfo = new MgUserInformation?("Administrator", "" + System.Configuration.ConfigurationManager?.AppSettings?mgP? + ""); MgSite? site = new MgSite?(); site.Open(userInfo); String mgSessionId = site.CreateSession?(); MgUserInformation? userInfoNew = new MgUserInformation?(mgSessionId); MgSiteConnection? siteConnection = new MgSiteConnection?(); siteConnection.Open(userInfoNew); MgResourceService? resourceService = siteConnection.CreateService?(MgServiceType?.ResourceService?) as MgResourceService?; MgFeatureService? featureService = siteConnection.CreateService?(MgServiceType?.FeatureService?) as MgFeatureService?; MgResourceIdentifier? resId = new MgResourceIdentifier?("Library://PROJEKTI/ZLZ/PODACI/DIONICE.FeatureSource"); MgFeatureCommandCollection? commands = new MgFeatureCommandCollection?(); string layerClassName = "DIONICE"; MgDeleteFeatures? deleteCmd = new MgDeleteFeatures?(layerClassName , "(FeatId?=1)"); commands.Add(deleteCmd); featureService.UpdateFeatures?(resId, commands, false); }}}

Hello,

I wolud like fill up my report. Thanks to Zac i have solved my problem. The reason was read only feature source.