Closed
Bug 356321
Opened 18 years ago
Closed 18 years ago
unable to open all items of a bookmarked feed, due to a tag:... guid
Categories
(Firefox :: Bookmarks & History, defect)
Tracking
()
RESOLVED
WORKSFORME
Firefox 3 alpha2
People
(Reporter: moco, Assigned: sayrer)
References
()
Details
Attachments
(2 files)
unable to open all items of a bookmarked feed, due to a tag:... guid
to reproduce, bookmark http://planet.mozilla.org/rss20.xml and then try to open all the livemark children at once with "open all in tabs"
we'll bail out, because of cbeard blog:
<guid>tag:typepad.com,2003:post-13205879</guid>
I'll attach the static rss20.xml and a screen shot
Reporter | ||
Comment 1•18 years ago
|
||
Reporter | ||
Comment 2•18 years ago
|
||
Reporter | ||
Comment 3•18 years ago
|
||
I'm using Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.8.1) Gecko/20061011 BonEcho/2.0
OS: Mac OS X 10.3 → Mac OS X 10.4
Assignee | ||
Comment 4•18 years ago
|
||
<guid> has one attribute, isPermaLink, defaulting to true. That doesn't mean we should pay so much heed to it, though.
OS: Mac OS X 10.4 → Mac OS X 10.3
Comment 5•18 years ago
|
||
I see this as well using Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8.1) Gecko/20061010 Firefox/2.0, which is the RC3 candidate build we are testing.
Comment 6•18 years ago
|
||
Confirmed on Windows Firefox 2 RC3 with Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8.1) Gecko/20061010 Firefox/2.0
OS: Mac OS X 10.3 → All
Assignee | ||
Comment 7•18 years ago
|
||
* Broken in 1.5 as well, not a regression
* Broken feed (bogus rewriting by Planet)
* toolkit parser not fooled (see feed preview)
* will go away on trunk with places bookmarks, so targeting Fx3a2.
separate bug for the fact that the tab loads stop?
Assignee: nobody → sayrer
Target Milestone: --- → Firefox 3 alpha2
Comment 8•18 years ago
|
||
"see also" bug 260918 (of which this is a duplicate, but whatever/wherever).
What heuristic is the toolkit parser using to not get fooled?
Assignee | ||
Comment 9•18 years ago
|
||
(In reply to comment #8)
> "see also" bug 260918 (of which this is a duplicate, but whatever/wherever).
>
> What heuristic is the toolkit parser using to not get fooled?
No heuristic. <link> is the item.link if present. The toolkit parser would have used the tag: url if there were no <link> element.
Comment 10•18 years ago
|
||
Reopened bug 302548 for getting planet.m.o fixed again.
Hrm, preferring <link> to <guid isPermaLink="true">? If you're certain you want to do that, I think you'll be needing to wontfix the bug I'll be filing on you. But, yay! for the opportunity to argue about interpretations of the RSS spec based on weblog posts, right?
Assignee | ||
Comment 11•18 years ago
|
||
(In reply to comment #10)
> Reopened bug 302548 for getting planet.m.o fixed again.
>
> Hrm, preferring <link> to <guid isPermaLink="true">? If you're certain you want
> to do that, I think you'll be needing to wontfix the bug I'll be filing on you.
Show me real broken feeds or IE7 behavior (I should have checked that).
> But, yay! for the opportunity to argue about interpretations of the RSS spec
> based on weblog posts, right?
Won't be arguing those.
Assignee | ||
Comment 12•18 years ago
|
||
P.S. -- RSS1 and Atom don't have this problem (as phil knows)
Comment 13•18 years ago
|
||
Robert
By this going away I assume you mean thanks to bug 353434?
I had noticed this problem recently but had made the assumption above because feed preview had no such problem.
Comment 14•18 years ago
|
||
Thank Atom, everyone I knew who would be broken by preferring <link> (link blogs with comments enabled, with <link> being the linked-to URL and <guid> the permalink for the post about it) is no longer producing RSS, so parse them as you like, I can't be bothered to file on hypothetical wrongness.
Assignee | ||
Comment 15•18 years ago
|
||
(In reply to comment #13)
>
> By this going away I assume you mean thanks to bug 353434?
Yes. That will fix it for Places.
Reporter | ||
Comment 16•18 years ago
|
||
> separate bug for the fact that the tab loads stop?
see bug #356341
Assignee | ||
Comment 17•18 years ago
|
||
this should be fixed now that we use the toolkit feed parser for livemarks. true?
Assignee | ||
Updated•18 years ago
|
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•