יחידה:NUMBEROF – הבדלי גרסאות

עיון אינטראקטיבי בהיסטוריה

→ העריכה הקודמת העריכה הבאה ←

@@ שורה 1: / שורה 1: @@
+local p = {}
 local function trimArg(arg, i)
 	arg = mw.text.trim(arg or '')
@@ שורה 12: / שורה 14: @@
 local function getValue(stats, action, map)
 	if action == 'depth' then
-		-- https://chabadpedia.co.il/index.php/חב%22דפדיה
-		-- This gives silly results if, for example, the number of articles is small.
 		local n = { 'articles', 'edits', 'pages' }
 		if map then
@@ שורה 36: / שורה 36: @@
 local function getIfLocal(site, action)
-	-- If wanted site is the local site where module is running,
-	-- return numberof result for given action, or nil.
-	-- This is faster than reading the cached table, and gives the current value.
 	local localSite = string.match(mw.site.server, 'https?://(.-)%.com$')
-              or string.match(mw.site.server, 'https?://(.-)%.co%.il$')  -- examples: 'chabadpedia', 'text.chabadpedia'
+		or string.match(mw.site.server, 'https?://(.-)%.co%.il$')
 	if site == localSite then
 		if action == 'activeusers' then
@@ שורה 49: / שורה 46: @@
 end
-local function main(frame)
+function p.main(frame)
 	local args = frame:getParent().args
-	local action = trimArg(args[1], 1)  -- activeusers, admins, articles, edits, files, pages, users, depth
+	local action = trimArg(args[1], 1) -- כמו "articles"
-	if action:sub(1, 8) == 'numberof' then  -- numberofX is an alias for X
+	if not action then return -1 end
+	if action:sub(1, 8) == 'numberof' then
 		action = trimArg(action:sub(9), 1)
 	end
-	local site = trimArg(args[2], 2)  -- "af" or "text.chabadpedia" or "zitut.chabadpedia" etc., including "total"
+	local site = trimArg(args[2], 2) or 'chabadpedia.co.il'
 	if not site:find('.', 1, true) then
 		site = site .. '.chabadpedia'
 	end
-	local wantComma = trimArg(args[3])  -- nil for no commas in output; "N" or anything nonblank inserts commas
+	local wantComma = trimArg(args[3])
+	-- בדיקה אם מדובר באתר המקומי
 	local result = getIfLocal(site, action)
+	-- אם לא נמצא מקומית – ננסה מהקובץ data
 	if not result then
 		local data = mw.loadData('Module:NUMBEROF/data')
 		local map = data.map
 		data = data.data
-		result = data[site]
+		local siteData = data[site]
-		if result then
+		if siteData then
-			result = getValue(result, action, map)
+			result = getValue(siteData, action, map)
 		end
 	end
 	if result then
 		if wantComma then
 			result = mw.getContentLanguage():formatNum(result)
 		end
-		return result  -- number or formatted string
+		return result
 	end
 	return -1
 end
-local function rank(frame)
+function p.rank(frame)
-	-- Rank sites in a specified sister project by their number of articles.
 	local args = frame:getParent().args
-	local parm = trimArg(args[1], 1)  -- a number like 12 or a site name like "text" (not "text.chabadpedia")
+	local parm = trimArg(args[1], 1)
-	local base = trimArg(args[2]) or 'chabadpedia'  -- base of full site name like "chabadpedia"
+	local base = trimArg(args[2]) or 'chabadpedia'
 	local wantComma = trimArg(args[3])
 	local data = mw.loadData('Module:NUMBEROF/' .. (base == 'chabadpedia' and 'rank' or 'other'))
@@ שורה 94: / שורה 100: @@
 		else
 			result = data.rankBySite[parm]
-			if result and wantComma then
-				result = mw.getContentLanguage():formatNum(result)
-			end
 		end
-		if result then
+		if result and wantComma then
-			return result  -- number or string
+			result = mw.getContentLanguage():formatNum(result)
 		end
+		return result or -1
 	end
 	return -1
 end
-return {
+return p
-	main = main,
-	rank = rank,
-}