View source for Module talk:JCW

{{WikiProject banner shell|
{{WikiProject Academic Journals}}
}}
== p.selected ==

I'm looking at the request at [[Wikipedia:Village pump (technical)#Need a LUA person for something (probably) pretty simple|WP:VPT]]. However I can't help but fiddle with code and I'm afraid I would also adjust several mostly style issues. First I need to understand what p.selected and the following are supposed to do.
<syntaxhighlight lang="lua">
if doi then
    text = text..string.format("\n** <code>\{\{doi\|[https://en.wikipedia.org/w/index.php?sort=relevance&title=Special%%3ASearch&profile=advanced&fulltext=1&advancedSearch-current={}&ns0=1&ns118=1&search=insource%%3A%%2F%s%%5C%%2F%%20*%%2F %s]\}\}<\/code>", doi, doi)
    text = mw.ustring.gsub(text, "%%2F10%.(%d)", "\/10\\.%1")
end
</syntaxhighlight>
Is there an example of p.selected being used and its output (possibly from [[Special:ExpandTemplates]])?

The above code puts a URL into text with the doi value inserted in two places. It then does the gsub to possibly insert <code>/10\</code> in the URL (I think I decoded that correctly, perhaps not). What is a typical doi value? Surely it never includes <code>%2F</code>? I'm thinking that the doi value, if necessary, should be changed with a gsub, and the result inserted into the URL in two locations. There is no need to put the URL through the gsub? Would that work? [[User:Johnuniq|Johnuniq]] ([[User talk:Johnuniq|talk]]) 02:59, 5 January 2020 (UTC)

{{re|Johnuniq}} Basically, <code><nowiki>{{JCW-selected|Foobar|Rawr|doi1=10.4321|doi2=10.3216|doi3=10.5446}}</nowiki></code> gives {{JCW-selected|Foobar|Rawr|doi1=10.4321|doi2=10.3216|doi3=10.5446}}

This could be equivalently be rendered with {{tl|doi prefix}} like so
*[[Foobar]]
**[[Rawr]]
**{{doi prefix|10.4321|mode=t}}
**{{doi prefix|10.3216|mode=t}}
**{{doi prefix|10.5446|mode=t}}
But the module choked on putting stuff templates last time around, so I had to URL encode a bunch of stuff so <code>insource:/10\.1234/\ *</code> search links work. &#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> 03:06, 5 January 2020 (UTC)

== Imprint/Parents ==

{{Ping|Johnuniq|Galobtter}}, I'm trying to introduce two new iterative parameters {{para|imprint#}} and {{para|parent#}} in {{tl|JCW-selected}}

The idea being that 
<pre>
{{JCW-selected|Foobar|YoYeah|imprint1=Imprint1|imprint2=Imprint2|imprint3=...|doi1=10.4321}}
{{JCW-selected|Foobar|parent1=Barfoo1|parent2=Barfoo2|parent3=...|doi1=10.4321}}
</pre>
would give
*[[Foobar]] / ''[[Imprint1]]'' /  ''[[Imprint2]]'' / ''[[...]]''
**[[YoYeah]]
**{{doi prefix|10.4321|mode=t}}
and
*[[Foobar]] / ''[[Parent1]]'' /  ''[[Parent2]]'' / ''[[...]]''
**[[YoYeah]]
**{{doi prefix|10.4321|mode=t}}

