Module:Pistes - Historique des versions

Wreck : 56 versions importées

2026-03-11T22:58:55Z

56 versions importées

← Version précédente	Version du 12 mars 2026 à 00:58
(Aucune différence)

Modèle:Pistes>Od1n : le nom de cette variable était trompeur : elle ne contient pas le nombre de pistes, mais le numéro de la dernière piste

2024-10-20T18:56:29Z

le nom de cette variable était trompeur : elle ne contient pas le nombre de pistes, mais le numéro de la dernière piste

← Version précédente		Version du 20 octobre 2024 à 20:56
Ligne 9 :		Ligne 9 :
	local args = {}		local args = {}
	local rowArgsTable = {}		local rowArgsTable = {}
	local ~~nbPistes~~ = 0		local maxNumPiste = 0

	-- map de noms de paramètres "alias → nom canonique" (attention à bien utiliser le nom canonique dans le code du module)		-- map de noms de paramètres "alias → nom canonique" (attention à bien utiliser le nom canonique dans le code du module)
Ligne 56 :		Ligne 56 :
	num = tonumber(num)		num = tonumber(num)
	if rowArgsTable[num] == nil then		if rowArgsTable[num] == nil then
	if num > ~~nbPistes~~ then		if num > maxNumPiste then
	~~nbPistes~~ = num		maxNumPiste = num
	end		end
	rowArgsTable[num] = {}		rowArgsTable[num] = {}
Ligne 184 :		Ligne 184 :

	-- 5. Tracé des lignes du tableau		-- 5. Tracé des lignes du tableau
	for i = 0, ~~nbPistes~~ do -- on commence à 0, pour éventuel [[morceau caché]] dans le [[prégap]]		for i = 0, maxNumPiste do -- on commence à 0, pour éventuel [[morceau caché]] dans le [[prégap]]
	local rowArgs = rowArgsTable[i]		local rowArgs = rowArgsTable[i]

Modèle:Pistes>Od1n : remplacement de "nb" (pour "nombre") par "num" (pour "numéro")

2024-10-20T18:53:46Z

remplacement de "nb" (pour "nombre") par "num" (pour "numéro")

← Version précédente		Version du 20 octobre 2024 à 20:53
Ligne 45 :		Ligne 45 :
	alternative pattern: ^(.+)%f[%d]([1-9]%d*)$		alternative pattern: ^(.+)%f[%d]([1-9]%d*)$
	--]]		--]]
	local param, nb = string.match( k, '^(.%D)([1-9]%d)$' ) -- in this specific case, multibyte mw.ustring.match() is not needed		local param, num = string.match( k, '^(.%D)([1-9]%d)$' ) -- in this specific case, multibyte mw.ustring.match() is not needed
	if not param then		if not param then
	param, nb = string.match( k, '^(.*%D)(0)$' ) -- also search for track zero		param, num = string.match( k, '^(.*%D)(0)$' ) -- also search for track zero
	end		end
	if param then		if param then
Ligne 54 :		Ligne 54 :
	end		end
	if v ~= "" or param == "numero" then -- il est possible de renseigner des paramètres « numeroN » vides		if v ~= "" or param == "numero" then -- il est possible de renseigner des paramètres « numeroN » vides
	nb = tonumber(nb)		num = tonumber(num)
	if rowArgsTable[nb] == nil then		if rowArgsTable[num] == nil then
	if nb > nbPistes then		if num > nbPistes then
	nbPistes = nb		nbPistes = num
	end		end
	rowArgsTable[nb] = {}		rowArgsTable[num] = {}
	end		end
	rowArgsTable[nb][param] = v		rowArgsTable[num][param] = v
	end		end
	else		else

Modèle:Pistes>Od1n : correction

2024-10-20T17:15:53Z

correction

← Version précédente		Version du 20 octobre 2024 à 19:15
Ligne 47 :		Ligne 47 :
	local param, nb = string.match( k, '^(.%D)([1-9]%d)$' ) -- in this specific case, multibyte mw.ustring.match() is not needed		local param, nb = string.match( k, '^(.%D)([1-9]%d)$' ) -- in this specific case, multibyte mw.ustring.match() is not needed
	if not param then		if not param then
	~~local~~ param, nb = string.match( k, '^(.*%D)(0)$' ) -- also search for track zero		param, nb = string.match( k, '^(.*%D)(0)$' ) -- also search for track zero
	end		end
	if param then		if param then

Modèle:Pistes>Od1n : correction régression, signalée dans la discussion : il faut aussi détecter la piste zéro (pregap hidden track ?) (vraiment dommage que les patterns Lua ne supportent pas les alternations…) ; noter que j'ai détecté une seule utilisation avec piste zéro sur tout le wiki

2024-10-20T16:40:21Z

