Module:Studio sessions - Revision history

Magent: Point at the copy in the repository, where it can be reviewed and tested (via update-page on MediaWiki MCP Server)

2026-09-20T23:03:03Z

Point at the copy in the repository, where it can be reviewed and tested (via update-page on MediaWiki MCP Server)

Magent: A cut's lineup reads down: one player per row, instruments in their own column (via update-page on MediaWiki MCP Server)

2026-09-20T03:40:19Z

A cut's lineup reads down: one player per row, instruments in their own column (via update-page on MediaWiki MCP Server)

← Older revision		Revision as of 03:40, 20 September 2026
Line 145:		Line 145:
	-- musician's page calls their main one.		-- musician's page calls their main one.
	local first = split( player.instruments )[1] or ""		local first = split( player.instruments )[1] or ""
	~~local link~~ = frame:expandTemplate{ title = "m", args = { player.name, first } }		table.insert( shown, {
	~~if player.~~instruments ~~~= "" then~~		who = frame:expandTemplate{ title = "m", args = { player.name, first } },
	~~link~~ = ~~link .. ' <span style="color:#6b7566;">' ..~~ player.instruments ~~.. "</span>"~~		instruments = player.instruments,
	~~end~~		} )
	~~table.insert( shown, link~~ )
	end		end

Line 185:		Line 184:
	local out = { '<div style="padding:.5em .2em; border-bottom:1px solid #d4dacd;">' }		local out = { '<div style="padding:.5em .2em; border-bottom:1px solid #d4dacd;">' }
	table.insert( out, '<div style="font-weight:600;">' .. heading .. "</div>" )		table.insert( out, '<div style="font-weight:600;">' .. heading .. "</div>" )
			-- A lineup reads down, not across. On a composition's page the personnel
			-- are the substance — who was in the room, on what — and a run-on line of
			-- names separated by dots makes them a caption. One player per row, with
			-- the instruments in their own column, is a session sheet.
	if #shown > 0 then		if #shown > 0 then
	table.insert( out, '<~~div~~ style="font-size:.95em; line-height:1.9;">'		local rows = {}
	.. table.concat( ~~shown~~, " ~~·~~ " ) .. "</~~div~~>" )		for _, player in ipairs( shown ) do
			table.insert( rows,
			'<tr><td style="padding:.1em 1.2em .1em 0; white-space:nowrap;'
			.. ' vertical-align:baseline;">' .. player.who .. "</td>"
			.. '<td style="padding:.1em 0; color:#3d4a3e;'
			.. ' vertical-align:baseline;">' .. player.instruments
			.. "</td></tr>" )
			end
			-- One line: a newline inside raw table markup gives the parser an
			-- opening to close the table early and leave the rest as text.
			table.insert( out, '<table style="margin:.3em 0 .4em 0;'
			.. ' border-collapse:collapse; font-size:.95em; line-height:1.8;">'
			.. table.concat( rows, "" ) .. "</table>" )
	end		end
	if #session > 0 then		if #session > 0 then
Line 549:		Line 564:
	local links = {}		local links = {}
	for _, player in ipairs( shown ) do		for _, player in ipairs( shown ) do
	table.insert( links, frame:expandTemplate{		-- Nowrap, so a name never ends a line with its icon stranded at
	title = "m", args = { player.name, player.instrument } } )		-- the start of the next one.
			table.insert( links, '<span style="white-space:nowrap;">'
			.. frame:expandTemplate{
			title = "m", args = { player.name, player.instrument } }
			.. "</span>" )
	end		end

Magent: A cut is identified within its composition: joining on the id alone credited a player with tracks they were not on (via update-page on MediaWiki MCP Server)

2026-09-19T21:09:03Z

A cut is identified within its composition: joining on the id alone credited a player with tracks they were not on (via update-page on MediaWiki MCP Server)

