FreshRSS / FreshRSS

A free, self-hostable news aggregator…
https://freshrss.org
GNU Affero General Public License v3.0
8.77k stars 780 forks source link

[Bug] opml/xml export does not contain css selector #6433

Closed sotn3m closed 1 month ago

sotn3m commented 1 month ago

Describe the bug

When downloading opml as xml from the feed management screen, the export misses css selector (frss:cssFullContent). image

It works correctly when exporting all subscriptions though.

To Reproduce

  1. Go to subscription management
  2. Choose a cog of a single feed that has CSS selector chosen.
  3. Click on "Download" button at the end of the sidebar.

Expected behavior

Complete file

FreshRSS version

1.23.2-dev

Environment information

Additional context

No response

math-GH commented 1 month ago

There are some settings missed in the export

Alkarex commented 1 month ago

Please try https://github.com/FreshRSS/FreshRSS/pull/6435

math-GH commented 1 month ago

fields, that will not be exported:

sotn3m commented 1 month ago

Two fields/attributes were added in comparison to my previous export:

Alkarex commented 1 month ago

I am not sure it makes sense to export all the parameters in OPML, but maybe a few more (but let's keep this issue focussed on bug fixing for 1.24 and open a new one for other aspects). The full SQLite-based export is there when there is a need to export everything. @math-GH Could you please provide more details for the CURLOPT_PROXY bug? (here or in another issue)

math-GH commented 1 month ago

I am not sure it makes sense to export all the parameters in OPML, but maybe a few more (but let's keep this issue focussed on bug fixing for 1.24 and open a new one for other aspects). The full SQLite-based export is there when there is a need to export everything.

IMHO we should have an overview which options will be exported and which not. Currently it looks like that all options will be exported to the OPML file. What do you think about this idea: add a link next to the OPML export button. The link goes to the documentation, that gives a better overview of exported fields.

math-GH commented 1 month ago

@math-GH Could you please provide more details for the CURLOPT_PROXY bug? (here or in another issue)

see #6438

Alkarex commented 1 month ago

What do you think about this idea: add a link next to the OPML export button

Good idea