correction régression, signalée dans la discussion : il faut aussi détecter la piste zéro (pregap hidden track ?) (vraiment dommage que les patterns Lua ne supportent pas les alternations…) ; noter que j'ai détecté une seule utilisation avec piste zéro sur tout le wiki

← Version précédente		Version du 20 octobre 2024 à 18:40
Ligne 36 :		Ligne 36 :
	* foo21bar42 (digits in base name)		* foo21bar42 (digits in base name)
	* 21foo42 (leading digits in base name)		* 21foo42 (leading digits in base name)
			* foobar0 (track zero)

	do not match:		do not match:
Ligne 45 :		Ligne 46 :
	--]]		--]]
	local param, nb = string.match( k, '^(.%D)([1-9]%d)$' ) -- in this specific case, multibyte mw.ustring.match() is not needed		local param, nb = string.match( k, '^(.%D)([1-9]%d)$' ) -- in this specific case, multibyte mw.ustring.match() is not needed
			if not param then
			local param, nb = string.match( k, '^(.*%D)(0)$' ) -- also search for track zero
			end
	if param then		if param then
	if aliasesN[param] then		if aliasesN[param] then

Modèle:Pistes>Od1n : suite à l'edit précédent, ajustement pour une valeur permettant une plus grande détection de bug en testant les patterns

2024-10-20T02:59:15Z

suite à l'edit précédent, ajustement pour une valeur permettant une plus grande détection de bug en testant les patterns

← Version précédente		Version du 20 octobre 2024 à 04:59
Ligne 40 :		Ligne 40 :
	* 42 (only digits)		* 42 (only digits)
	* foobar (no digits suffix)		* foobar (no digits suffix)
	* ~~foobar04~~ (leading zeroes)		* foobar042 (leading zeroes)

	alternative pattern: ^(.+)%f[%d]([1-9]%d*)$		alternative pattern: ^(.+)%f[%d]([1-9]%d*)$

Modèle:Pistes>Od1n : c'était encore erroné : "foobar04" était bien rejeté, mais "foobar042" donnait param = "foobar04" et nb = "2" ; remplacement par l'un des deux autres patterns, qui étaient de toute façon devenus préférables

2024-10-20T02:44:26Z

c'était encore erroné : "foobar04" était bien rejeté, mais "foobar042" donnait param = "foobar04" et nb = "2" ; remplacement par l'un des deux autres patterns, qui étaient de toute façon devenus préférables

← Version précédente		Version du 20 octobre 2024 à 04:44
Ligne 42 :		Ligne 42 :
	* foobar04 (leading zeroes)		* foobar04 (leading zeroes)

	alternative ~~patterns~~: ~~^(.%D)([1-9]%d)$ and~~ ^(.+)%f[%d]([1-9]%d*)$		alternative pattern: ^(.+)%f[%d]([1-9]%d*)$
	--]]		--]]
	local param, nb = string.match( k, '^(.~~-[^0]~~)([1-9]%d*)$' ) -- in this specific case, multibyte mw.ustring.match() is not needed		local param, nb = string.match( k, '^(.%D)([1-9]%d)$' ) -- in this specific case, multibyte mw.ustring.match() is not needed
	if param then		if param then
	if aliasesN[param] then		if aliasesN[param] then

Modèle:Pistes>Od1n : du coup, le pattern utilisé ne fait plus usage de la condition suivante « param ~= '' », comme les deux autres patterns proposés ; je simplifie en ne mentionnant plus cette considération

2024-10-20T02:37:02Z

du coup, le pattern utilisé ne fait plus usage de la condition suivante « param ~= '' », comme les deux autres patterns proposés ; je simplifie en ne mentionnant plus cette considération

← Version précédente		Version du 20 octobre 2024 à 04:37
Ligne 38 :		Ligne 38 :

	do not match:		do not match:
	* 42 (only digits~~; the regex matches actually, but we reject in the following « param ~= '' » conditional~~)		* 42 (only digits)
	* foobar (no digits suffix)		* foobar (no digits suffix)
	* foobar04 (leading zeroes)		* foobar04 (leading zeroes)

	alternative patterns~~, that don't need the « param ~= '' » conditional~~: ^(.%D)([1-9]%d)$ and ^(.+)%f[%d]([1-9]%d*)$		alternative patterns: ^(.%D)([1-9]%d)$ and ^(.+)%f[%d]([1-9]%d*)$
	--]]		--]]
	local param, nb = string.match( k, '^(.-[^0])([1-9]%d*)$' ) -- in this specific case, multibyte mw.ustring.match() is not needed		local param, nb = string.match( k, '^(.-[^0])([1-9]%d*)$' ) -- in this specific case, multibyte mw.ustring.match() is not needed
	if param ~~and param ~= ''~~ then		if param then
	if aliasesN[param] then		if aliasesN[param] then
	param = aliasesN[param]		param = aliasesN[param]

