Note: There are a few cases of duplicates in user autocompletion which are being worked on.

'Error parsing template: expected <content> to be first' when opening virtualFolderListDialog.xul

RESOLVED FIXED in Thunderbird 22.0

Status

MailNews Core
Search
--
minor
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: florian, Assigned: aceman)

Tracking

Trunk
Thunderbird 22.0

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

2.64 KB, patch
neil@parkwaycc.co.uk
: review+
Details | Diff | Splinter Review
(Reporter)

Description

5 years ago
Each time I open the virtualFolderListDialog.xul dialog, I see this message twice in my error console:
Error parsing template: expected <content> to be first

This message is printed from http://mxr.mozilla.org/comm-central/source/mozilla/content/xul/templates/src/nsXULContentUtils.cpp#498 without any location information, but it seems clear that the problem is something in http://mxr.mozilla.org/comm-central/source/mailnews/base/content/virtualFolderListDialog.xul#74 that the parser dislikes.

Steps to open that dialog:
1. View -> Folders -> Unified
2. Open the context menu of the Inbox special folder, and select "Properties…"
3. Click the "Choose…" button in the dialog that appeared.

Comment 1

5 years ago
WFM over here
(Reporter)

Comment 2

5 years ago
I can still reproduce (tested on a Mac debug build).
(Assignee)

Comment 3

5 years ago
Still happening. Maybe it is something from RDF?
Component: Folder and Message Lists → Search
Product: Thunderbird → MailNews Core

Comment 4

5 years ago
There's a duplicate <content> tag in virtualFolderListDialog.xul, only the first child of <conditions> can be <content>, thus the error.
(Assignee)

Comment 5

5 years ago
Created attachment 716145 [details] [diff] [review]
patch

Thanks Neil.
Assignee: nobody → acelists
Status: NEW → ASSIGNED
Attachment #716145 - Flags: review?(neil)

Comment 6

5 years ago
Comment on attachment 716145 [details] [diff] [review]
patch

>+            <member container="?container" child="?member"
>+                    isDeferred="?isDeferred"/>
Well, if you're going to fiddle, I don't think this is necessary. But I don't really want to create a virtual folder just to find out...
(Assignee)

Comment 7

5 years ago
Created attachment 716211 [details] [diff] [review]
patch v2

If that one is to hide deferred accounts in the list, then I have tried it now and with that line removed deferred accounts are still hidden.
Attachment #716145 - Attachment is obsolete: true
Attachment #716145 - Flags: review?(neil)
Attachment #716211 - Flags: review?(neil)

Comment 8

5 years ago
Comment on attachment 716211 [details] [diff] [review]
patch v2

OK, r=me based on code inspection.
Attachment #716211 - Flags: review?(neil) → review+
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/33a80322430e
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 22.0
You need to log in before you can comment on or make changes to this bug.