BC break in Horde_Share::listShares() method signature

We have Horde version 3.1.5. ----8<---- Dec 31 13:20:40

Wed, 31 Dec 2008 12:07:25 +0000

We have Horde version 3.1.5. ----8<---- Dec 31 13:20:40 HORDE [error] [nag] 0 does not exist [on line 281 of "/home/httpd/html/horde/nag-h3-2.3.1/lib/Nag.php"] ----8<---- There is listShares function call in Nag.php at line 281. There seems to be wrong parameters for listShares function. Original version of line 282: ----8<---- $tasklists = $GLOBALS['nag_shares']->listShares(Auth::getAuth(), $permission, $owneronly ? Auth::getAuth() : null, 0, 0, 'name'); ----8<---- I added 'parent' and 'allLevels' parameters in the middle on parameters: ----8<---- $tasklists = $GLOBALS['nag_shares']->listShares(Auth::getAuth(), $permission, $owneronly ? Auth::getAuth() : null, DATATREE_ROOT, true, 0, 0, 'name'); ----8<---- The behaves now quite well. But I don't have any idea what 'parent' and 'allLevels' parameters should be.

You're going to need a different Horde version or a differen

Sun, 04 Jan 2009 06:24:32 +0000

You're going to need a different Horde version or a different Nag version. At this point I'm not sure if any Nag version should have worked with that version of listShares; it's more likely to be an unintentional BC break when going from x.1.x to x.2.x.

I don't think there's anything we can do about this at this

Mon, 12 Jan 2009 04:19:40 +0000

I don't think there's anything we can do about this at this point - anyone else?

Reverting to the original signature in Horde_Share would bre

Mon, 12 Jan 2009 10:53:17 +0000

Reverting to the original signature in Horde_Share would break forward compatibility, so I guess the cheapest solution is to test for the Horde version in the three places where we actually use the additional parameters and add a switch there.

The three places would be: nag/lib/Nag.php mnemo/lib/Mnemo

Mon, 12 Jan 2009 10:56:41 +0000

The three places would be: nag/lib/Nag.php mnemo/lib/Mnemo.php kronolith/lib/Kronolith.php and additionally: incubator/spread/lib/Spread.php

For posterity, this happened because I removed the listShare

Tue, 13 Jan 2009 04:28:45 +0000

For posterity, this happened because I removed the listShares() method completely in this commit: http://cvs.horde.org/diff.php/framework/Share/Share.php?r1=1.142&r2=1.143 And then when the extended parameters were added back in, $parent and $allLevels weren't there since this was no longer DataTree-specific code: http://cvs.horde.org/diff.php/framework/Share/Share.php?r1=1.162&r2=1.163

Proposed patch for Nag, I'll add Mnemo and Kronolith (FW_3 o

Tue, 13 Jan 2009 04:36:55 +0000

Proposed patch for Nag, I'll add Mnemo and Kronolith (FW_3 only) if this looks good. No need for spread to be compatible with Horde 3.1.

Looks good.

Tue, 13 Jan 2009 12:07:22 +0000

Looks good.

Changes have been made in CVS for this ticket: http://cvs.h

Tue, 13 Jan 2009 15:46:36 +0000

Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/kronolith/docs/CHANGES?rt=horde&r1=1.165.2.246&r2=1.165.2.247&ty=u http://cvs.horde.org/diff.php/kronolith/lib/Kronolith.php?rt=horde&r1=1.263.2.79&r2=1.263.2.80&ty=u http://cvs.horde.org/diff.php/nag/docs/CHANGES?rt=horde&r1=1.115.2.109&r2=1.115.2.110&ty=u http://cvs.horde.org/diff.php/nag/lib/Nag.php?rt=horde&r1=1.124.2.32&r2=1.124.2.33&ty=u

Changes have been made in CVS for this ticket: http://cvs.h

Tue, 13 Jan 2009 15:47:36 +0000

Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/mnemo/docs/CHANGES?rt=horde&r1=1.63.2.76&r2=1.63.2.77&ty=u http://cvs.horde.org/diff.php/mnemo/lib/Mnemo.php?rt=horde&r1=1.52.2.14&r2=1.52.2.15&ty=u http://cvs.horde.org/diff.php/mnemo/lib/api.php?rt=horde&r1=1.53.2.31&r2=1.53.2.32&ty=u http://cvs.horde.org/diff.php/mnemo/lib/prefs.php?rt=horde&r1=1.2.10.9&r2=1.2.10.10&ty=u http://cvs.horde.org/diff.php/mnemo/memo.php?rt=horde&r1=1.42.2.12&r2=1.42.2.13&ty=u http://cvs.horde.org/diff.php/mnemo/templates/data/import.inc?rt=horde&r1=1.11.10.4&r2=1.11.10.5&ty=u http://cvs.horde.org/diff.php/mnemo/templates/panel.inc?rt=horde&r1=1.6.2.4&r2=1.6.2.5&ty=u http://cvs.horde.org/diff.php/mnemo/templates/prefs/notepadselect.inc?rt=horde&r1=1.4.12.2&r2=1.4.12.3&ty=u

Committed.

Tue, 13 Jan 2009 15:50:47 +0000

Committed.