← Older revision		Revision as of 21:09, 19 September 2026
Line 84:		Line 84:
	--		--
	-- Block height and studio, per the convention: a block height is a time nobody		-- Block height and studio, per the convention: a block height is a time nobody
	-- has to agree on the name of~~, and two sessions in the same studio at the same~~		-- has to agree on the name of.
	~~-- height do not happen~~.
	--		--
	-- Failing that, an explicit label, and failing that the record it was cut for		-- Failing that, an explicit label, and failing that the record it was cut for
Line 91:		Line 90:
	-- that way cannot be told they are the same performance; that is exactly what		-- that way cannot be told they are the same performance; that is exactly what
	-- recording the block height fixes.		-- recording the block height fixes.
			--
			-- Either way it identifies the cut within this composition, never across
			-- compositions: five tunes cut in one session share a block height, and every
			-- imported cut on a record shares that record's name. See key().
	local function cutId( args )		local function cutId( args )
	local given = trim( args.cut )		local given = trim( args.cut )
Line 274:		Line 277:
	end		end
	return out		return out
			end

			-- A cut's identity is the composition it belongs to plus its own id.
			--
			-- Never the id alone. Two tunes cut in one session share a block height and a
			-- studio, and every imported cut on one record falls back to that record's
			-- name — so joining on the id by itself put Harry Clark on a Barlows Jig he
			-- never played, because he was on another 4masks track.
			local function key( song, cut )
			return song .. "\0" .. cut
	end		end

Line 279:		Line 292:
	--		--
	-- A record with a dozen tracks would otherwise mean a dozen queries; SMW takes		-- A record with a dozen tracks would otherwise mean a dozen queries; SMW takes
	-- alternatives with \|\|, so it is one.		-- alternatives with \|\|, so it is one. The results are then matched back by
			-- composition and cut together.
	local function appearancesOf( frame, ids )		local function appearancesOf( frame, ids )
	local wanted = {}		local wanted, seen = {}, {}
	for id in pairs( ids ) do		for _, cut in pairs( ids ) do
	table.insert( wanted, id )		if not seen[ cut ] then
			seen[ cut ] = true
			table.insert( wanted, cut )
			end
	end		end
	if #wanted == 0 then		if #wanted == 0 then
Line 299:		Line 316:
	for _, row in ipairs( found ) do		for _, row in ipairs( found ) do
	if row.record ~= "" then		if row.record ~= "" then
	~~byCut[~~ row.cut ] = byCut[ ~~row.cut~~ ] or {}		local id = key( row.song, row.cut )
	table.insert( byCut[ ~~row.cut~~ ], {		byCut[ id ] = byCut[ id ] or {}
			table.insert( byCut[ id ], {
	song = row.song, record = row.record, number = tonumber( row.number ),		song = row.song, record = row.record, number = tonumber( row.number ),
	} )		} )
Line 328:		Line 346:
	}, "song" )		}, "song" )

	local ids, instruments, songOf = {}, {}, {}		local ids, instruments, songOf, cutOf = {}, {}, {}, {}
	for _, row in ipairs( mine ) do		for _, row in ipairs( mine ) do
	if row.cut ~= "" then		if row.cut ~= "" then
	ids[ row.cut ~~] = true~~		local id = key( row.song, row.cut )
	songOf[ row.~~cut~~ ] = row.~~song~~		ids[ id ] = row.cut
			songOf[ id ] = row.song
			cutOf[ id ] = row.cut
	for _, one in ipairs( split( row.instrument ) ) do		for _, one in ipairs( split( row.instrument ) ) do
	instruments[ ~~row.cut~~ ] = instruments[ ~~row.cut~~ ] or {}		instruments[ id ] = instruments[ id ] or {}
	table.insert( instruments[ ~~row.cut~~ ], one )		table.insert( instruments[ id ], one )
	end		end
	end		end
