Jump to content

Module talk:Gadgets

Page contents not supported in other languages.
From Wikipedia, the free encyclopedia
(Redirected from Module talk:Gadgets/sandbox)

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]