View source for Check for unknown parameters

{{Permanently protected}}
{{oldtfdfull|date= 2019 December 30 |result=no consensus |disc=Module:Check for deprecated parameters|merge=self}}
{{talk header}}
{{WikiProject banner shell|
{{WikiProject Templates|module_for_template_maintenance=y}}
}}
{{User:MiszaBot/config
| algo=old(90d)
| archive=Module talk:Check for unknown parameters/Archive %(counter)d
| counter=1
| maxarchivesize=900K
| archiveheader={{aan}}
| minthreadsleft=2
| minthreadstoarchive=1
}}

== Suggested enhancement ==

{{see also|Module_talk:WikiProject banner}}
We are using this module on [[Module:WikiProject banner]]. We first check if [[:Category:Pages using WikiProject PROJECT with unknown parameters]] exists and if not, then we use [[:Category:WikiProject templates with unknown parameters]] instead. The problem is that this is causing thousands of links to non-existent categories to be recorded, which can be seen in [[Special:WantedPages]]. My suggestion is as follows. The module can accept an additional argument called <code>fallback</code> which is a category which will be used if the one specified in <code>unknown</code> does not exist. In this way we can check existence of that category only when unknown parameters are discovered, not in every single case. &mdash;&nbsp;Martin <small>([[User:MSGJ|MSGJ]]&nbsp;·&nbsp;[[User talk:MSGJ|talk]])</small> 14:19, 4 September 2024 (UTC)
:What is it that I'm missing?  If the problem is caused by something that happens in [[Module:WikiProject banner]], that is where the fix should be applied.  Adding miscellaneous one-off patches to this module is not a good idea.  When you switch to the default category, you know that the preferred category does not exist so why link to it?  Link to the preferred category only when it exists.
:—[[User:Trappist the monk|Trappist the monk]] ([[User talk:Trappist the monk|talk]]) 15:35, 4 September 2024 (UTC)
::I'll try and explain better. Or you can look at the code at around [[Module:WikiProject banner#L-831]]. We have to check existence before calling this module. Checking existence adds a link to the page. Therefore every transclusion of {{tl|WikiProject Lepidoptera}} is generating a link to [[:Category:Pages using WikiProject Lepidoptera with unknown parameters]] which is not so good. I would prefer to only check if the category exists if there are some unknown parameters. That check can only happen in this module. Or perhaps you have a better idea &mdash;&nbsp;Martin <small>([[User:MSGJ|MSGJ]]&nbsp;·&nbsp;[[User talk:MSGJ|talk]])</small> 21:02, 4 September 2024 (UTC)
:::I haven't taken any real time to study the code (it has taken me more time to write this than I spent looking at the code) but [[Module:Check_for_unknown_parameters#L-113|Module:Check for unknown parameters line 113]] appears to return a concatenated string of unknown parameters (apparently without delimiters).  If there are no unknowns then the <syntaxhighlight lang="lua" inline="1">table.concat(res)</syntaxhighlight> returns an empty string.  So, it looks like <syntaxhighlight lang="lua" inline="1">p._check(args, pargs)</syntaxhighlight> returns:
::::<code>nil</code> when either of <code>args</code> and <code>pargs</code> is not a table
::::<code>''empty string''</code> when there are no unknown parameters
::::some sort of list of unknown parameters
:::If this is true then you should be checking the returned value from <syntaxhighlight lang="lua" inline="1">require('Module:Check for unknown parameters')._check(parameters, parent_args)</syntaxhighlight> before you fiddle about with categories.
:::—[[User:Trappist the monk|Trappist the monk]] ([[User talk:Trappist the monk|talk]]) 22:22, 4 September 2024 (UTC)
::::Great, thanks. I'll look into that &mdash;&nbsp;Martin <small>([[User:MSGJ|MSGJ]]&nbsp;·&nbsp;[[User talk:MSGJ|talk]])</small> 08:55, 5 September 2024 (UTC)
::::Yes that works. Unfortunately it means that I need to call the module twice: the second time after deciding which category to use. But this will only happen if there are any unknown parameters so will not affect performance unduly. &mdash;&nbsp;Martin <small>([[User:MSGJ|MSGJ]]&nbsp;·&nbsp;[[User talk:MSGJ|talk]])</small> 09:06, 5 September 2024 (UTC)
:::::Add comments to your code.  When you are crushed by a steamroller while jaywalking Main Street, whoever comes after you to maintain that module can then know why you did what you did.
:::::—[[User:Trappist the monk|Trappist the monk]] ([[User talk:Trappist the monk|talk]]) 11:48, 5 September 2024 (UTC)

== Bulk include all mapframe parameters ==

Rather than having to copy and paste all 20+ [[Module:Infobox mapframe]] parameters into the check at the bottom of the page, I’m wondering if we can add a hook of some kind? Something like {{para|mapframe_params|on}} that will automatically include the hardcoded mapframe parameters? Alternatively could we create a {{tl|Infobox mapframe/parameters}} that is just a list of parameters and then pass that in? IF the parameters for [[Module:Infobox mapframe]] were to change for some reason, that would involve updating a LOT of infoboxes. This would make that process easier. —'''[[User:Zackmann08|<span style="color:#00ced1">Zack</span><span style="color:#007F94">mann</span>]]''' (<sup>[[User_talk:Zackmann08|Talk to me]]</sup>/<sub>[[Special:Contributions/Zackmann08|<span style="color:orange;">What I been doing</span>]]</sub>) 22:23, 30 September 2025 (UTC)