Line 367:		Line 387:
	end		end
	else		else
	table.insert( loose, { song = songOf[ id ], id = id,		table.insert( loose, { song = songOf[ id ], id = cutOf[ id ],
	instruments = instruments[ id ] or {} } )		instruments = instruments[ id ] or {} } )
	end		end
Line 463:		Line 483:
	for _, row in ipairs( places ) do		for _, row in ipairs( places ) do
	if row.cut ~= "" then		if row.cut ~= "" then
	ids[ row.cut ] = ~~true~~		ids[ key( row.song, row.cut ) ] = row.cut
	if not bySong[ row.song ] then		if not bySong[ row.song ] then
	bySong[ row.song ] = { number = tonumber( row.number ), cuts = {} }		bySong[ row.song ] = { number = tonumber( row.number ), cuts = {} }
Line 479:		Line 499:

	-- Who played, for every cut on this record, in one more query.		-- Who played, for every cut on this record, in one more query.
	local wanted = {}		local wanted, seen = {}, {}
	for id in pairs( ids ) do		for _, cut in pairs( ids ) do
	table.insert( wanted, id )		if not seen[ cut ] then
			seen[ cut ] = true
			table.insert( wanted, cut )
			end
	end		end
	table.sort( wanted )		table.sort( wanted )
Line 494:		Line 517:
	for _, row in ipairs( personnel ) do		for _, row in ipairs( personnel ) do
	if row.cut ~= "" and row.performer ~= "" then		if row.cut ~= "" and row.performer ~= "" then
	~~byCut[~~ row.cut ] = byCut[ ~~row.cut~~ ] or {}		local id = key( row.song, row.cut )
	table.insert( byCut[ ~~row.cut~~ ],		byCut[ id ] = byCut[ id ] or {}
			table.insert( byCut[ id ],
	{ name = row.performer, instrument = split( row.instrument )[1] or "" } )		{ name = row.performer, instrument = split( row.instrument )[1] or "" } )
	end		end
Line 513:		Line 537:

	local seen, shown = {}, {}		local seen, shown = {}, {}
	for _, id in ipairs( entry.cuts ) do		for _, cut in ipairs( entry.cuts ) do
	for _, player in ipairs( byCut[ id ] or {} ) do		for _, player in ipairs( byCut[ key( song, cut ) ] or {} ) do
	if not seen[ player.name ] then		if not seen[ player.name ] then
	seen[ player.name ] = true		seen[ player.name ] = true

Magent: Query rows come back keyed by the names the query gave its columns, so callers stop saying row.b (via update-page on MediaWiki MCP Server)

2026-09-19T15:58:24Z

Query rows come back keyed by the names the query gave its columns, so callers stop saying row.b (via update-page on MediaWiki MCP Server)

← Older revision		Revision as of 15:58, 19 September 2026
Line 228:		Line 228:
	end		end

	-- ~~Rows of "subject\|\|b\|\|c\|\|d"~~, ~~from~~ Template:Studio appearance row.		-- Ask a query, and get back rows keyed by the names the query gave its
			-- columns.
			--
			-- Template:Studio appearance row emits the values joined by \|\|, positionally,
			-- because that is all a result template can do. Reading them back as row[1]..
			-- row[4] — or worse, row.b and row.c — makes every caller a puzzle, so the
			-- labels declared in the printouts ("?Has cut=cut") are used as the keys.
	local function rows( frame, query, printouts, mainlabel )		local function rows( frame, query, printouts, mainlabel )
			local names = { mainlabel }
			for _, printout in ipairs( printouts ) do
			table.insert( names, printout:match( "=([^=]*)$" ) or printout )
			end

	local args = { query }		local args = { query }
	for _, printout in ipairs( printouts ) do		for _, printout in ipairs( printouts ) do