I tried [https://en.wikipedia.org/w/index.php?title=Module%3AJCW&type=revision&diff=941050031&oldid=936809784], but it made things explode and time out. A bit of help would be nice here. &#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> 09:06, 16 February 2020 (UTC)
:{{Ping|Johnuniq|Galobtter}} anything you can do to help here? &#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> 19:03, 6 March 2020 (UTC)
::@[[User:Headbomb|Headbomb]]: I'll have a look while hoping that [[User:Galobtter|Galobtter]] either says they will do it, or gives me a couple of days to play around (I imagine neither of us wants to do some work and then find the other finishes first). What is the following trying to do?
<pre>
if mw.title.getCurrentTitle().fullText == 'User:JL-Bot/Publishers.cfg' or 'User:JL-Bot/Maintenance.cfg' then
    source = nil
end
</pre>
::Is it trying to see if the current page is one of the two user cfg pages? The <code>if</code> condition evaluates to <code>User:JL-Bot/Maintenance.cfg</code> if the <code>==</code> is true, or to <code>false</code> otherwise. I need to understand stuff before diving in. [[User:Johnuniq|Johnuniq]] ([[User talk:Johnuniq|talk]]) 00:44, 7 March 2020 (UTC)
:::{{u|Johnuniq}}, feel free to (and thanks for) work on this - I really don't have much time for wikistuff atm so won't really be able to help with this. [[User:Galobtter|Galobtter]] ([[User talk:Galobtter|pingó mió]]) 04:10, 7 March 2020 (UTC)
::Where can I play? What-links-here suggests [[Module:JCW]] is not used much. What is its status? Can I just edit it without worrying if something will break? Are there any testcases? There should be at least one test in a sandbox with the wikitext of a full example call to {{tl|JCW-selected}} using all wanted bells and whistles, and the wikitext of what the output should be. That would help me understand what it's trying to do, and would be needed for testing. Preferably without red links. [[User:Johnuniq|Johnuniq]] ([[User talk:Johnuniq|talk]]) 00:58, 7 March 2020 (UTC)
{{Re|Johnuniq}} the general idea is that there are four pages where this is called
*[[User:JL-Bot/Citations.cfg]] via {{tl|JCW-exclude}}
*[[User:JL-Bot/Questionable.cfg]] via {{tl|JCW-selected}} and {{tl|JCW-pattern}}
*[[User:JL-Bot/Publishers.cfg]] via {{tl|JCW-selected}} and {{tl|JCW-pattern}}
*[[User:JL-Bot/Maintenance.cfg]] via {{tl|JCW-pattern}}

I believe that piece of code is simply telling that {{para|source}} of {{tl|JCW-selected}} is simply irrelevant on [[User:JL-Bot/Publishers.cfg]] and [[User:JL-Bot/Maintenance.cfg]]. I think this is mostly because {{tl|JCW-selected}} ''could'' be used on that page, but currently isn't. 

As far as playing is concern, you can play pretty much on the live versions of the templates and modules and the below examples generated from the above code.
{{JCW-selected|Foobar|YoYeah|imprint1=Imprint1|imprint2=Imprint2|imprint3=...|doi1=10.4321}}
{{JCW-selected|Foobar|parent1=Barfoo1|parent2=Barfoo2|parent3=Barfoo3|doi1=10.4321}}
JL-Bot only cares about the plain  text of the .cfg pages. Only the people maintaining the pages (i.e. me) might care about how they render, and it's really not a big deal if they're temporarily borked. &#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> 01:39, 7 March 2020 (UTC)

Or if you want a more concrete example
{|-
|Code || Live || Goal
|-
|
<pre>
{{JCW-selected
 |Hindawi Publishing Corporation
 |imprint1=International Scholarly Research Notices
 |doi1=10.1155
 |doi2=10.3814
 |doi3=10.4061
 |doi4=10.5402
 |doi5=10.6064
 |doi6=10.7167
 |doi7=10.7217
}}
{{JCW-selected
 |International Scholarly Research Notices
 |parent1=Hindawi Publishing Corporation
 |doi1=10.5402
}}
</pre>
|
{{JCW-selected |Hindawi Publishing Corporation |imprint1=International Scholarly Research Notices |doi1=10.1155 |doi2=10.3814 |doi3=10.4061 |doi4=10.5402 |doi5=10.6064 |doi6=10.7167 |doi7=10.7217}}
{{JCW-selected |International Scholarly Research Notices |parent1=Hindawi Publishing Corporation |doi1=10.5402}}
|
*[[Hindawi Publishing Corporation]] / ''[[International Scholarly Research Notices]]''
**{{doi-prefix|10.1155|mode=t}}
**{{doi-prefix|10.3814|mode=t}}
**{{doi-prefix|10.4061|mode=t}}
**{{doi-prefix|10.5402|mode=t}}
**{{doi-prefix|10.6064|mode=t}}
**{{doi-prefix|10.7167|mode=t}}
**{{doi-prefix|10.7217|mode=t}}
* [[International Scholarly Research Notices]] / ''[[Hindawi Publishing Corporation]]''
**{{doi-prefix|10.5402|mode=t}}
|}
&#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> 01:48, 7 March 2020 (UTC)
:@[[User:Headbomb|Headbomb]]: I'll have a look at it but I will also refactor the code. Details matter to me and I want to know if you really want the first of the following for one of the results, or should it be the second? The first has the &lt;code> font for the 10.1155 search link, while the second uses the normal font.
:*<code>&#123;&#123;doi|[https://en.wikipedia.org/w/index.php?sort=relevance&title=Special%3ASearch&profile=advanced&fulltext=1&advancedSearch-current={}&ns0=1&ns118=1&search=insource%3A/10\.1155%5C%2F%20*%2F 10.1155]}}</code>
:*<code>&#123;&#123;doi|</code>[https://en.wikipedia.org/w/index.php?sort=relevance&title=Special%3ASearch&profile=advanced&fulltext=1&advancedSearch-current={}&ns0=1&ns118=1&search=insource%3A/10\.1155%5C%2F%20*%2F 10.1155]<code>}}</code>
:I will have some other questions, and will put a version of what you posted above at [[User:Johnuniq/sandbox2]] adjusted to suit the tests I will use. [[User:Johnuniq|Johnuniq]] ([[User talk:Johnuniq|talk]]) 02:55, 7 March 2020 (UTC)
::{{Re|Johnuniq}} Fixed the fonts. I think? There was an inconsistency, but it didn't look like your example. &#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> 02:59, 7 March 2020 (UTC)
:::I think you're saying you want the first. [[User:Johnuniq|Johnuniq]] ([[User talk:Johnuniq|talk]]) 03:36, 7 March 2020 (UTC)

=== Questions ===
@[[User:Headbomb|Headbomb]]: The search URL from the module is slightly different from {{tl|doi-prefix}}'s output. To see this, copy the URL shown in the above examples and paste it into an editor. The first of the following lines is the URL parameter from the module, and the second is from the template. I've tried to highlight the two differences.
:?sort=relevance&title=Special{{red|%3A}}Search&profile=advanced&fulltext=1&advancedSearch-current={}&ns0=1&ns118=1&search=insource%3A{{red|/10\}}.1155%5C%2F%20*%2F
:?sort=relevance&title=Special{{red|%%3A}}Search&profile=advanced&fulltext=1&advancedSearch-current={}&ns0=1&ns118=1&search=insource%3A{{red|%2F10%5C}}.1155%5C%2F%20*%2F
Which is wanted? Perhaps there is a good reason for what the template outputs and the second should be used (although they seem to do the same thing)? [[User:Johnuniq|Johnuniq]] ([[User talk:Johnuniq|talk]]) 07:14, 7 March 2020 (UTC)
::I believe the difference is mostly in how urls are handled by modules and templates, but that they both produce the same thing in the end? It's also possible that there's a mistake somewhere. &#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> 07:17, 7 March 2020 (UTC)
:::The goal is a <code>insource:/10\.1155\/ */</code> search in the main and draft spaces, so whatever achieves 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> 07:22, 7 March 2020 (UTC)
:::There was an error in the doi-prefix template ({{red|%%3A}}→{{red|%3A}}), although it didn't affect anything. The second part is a module vs template difference, I think. {{red|%2F10%5C}} is simply the html encoded version of {{red|/10\}}. &#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> 07:27, 7 March 2020 (UTC)
::::Also I notice that your recent changes to the module have made (Unknown) appear when it wasn't there before. Might not be important, since this is just the talk page of the module and is still suppressed on the relevant user pages. Might be worth switching to a whitelist to allow, instead of a blacklist to disallowed. &#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> 07:33, 7 March 2020 (UTC)
:::::After my edit, the source is nil (not displayed) at [[User:JL-Bot/Publishers.cfg]] and [[User:JL-Bot/Maintenance.cfg]] and is either the given source or "Unknown" at other pages. I can easily do a whitelist instead. On what pages should the given source or "Unknown" be displayed? [[User:Johnuniq|Johnuniq]] ([[User talk:Johnuniq|talk]]) 08:16, 7 March 2020 (UTC)
::::::On [[User:JL-Bot/Questionable.cfg]] is the only place it's relevant. &#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> 17:58, 7 March 2020 (UTC)
:::::I have done a first pass at implementing the requested changes. I will do a more thorough check in a day or two. Some behavior is different, for example the module will now show an error if there is no target or if both doi and doi1 parameters are used. Testing needed! [[User:Johnuniq|Johnuniq]] ([[User talk:Johnuniq|talk]]) 08:38, 7 March 2020 (UTC)
::::::From a brief look, everything seems to work just fine. I haven't tried intentionally breaking things though. &#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> 18:04, 7 March 2020 (UTC)
:::::::I have finished tweaking the code. More testing soon would be good because if more than a few days elapse before another request I will have forgotten what it's all about and will be harder to move. [[User:Johnuniq|Johnuniq]] ([[User talk:Johnuniq|talk]]) 00:00, 8 March 2020 (UTC)
:::::::::Looks good, thanks. &#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> 00:19, 8 March 2020 (UTC)

== Update for JCW-doi-redirects ==

{{ping|Johnuniq}}, could you do a tweak where the DOI prefix links generated by <code><nowiki>{{JCW-doi-redirects|Foobar|10.1001|10.1002}}</nowiki></code>
{{JCW-doi-redirects|Foobar|10.1001|10.1002}}
and those generated by <code><nowiki>{{JCW-selected|Foobar|10.1001|10.1002}}</nowiki></code>
{{JCW-selected|Foobar|doi1=10.1001|doi2=10.1002}}

are the same (i.e. in the style of JCW-selected style), this way it's easy to search for which articles are using those DOIs prefixes? In {{tl|JCW-doi-redirects}}, {{para|1}} will always be a regular article link ([[Foobar]]), while {{para|2/3/4...}} will always be DOIs prefixes.
&#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> 01:42, 29 October 2020 (UTC)
:I have totally forgotten what this is all about and need simple instructions. The first example shows 10.1001 as a link to an article (which is a redirect). The second shows the doi template syntax and links to an enwiki search. I don't know what "DOI prefix links" are generated. It would be clearer for me if you were to show the wikitext for what the first example should output. Also show any other changes needed with sample input wikitext and the wanted output. I can use that to understand the problem and as testcases. [[User:Johnuniq|Johnuniq]] ([[User talk:Johnuniq|talk]]) 06:43, 29 October 2020 (UTC)
::{{Re|Johnuniq}} Well, there's the examples above. The current version of {{tl|JCW-doi-redirects}}, which currently outputs the non-desired [[10.1001]]. The desired output is <code><nowiki>{{</nowiki>doi{{!}}[https://en.wikipedia.org/w/index.php?sort=relevance&title=Special%3ASearch&profile=advanced&fulltext=1&advancedSearch-current={}&ns0=1&ns118=1&search=insource%3A/10\.1001%5C%2F%20*%2F 10.1001]}}</code>, from a template that already has the functionality ({{tl|JCW-selected}}). &#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> 06:48, 29 October 2020 (UTC)
::If you want to see it used in concrete examples, see [[User:JL-Bot/Questionable.cfg/Publishers#B]]. Compare the entries for the 'Baishideng Publishing Group', with plain links, with those from Bentham Science Publishers, with search links. Or those from the two entries of the European Mathematical Society in [[User:JL-Bot/Publishers.cfg#E]] (one with plain links, the other with search links). &#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> 06:53, 29 October 2020 (UTC)
:::I'll try to look in about 24 hours. Remind me if I'm not back in 36 hours. [[User:Johnuniq|Johnuniq]] ([[User talk:Johnuniq|talk]]) 08:17, 29 October 2020 (UTC)
{{od|3}}Both {{tl|JCW-doi-redirects}} and {{tl|JCW-selected}} call <code><nowiki>{{#invoke:JCW|selected}}</nowiki></code> and the result from one is exactly the same as from the other. Currently, there is a difference between positional parameters and named parameters used to specify doi values. Examples:
*<code><nowiki>{{JCW-doi-redirects|Foobar|10.1001|10.1002}}</nowiki></code> → {{JCW-doi-redirects|Foobar|10.1001|10.1002}}
*<code><nowiki>{{JCW-doi-redirects|Foobar|doi1=10.1001|doi2=10.1002}}</nowiki></code> → {{JCW-doi-redirects|Foobar|doi1=10.1001|doi2=10.1002}}
*<code><nowiki>{{JCW-selected|Foobar|10.1001|10.1002}}</nowiki></code> → {{JCW-selected|Foobar|10.1001|10.1002}}
*<code><nowiki>{{JCW-selected|Foobar|doi1=10.1001|doi2=10.1002}}</nowiki></code> → {{JCW-selected|Foobar|doi1=10.1001|doi2=10.1002}}
I could change function <code>selected</code> so the positional parameters give the same results as the named parameters. Doing that in <code>selected</code> would mean both templates continue to give exactly the same results. Is that what is wanted? [[User:Johnuniq|Johnuniq]] ([[User talk:Johnuniq|talk]]) 05:58, 31 October 2020 (UTC)

For the DOIs, yes. But JCW-selected is also used for things like
{{JCW-selected|Foobar|Barfoo|Category:Foobar|doi=10.1234}}
So as long as that still works, it should be good, I think. Tweaking the templates won't screw up the bot, the bot only looks at the raw text of these pages. So feel free to iterate. The visual output is only for human convenience. &#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> 06:11, 31 October 2020 (UTC)
:So the module needs a simple rule to guess when a positional parameter is a name to be linked. Is a DOI '''always''' "digits dot digits"? If so, that could be the rule to decide when to display as a DOI. Any other text would just be linked. [[User:Johnuniq|Johnuniq]] ([[User talk:Johnuniq|talk]]) 08:15, 31 October 2020 (UTC)
::DOIs are always either <code>10.####</code> or <code>10.#####</code>. The rest of positional parameters can be plainlinked (with a : at the start of the link to handle categories and such). &#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> 14:47, 31 October 2020 (UTC)
{{od|2}}I think that's done. Please check results. [[User:Johnuniq|Johnuniq]] ([[User talk:Johnuniq|talk]]) 06:12, 1 November 2020 (UTC)
:Seems fine, although makes the template expansion size explode. &#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> 16:42, 1 November 2020 (UTC)
::See [[User:JL-Bot/Publishers.cfg]], where things blow up in the U section. I already bypassed the columns-list. &#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> 16:56, 1 November 2020 (UTC)
:::I fixed it by pruning the search URL: [[Special:Diff/986657727|diff]]. The search still seems to work, although how is unclear. More fat could be trimmed by removing the code tags, and then the {{tlf|doi}} markup. After that, a really major improvement would result from replacing the template calls with direct invokes of the module. That would give slightly uglier wikitext (and would require signifiant tweaks to the bot), and I would need to check and possibly fix the module so directly invoking it worked. [[User:Johnuniq|Johnuniq]] ([[User talk:Johnuniq|talk]]) 06:14, 2 November 2020 (UTC)
::::The bot tweaks would be pretty easy to implement by switching to <code>#invoke</code>, but it would be very ugly in wikicode. If things work right now, that's really all that matters. The publisher page is fairly unlikely to significantly blow up in size (meaning number of entries on the page) any time soon. Anyway, thanks a bunch. &#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> 06:22, 2 November 2020 (UTC)

== Exclude ==

{{ping|Headbomb}} While trying to get an overview of what [[Module:JCW]] is all about, I found [[WP:JCW/EXCLUDE]] which redirects to [[User:JL-Bot/Citations.cfg]]. That page is in [[:Category:Pages where template include size is exceeded]]. Do you want me to fix that error? The problem is the use of {{tl|columns-list}} which means that virtually the whole page is transcluded twice. The fix is to replace each template call with ugly wikitext. I don't know if that page is used or if a change would interfere with the bot. [[User:Johnuniq|Johnuniq]] ([[User talk:Johnuniq|talk]]) 06:12, 30 October 2020 (UTC)
::It wouldn't interfere with the bot, but if the columns-lists are the issue (I think figured that a while back on other pages, just never cared to bother with those), then feel free to subst/hardcode them. It won't inteference with the bot. &#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> 06:21, 30 October 2020 (UTC)
:::Actually, I took care of it. Thanks for pointing out the solution. &#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> 06:33, 30 October 2020 (UTC)
::::That's quite irritating because I got an edit conflict after doing the edit myself. Please check my edit which fixed an error and tweaked the doc. [[User:Johnuniq|Johnuniq]] ([[User talk:Johnuniq|talk]]) 06:42, 30 October 2020 (UTC)
::::::Yup it's fine. &#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> 06:44, 30 October 2020 (UTC)

== Pattern ==

Function <code>p.pattern</code> in [[Module:JCW]] was giving "Lua error: not enough memory" at [[User:JL-Bot/Maintenance.cfg]] and was calling gsub excessively (each parameter caused the function to gsub the entire string again). That's the reason it was hitting memory limits as shown by running <code><nowiki>{{JCW-pattern|Example|.*Abc.*|!Def!|G}}</nowiki></code> in [[Special:ExpandTemplates]] with the module before I fixed it just now. The output follows.
<pre>
*[[Example]]
** <code><b><font style=color:#006400;><b><font style=color:#006400;><b><font style=color:#006400;>.*</font></b></font></b></font></b>Abc<b><font style=color:#006400;><b><font style=color:#006400;><b><font style=color:#006400;>.*</font></b></font></b></font></b></code>
** <code><b><font style=color:#8B0000;><b><font style=color:#8B0000;>!</font></b></font></b>Def<b><font style=color:#8B0000;><b><font style=color:#8B0000;>!</font></b></font></b></code>
** <code>G</code>
</pre>
After fixing the module, the output is:
<pre>
*[[Example]]
**<code><b><font style=color:#006400;>.*</font></b>Abc<b><font style=color:#006400;>.*</font></b></code>
**<code><b><font style=color:#8B0000;>!</font></b>Def<b><font style=color:#8B0000;>!</font></b></code>
**<code>G</code>
</pre>
Checking [[User:JL-Bot/Maintenance.cfg]] shows "BREAK .*Palaeontologie.*" which is due to a missing pipe. You might like to fix that and contemplate whether the bot has been doing anything unwanted because of the typo (I have no idea what the bot is doing).

'''Question''' I have removed TableTools.compressSparseArray from the pattern function because I was running into memory and then time limits while working out the problem. The new module assumes that the template is used in a reasonable manner. For example:
* <code><nowiki>{{JCW-pattern|Example|.*Abc.*|!Def!|G}}</nowiki></code> {{green|(this works)}}
* <code><nowiki>{{JCW-pattern|Example|3=.*Abc.*|4=!Def!|9=G}}</nowiki></code> {{green|(this fails because numbered parameters are skipped)}}
I assume that skipping numbers will never occur and can be ignored. [[User:Johnuniq|Johnuniq]] ([[User talk:Johnuniq|talk]]) 01:39, 31 October 2020 (UTC)
::Yeah that's fine. &#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> 01:56, 31 October 2020 (UTC)

== JCW tweak ==

{{ping|Johnuniq}}, the next tweak would be to have [[Template:JCW-PUB-rank]] (same in [[Template:JCW-CRAP-rank]]) handle an arbitrary number of DOI prefixes (in the 'Publisher column').

{{JCW-PUB-top}}
{{JCW-PUB-rank|rank=618|target=[[University of Zagreb]]|citations=23|articles=23|note=|doi1=10.15233|doi2=10.15644|doi3=10.20532|doi4=10.20901|doi5=10.21278|doi6=10.21861|doi7=10.22598|doi8=10.24099|doi9=10.26582|doi10=10.3336|doi11=10.5552|doi12=10.5592|l-count=17|e-count=11|entries=
* {{doi prefix|10.15644}}
** [[Acta Stomatologica Croatica]] (4 in [[Ali Kosh|1]],&nbsp;[[Alveolar cleft grafting|2]],&nbsp;[[Dental avulsion|3]],&nbsp;[[Teeth blackening|4]])
* {{doi prefix|10.20901}}
** [[Anali Hrvatskog politološkog društva]] (2 in [[Ante Paradžik|1]],&nbsp;[[Croatian Spring|2]])
** [[Politička Misao : časopis Za Politologiju]] (1 in [[Jasenovac concentration camp|1]])
** [[Politička Misao: Časopis za Politologiju]] (3 in [[Denial of genocide of Serbs in the Independent State of Croatia|1]],&nbsp;[[Genocide of Serbs in the Independent State of Croatia|2]],&nbsp;[[Za dom spremni|3]])
** ''[[Politička misao]]'' (1 in [[Social Democratic Party of Croatia|1]])
* {{doi prefix|10.21861}}
** [[Hrvatski Geografski Glasnik]] (3 in [[Eastern Slavonia, Baranja and Western Syrmia (1995–1998)|1]],&nbsp;[[Geography of Croatia|2]],&nbsp;[[Sava|3]])
** [[Hrvatski geografski glasnik]] (5 in [[Bosanska Krajina|1]],&nbsp;[[Cres|2]],&nbsp;[[Lake Vrana (Cres)|3]],&nbsp;[[Turkish Croatia|4]],&nbsp;[[Una (Sava)|5]])
* {{doi prefix|10.22598}}
** [[Market-Tržište]] (1 in [[Mindaugas Sinkevičius|1]])
* {{doi prefix|10.24099}}
** [[Vet. Arhiv]] (1 in [[Red-wattled lapwing|1]])
* {{doi prefix|10.3336}}
** [[Glas. Mat. Ser. III]] (1 in [[Andrej Dujella|1]])
** [[Glasnik Matematicki]] (1 in [[Lonely runner conjecture|1]])
}}
{{JCW-bottom|legend=no|type=no|date=no}}


The maximum case currently in use in the compilation is {{para|doi47}}, but I'm sure there's a smarter way than a dumb

<pre>
-->{{#if:{{{doi1|}}}|{{·}}{{doi prefix|{{{doi5|}}}|mode=t}}}}<!--

...

-->{{#if:{{{doi50|}}}|{{·}}{{doi prefix|{{{doi47|}}}|mode=t}}}}<!--
</pre>
type of [https://en.wikipedia.org/w/index.php?title=Template:JCW-PUB-rank&curid=60512524&diff=986661018&oldid=965386053 copy-paste] in the template to handle this. &#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> 06:31, 2 November 2020 (UTC)
:I'll take a couple of days off but will have a look at this later. If I'm not back by Friday, remind me. [[User:Johnuniq|Johnuniq]] ([[User talk:Johnuniq|talk]]) 06:39, 2 November 2020 (UTC)
::{{Re|Johnuniq}} No worries, this is hardly urgent. AFAICT, there's no issue anywhere with this, save for it not handling something arbitrary. &#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> 06:44, 2 November 2020 (UTC)
:::{{Re|Johnuniq}} courtesy ping, as requested. Any updates on this? It's still not urgent, so no worries if you don't have time for a while. &#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> 03:25, 24 November 2020 (UTC)
::::Oops, I totally forgot. I've noted it now and should have a look in the next few days. You'd better remind me again if I miss. [[User:Johnuniq|Johnuniq]] ([[User talk:Johnuniq|talk]]) 09:45, 24 November 2020 (UTC)

{{ping|Headbomb}} I am planning a new function in [[Module:JCW]] which might be called DOI_LIST. In {{tl|JCW-PUB-rank}} and {{tl|JCW-CRAP-rank}}, you would replace the guts of each template (all the stuff that refers to doi1, doi2, etc.) with <code><nowiki>{{#invoke:JCW|DOI_LIST}}</nowiki></code>. The new function would output equivalent wikitext. It would not call {{tl|doi prefix}} but would output equivalent wikitext assuming mode=t is wanted. I assume you would be happy with this. What should the function be called? The module currently has functions <code>selected</code>, <code>exclude</code> and <code>pattern</code>. Would <code>list</code> be enough? <code>doilist</code>? [[User:Johnuniq|Johnuniq]] ([[User talk:Johnuniq|talk]]) 03:00, 28 November 2020 (UTC)
: Name it whatever you feel is natural. It should be fairly trivial to update if I decide I don't like the name down the road. The only real usability criteria is that the current output remains unchanged. &#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> 03:05, 28 November 2020 (UTC)
::As you saw, I have finished and it should be ok for any number of doiN parameters. You might like to edit {{tl|JCW-PUB-rank}} and remove the blank line at the bottom before the category. I suspect that the category should be moved into the noinclude tags which are already there. [[User:Johnuniq|Johnuniq]] ([[User talk:Johnuniq|talk]]) 00:11, 29 November 2020 (UTC)

==Issue found==
{{re|Johnuniq}} Found a small issue. In something like, where {{para|doi1=}} is empty
{{JCW-CRAP-top}}
{{JCW-CRAP-rank|rank=405|target=[[Communications on Applied Electronics]]|citations=1|articles=1|source=BLJ|note=|doi1=|l-count=1|e-count=1|entries=
* [[Communications on Applied Electronics]] (1 in [[Mivar-based approach|1]])
}}
|}
It shows a '''{{doi-prefix|[https://en.wikipedia.org/w/index.php?ns0=1&ns118=1&search=insource%3A%5C%2F%20*%2F 1]|mode=t}}''' instead of just nothing. &#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:45, 12 December 2020 (UTC)
:Ouch, I'm writing some shitty code lately. I think I fixed that. [[User:Johnuniq|Johnuniq]] ([[User talk:Johnuniq|talk]]) 00:06, 13 December 2020 (UTC)
::It's OK, no babies, giraffes, or armadillos were harmed. &#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> 00:23, 13 December 2020 (UTC)