import subscriptions and export subscriptions differ about opml extension

VERIFIED FIXED in Thunderbird 11.0

Status

MailNews Core
Feed Reader
P2
minor
VERIFIED FIXED
6 years ago
6 years ago

People

(Reporter: Matthew L Daniel, Assigned: aceman)

Tracking

({polish})

Thunderbird 11.0
polish

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [duptome])

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
User-Agent:       Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/534.30 (KHTML, like Gecko) Chrome/12.0.742.91 Safari/534.30
Build Identifier: 5.0b1

When one uses "Manage Subscriptions", "Export", TB will save a file with the .opml extension. 

However, when one uses "Manage Subscriptions", "Import", the file selection dialog does not show the same .opml file. It works fine if one types the filename manually, but this is unexpected behavior. 

Reproducible: Always

Steps to Reproduce:
1. Please see the description
Component: Migration → Feed Reader
Product: Thunderbird → MailNews Core
QA Contact: migration → feed.reader
Whiteboard: [good first bug]
Status: UNCONFIRMED → NEW
Ever confirmed: true
Whiteboard: [good first bug] → [good first bug][duptome]
(Assignee)

Comment 1

6 years ago
So what would be the expected behavior here?
This happens because the exported file is saved as .opml. But the import dialog has 3 filters for the file list. The default happens to be "XML files" so the previously created file is not directly visible. Changing the filter to "OPML files" or "All Files" makes the file visible.

So what is requested here? Should OPML be the default filter? Reorder the filter list to "OPML, XML, All" ?
Keywords: uiwanted
Version: unspecified → 5.0
(Reporter)

Comment 2

6 years ago
(In reply to :aceman from comment #1)
> So what would be the expected behavior here?

The expected behavior is that if one exports from TB, one should be able to import from TB without having to alter something on the TB import dialog. The current setup seems to imply that TB feels .xml is the "primary" import type, and .opml is an alternative. However, the export emits .opml files.

> So what is requested here? Should OPML be the default filter? Reorder the
> filter list to "OPML, XML, All" ?

My suggestion is to choose from the most attractive of the following two choices:

1. Change the import filter order such that the default export format appears first
2. Change the export extension to be .xml to be aligned with the import file list
(Assignee)

Comment 3

6 years ago
I do not know what format the exported file actually is and what is the difference in content between OPML and XML.

I am able to code choice 1. from comment 2. But need confirmation from module owner that this would be accepted.
Ludovic, can you arrange it?
go for it.
(Assignee)

Updated

6 years ago
Assignee: nobody → acelists
Severity: normal → minor
Status: NEW → ASSIGNED
Keywords: polish
Priority: -- → P2
(Assignee)

Comment 5

6 years ago
Created attachment 572514 [details] [diff] [review]
changes the order of file selection dialog filters to "OMPL, XML, All"
Attachment #572514 - Flags: review?(myk)
(Assignee)

Updated

6 years ago
OS: Windows 7 → All
Hardware: x86 → All
Comment on attachment 572514 [details] [diff] [review]
changes the order of file selection dialog filters to "OMPL, XML, All"

I double-checked the recommended file extension for OPML files, and it seems to be .opml, per the OPML 1 and 2 specs (which do not explicitly state this but do link to examples with that extension).

So this is the right fix and works well in my testing (although importing, exporting, and then re-importing an OPML file throws: "Either MyDailyFeeds.opml is not a valid OPML file or there was an error importing the file."; but that's a different bug!).
Attachment #572514 - Flags: review?(myk) → review+
Ludovic: does this need super-review, or can it be committed?
(Assignee)

Comment 8

6 years ago
reply to Myk Melez [:myk] [@mykmelez] from comment #6)
> So this is the right fix and works well in my testing (although importing,
> exporting, and then re-importing an OPML file throws: "Either
> MyDailyFeeds.opml is not a valid OPML file or there was an error importing
> the file."; but that's a different bug!).

Maybe has problems with the fact that there already exist the same feeds.
Does importing into another profile show the same error?
(In reply to Myk Melez [:myk] [@mykmelez] from comment #7)
> Ludovic: does this need super-review, or can it be committed?

This can be committed as there is no API change we don't need sr.
Keywords: checkin-needed
(In reply to :aceman from comment #8)
> Maybe has problems with the fact that there already exist the same feeds.

Yes, I suspect that's the problem, as re-importing works after I unsubscribe from those feeds.
(Assignee)

Comment 11

6 years ago
(In reply to Myk Melez [:myk] [@mykmelez] from comment #6)
> So this is the right fix and works well in my testing (although importing,
> exporting, and then re-importing an OPML file throws: "Either
> MyDailyFeeds.opml is not a valid OPML file or there was an error importing
> the file."; but that's a different bug!).

Somebody filed this as bug 702527.
Keywords: uiwanted
Whiteboard: [good first bug][duptome] → [duptome]
(Assignee)

Updated

6 years ago
Duplicate of this bug: 516949
Checked in: http://hg.mozilla.org/comm-central/rev/92b016a6e279
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 11.0
(Assignee)

Comment 14

6 years ago
(In reply to :aceman from comment #11)
> (In reply to Myk Melez [:myk] [@mykmelez] from comment #6)
> > So this is the right fix and works well in my testing (although importing,
> > exporting, and then re-importing an OPML file throws: "Either
> > MyDailyFeeds.opml is not a valid OPML file or there was an error importing
> > the file."; but that's a different bug!).
> 
> Somebody filed this as bug 702527.

Typo, it is 702542 -> 307629.
(Assignee)

Updated

6 years ago
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.