Line 246:		Line 257:
	return out		return out
	end		end
	for _, ~~row~~ in ipairs( mw.text.split( raw, "@@", true ) ) do		for _, line in ipairs( mw.text.split( raw, "@@", true ) ) do
	local ~~a, b, c, d~~ = ~~row:match( "^(~~.~~-)\|\|(~~.~~-)\|\|~~(~~.-)~~\|\|~~(.*)$~~" )		local values = mw.text.split( line, "\|\|", true )
	if ~~a and~~ trim( a ) ~= "" then		if trim( values[1] or "" ) ~= "" then
	~~table.insert~~( ~~out, {~~		local row = {}
	subject = trim( ~~( a~~:match( "^([^#]*)" ) or a ) ),		for index, name in ipairs( names ) do
	b = ~~trim~~( ~~b )~~, ~~c = trim( c ), d = trim( d ),~~		local value = trim( values[ index ] or "" )
	} )		if index == 1 then
			-- A subobject's subject prints as "Song:Barlows#_abc123";
			-- the page is the part before the fragment.
			value = trim( value:match( "^([^#]*)" ) or value )
			end
			row[ name ] = value
			end
			table.insert( out, row )
	end		end
	end		end
Line 280:		Line 298:
	local byCut = {}		local byCut = {}
	for _, row in ipairs( found ) do		for _, row in ipairs( found ) do
	if row.c ~= "" then		if row.record ~= "" then
	byCut[ row.b ] = byCut[ row.b ] or {}		byCut[ row.cut ] = byCut[ row.cut ] or {}
	table.insert( byCut[ row.b ], {		table.insert( byCut[ row.cut ], {
	song = row.~~subject~~, record = row.c, number = tonumber( row.d ),		song = row.song, record = row.record, number = tonumber( row.number ),
	} )		} )
	end		end
Line 312:		Line 330:
	local ids, instruments, songOf = {}, {}, {}		local ids, instruments, songOf = {}, {}, {}
	for _, row in ipairs( mine ) do		for _, row in ipairs( mine ) do
	if row.b ~= "" then		if row.cut ~= "" then
	ids[ row.b ] = true		ids[ row.cut ] = true
	songOf[ row.b ] = row.~~subject~~		songOf[ row.cut ] = row.song
	for _, one in ipairs( split( row.c ) ) do		for _, one in ipairs( split( row.instrument ) ) do
	instruments[ row.b ] = instruments[ row.b ] or {}		instruments[ row.cut ] = instruments[ row.cut ] or {}
	table.insert( instruments[ row.b ], one )		table.insert( instruments[ row.cut ], one )
	end		end
	end		end
Line 444:		Line 462:
	local ids, order, bySong = {}, {}, {}		local ids, order, bySong = {}, {}, {}
	for _, row in ipairs( places ) do		for _, row in ipairs( places ) do
	if row.b ~= "" then		if row.cut ~= "" then
	ids[ row.b ] = true		ids[ row.cut ] = true
	if not bySong[ row.~~subject~~ ] then		if not bySong[ row.song ] then
	bySong[ row.~~subject~~ ] = { number = tonumber( row.c ), cuts = {} }		bySong[ row.song ] = { number = tonumber( row.number ), cuts = {} }
	table.insert( order, row.~~subject~~ )		table.insert( order, row.song )
	end		end
	local entry = bySong[ row.~~subject~~ ]		local entry = bySong[ row.song ]
	entry.number = entry.number or tonumber( row.c )		entry.number = entry.number or tonumber( row.number )
	table.insert( entry.cuts, row.b )		table.insert( entry.cuts, row.cut )
	end		end
	end		end
Line 475:		Line 493:
	local byCut = {}		local byCut = {}
	for _, row in ipairs( personnel ) do		for _, row in ipairs( personnel ) do
	if row.b ~= "" and row.c ~= "" then		if row.cut ~= "" and row.performer ~= "" then
	byCut[ row.b ] = byCut[ row.b ] or {}		byCut[ row.cut ] = byCut[ row.cut ] or {}
	table.insert( byCut[ row.b ], { name = row.c, instrument = split( row.d )[1] or "" } )		table.insert( byCut[ row.cut ],
			{ name = row.performer, instrument = split( row.instrument )[1] or "" } )
	end		end
	end		end

