6.0.0
7/2/26

[#12508] Don't lose isModSeq flag on recursive listBy() call
Summary Don't lose isModSeq flag on recursive listBy() call
Queue Kronolith
Queue Version Git master
Type Bug
State Resolved
Priority 1. Low
Owners mrubinsk (at) horde (dot) org
Requester thomas.jarosch (at) intra2net (dot) com
Created 7/29/13 (4721 days ago)
Due
Updated 7/29/13 (4721 days ago)
Assigned
Resolved 7/29/13 (4721 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch Yes

History
4910 Michael Rubinsky Comment #4
Assigned to Michael Rubinsky
State ⇒ Resolved
Reply to this comment
Committed, thanks!
2610 Git Commit Comment #3 Reply to this comment
Changes have been made in Git (master):

commit cf49c1ea7dc39728a07bad4fc6cdc0275a9ff975
Author: Thomas Jarosch <thomas.jarosch@intra2net.com>
Date:   Mon Jul 29 12:02:39 2013 +0200

     Don't lose isModSeq flag on recursive listBy() call (bug #12508)

     Signed-off-by: Thomas Jarosch <thomas.jarosch@intra2net.com>
     Signed-off-by: Michael J Rubinsky <mrubinsk@horde.org>

  kronolith/lib/Api.php |    2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)

http://git.horde.org/horde-git/-/commit/cf49c1ea7dc39728a07bad4fc6cdc0275a9ff975
910 Thomas Jarosch Comment #2 Reply to this comment
Side note: I've grepped the horde source tree for listBy() calls and 
isModSeq use, the rest of the code looks fine.

1110 Thomas Jarosch Patch ⇒ Yes
New Attachment: 0002-Don-t-lose-isModSeq-flag-on-recursive-listBy-call-bu.patch Download
 
1010 Thomas Jarosch Comment #1
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ Don't lose isModSeq flag on recursive listBy() call
Queue ⇒ Kronolith
Milestone ⇒
Patch ⇒ No
State ⇒ Unconfirmed
Reply to this comment
Hi,

attached patch fixes passing the isModSeq flag in listBy() correctly.

Thomas

Saved Queries