luci-base: remove unused i18n functions
authorJo-Philipp Wich <jo@mein.io>
Thu, 18 Oct 2018 11:32:52 +0000 (13:32 +0200)
committerJo-Philipp Wich <jo@mein.io>
Mon, 5 Nov 2018 10:01:45 +0000 (11:01 +0100)
Drop load(), loadc(), string() and stringf() from the luci.i18n class since
these functions are either no longer unused or were never used to begin with.

Also slightly rework the module to only use local symbols and unify the
module require style.

Signed-off-by: Jo-Philipp Wich <jo@mein.io>
modules/luci-base/luasrc/i18n.lua
modules/luci-base/luasrc/i18n.luadoc

index 42de832f7b95ab5b045041264ad7d133aacd5aba..323912b65022ef10cfe3dd63315cca8c99896f38 100644 (file)
@@ -1,26 +1,16 @@
 -- Copyright 2008 Steven Barth <steven@midlink.org>
 -- Licensed to the public under the Apache License 2.0.
 
-module("luci.i18n", package.seeall)
-require("luci.util")
+local tparser  = require "luci.template.parser"
+local util     = require "luci.util"
+local tostring = tostring
 
-local tparser = require "luci.template.parser"
+module "luci.i18n"
 
-table   = {}
-i18ndir = luci.util.libpath() .. "/i18n/"
-loaded  = {}
-context = luci.util.threadlocal()
+i18ndir = util.libpath() .. "/i18n/"
+context = util.threadlocal()
 default = "en"
 
-function clear()
-end
-
-function load(file, lang, force)
-end
-
--- Alternatively load the translation of the fallback language.
-function loadc(file, force)
-end
 
 function setlanguage(lang)
        local code, subcode = lang:match("^([A-Za-z][A-Za-z])[%-_]([A-Za-z][A-Za-z])$")
@@ -58,18 +48,6 @@ function translatef(key, ...)
        return tostring(translate(key)):format(...)
 end
 
--- and ensure that the returned value is a Lua string value.
--- This is the same as calling <code>tostring(translate(...))</code>
-function string(key)
-       return tostring(translate(key))
-end
-
--- Ensure that the returned value is a Lua string value.
--- This is the same as calling <code>tostring(translatef(...))</code>
-function stringf(key, ...)
-       return tostring(translate(key)):format(...)
-end
-
 function dump()
        local rv = {}
        tparser.get_translations(function(k, v) rv[k] = v end)
index df6e38e5de38ca8b25afe5c6cff3d13e75b7c14a..b76c298565533b8c53d378cacb326155ab05a229 100644 (file)
@@ -3,35 +3,6 @@ LuCI translation library.
 ]]
 module "luci.i18n"
 
----[[
-Clear the translation table.
-
-@class function
-@name clear
-]]
-
----[[
-Load a translation and copy its data into the translation table.
-
-@class function
-@name load
-@param file    Language file
-@param lang    Two-letter language code
-@param force   Force reload even if already loaded (optional)
-@return                Success status
-]]
-
----[[
-Load a translation file using the default translation language.
-
-Alternatively load the translation of the fallback language.
-
-@class function
-@name loadc
-@param file    Language file
-@param force   Force reload even if already loaded (optional)
-]]
-
 ---[[
 Set the context default translation language.
 
@@ -60,32 +31,6 @@ Return the translated value for a specific translation key and use it as sprintf
 @return                        Translated and formatted string
 ]]
 
----[[
-Return the translated value for a specific translation key
-and ensure that the returned value is a Lua string value.
-
-This is the same as calling <code>tostring(translate(...))</code>
-
-@class function
-@name string
-@param key             Default translation text
-@return                        Translated string
-]]
-
----[[
-Return the translated value for a specific translation key and use it as sprintf pattern.
-
-Ensure that the returned value is a Lua string value.
-
-This is the same as calling <code>tostring(translatef(...))</code>
-
-@class function
-@name stringf
-@param key             Default translation text
-@param ...             Format parameters
-@return                        Translated and formatted string
-]]
-
 ---[[
 Return all currently loaded translation strings as a key-value table. The key is the
 hexadecimal representation of the translation key while the value is the translated