Magent: A cut is its own thing now: identified by block height and studio, with zero or more appearances on records (via update-page on MediaWiki MCP Server)

2026-09-19T00:16:39Z

A cut is its own thing now: identified by block height and studio, with zero or more appearances on records (via update-page on MediaWiki MCP Server)

Show changes

Magent: Move recordings onto composition pages: records and musicians now read them rather than holding them (via update-page on MediaWiki MCP Server)

2026-09-18T23:42:35Z

Move recordings onto composition pages: records and musicians now read them rather than holding them (via update-page on MediaWiki MCP Server)

Show changes

Magent: Split into Creative Commons and other studio work, and name whose record it is (via update-page on MediaWiki MCP Server)

2026-09-18T17:32:22Z

Split into Creative Commons and other studio work, and name whose record it is (via update-page on MediaWiki MCP Server)

← Older revision		Revision as of 17:32, 18 September 2026
Line 163:		Line 163:
	table.insert( out, "</div>" )		table.insert( out, "</div>" )
	return table.concat( out, "\n" )		return table.concat( out, "\n" )
			end

			-- One property of one page, as plain text. Asked of the record's own page, so
			-- that the licence and the credited artist live where the record is described
			-- rather than being repeated on every track.
			local function property( frame, page, name )
			return trim( plain( frame:callParserFunction( "#show", {
			page, "?" .. name, "link=none", "default=",
			} ) ) )
	end		end

	-- Whether a record is under a licence that lets people share it.		-- Whether a record is under a licence that lets people share it.
	--
	~~-- Asked of the record's own page, so the answer lives where the record is~~
	~~-- described rather than being repeated on every track.~~
	local function freelyLicensed( frame, record )		local function freelyLicensed( frame, record )
	local licence = ~~plain~~( frame~~:callParserFunction( "#show"~~, {		local licence = property( frame, record, "Has license" )
	record, "?Has license"~~, "link=none", "default=",~~
	~~} )~~ )
	local lower = licence:lower()		local lower = licence:lower()
	local free = lower:find( "cc " ) or lower:find( "creative commons" )		local free = lower:find( "cc " ) or lower:find( "creative commons" )
	or lower:find( "cc0" ) or lower:find( "public domain" )		or lower:find( "cc0" ) or lower:find( "public domain" )
	return ( free ~= nil ), ~~trim(~~ licence )		return ( free ~= nil ), licence
	end		end

	-- Every track a musician appears on, grouped by record.		-- Every track a musician appears on, grouped by record.
	--		--
	-- ~~Records~~ anybody may share ~~come first and in full;~~ the ~~rest are listed after,~~		-- @param frame
	~~-- more quietly~~. ~~That ordering is a claim about what this wiki is for: a~~		-- @param wanted "free" for records anybody may share, "rest" for the others.
	-- ~~recording you can pass~~ to ~~somebody is worth more here than one you can only~~		-- @param heading the section heading to put above them.
	~~-- point at~~.		local function listing( frame, wanted, heading )
	function ~~p.appearances~~( frame )
	local who = trim( callArgs( frame )[1] or frame.args[1] )		local who = trim( callArgs( frame )[1] or frame.args[1] )
	if who == "" then		if who == "" then
Line 236:		Line 239:
	end		end

	local ~~free, paid~~ = ~~{},~~ {}		local blocks = {}
	for _, record in ipairs( order ) do		for _, record in ipairs( order ) do
	local entry = byRecord[ record ]		local entry = byRecord[ record ]
