error in repeat-index for nested repeats

RESOLVED FIXED

Status

Core Graveyard
XForms
RESOLVED FIXED
12 years ago
a year ago

People

(Reporter: Allan Beaufour, Assigned: Allan Beaufour)

Tracking

({fixed1.8})

Trunk
fixed1.8
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(2 attachments)

(Assignee)

Description

12 years ago
 
(Assignee)

Comment 1

12 years ago
Created attachment 191183 [details]
Testcase

The repeat index is set for all the nested repeats, only one of them should
have the repeat index (the top one, on load)
(Assignee)

Updated

12 years ago
Blocks: 264329
(Assignee)

Comment 2

12 years ago
Created attachment 197406 [details] [diff] [review]
Patch

Problem was that inner repeats initializes themselves before their parent is
set. This patch deselects the repeat index if the parent is set, fixing the
problem.
Assignee: aaronr → allan
Status: NEW → ASSIGNED
Attachment #197406 - Flags: review?(smaug)
(Assignee)

Updated

12 years ago
Blocks: 301998

Comment 3

12 years ago
Comment on attachment 197406 [details] [diff] [review]
Patch

Does this depend somehow on Bug 301998 too? Well, looks ok to me, r+
Attachment #197406 - Flags: review?(smaug)
Attachment #197406 - Flags: review?(aaronr)
Attachment #197406 - Flags: review+

Updated

12 years ago
Attachment #197406 - Flags: review?(aaronr) → review+
(Assignee)

Comment 4

12 years ago
(In reply to comment #3)
> (From update of attachment 197406 [details] [diff] [review] [edit])
> Does this depend somehow on Bug 301998 too? Well, looks ok to me, r+

Yeah, they are probably mutually dependent. I did a code split to put then in
different bugs.
(Assignee)

Comment 5

12 years ago
Checked in to trunk
Whiteboard: xf-to-branch

Comment 6

12 years ago
checked into branch 20051004
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
Whiteboard: xf-to-branch

Updated

12 years ago
Keywords: fixed1.8
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.