Template talk:In string
Edit request 4 September 2024
[edit] | It is requested that an edit be made to the fully protected template at Template:In string. (edit · history · last · links · transclusion count · protection log) This template must be followed by a complete and specific description of the request, so that an editor unfamiliar with the subject matter could complete the requested edit immediately.
Edit requests to fully protected pages should only be used for edits that are either uncontroversial or supported by consensus. If the proposed edit might be controversial, discuss it on the protected page's talk page before using this template. To request that a page be protected or unprotected, make a protection request. When the request has been completed or denied, please add the |
Follow-up to this topic.
{{In string}} is also affected by the Module:String bug: "plain mode" is disabled by default, whereas it was expected to be enabled by default.
We could modify this template to ensure "plain mode" stays disabled by default after the module is fixed, but I have better: let's right now switch this template to "plain mode" enabled by default.
I have reviewed the uses of this templates, and in all cases, "|plain=false" is present when required, or the pattern doesn't have special characters, so both enabled and disabled plain modes work. The only fixes I had to make are 1243726361 and 1243727056.
I think plain mode is better to be enabled by default, as it is less surprising (no mistakes because the user doesn't know about special characters), and it always has been the documented behavior. Also, plain mode should now be only marginally slower, thanks to this change.
So, here I'm asking to replace:
|source={{#invoke:String|find|source={{{source|{{{1|}}}}}}|target={{{target|{{{2|}}}}}}|start={{{start|{{{3|}}}}}}|plain={{{plain|{{{4|}}}}}}}}
with:
|source={{#invoke:String|find|source={{{source|{{{1|}}}}}}|target={{{target|{{{2|}}}}}}|start={{{start|{{{3|}}}}}}|plain={{#if:{{{plain|{{{4|}}}}}}|{{{plain|{{{4}}}}}}|true}}}}