6.0.0-alpha14
7/2/25

[#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 07/29/2013 (4356 days ago)
Due
Updated 07/29/2013 (4356 days ago)
Assigned
Resolved 07/29/2013 (4356 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch Yes

History
07/29/2013 10:00:49 PM Michael Rubinsky Comment #4
Assigned to Michael Rubinsky
State ⇒ Resolved
Reply to this comment
Committed, thanks!
07/29/2013 10:00:26 PM 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
07/29/2013 10:18:09 AM 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.

07/29/2013 10:12:11 AM Thomas Jarosch Patch ⇒ Yes
New Attachment: 0002-Don-t-lose-isModSeq-flag-on-recursive-listBy-call-bu.patch Download
 
07/29/2013 10:07:10 AM 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