Module:Is instance - Revision history

MusikBot II: Changed protection settings for "Module:Is instance": High-risk template or module: 8007 transclusions (more info) ([Edit=Require template editor access] (indefinite) [Move=Require template editor access] (indefinite))

2022-09-13T18:01:44Z

Changed protection settings for "Module:Is instance": High-risk template or module: 8007 transclusions (more info) ([Edit=Require template editor access] (indefinite) [Move=Require template editor access] (indefinite))

← Previous revision	Revision as of 18:01, 13 September 2022
(No difference)

MSGJ: check that snaktype=value

2022-04-02T21:41:18Z

check that snaktype=value

← Previous revision		Revision as of 21:41, 2 April 2022
Line 13:		Line 13:
	local subclasses = mw.wikibase.getBestStatements(checklist[depth][j],property) -- get P31 or P279 statements		local subclasses = mw.wikibase.getBestStatements(checklist[depth][j],property) -- get P31 or P279 statements
	for i,statement in ipairs(subclasses) do -- loop through items at next depth		for i,statement in ipairs(subclasses) do -- loop through items at next depth
	if statement.mainsnak.~~datavalue~~ then		if statement.mainsnak.snaktype == 'value' then
	local newitem = statement.mainsnak.datavalue.value.id		local newitem = statement.mainsnak.datavalue.value.id
	if newitem == value then		if newitem == value then

MSGJ: return 0 if no qid

2022-04-02T21:03:35Z

return 0 if no qid

← Previous revision		Revision as of 21:03, 2 April 2022
Line 35:		Line 35:
	end		end
	if not qid then		if not qid then
	return ~~"No qid defined!"~~		return 0
	end		end
	checklist = {{qid}}		checklist = {{qid}}

MusikBot II: Protected "Module:Is instance": High-risk template or module: 2426 transclusions (more info) ([Edit=Require autoconfirmed or confirmed access] (indefinite))

2022-03-23T17:59:53Z

Protected "Module:Is instance": High-risk template or module: 2426 transclusions (more info) ([Edit=Require autoconfirmed or confirmed access] (indefinite))

← Previous revision	Revision as of 17:59, 23 March 2022
(No difference)

Frietjes: check to see if statement.mainsnak.datavalue is nil before using it, fixes script error in Kugelbake and doesn't appear to break anything since the working cases would have had a non-nil statement.mainsnak.datavalue

2022-03-23T15:44:36Z

check to see if statement.mainsnak.datavalue is nil before using it, fixes script error in Kugelbake and doesn't appear to break anything since the working cases would have had a non-nil statement.mainsnak.datavalue

← Previous revision		Revision as of 15:44, 23 March 2022
Line 13:		Line 13:
	local subclasses = mw.wikibase.getBestStatements(checklist[depth][j],property) -- get P31 or P279 statements		local subclasses = mw.wikibase.getBestStatements(checklist[depth][j],property) -- get P31 or P279 statements
	for i,statement in ipairs(subclasses) do -- loop through items at next depth		for i,statement in ipairs(subclasses) do -- loop through items at next depth
	~~local newitem =~~ statement.mainsnak.datavalue~~.value.id~~		if statement.mainsnak.datavalue then
	if newitem == value ~~then~~		local newitem = statement.mainsnak.datavalue.value.id
	~~isInstance~~ = ~~true~~		if newitem == value then
			isInstance = true
	else		else
	checklist[depth+1][#checklist[depth+1]+1] = newitem -- add item to next depth of checklist table		checklist[depth+1][#checklist[depth+1]+1] = newitem -- add item to next depth of checklist table
			end
	end		end
	end		end

MSGJ at 13:56, 23 March 2022

2022-03-23T13:56:34Z

← Previous revision		Revision as of 13:56, 23 March 2022
Line 28:		Line 28:
	local pargs = frame:getParent().args		local pargs = frame:getParent().args
	value = args.value or pargs.value		value = args.value or pargs.value
	local qid = args.qid or pargs.qid		local qid = args.qid or pargs.qid or ""
	if qid == "" then		if qid == "" then
	qid = mw.wikibase.getEntityIdForCurrentPage()		qid = mw.wikibase.getEntityIdForCurrentPage()

MSGJ: default to current page id

2022-03-23T13:54:27Z

default to current page id

@@ Line 28: / Line 28: @@
 	local pargs = frame:getParent().args
 	value = args.value or pargs.value
-	local qid = args.qid or pargs.qid or ""
+	local qid = args.qid or pargs.qid
-	if qid=="" then
+	if qid == "" then
 		return "No qid defined!"
 	else
-		checklist = {{qid}}
+		maxdepth = 5
-			maxdepth = 5
+		depth = depth+1
 	end
 end

MSGJ at 13:36, 23 March 2022

2022-03-23T13:36:23Z

← Previous revision		Revision as of 13:36, 23 March 2022
Line 28:		Line 28:
	local pargs = frame:getParent().args		local pargs = frame:getParent().args
	value = args.value or pargs.value		value = args.value or pargs.value
	local qid = args.qid or pargs.qid		local qid = args.qid or pargs.qid or ""
	if qid then		if qid=="" then
		⚫	return "No qid defined!"
		⚫	else
	checklist = {{qid}}		checklist = {{qid}}
	local md = args.maxdepth or pargs.maxdepth		local md = args.maxdepth or pargs.maxdepth
Line 47:		Line 49:
	end		end
	return depth -- return depth that item was found		return depth -- return depth that item was found
⚫	else
⚫	return "No qid defined!"
	end		end
	end		end

MSGJ: check for qid

2022-03-23T13:34:15Z

check for qid

@@ Line 27: / Line 27: @@
 	local args = frame.args
 	local pargs = frame:getParent().args
 	local qid = args.qid or pargs.qid
-	checklist = {{qid}}
+	if qid then
-	local md = args.maxdepth or pargs.maxdepth
+		local md = args.maxdepth or pargs.maxdepth
-		maxdepth = tonumber(md)
+		if md then
 	else
 	end
 end

MSGJ: simpler

2022-03-22T21:45:52Z

simpler

← Previous revision		Revision as of 21:45, 22 March 2022
Line 4:		Line 4:
	checklist[depth+1] = {} -- setup next layer of table		checklist[depth+1] = {} -- setup next layer of table
	local j = 0		local j = 0
	~~while~~ j<#checklist[depth] ~~and~~ ~~not~~ ~~isInstance~~ do		for j = 1,#checklist[depth] do -- loop over items at current depth
	--for j=1,#checklist[depth],1 do -- loop over items at current depth
	j = j+1
	local property		local property
	if depth == 1 then		if depth == 1 then
Line 18:		Line 16:
	if newitem == value then		if newitem == value then
	isInstance = true		isInstance = true
	break -- break away from loop if instance found
	else		else
	checklist[depth+1][#checklist[depth+1]+1] = newitem -- add item to next depth of checklist table		checklist[depth+1][#checklist[depth+1]+1] = newitem -- add item to next depth of checklist table