Module talk:DidYouKnow (edit)
Latest revision as of 14:11, 2 January 2024 (edit) (undo)
, 2 January 2024→Sandbox: reply
MarkusRost (talk | contribs) (→Sandbox: endless loop) |
Dinoguy1000 (talk | contribs) (→Sandbox: reply) |
||
Line 19:
Please copy the [[Module:DidYouKnow/sandbox|sandbox]] to the live module; this removes the <code>isPresent</code> variable from <code>p.getFacts()</code> and instead just directly stores the fact to the <code>chosen</code> table if it isn't already present there. <span class="nowrap">「[[User:Dinoguy1000|<span style="color:#00f">ディノ</span><span style="color:#080">奴</span>]][[Special:Contributions/Dinoguy1000|<span style="color:#F90">千？！</span>]]」<sup>[[User talk:Dinoguy1000#top|☎ Dinoguy1000]]</sup></span> 14:56, 1 January 2024 (UTC)
: Your change does the exact opposite of the current code, adding the number only when it already exists. This also results in an endless loop, as you start with an empty list so there won't ever be any facts added. -- [[File:Markus Rost head.png|16px|text-top|link=]] [[User:MarkusRost|MarkusRost]] ([[User talk:MarkusRost|talk]]) 15:03, 1 January 2024 (UTC)
::...Aah, I see it now. No idea how I missed that before. =/ <span class="nowrap">「[[User:Dinoguy1000|<span style="color:#00f">ディノ</span><span style="color:#080">奴</span>]][[Special:Contributions/Dinoguy1000|<span style="color:#F90">千？！</span>]]」<sup>[[User talk:Dinoguy1000#top|☎ Dinoguy1000]]</sup></span> 14:11, 2 January 2024 (UTC)
| |||