Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Recent changes filter improvement (redirections) #228

Open
Kycko opened this issue Jul 22, 2017 · 2 comments
Open

Recent changes filter improvement (redirections) #228

Kycko opened this issue Jul 22, 2017 · 2 comments

Comments

@Kycko
Copy link

Kycko commented Jul 22, 2017

Hi again ^_^
I doesn't know how your code work, but it will be great if you add next if:

  • if a particular page has no language tag and it is a redirection to a page with language tag then show it under corresponding header

e.g. show USB-накопители in ru section.
Thanks

@kynikos kynikos added this to the 3.1.0 milestone Jul 22, 2017
@kynikos
Copy link
Owner

kynikos commented Jul 22, 2017

Hi, it's a good idea, but it would practically require to completely rewrite the plugin, because right now I'm simply reordering the list on the client, while to implement this I should first collect all the redirects in the list, then separately query the server to find their targets (which aren't shown in the RecentChanges), and finally filter the list using their targets.

I have some other reasons to rewrite the RecentChanges filter, e.g. displaying the entries in a table with some added functionality, so I'll keep this request, and probably eventually implement it as optional.

For completeness' sake, my secret i18n plan has always also included to enforce using a language tag on redirects too: it's not currently enforced, but the current guidelines don't forbid adding language tags to redirects, so you can consider adding them to the Russian redirects for forward compatibility :)

@lahwaacz
Copy link
Contributor

The funny thing about MediaWiki is that you can't list all redirecting pages and request their targets to be added to the query. You can only list the redirect targets and have the source pages added as their properties, such as this. Then you need to manually query all namespaces to get everything you need and transform the data into some usable form. I think that I still haven't wrapped my head around this properly... I have some old notes on my wiki page and in the code.

If these fancy things start to slow down, we could try to get through the cross-site request to localhost and have wiki-scripts serve its (future) local database/cache 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants