Diff

plugins/mod_version.lua @ 9435:33301038d3e4

mod_version: Rename confusingly named variable Maybe this meant the version of the OS, but it's still confusing.
author Kim Alvefur <zash@zash.se>
date Sat, 06 Oct 2018 16:32:37 +0200
parent 9434:b502766a10d7
child 9436:a950f9fa9137
line wrap: on
line diff
--- a/plugins/mod_version.lua	Sat Oct 06 16:29:42 2018 +0200
+++ b/plugins/mod_version.lua	Sat Oct 06 16:32:37 2018 +0200
@@ -10,7 +10,7 @@
 
 module:add_feature("jabber:iq:version");
 
-local version;
+local platform;
 
 local query = st.stanza("query", {xmlns = "jabber:iq:version"})
 	:text_tag("name", "Prosody")
@@ -18,24 +18,24 @@
 
 if not module:get_option_boolean("hide_os_type") then
 	if os.getenv("WINDIR") then
-		version = "Windows";
+		platform = "Windows";
 	else
 		local os_version_command = module:get_option_string("os_version_command");
 		local ok, pposix = pcall(require, "util.pposix");
 		if not os_version_command and (ok and pposix and pposix.uname) then
-			version = pposix.uname().sysname;
+			platform = pposix.uname().sysname;
 		end
-		if not version then
+		if not platform then
 			local uname = io.popen(os_version_command or "uname");
 			if uname then
-				version = uname:read("*a");
+				platform = uname:read("*a");
 			end
 			uname:close();
 		end
 	end
-	if version then
-		version = version:match("^%s*(.-)%s*$") or version;
-		query:text_tag("os", version);
+	if platform then
+		platform = platform:match("^%s*(.-)%s*$") or platform;
+		query:text_tag("os", platform);
 	end
 end