6.0.0-git
2019-03-24

[#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 2013-07-29 (2064 days ago)
Due
Updated 2013-07-29 (2064 days ago)
Assigned
Resolved 2013-07-29 (2064 days ago)
Milestone
Patch Yes

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

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

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

Thomas

Saved Queries