Line 250:		Line 253:

	local isFree, licence = freelyLicensed( frame, record )		local isFree, licence = freelyLicensed( frame, record )
	local instruments = table.concat( entry.instruments, ", " )		if ( wanted == "free" ) == isFree then
	local line = {		local artist = property( frame, record, "Has record artist" )
	'<div style="padding:.45em .2em; border-bottom:1px solid #d4dacd;">',		local instruments = table.concat( entry.instruments, ", " )
	'<div style="font-weight:600;">[[' .. record .. "]]",		local line = {
	}		'<div style="padding:.45em .2em; border-bottom:1px solid #d4dacd;">',
	if isFree then		'<div style="font-weight:600;">[[' .. record .. "]]",
	table.insert( line, ' <span style="font-size:.72em; letter-spacing:.08em;'		}
	.. ' text-transform:uppercase; color:#3d6b3d; border:1px solid #9bb89b;'		-- Whose record it is. A picker's page should say whose session
	.. ' border-radius:3px; padding:0 .35em;">' .. ( ~~licence~~ ~= "" and ~~licence~~ or "~~free to share~~" )		-- they played on without making the reader open the record.
	.. "</~~span~~>" )		if artist ~= "" then
			table.insert( line, ' <span style="font-weight:400; color:#5a6858;">by [['
			.. artist .. "]]</span>" )
			end
			if isFree and licence ~= "" then
			table.insert( line, ' <span style="font-size:.72em; letter-spacing:.08em;'
			.. ' text-transform:uppercase; color:#3d6b3d; border:1px solid #9bb89b;'
			.. ' border-radius:3px; padding:0 .35em;">' .. licence .. "</span>" )
			end
			table.insert( line, "</div>" )
			table.insert( line, '<div style="font-size:.9em; color:#3d4a3e;">'
			.. ( instruments ~= "" and instruments .. " · " or "" )
			.. ( #titles == 1 and titles[1] or ( #titles .. " tracks: " .. table.concat( titles, ", " ) ) )
			.. "</div>" )
			table.insert( line, "</div>" )

			table.insert( blocks, table.concat( line, "" ) )
	end		end
	~~table.insert( line, "</div>" )~~		end
	~~table.insert( line, '<div style="font-size:.9em; color:#3d4a3e;">'~~
	~~.. ( instruments ~= "" and instruments .. " · " or "" )~~
	~~.. ( #titles == 1 and titles[1] or ( #titles .. " tracks: " .. table.concat( titles, ", " ) ) )~~
	~~.. "</div>" )~~
	~~table.insert( line, "</div>" )~~

	~~table.insert( isFree and free or paid, table.concat( line,~~ "" ~~) )~~		if #blocks == 0 then
			return ""
	end		end

	~~local out = {~~ "== ~~In the studio~~ ==" }		table.insert( blocks, 1, "== " .. heading .. " ==" )
	~~for _, block in ipairs( free ) do~~		return table.concat( blocks, "\n" )
	table.~~insert~~( ~~out~~, ~~block~~ )		end
	end
	~~if #paid > 0 then~~		-- The records a reader may share, which is what this wiki is for.
	~~if #free > 0 then~~		function p.appearances( frame )
	~~table~~.~~insert~~( ~~out~~, ~~'<div style=~~"~~font-size:.72em; letter-spacing:.14em;'~~		return listing( frame, "free",
	~~.. ' text-transform:uppercase; color:#6b7566; margin:.8em 0 .2em;~~">'		"[[PickiPedia:Creative Commons\|Creative Commons]] Studio Work" )
	.. "~~Also on</div>~~" )		end
	end
	~~for _~~, ~~block in ipairs( paid )~~ do		-- Everything else the picker played on. Worth recording, and kept out of the
	~~table~~.~~insert~~( ~~out, block~~ )		-- section above, which is a promise about what a reader can do with what they
	~~end~~		-- find there.
	~~end~~		function p.other( frame )
	return ~~table.concat~~( ~~out~~, "\n" )		return listing( frame, "rest", "Other studio work" )
	end		end

	return p		return p

Magent: Strip SMW's SMW::off switch out of query results, which was becoming part of the first record's name (via update-page on MediaWiki MCP Server)

2026-09-17T21:45:03Z

Strip SMW's SMW::off switch out of query results, which was becoming part of the first record's name (via update-page on MediaWiki MCP Server)

← Older revision		Revision as of 21:45, 17 September 2026
Line 53:		Line 53:
	-- pairs(), not next(). Scribunto's args table is lazy: values arrive		-- pairs(), not next(). Scribunto's args table is lazy: values arrive
	-- through a metatable and the table itself is empty, so next() reports		-- through a metatable and the table itself is empty, so next() reports
	-- no arguments on a call that has plenty~~. That cost a second round of~~		-- no arguments on a call that has plenty.
	~~-- "no track name given"~~.
	for _ in pairs( parent.args ) do		for _ in pairs( parent.args ) do
	return parent.args		return parent.args
Line 81:		Line 80:
	local function escape( value )		local function escape( value )
	return ( value:gsub( "\|", "{{!}}" ):gsub( "=", "{{=}}" ) )		return ( value:gsub( "\|", "{{!}}" ):gsub( "=", "{{=}}" ) )
			end

			-- Everything the parser wraps around a value that is not part of the value.
			--
			-- Three layers, each found the hard way: the strip marker the parser hands
			-- back, the span markup SMW adds even with link=none, and SMW's own
			-- [[SMW::off]] switch, which brackets template-formatted results so a result
			-- cannot annotate the page showing it. That last one made the first record's
			-- name begin "[[SMW::off]]", so "[[" .. name .. "]]" rendered as a warning
			-- about a property label containing "[" rather than as a link.
			local function plain( text )
			text = mw.text.unstrip( text or "" )
			text = text:gsub( "\127[^\127]*\127", "" ):gsub( "<[^>]+>", "" )
			return ( text:gsub( "%[%[SMW::o[nf]f?%]%]", "" ) )
	end		end

Line 157:		Line 170:
	-- described rather than being repeated on every track.		-- described rather than being repeated on every track.
	local function freelyLicensed( frame, record )		local function freelyLicensed( frame, record )
	local licence = frame:callParserFunction( "#show", {		local licence = plain( frame:callParserFunction( "#show", {
	record, "?Has license", "link=none", "default=",		record, "?Has license", "link=none", "default=",
	} )		} ) )
	~~licence = mw.text.unstrip( licence or "" )~~
	~~licence = licence:gsub( "\127[^\127]*\127", "" ):gsub( "<[^>]+>", ""~~ )
	local lower = licence:lower()		local lower = licence:lower()
	local free = lower:find( "cc " ) or lower:find( "creative commons" )		local free = lower:find( "cc " ) or lower:find( "creative commons" )
Line 180:		Line 191:
	end		end

	local raw = frame:callParserFunction( "#ask", {		local raw = plain( frame:callParserFunction( "#ask", {
	"[[Has performer::" .. who .. "]]",		"[[Has performer::" .. who .. "]]",
	"?Has track name=track",		"?Has track name=track",
Line 192:		Line 203:
	"searchlabel=",		"searchlabel=",
	"limit=200",		"limit=200",
	} )		} ) )
	~~-- Two cleanups, both learned the hard way. unstrip() resolves the strip~~
	~~-- marker the parser hands back, but markers can survive nested inside it,~~
	~~-- and SMW wraps values in span markup even with link=none. Left in, the~~
	~~-- record's name is not a name: "[[" .. that .. "]]" rendered as a warning~~
	~~-- about a property label containing "[", followed by a stray "]]".~~
	~~raw = mw.text.unstrip( raw or "" )~~
	~~raw = raw:gsub( "\127[^\127]*\127", "" ):gsub( "<[^>]+>", ""~~ )
	if trim( raw ) == "" then		if trim( raw ) == "" then
	return ""		return ""

Magent: Strip leftover strip markers and SMW span markup from query results, or the record name is not a name (via update-page on MediaWiki MCP Server)

2026-09-17T21:39:02Z

Strip leftover strip markers and SMW span markup from query results, or the record name is not a name (via update-page on MediaWiki MCP Server)

← Older revision		Revision as of 21:39, 17 September 2026
Line 160:		Line 160:
	record, "?Has license", "link=none", "default=",		record, "?Has license", "link=none", "default=",
	} )		} )
	licence = mw.text.unstrip( licence or "" ):gsub( "<[^>]+>", "" )		licence = mw.text.unstrip( licence or "" )
			licence = licence:gsub( "\127[^\127]*\127", "" ):gsub( "<[^>]+>", "" )
	local lower = licence:lower()		local lower = licence:lower()
	local free = lower:find( "cc " ) or lower:find( "creative commons" )		local free = lower:find( "cc " ) or lower:find( "creative commons" )