Modèle:Pistes>Od1n : correction d'un bug traître : le « (.-) » capturait ces zéros, ainsi ça acceptait encore le paramètre, et le découpage était incorrect, par exemple "foobar04" aurait donné param = "foobar0" et nb = "4"

2024-10-20T02:29:25Z

correction d'un bug traître : le « (.-) » capturait ces zéros, ainsi ça acceptait encore le paramètre, et le découpage était incorrect, par exemple "foobar04" aurait donné param = "foobar0" et nb = "4"

← Version précédente		Version du 20 octobre 2024 à 04:29
Ligne 44 :		Ligne 44 :
	alternative patterns, that don't need the « param ~= '' » conditional: ^(.%D)([1-9]%d)$ and ^(.+)%f[%d]([1-9]%d*)$		alternative patterns, that don't need the « param ~= '' » conditional: ^(.%D)([1-9]%d)$ and ^(.+)%f[%d]([1-9]%d*)$
	--]]		--]]
	local param, nb = string.match( k, '^(.-)([1-9]%d*)$' ) -- in this specific case, multibyte mw.ustring.match() is not needed		local param, nb = string.match( k, '^(.-[^0])([1-9]%d*)$' ) -- in this specific case, multibyte mw.ustring.match() is not needed
	if param and param ~= '' then		if param and param ~= '' then
	if aliasesN[param] then		if aliasesN[param] then

Modèle:Pistes>Od1n : à la réflexion, je préfère ne pas supporter les leading zeroes dans ces noms de paramètres ; ça fait un truc magique en moins, comme ça il y a un seul nom pour le paramètre, pas de surprise ; et à la rigueur, il reste toujours possible de remettre cet assouplissement plus tard ; j'ai vérifié, il n'y avait aucune utilisation avec de tels leading zeroes

2024-10-20T02:29:12Z

à la réflexion, je préfère ne pas supporter les leading zeroes dans ces noms de paramètres ; ça fait un truc magique en moins, comme ça il y a un seul nom pour le paramètre, pas de surprise ; et à la rigueur, il reste toujours possible de remettre cet assouplissement plus tard ; j'ai vérifié, il n'y avait aucune utilisation avec de tels leading zeroes

← Version précédente		Version du 20 octobre 2024 à 04:29
Ligne 36 :		Ligne 36 :
	* foo21bar42 (digits in base name)		* foo21bar42 (digits in base name)
	* 21foo42 (leading digits in base name)		* 21foo42 (leading digits in base name)
	* foobar04 (leading zeroes)

	do not match:		do not match:
	* 42 (only digits; the regex matches actually, but we reject in the following « param ~= '' » conditional)		* 42 (only digits; the regex matches actually, but we reject in the following « param ~= '' » conditional)
	* foobar (no digits suffix)		* foobar (no digits suffix)
			* foobar04 (leading zeroes)

	alternative patterns, that don't need the « param ~= '' » conditional: ^(.*%D)(%d+)$ and ^(.+)%f[%d](%d+)$		alternative patterns, that don't need the « param ~= '' » conditional: ^(.%D)([1-9]%d)$ and ^(.+)%f[%d]([1-9]%d*)$
	--]]		--]]
	local param, nb = string.match( k, '^(.-)(%d+)$' ) -- in this specific case, multibyte mw.ustring.match() is not needed		local param, nb = string.match( k, '^(.-)([1-9]%d*)$' ) -- in this specific case, multibyte mw.ustring.match() is not needed
	if param and param ~= '' then		if param and param ~= '' then
	if aliasesN[param] then		if aliasesN[param] then

Module:Pistes - Historique des versions

Wreck : 56 versions importées

Modèle:Pistes>Od1n : le nom de cette variable était trompeur : elle ne contient pas le nombre de pistes, mais le numéro de la dernière piste

Modèle:Pistes>Od1n : remplacement de "nb" (pour "nombre") par "num" (pour "numéro")

Modèle:Pistes>Od1n : correction

Modèle:Pistes>Od1n : suite à l'edit précédent, ajustement pour une valeur permettant une plus grande détection de bug en testant les patterns

Modèle:Pistes>Od1n : c'était encore erroné : "foobar04" était bien rejeté, mais "foobar042" donnait param = "foobar04" et nb = "2" ; remplacement par l'un des deux autres patterns, qui étaient de toute façon devenus préférables

Modèle:Pistes>Od1n : du coup, le pattern utilisé ne fait plus usage de la condition suivante « param ~= '' », comme les deux autres patterns proposés ; je simplifie en ne mentionnant plus cette considération

Modèle:Pistes>Od1n : correction d'un bug traître : le « (.-) » capturait ces zéros, ainsi ça acceptait encore le paramètre, et le découpage était incorrect, par exemple "foobar04" aurait donné param = "foobar0" et nb = "4"