Content deleted Content added
| (6 intermediate revisions by 2 users not shown) | |||
Line 96:
:Regarding the specific errors you have mentioned, the problem was that Wikidata says that references citing online databases need to contain both {{prop|248}} and a property of the {{mono|External identifier}} data type, while the references causing errors contained only the latter. This may seem like a minor technicality, but database references are handled by an ugly hack in the module, and I do not want to make this hacky code more complicated to support references like this. I plan to fix this issue by creating a separate template for citing database references; this will hopefully also eliminate cryptic error messages like these.
:As for re-introducing the cywiki fix, the old code is probably of little use, because the code has changed quite a lot. If it is really needed, I can try to reimplement it, but I would strongly prefer if you waited until I make the fix I have mentioned in the previous paragraph.
:Also, regarding the changes <u>to this module</u> since I made the cywiki fix, there were two of them that are especially relevant here:
:* I added verbose error messages. They should mostly be comprehensible after reading the documentation. I agree though that the errors you have mentioned are hard to understand; I will fix that.
:* {{mono|1={{!}}title=}} is no longer a mandatory param for {{tl|Cite web}}. In practice, this means that missing titles in references do not cause errors in this module, but in the citation template, like you see in the article you have linked. This hopefully made the errors more understandable. When I create the database-citation template, many of these missing-title errors could be eliminated.
:—[[User:Janhrach|Janhrach]] ([[User talk:Janhrach|talk]]) 14:19, 26 August 2025 (UTC)
::{{tq|{{mono|1={{!}}title=}} is no longer a mandatory param for {{tl|Cite web}}.}} False. I don't know where you got that idea but you are mistook. {{para|title}} {{em|is}}, and likely will always be, required by {{tld|cite web}}. If you have found some place where it is stated that {{para|title}} is {{em|not}} required, name that place so that it can be visited and fixed.
::—[[User:Trappist the monk|Trappist the monk]] ([[User talk:Trappist the monk|talk]]) 14:40, 26 August 2025 (UTC)
:::I meant that ''this module'' does not consider it mandatory. I wrote that {{tq|this means that missing titles in references do not cause errors in this module, but in the citation template[...]}} [[User:Janhrach|Janhrach]] ([[User talk:Janhrach|talk]]) 14:43, 26 August 2025 (UTC)
::::Yes, I saw that, but your initial declaration that {{para|title}} is not required by {{tlx|cite web}} is false. That declaration cannot be allowed to stand unchallenged.
::::—[[User:Trappist the monk|Trappist the monk]] ([[User talk:Trappist the monk|talk]]) 14:52, 26 August 2025 (UTC)
:::::I should have made it clear that I refered to ''my'' changes to ''this'' module. I now see that {{tq|changes since I made the cywiki fix}} could have been interpreted as including changes to CS1. [[User:Janhrach|Janhrach]] ([[User talk:Janhrach|talk]]) 14:59, 26 August 2025 (UTC)
::Thanks for the reply! I will wait until your changes are made and may look at reverting our copy of Wd until that's done. [[User:Dafyddt|Dafyddt]] ([[User talk:Dafyddt|talk]]) 18:20, 26 August 2025 (UTC)
| |||