Module talk:Gadgets

Latest comment: 4 days ago by SD0001 in topic Protected edit request on 9 October 2024

Protected edit request on 26 January 2024

edit

Please sync the changes from Module:Gadgets/sandbox (diff). The following change(s) have been done:

SD0001 (talk) 17:50, 26 January 2024 (UTC)Reply

  Done * Pppery * it has begun... 19:59, 26 January 2024 (UTC)Reply

Protected edit request on 7 May 2024

edit

Please sync from Module:Gadgets/sandbox (diff). This fixes "used by −1 users" at MediaWiki:Gadget-popups.js. Nardog (talk) 11:07, 7 May 2024 (UTC)Reply

  Donexaosflux Talk 20:25, 7 May 2024 (UTC)Reply

Some issues in module

edit

@SD0001: currently, this module seems to remove all spaces, even where they are now legitimate, like the new |categories= keyword. See, for example, ru:MediaWiki:Gadgets-definition#ondemand. I tried fixing it locally by replacing the trimming with mw.text.trim but that makes Module:Validate gadgets to no longer work correctly with items containing spaces, see the top of ru:MediaWiki:Gadgets-definition.

Additionally, Wikipedia:GUS2Wiki seems to be updated differently in other wikis, where CSV also includes info on active users. I believe that can be fixed with this change. stjn 01:07, 9 October 2024 (UTC)Reply

Feel free to propose the edits here using {{sudo}}. I don't have direct edit access either. – SD0001 (talk) 03:43, 9 October 2024 (UTC)Reply
The problem was that I didn’t really know how to solve this issue. Now I think it just requires multiple edits: remove trimming in Module:Gadgets and then add trimming to Module:Validate gadgets. But maybe you as the author can propose a better solution. stjn 23:44, 9 October 2024 (UTC)Reply

Protected edit request on 9 October 2024

edit

Please merge this edit into the module code. It supports the re-use of the module in other wikis, as clarified by the comment in ru:Обсуждение участника:Alexis Jazz. stjn 23:46, 9 October 2024 (UTC)Reply

Stjn, this could be done more neatly by adjusting regular expression above it. I'm unsure about the regex syntax within Lua, but I'm thinking you could replace ,(%S+)\n with something like ,([0-9]*)[\n,]. (untested)Alexis Jazz (talk or ping me) 03:19, 10 October 2024 (UTC)Reply
Yes, just changing (%S+)\n in the Lua pattern to (%d+) should be enough – SD0001 (talk) 12:06, 10 October 2024 (UTC)Reply