407965 2019-05-26 20:03:06 +0000 Applying two changes in a row in the event editor fails with "Only resources can modify remote identifiers" 2020-04-11 07:45:34 +0000 1 1 2 Applications korganizer incidence editors 5.7.3 Kubuntu Linux RESOLVED FIXED NOR normal --- 1 rwestman faure faure sander https://commits.kde.org/incidenceeditor/7ba473fef78978aee6cdda0cdcead3fa65d3c62b 0 oldest_to_newest 1858908 0 rwestman 2019-05-26 20:03:06 +0000 SUMMARY The event editor UI allows you to apply a change immediately by pressing "Apply". But if you make more than one change without closing the editor in-between, the second one fails with an error dialog. You then have to cancel out of the dialog to exit it. This happens whether you try to apply the second change using the "Apply" button or the "OK" button. STEPS TO REPRODUCE 1. Press "New Event..." button and edit the Title field to say "Test" 2. Press "Apply" 3. Edit the Title field to say "Foo". 4. Press "Apply". OBSERVED RESULT An error dialog appears after the last step. The title is "Sorry - KOrganizer" and the message text is: "Error while trying to modify calendar item. Error was: Only resources can modify remote identifiers" When you click OK, you get a warning dialog saying "Unable to store the incidence in the calendar. Try again?". If you try again, the first error dialog will show again. It is possible to work around the bug by exiting the event editor, selecting the event created in step 2 for modification, and then redoing steps 3 to 4. EXPECTED RESULT I expected to be able to make many edits to an event, pressing "Apply" after each one, without getting the errors dialogs described. SOFTWARE/OS VERSIONS Linux distribution: Kubuntu 18.04 KDE Plasma Version: 5.12.7 KDE Frameworks Version: 5.44.0 Qt Version: 5.95 Kernel Version: 4.18.0-20-generic ADDITIONAL INFORMATION Maybe related to bug #307383, marked as fixed in 2012? 1858909 1 rwestman 2019-05-26 20:10:08 +0000 > You then have to cancel out of the dialog to exit it. That came out wrong - I meant that you have to cancel out of the editor since pressing "OK" (re-)triggers the problem. 1920007 2 faure 2020-04-04 07:34:12 +0000 *** Bug 408898 has been marked as a duplicate of this bug. *** 1920079 3 faure 2020-04-04 12:37:17 +0000 I think I fixed this in https://phabricator.kde.org/D16917 which landed in akonadi commit fbcd065c0dbb, part of tag v18.11.90, i.e. the Applications 18.12 release. (This report comes from KOrganizer 5.7.3 which Applications/17.12.3.) I'm just not 100% sure it's fully fixed though, because I had that error message on 03-Jan-2020. But today I can't reproduce it, no matter what I try (with or without the resource running, etc). Puzzling... 1920091 4 faure 2020-04-04 13:41:35 +0000 Nowadays, if I could find a way to reproduce this error, I would try itemFetchScope().setFetchRemoteIdentification(false); like I did in FatCRM. Hmm maybe one solution is to revert the D16917 patch to trigger the error more easily... 1920092 5 127267 faure 2020-04-04 13:41:56 +0000 Created attachment 127267 untested incidenceeditor fetchscope patch 1921554 6 faure 2020-04-11 07:45:34 +0000 Git commit 7ba473fef78978aee6cdda0cdcead3fa65d3c62b by David Faure. Committed on 11/04/2020 at 07:45. Pushed by dfaure into branch 'release/20.04'. Fix "Only resources can modify remote identifiers" when re-editing event Summary: ... before the resource had time to create it and notify korganizer about it. This seems to be a better fix than akonadi commit fbcd065c0dbb, which for some reason isn't enough (I had the error again in January 2020). Test Plan: Create event in korganizer, double-click it immediately, change something, click OK. Reviewers: dvratil Reviewed By: dvratil Differential Revision: https://phabricator.kde.org/D28559 M +1 -0 src/editoritemmanager.cpp https://commits.kde.org/incidenceeditor/7ba473fef78978aee6cdda0cdcead3fa65d3c62b 127267 2020-04-04 13:41:56 +0000 2020-04-04 13:41:56 +0000 untested incidenceeditor fetchscope patch incidenceeditor.diff text/plain 592 faure ZGlmZiAtLWdpdCBpL3NyYy9lZGl0b3JpdGVtbWFuYWdlci5jcHAgdy9zcmMvZWRpdG9yaXRlbW1h bmFnZXIuY3BwCmluZGV4IDdiYmY0MTMuLjhhMDY3MWMgMTAwNjQ0Ci0tLSBpL3NyYy9lZGl0b3Jp dGVtbWFuYWdlci5jcHAKKysrIHcvc3JjL2VkaXRvcml0ZW1tYW5hZ2VyLmNwcApAQCAtODIsNiAr ODIsNyBAQCBJdGVtRWRpdG9yUHJpdmF0ZTo6SXRlbUVkaXRvclByaXZhdGUoQWtvbmFkaTo6SW5j aWRlbmNlQ2hhbmdlciAqY2hhbmdlciwgRWRpdG9ySQogICAgIG1GZXRjaFNjb3BlLnNldEFuY2Vz dG9yUmV0cmlldmFsKEFrb25hZGk6Okl0ZW1GZXRjaFNjb3BlOjpQYXJlbnQpOwogICAgIG1GZXRj aFNjb3BlLnNldEZldGNoVGFncyh0cnVlKTsKICAgICBtRmV0Y2hTY29wZS50YWdGZXRjaFNjb3Bl KCkuc2V0RmV0Y2hJZE9ubHkoZmFsc2UpOworICAgIG1GZXRjaFNjb3BlLnNldEZldGNoUmVtb3Rl SWRlbnRpZmljYXRpb24oZmFsc2UpOwogCiAgICAgbUNoYW5nZXIKICAgICAgICAgPSBjaGFuZ2Vy ID8gY2hhbmdlciA6IG5ldyBBa29uYWRpOjpJbmNpZGVuY2VDaGFuZ2VyKG5ldyBJbmRpdmlkdWFs TWFpbENvbXBvbmVudEZhY3RvcnkoCg==