Module talk:Data: Difference between revisions

Revision as of 06:54, 15 July 2023

Latest revision as of 12:01, 5 October 2023

(5 intermediate revisions by 2 users not shown)

Line 14:

Would it be possible to develop a version of this module which would read a JSON pages? Or do we have something that can do this already? &mdash;&nbsp;Martin <small>([[User:MSGJ|MSGJ]]&nbsp;·&nbsp;[[User talk:MSGJ|talk]])</small> 06:54, 15 July 2023 (UTC)

:Well, I attempted to modify this module but was unsuccessful. So I created a bespoke module at [[Module:Women in Green event]] which serves my immediate needs. If this ever gets implemented here, then that module could be deleted. &mdash;&nbsp;Martin <small>([[User:MSGJ|MSGJ]]&nbsp;·&nbsp;[[User talk:MSGJ|talk]])</small> 12:14, 29 September 2023 (UTC)

::{{ping|MSGJ}} I've decided to give this a crack in the sandbox. The module will attempt to load the page as if it was a module page by default, and if this doesn't work, it'll instead try to load it as a Json page. If neither of these works, it throws an error like current functionallity (yet worded to mean something a bit more specific for this case). I don't imagine this pcall logic will cause any overhead (and could even maybe be made smart by checking if the input is in the Module namespace before even bothering trying to do loadData). You can see a test of it [[Special:Permalink/1178703748|here]]. Does this seem fine? [[User:Aidan9382|Aidan9382]] _{([[User talk:Aidan9382|talk]])} 09:51, 5 October 2023 (UTC)

:::Thanks Aidan! Can you see why [[Module talk:Data/testcases]] is not working? To get this working in [[Module:Women in Green event]] I had to use <code>tonumber</code> to convert the 1 to a number. (I don't understand why because I used "1" in the JSON file, which should be a string.) It seems this module can only handle keys which are strings. &mdash;&nbsp;Martin <small>([[User:MSGJ|MSGJ]]&nbsp;·&nbsp;[[User talk:MSGJ|talk]])</small> 11:23, 5 October 2023 (UTC)

::::The issue here is that the JSON loader is probably automatically converting these to number keys for whatever reason. I've fixed it in this module by going for the numbered key if and only if it has a tonumber evaluation and the string key returns nil, which fixes your testcase. [[User:Aidan9382|Aidan9382]] _{([[User talk:Aidan9382|talk]])} 11:50, 5 October 2023 (UTC)

:::::Thanks, that looks good &mdash;&nbsp;Martin <small>([[User:MSGJ|MSGJ]]&nbsp;·&nbsp;[[User talk:MSGJ|talk]])</small> 12:01, 5 October 2023 (UTC)