Fri, 10 Apr 2026 09:04:29 +0000 https://bugs.horde.org/ticket/6770 Nag task lists over WebDAV don't work Somewhere between 2.2 RC3 and 2.2 final this feature stopped working. I made a clean 2.2 installation on a test server and tried to access the /nag/user/user.ics (and /nag/user.ics) files using Kontact and curl. In both cases, no content is given back; the right URL gives 200 and 'HORDE [error] [horde] Invalid tasklist requested. [pid 28497 on line 361 of "/home/bb/src/groupware/horde/horde-webmail-1.1/lib/Horde/RPC/webdav.php"]' in the log; the wrong one gives 404. Wed, 28 May 2008 14:17:53 +0000 https://bugs.horde.org/ticket/6770#t45649 Bug #6947 has some additional information. Wed, 25 Jun 2008 09:35:30 +0000 https://bugs.horde.org/ticket/6770#t46799 > Somewhere between 2.2 RC3 and 2.2 final this feature stopped working. > > I made a clean 2.2 installation on a test server and tried to access > the /nag/user/user.ics (and /nag/user.ics) files using Kontact and > curl. In both cases, no content is given back; the right URL gives > 200 and 'HORDE [error] [horde] Invalid tasklist requested. [pid 28497 > on line 361 of > "/home/bb/src/groupware/horde/horde-webmail-1.1/lib/Horde/RPC/webdav.php"]' > in the log; the wrong one gives 404. > I can confirm that it doesnt work under 2.2. Temporal fix? I changed nag/lib/api.php to the 2.2_RC2 versions and now it "works"... Wed, 25 Jun 2008 16:49:21 +0000 https://bugs.horde.org/ticket/6770#t46811 I can confirm this bug. The mentioned workaround wont work correct when it comes to uploading and does not confirm to the url nag tolds to use. Any progress here to get it working? Thu, 26 Jun 2008 09:28:03 +0000 https://bugs.horde.org/ticket/6770#t46829 Whoops, there I go updating the wrong bug again... Please try what I just committed to CVS. Two features have been fixed: the ability to download .ics files and the ability to browse within tasklists. http://cvs.horde.org/diff.php/nag/lib/api.php?r1=1.183&r2=1.184&ty=u Thu, 26 Jun 2008 13:45:02 +0000 https://bugs.horde.org/ticket/6770#t46865 Apllied the patch, but now i got: The method "browse" is not defined in the API for nag. [pid 4870 on line 361 of "/var/www/test1/horde/lib/Horde/RPC/webdav.php"] Thu, 26 Jun 2008 14:33:49 +0000 https://bugs.horde.org/ticket/6770#t46870 Thanks! This fixes loading the .ics file in Kontact for me, and also saving new entries onto the server. Editing existing entries and publishing that to the server doesn't work, I get "Access denied when uploading .....ics". This is in the horde log: DB Error: null value violates not-null constraint: UPDATE nag_tasks SET task_creator = 'bb', task_assignee = 'MAILTO:bb', task_name = 'Teste Nag@baby', task_desc = 'Mehrzeilige Beschreibung mit neuem Text', task_start = 1214431200, task_due = 0, task_priority = 5, task_estimate = '0.00', task_completed = 0, task_completed_date = 0, task_category = '', task_alarm = 0, task_parent = NULL, task_private = 0 WHERE task_owner = 'bb' AND task_id = 'fe4b7e6e2d4e367ea68f036b72c1bfc0' [nativecode=ERROR: null value in column "task_parent" violates not-null constraint] [pid 5176 on line 280 of "/home/bb/src/groupware/horde/horde-webmail-1.1.1/nag/lib/Driver/sql.php"] All the other entries in the DB have task_parent = '' but not NULL. So I changed line 619 of nag/lib/api.php from isset($task->parent) ? $task->parent : $existing->parent, to: isset($task->parent) ? $task->parent : '', and now it seems to work. Thank you for fixing this! Thu, 26 Jun 2008 14:36:59 +0000 https://bugs.horde.org/ticket/6770#t46873 Forget my last comment, solved by correcting file ownership after patching ;). Whats missing: Nag still displays the wrong abonnent url, the "user" part is missing - you have to add this by hand for now. And the patch with "existing->parent" is needed to create new "Root" tasks, but it breaks creation of subtasks, so this is not solved yet. Thu, 26 Jun 2008 14:53:52 +0000 https://bugs.horde.org/ticket/6770#t46876 > Apllied the patch, but now i got: > > The method "browse" is not defined in the API for nag. [pid 4870 on > line 361 of "/var/www/test1/horde/lib/Horde/RPC/webdav.php"] > Sometimes I have seen this if I change the file in the middle of a session. I would suggest you log out and start a new session. It also would not hurt to check the filesystem permissions as sometimes patching or CVS updating can "adjust" them for you. Thu, 26 Jun 2008 14:54:07 +0000 https://bugs.horde.org/ticket/6770#t46877 To clarify things: Used your patch and now i can subsribe to the tasklist. Adding tasks and subtasks works fine with webinterface. Adding them via webdav is broken with some the null-constraint message below, the workaround provided by bb@apc.ag can not be used, subtasks are broken than. Thu, 26 Jun 2008 15:10:04 +0000 https://bugs.horde.org/ticket/6770#t46879 > Whats missing: Nag still displays the wrong abonnent url, the "user" > part is missing - you have to add this by hand for now. This had been fixed in CVS already. Thu, 26 Jun 2008 15:25:37 +0000 https://bugs.horde.org/ticket/6770#t46883 OK, I have a new solution for the subtask problem. Inserted on line 605 of nag/lib/api.php: //apc: avoid NOT NULL constraint of the database if (! $existing->parent) $existing->parent = ''; and reverted the old change. Creating tasks and subtasks works; they are displayed correctly (with hierarchy) in Kontact but when I change something there, the hierarchy is broken. I suspect that the problem is with Kontact because it didn't work in -RC3, either. I'll try to track this down further. Fri, 27 Jun 2008 08:58:19 +0000 https://bugs.horde.org/ticket/6770#t46924 Looked at the ics contact uploads and it looks fine - is does have RELATED-TO fields pointing to the parent. Looked at the sql queries done and there is no parent set where subtasks are added. I guess on the way from parsing (the parent is set in the parser, did some logging) to persisting it, the parent gets lost, but i don't know where yet. Fri, 27 Jun 2008 09:57:36 +0000 https://bugs.horde.org/ticket/6770#t46926 I finally tracked the subtask problem and solved it. In Driver.php, fromiCalendar($vTodo) reads the parent_id from the vTodo file and returns that in parent_id. However, line 619 of api.php uses the parent property for saving, not parent_id. Modifying that line in this way: isset($task->parent_id) ? $task->parent_id : $existing->parent_id, solves saving subtasks from Kontact. I tested all cases (creating new task in Horde and in Kontact, editing existing ones) without error messages or the tasks getting out of sync. So I propose this change to be made in CVS. Fri, 27 Jun 2008 10:07:42 +0000 https://bugs.horde.org/ticket/6770#t46927 lol - nice ;-) i wanted to write the same, that driver.php uses "parent_id" and api.php "parent" - should have known that, i may have used my time to track down this annoying VALARM problem ^^. Fri, 27 Jun 2008 10:16:44 +0000 https://bugs.horde.org/ticket/6770#t46928 _modify and _add needs this $task->parent_id fix. After doing that, you can remove the patch from line 605, setting '' to existing->parent, am i right - it works fine here without. Fri, 27 Jun 2008 10:23:26 +0000 https://bugs.horde.org/ticket/6770#t46929 Here a patchfile. Fri, 27 Jun 2008 10:31:41 +0000 https://bugs.horde.org/ticket/6770#t46931 > After doing that, you can remove the patch from line 605, setting '' > to existing->parent, am i right - it works fine here without. yes, I removed that. There's also an iCal import function that uses ->parent instead of parent_id, too. I assume that function must also be changed. Fri, 27 Jun 2008 10:35:41 +0000 https://bugs.horde.org/ticket/6770#t46932 Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/nag/lib/api.php?r1=1.188&r2=1.189&ty=u Wed, 09 Jul 2008 21:56:51 +0000 https://bugs.horde.org/ticket/6770#t47416 Really nice work guys! Correct analysis, correct fix, working patch. A bug report for the hall of fame. Wed, 09 Jul 2008 21:59:32 +0000 https://bugs.horde.org/ticket/6770#t47417