View source for Module talk:Check DYK hook

==Script error==
{{ping|Frietjes|Mr. Stradivarius}}

This [https://en.wikipedia.org/w/index.php?title=Module:Check_DYK_hook&diff=990847290&oldid=985376245] apparently 'fixed' some script errors, but broke the checks. At least, for entries ending with <code><nowiki>{{-?}}</nowiki></code> and made [[:Category:Pages with a malformed DYK entry]] balloon to 160+ 'malformed' entries, which were perfectly well-formed. &#32;<span style="font-variant:small-caps; whitespace:nowrap;">[[User:Headbomb|Headbomb]] {[[User talk:Headbomb|t]] · [[Special:Contributions/Headbomb|c]] · [[WP:PHYS|p]] · [[WP:WBOOKS|b]]}</span> 15:17, 27 November 2020 (UTC)
:It's possible that it did fix the script, but that the check itself didn't handle those entries. But this is LUA, so I can't diagnose anything here much. &#32;<span style="font-variant:small-caps; whitespace:nowrap;">[[User:Headbomb|Headbomb]] {[[User talk:Headbomb|t]] · [[Special:Contributions/Headbomb|c]] · [[WP:PHYS|p]] · [[WP:WBOOKS|b]]}</span> 15:18, 27 November 2020 (UTC)
:: [[User:Headbomb|Headbomb]], yes, before the fix those 160+ entries were in [[:Category:Pages with script errors]] instead.  I think the regular expression requires some changes you may be able to test it with
Live
* <code><nowiki>"{{#invoke:Check DYK hook|isValidHook|1=... that director and screenwriter Travis Stevens paused renovations on his production company's house to film '''''[[Girl on the Third Floor]]'''''?}}"</nowiki></code> = "{{#invoke:Check DYK hook|isValidHook|1=... that director and screenwriter Travis Stevens paused renovations on his production company's house to film '''''[[Girl on the Third Floor]]'''''?}}"
* <code><nowiki>"{{#invoke:Check DYK hook|isValidHook|1=... that director and screenwriter Travis Stevens paused renovations on his production company's house to film '''''[[Girl on the Third Floor]]'''''{{-?}}}"</nowiki></code> = "{{#invoke:Check DYK hook|isValidHook|1=... that director and screenwriter Travis Stevens paused renovations on his production company's house to film '''''[[Girl on the Third Floor]]'''''{{-?}}}}"
Sandbox
* <code><nowiki>"{{#invoke:Check DYK hook/sandbox|isValidHook|1=... that director and screenwriter Travis Stevens paused renovations on his production company's house to film '''''[[Girl on the Third Floor]]'''''?}}"</nowiki></code> = "{{#invoke:Check DYK hook/sandbox|isValidHook|1=... that director and screenwriter Travis Stevens paused renovations on his production company's house to film '''''[[Girl on the Third Floor]]'''''?}}"
* <code><nowiki>"{{#invoke:Check DYK hook/sandbox|isValidHook|1=... that director and screenwriter Travis Stevens paused renovations on his production company's house to film '''''[[Girl on the Third Floor]]'''''{{-?}}}}"</nowiki></code> = "{{#invoke:Check DYK hook/sandbox|isValidHook|1=... that director and screenwriter Travis Stevens paused renovations on his production company's house to film '''''[[Girl on the Third Floor]]'''''{{-?}}}}"
: not sure why we are getting a script error for the live template even with the {{para|1}}. [[User:Frietjes|Frietjes]] ([[User talk:Frietjes|talk]]) 15:26, 27 November 2020 (UTC)
::[[User:Headbomb|Headbomb]], it looks like the "match" used for whitespace cleaning was returning "nil" in those cases but changing it to a mw.ustring.gsub ([//en.wikipedia.org/w/index.php?title=Module%3ACheck_DYK_hook&type=revision&diff=990969954&oldid=990966539 see here]) fixed it.  or at least fixed it so both cases are showing valid and no script errors. [[User:Frietjes|Frietjes]] ([[User talk:Frietjes|talk]]) 15:42, 27 November 2020 (UTC)
:::{{Re|Frietjes}} seems to work yup. Thanks. I could never have caught that. &#32;<span style="font-variant:small-caps; whitespace:nowrap;">[[User:Headbomb|Headbomb]] {[[User talk:Headbomb|t]] · [[Special:Contributions/Headbomb|c]] · [[WP:PHYS|p]] · [[WP:WBOOKS|b]]}</span> 15:46, 27 November 2020 (UTC)
::::[[User:Headbomb|Headbomb]], unfortunately, for the "#if:" parser an error is the same as "yes" which is probably why it wasn't noticed earlier. [[User:Frietjes|Frietjes]] ([[User talk:Frietjes|talk]]) 15:52, 27 November 2020 (UTC)
:::::{{ping|Frietjes|Headbomb}} Sorry, this was my mistake; I used <code>.*?</code> in the whitespace-trimming pattern, when it should have been <code>.-</code>. The <code>.*?</code> is how non-greedy matching works in PCRE regex, but Lua was interpreting that as a greedy match for any character, plus a literal question mark. So hooks that didn't have a literal question mark in (like ones containing {{tl|-?}}) didn't match the pattern, meaning the whitespace-trimming code returned nil. Unfortunately, all the test cases that tested the whitespace trimming code contained question marks, so none of them picked this up. I've fixed the problem in the module and added some extra test cases so that this will get caught next time. I've also added a {{para|hook}} parameter, which I thought seemed neater than {{para|1}} for dealing with hooks that might contain an equals sign. — '''''[[User:Mr. Stradivarius|<span style="color: #194D00; font-family: Palatino, Times, serif">Mr.&nbsp;Stradivarius</span>]]''''' <sup>[[User talk:Mr. Stradivarius|♪&nbsp;talk&nbsp;♪]]</sup> 12:42, 29 November 2020 (UTC)