Jump to content

Module talk:Infobox military conflict

Page contents not supported in other languages.
From Wikipedia, the free encyclopedia
This is the current revision of this page, as edited by Zackmann08 (talk | contribs) at 03:17, 28 October 2025 (Embed/child: Reply). The present address (URL) is a permanent link to this version.
(diff) ← Previous revision | Latest revision (diff) | Newer revision → (diff)

Use of map_type= without coordinates causes script error to display instead of infobox

[edit]
Resolved

Please see this test case for an example of the infobox being replaced entirely by a script error when |map_type= is used without coordinates. That condition should result in an error-tracking category, not a giant red error message. – Jonesey95 (talk) 01:02, 9 May 2024 (UTC)[reply]

Never mind, I fixed it by adjusting the if statement to check for both map_type and coordinates before rendering a map. Why is this talk page separate from the Template's talk page? That is not the usual practice. – Jonesey95 (talk) 04:02, 9 May 2024 (UTC)[reply]

native_name

[edit]

Description of suggested change: Add native_name. Although I personally don't see how including a conflict's native name adds value to an infobox, many editors seem to disagree. As a result, the conflict parameter is often misused so as to include this. This new parameter is intended to prevent that.

Example: Template:Infobox military conflict/testcases#native_name

Code: Add from line 159:

	if self.args.native_name then
		builder:tag('tr')
			:tag('td')
				:attr('colspan', self.combatants)
				:cssText(infoboxStyle.header_raw)
				:wikitext(self.args.native_name)
	end

Jay D. Easy (t) 16:26, 7 October 2024 (UTC)[reply]

 Not done for now: please establish a consensus for this alteration before using the {{Edit template-protected}} template. @Jay D. Easy: Once consensus is established (or a reasonable amount of time has passed without objection), feel free to re-activate this edit request by changing it to |answered=no. --Ahecht (TALK
PAGE
)
17:01, 7 October 2024 (UTC)[reply]
I judged this to be sufficiently uncontroversial as not to require consensus. But alright, RfC noticed WP:MILHIST. Jay D. Easy (t) 17:20, 7 October 2024 (UTC)[reply]
Could you provide an example of previous disagreement or misuse of existing parameters? SamuelRiv (talk) 17:36, 7 October 2024 (UTC)[reply]
Sure, here's a random few:
Jay D. Easy (t) 18:03, 7 October 2024 (UTC)[reply]
|native_name= is typically accompanied by |native_name_lang=. See {{Infobox settlement}}, {{Infobox musical composition}}, and {{Infobox sheep breed}} for sample implementations. More examples are here.Jonesey95 (talk) 05:03, 8 October 2024 (UTC)[reply]
I would have proposed adding |native_name_lang= were it not for the fact that this infobox is Lua-based and the suggested change is the current extent of my code scavenging ability. Jay D. Easy (t) 15:19, 11 October 2024 (UTC)[reply]

check for unknown parameters missing

[edit]

We should add Module:Check for unknown parameters here. --Joy (talk) 21:54, 22 October 2025 (UTC)[reply]

@Joy: funny story... I just did that in the template. Still working out the bugs. Obviously feel free to migrate it to the module instead. I'm less familiar with implementing it via LUA then I am will doing it in the template code.... Zackmann (Talk to me/What I been doing) 20:03, 26 October 2025 (UTC)[reply]

mapframe implementation

[edit]

So my initial tests of mapframe here were based on not touching anything at all if map_type is set, but not through mapframe onByDefault. Then I ran into a couple of articles where both mapframe and location map were added by editors, but mapframe=yes gets ignored. So I utilized onByDefault to get that to work, and it did, but everything else broke horribly. That change is now reverted.

I also have an issue of not knowing how to set variable defaults in this context - I want to move mapframe-zoom from the default 10 to 7 (cf. sandbox), but it's just not budging. --Joy (talk) 22:03, 22 October 2025 (UTC)[reply]

Embed/child

[edit]

So I just came across Battle of Lyman's Wagon Train which tries to embed this infobox... Neither |child= nor |embed= seem to work. Is this something that can be added? Zackmann (Talk to me/What I been doing) 02:52, 28 October 2025 (UTC)[reply]

I will note that per the Param Report this is the ONLY instance of |embed= or |child= in over 23,000 transclusions... So if it cannot be easily done, maybe it isn't worth fixing. Zackmann (Talk to me/What I been doing) 03:17, 28 October 2025 (UTC)[reply]