Last Comment Bug 747325 - 'Error parsing template: expected <content> to be first' when opening virtualFolderListDialog.xul
: 'Error parsing template: expected <content> to be first' when opening virtual...
Product: MailNews Core
Classification: Components
Component: Search (show other bugs)
: Trunk
: All All
-- minor (vote)
: Thunderbird 22.0
Assigned To: :aceman
Depends on:
  Show dependency treegraph
Reported: 2012-04-20 02:37 PDT by Florian Quèze [:florian] [:flo] (PTO until February 27)
Modified: 2013-02-21 02:45 PST (History)
3 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---

patch (2.75 KB, patch)
2013-02-20 11:27 PST, :aceman
no flags Details | Diff | Splinter Review
patch v2 (2.64 KB, patch)
2013-02-20 13:28 PST, :aceman
neil: review+
Details | Diff | Splinter Review

Description User image Florian Quèze [:florian] [:flo] (PTO until February 27) 2012-04-20 02:37:11 PDT
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 without any location information, but it seems clear that the problem is something in 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 User image Magnus Melin 2012-06-09 04:30:29 PDT
WFM over here
Comment 2 User image Florian Quèze [:florian] [:flo] (PTO until February 27) 2012-06-12 02:10:20 PDT
I can still reproduce (tested on a Mac debug build).
Comment 3 User image :aceman 2013-02-19 05:47:01 PST
Still happening. Maybe it is something from RDF?
Comment 4 User image 2013-02-19 06:56:17 PST
There's a duplicate <content> tag in virtualFolderListDialog.xul, only the first child of <conditions> can be <content>, thus the error.
Comment 5 User image :aceman 2013-02-20 11:27:10 PST
Created attachment 716145 [details] [diff] [review]

Thanks Neil.
Comment 6 User image 2013-02-20 13:06:59 PST
Comment on attachment 716145 [details] [diff] [review]

>+            <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...
Comment 7 User image :aceman 2013-02-20 13:28:21 PST
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.
Comment 8 User image 2013-02-20 13:34:57 PST
Comment on attachment 716211 [details] [diff] [review]
patch v2

OK, r=me based on code inspection.
Comment 9 User image Ryan VanderMeulen [:RyanVM] 2013-02-21 02:45:23 PST

Note You need to log in before you can comment on or make changes to this bug.