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

Target page namespace missing for some log operations #63

Open
Ostrzyciel opened this issue Jan 30, 2022 · 3 comments
Open

Target page namespace missing for some log operations #63

Ostrzyciel opened this issue Jan 30, 2022 · 3 comments
Assignees
Labels

Comments

@Ostrzyciel
Copy link

Extension version: current main
MW version: 1.36.3

I've noticed that for the page restore log and page move log, the target titles in the Discord RC feed have missing namespaces. The links direct the user to non-existent pages in the main namespace.

Examples (unfortunately in Polish, sorry):

In both cases the Użytkownik (User in Polish) namespace prefix is missing.

Restoring a page
image

vs the regular RC feed:
image

Moving a page
In this case the former page title is correct, the new one is missing the namespace prefix.
image

vs the regular RC feed:
image

@lens0021
Copy link
Member

lens0021 commented Feb 2, 2022

Unfortunately, I couldn't reproduce this. The attempts were done on:

  • MediaWiki: 1.36.3 (e7d1e9f)
  • PHP: 7.4.22 (fpm-fcgi)
  • SQLite: 3.27.2
  • DiscordRCFeed: 1.0.1 (6263811)

image

I am suspecting some extension has a conflict with DiscordRCFeed, but I am not sure.

@lens0021
Copy link
Member

lens0021 commented Feb 2, 2022

I've reproduced it. I will do more tests.
image

@Ostrzyciel
Copy link
Author

I can confirm that the temporary fix does indeed work :)

@lens0021 lens0021 self-assigned this Feb 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: To do
Development

No branches or pull requests

2 participants