Last modified: 2012-11-11 16:17:32 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T42748, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 40748 - result format rss / not working as expected (1.8 beta 1)
result format rss / not working as expected (1.8 beta 1)
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
Semantic MediaWiki (Other open bugs)
unspecified
All All
: High major with 1 vote (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks: 39480
  Show dependency treegraph
 
Reported: 2012-10-03 21:13 UTC by [[kgh]]
Modified: 2012-11-11 16:17 UTC (History)
5 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description [[kgh]] 2012-10-03 21:13:08 UTC
Instead of creating a pop-up which allows to save the feed in the browser the user gets directed to Special:Ask
Comment 1 [[kgh]] 2012-10-04 09:39:34 UTC
A solution to this might as well result in withdrawing RSS from SMW entirely. Legacy support seems to be broken right now.
Comment 2 Jeroen De Dauw 2012-10-04 10:20:38 UTC
Well, we have the atom format now. If the atom format works with the same parameters as the rss one, we can just ditch the rss one and alias rss to atom. James, any thoughts?
Comment 3 MWJames 2012-10-04 12:06:41 UTC
The only additional option introduced with format=feed was the type parameter (which by default is set to rss) all others remained the same. The only thing to remember is that format=feed is deployed with SRF while format=rss is deployed with SMW.

But swapping the format should not cause any trouble for the output nor the parameters used. Just made the switch from format=rss to format=feed on [1].

[1] http://semantic-mediawiki.org/wiki/Semantic_MediaWiki
Comment 4 Nemo 2012-10-18 09:16:32 UTC
(In reply to comment #3)
> The only additional option introduced with format=feed was the type parameter
> (which by default is set to rss) all others remained the same. The only thing
> to remember is that format=feed is deployed with SRF while format=rss is
> deployed with SMW.

Not all installations do or can have SRF, so having a basic, working feed format in SMW seems a valid request. You should probably decide if you're able to provide and maintain it or you prefer to tell people to use SRF, transfering the burden on them.
Comment 5 MWJames 2012-11-07 04:00:14 UTC
Can we have a resolution on this one? 

The SMW inherent RSS printer is somewhat outdated (and does things in its own way) but we have a SRF feed printer which does pretty much the same job (but in a more sane way) and unless we migrate the SRF feed printer into the SWM RSS printer this bug will not solve itself.

This one goes either won't fix or someone invests the time the make SMW RSS work but I for one had my part in creating a working RSS/Atom printer, so I will no put any energy on the table to fix this one.
Comment 6 [[kgh]] 2012-11-07 08:37:53 UTC
In case of a WONTFIX, I come to believe that this is the preferred resolution for this bug, the result format RSS should be removed in SMW core.
Comment 7 Markus Krötzsch 2012-11-07 09:15:18 UTC
The builtin RSS printer has hopefully not been broken in any released version. I am using it for a long time, and I know that people subscribe to it. I don't mind changing this to atom if this has some improvements, but this seems unrelated to this bug (the internal improvements to the format do not account for being redirected to Special:Ask).

If this is the simplest solution, we can still move to atom instead. What would it mean for existing subscribers? Would their clients be unhappy?
Comment 8 [[kgh]] 2012-11-07 10:04:59 UTC
Currently we have:
* 1.7: RSS (no Atom) in SMW (working)
* 1.8: RSS (no Atom) in SMW (broken), RSS and Atom in SRF (working)

I guess this is now about providing working RSS and Atom either in SMW or in SRF. I do not think that it is necessary or advisable to have RSS both in SMW and SRF.

If RSS gets removed from SMW wikis will have to install SRF to use in. The queries will have to be adjusted anyway since the new name of the result format is feed instead of rss (which is good).
Comment 9 Jeroen De Dauw 2012-11-07 12:32:52 UTC
If I understand this correctly, people should be able to switch from the old to the new format without breaking stuff. If this is so, I vote for ditching the old one and putting the new one in SMW.
Comment 10 [[kgh]] 2012-11-07 14:34:24 UTC
Currently we have:
* 1.7: RSS (no Atom) in SMW (working)
* 1.8: RSS (no Atom) in SMW (broken), RSS and Atom in SRF (working)

I guess this is now about providing working RSS and Atom either in SMW or in SRF. I do not think that it is necessary or advisable to have RSS both in SMW and SRF.

If RSS gets removed from SMW wikis will have to install SRF to use in. The queries will have to be adjusted anyway since the new name of the result format is feed instead of rss (which is good).
Comment 11 [[kgh]] 2012-11-07 14:35:54 UTC
Pls disregard Comment 10. I do not know what happened here.
Comment 12 Markus Krötzsch 2012-11-08 07:59:58 UTC
I am happy with having the new code in SMW. Can we make it so that the format=rss works as before (using the new RSS printer)? Otherwise it will be a pain to upgrade this on wikis with such queries.
Comment 13 MWJames 2012-11-11 05:34:56 UTC
The new name is format=feed but format=rss still works as alias.

Move from SRF to SMW

https://gerrit.wikimedia.org/r/#/c/32847/

Eliminating the Feed printer from the SRF repository 

https://gerrit.wikimedia.org/r/#/c/32861/
Comment 14 [[kgh]] 2012-11-11 16:17:32 UTC
The change was merged [1]. Thus closing as RESOLVED FIXED. I will update the docu on smw.o soon. Thank you and cheers.

https://gerrit.wikimedia.org/r/gitweb?p=mediawiki/extensions/SemanticMediaWiki.git;a=commit;h=e678ada7331002748789811527bffc7797070337

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


Navigation
Links