:Sounds sensible &mdash;&nbsp;Martin <small>([[User:MSGJ|MSGJ]]&nbsp;·&nbsp;[[User talk:MSGJ|talk]])</small> 07:47, 1 October 2025 (UTC)
::{{ping|Frietjes}} I know you put a lot of work into this module. Any thoughts? 
::Also a quick search reveals that at least [https://en.wikipedia.org/w/index.php?title=Special:Search&limit=250&offset=0&ns10=1&search=insource%3A%2Finfobox+mapframe%2Fi+-intitle%3A%2F%5C%2F%2F 100] templates use [[Module:Infobox mapframe]] so it would definitely help. '''[[User:Zackmann08|<span style="color:#00ced1">Zack</span><span style="color:#007F94">mann</span>]]''' (<sup>[[User_talk:Zackmann08|Talk to me]]</sup>/<sub>[[Special:Contributions/Zackmann08|<span style="color:orange;">What I been doing</span>]]</sub>) 09:28, 1 October 2025 (UTC)
::: {{u|Zackmann08}} this seems reasonable, so long as this is the only module that we are supporting.  if there are going to be multiple modules that we want to support, we could provide a link to a data page.  something like {{para|params_data|Module:Infobox mapframe/parameters}} or {{para|params_data1|Module:Infobox mapframe/parameters}} or whatever.  back in the early days of this module, it was suggested that it would be more efficient if the list of parameters lived in module space and were loaded using <code>mw.loadData</code> but this would only really help if the template using this module were being called multiple times on each page, so not as big of an issue for infoboxes. [[User:Frietjes|Frietjes]] ([[User talk:Frietjes|talk]]) 14:54, 2 October 2025 (UTC)
::::I’m a fan of either solution. I guess the second solution provides support for future expansions better but that could always been performed as a [[refactor]] down the line if needed. '''[[User:Zackmann08|<span style="color:#00ced1">Zack</span><span style="color:#007F94">mann</span>]]''' (<sup>[[User_talk:Zackmann08|Talk to me]]</sup>/<sub>[[Special:Contributions/Zackmann08|<span style="color:orange;">What I been doing</span>]]</sub>) 18:06, 2 October 2025 (UTC)
:::::{{ping|Frietjes|MSGJ}} I have mocked up a solution in the sandbox. Not married to the parameter names of course but I did do some testing in my own sandbox and it seems to work just fine. All I did was {{mlx|Check for unknown parameters/sandbox|check|mapframe_args{{=}}y}}. 
:::::<s>Now there is a drawback I noticed... If you look, for example, at {{tl|Infobox zoo}} you will see in the implementation the following:
:::::The significance of this is that some of those parameters, {{para|mapframe-marker}}, {{para|mapframe-width}} and a few others are NOT valid parameters to be passed in to {{tl|Infobox zoo}} as they are hardcoded in the Infobox code. 
:::::So we are left with a few options
:::::# Use {{para|mapframe_args|y}} and just not care that there are a few extra params included (seems like a slippery slope to me but probably not a big deal)
:::::# Don't use the {{para|mapframe_args|y}} and go back to the old school way of copying and pasting all the params in the check for unknown list
:::::# Add an additional {{para|_exclude}} which takes a CSV of mapframe_args to NOT pass to the check. (This idea takes from [[Module:Template wrapper]].)
:::::Let me know your thoughts. Happy to experiment with option 3 if there is interest.</s> Also if this is implemented, I'll commit to updating the necessary documentation as well as taking on cleaning up the actual calls to the module so the new param is used. --'''[[User:Zackmann08|<span style="color:#00ced1">Zack</span><span style="color:#007F94">mann</span>]]''' (<sup>[[User_talk:Zackmann08|Talk to me]]</sup>/<sub>[[Special:Contributions/Zackmann08|<span style="color:orange;">What I been doing</span>]]</sub>) 07:06, 6 October 2025 (UTC)
::::::{{ping|Zackmann08}} you may not be understanding the obscure way that [[Module:Infobox mapframe]] is parsing the arguments with getArgs in autoArgs(). the <code>parentFirst=true</code> flag is set, so that the parameter passed into {{tl|Infobox zoo}} ''overrides'' the parameter provided in the template code itself. In other words, {{para|mapframe-marker|zoo}} is simply the default. If {{para|mapframe-marker}} is set to something else, it will override zoo. This is true with any of the other parameters. Whomever wrote the template code didn't realize this. I'll fix the code to make it less obscure, but you don't have to do anything. — [[User:Hike395|hike395]] ([[User talk:Hike395|talk]]) 12:18, 6 October 2025 (UTC)
:::::::{{ping|Hike395}} thanks for the explanation! That dramatically simplifies things! I have struck my previous comments as they are no longer relevant. - '''[[User:Zackmann08|<span style="color:#00ced1">Zack</span><span style="color:#007F94">mann</span>]]''' (<sup>[[User_talk:Zackmann08|Talk to me]]</sup>/<sub>[[Special:Contributions/Zackmann08|<span style="color:orange;">What I been doing</span>]]</sub>) 17:03, 6 October 2025 (UTC)