Line 192:		Line 193:
	"limit=200",		"limit=200",
	} )		} )
			-- Two cleanups, both learned the hard way. unstrip() resolves the strip
			-- marker the parser hands back, but markers can survive nested inside it,
			-- and SMW wraps values in span markup even with link=none. Left in, the
			-- record's name is not a name: "[[" .. that .. "]]" rendered as a warning
			-- about a property label containing "[", followed by a stray "]]".
	raw = mw.text.unstrip( raw or "" )		raw = mw.text.unstrip( raw or "" )
			raw = raw:gsub( "\127[^\127]*\127", "" ):gsub( "<[^>]+>", "" )
	if trim( raw ) == "" then		if trim( raw ) == "" then
	return ""		return ""

Magent: Store subobjects through the preprocessor: without Semantic Scribunto, a #subobject called from Lua renders and is then forgotten (via update-page on MediaWiki MCP Server)

2026-09-17T21:36:03Z

Store subobjects through the preprocessor: without Semantic Scribunto, a #subobject called from Lua renders and is then forgotten (via update-page on MediaWiki MCP Server)

← Older revision		Revision as of 21:36, 17 September 2026
Line 75:		Line 75:
	table.sort( named, function( a, b ) return a.name < b.name end )		table.sort( named, function( a, b ) return a.name < b.name end )
	return named		return named
			end

			-- A pipe or an equals sign in a value would end the parameter early, so they
			-- are written as the templates that stand for them.
			local function escape( value )
			return ( value:gsub( "\|", "{{!}}" ):gsub( "=", "{{=}}" ) )
	end		end

	-- Record one player on one track, as data a query can reach.		-- Record one player on one track, as data a query can reach.
			--
			-- Written as wikitext and run back through the preprocessor rather than called
			-- with frame:callParserFunction. This wiki has no Semantic Scribunto, and a
			-- #subobject invoked that way renders and is then forgotten: the page looked
			-- right and [[Has performer::+]] matched nothing at all. Preprocessing it is
			-- what actually stores it.
	local function annotate( frame, track, number, player )		local function annotate( frame, track, number, player )
	local ~~args~~ = {		local parts = {
	~~"",~~		"Has track name=" .. escape( track ),
	"Has track name=" .. track,		"Has performer=" .. escape( player.name ),
	"Has performer=" .. player.name,
	}		}
	if number ~= "" then		if number ~= "" then
	table.insert( ~~args~~, "Has track number=" .. number )		table.insert( parts, "Has track number=" .. escape( number ) )
	end		end
	for _, instrument in ipairs( split( player.instruments ) ) do		for _, instrument in ipairs( split( player.instruments ) ) do
	table.insert( ~~args~~, "Has instrument=" .. instrument )		table.insert( parts, "Has instrument=" .. escape( instrument ) )
	end		end
	frame:~~callParserFunction~~( "#subobject", ~~args~~ )		frame:preprocess( "{{#subobject:\|" .. table.concat( parts, "\|" ) .. "}}" )
	end		end