Merge pull request #980 from jgmdev/PR/plugins-load-priority

plugins: add load priority support with '--priority:###' closes #978

data/core/init.lua

@@ -859,7 +859,7 @@ function core.restart()
 end
 
 
-local function check_plugin_version(filename)
+local function get_plugin_details(filename)
   local info = system.get_file_info(filename)
   if info ~= nil and info.type == "dir" then
     filename = filename .. "/init.lua"
@@ -868,24 +868,35 @@ local function check_plugin_version(filename)
   if not info or not filename:match("%.lua$") then return false end
   local f = io.open(filename, "r")
   if not f then return false end
+  local priority = false
   local version_match = false
   for line in f:lines() do
-    local mod_version = line:match('%-%-.*%f[%a]mod%-version%s*:%s*(%d+)')
+    if not version_match then
-    if mod_version then
+      local mod_version = line:match('%-%-.*%f[%a]mod%-version%s*:%s*(%d+)')
-      version_match = (mod_version == MOD_VERSION)
+      if mod_version then
-      break
+        version_match = (mod_version == MOD_VERSION)
+      end
+      -- The following pattern is used for backward compatibility only
+      -- Future versions will look only at the mod-version tag.
+      local version = line:match('%-%-%s*lite%-xl%s*(%d+%.%d+)$')
+      if version then
+        -- we consider the version tag 2.0 equivalent to mod-version:2
+        version_match = (version == '2.0' and MOD_VERSION == "2")
+      end
     end
-    -- The following pattern is used for backward compatibility only
+    if not priority then
-    -- Future versions will look only at the mod-version tag.
+      priority = line:match('%-%-.*%f[%a]priority%s*:%s*(%d+)')
-    local version = line:match('%-%-%s*lite%-xl%s*(%d+%.%d+)$')
+      if priority then priority = tonumber(priority) end
-    if version then
+    end
-      -- we consider the version tag 2.0 equivalent to mod-version:2
+    if version_match then
-      version_match = (version == '2.0' and MOD_VERSION == "2")
       break
     end
   end
   f:close()
-  return true, version_match
+  return true, {
+    version_match = version_match,
+    priority = priority or 100
+  }
 end
 
 
@@ -899,33 +910,70 @@ function core.load_plugins()
   for _, root_dir in ipairs {DATADIR, USERDIR} do
     local plugin_dir = root_dir .. "/plugins"
     for _, filename in ipairs(system.list_dir(plugin_dir) or {}) do
-      if not files[filename] then table.insert(ordered, filename) end
+      if not files[filename] then
-      files[filename] = plugin_dir -- user plugins will always replace system plugins
+        table.insert(
+          ordered, {file = filename}
+        )
+      end
+      -- user plugins will always replace system plugins
+      files[filename] = plugin_dir
     end
   end
-  table.sort(ordered)
 
+  for _, plugin in ipairs(ordered) do
+    local dir = files[plugin.file]
+    local name = plugin.file:match("(.-)%.lua$") or plugin.file
+    local is_lua_file, details = get_plugin_details(dir .. '/' .. plugin.file)
+
+    plugin.valid = is_lua_file
+    plugin.name = name
+    plugin.dir = dir
+    plugin.priority = details and details.priority or 100
+    plugin.version_match = details and details.version_match or false
+  end
+
+  -- sort by priority or name for plugins that have same priority
+  table.sort(ordered, function(a, b)
+    if a.priority ~= b.priority then
+      return a.priority < b.priority
+    end
+    return a.name < b.name
+  end)
 
   local load_start = system.get_time()
-  for _, filename in ipairs(ordered) do
+  for _, plugin in ipairs(ordered) do
-    local plugin_dir, basename = files[filename], filename:match("(.-)%.lua$") or filename
+    if plugin.valid then
-    local is_lua_file, version_match = check_plugin_version(plugin_dir .. '/' .. filename)
+      if not config.skip_plugins_version and not plugin.version_match then
-    if is_lua_file then
+        core.log_quiet(
-      if not config.skip_plugins_version and not version_match then
+          "Version mismatch for plugin %q from %s",
-        core.log_quiet("Version mismatch for plugin %q from %s", basename, plugin_dir)
+          plugin.name,
-        local list = refused_list[plugin_dir:find(USERDIR, 1, true) == 1 and 'userdir' or 'datadir'].plugins
+          plugin.dir
-        table.insert(list, filename)
+        )
-      elseif config.plugins[basename] ~= false then
+        local rlist = plugin.dir:find(USERDIR, 1, true) == 1
+          and 'userdir' or 'datadir'
+        local list = refused_list[rlist].plugins
+        table.insert(list, plugin.file)
+      elseif config.plugins[plugin.name] ~= false then
         local start = system.get_time()
-        local ok = core.try(require, "plugins." .. basename)
+        local ok = core.try(require, "plugins." .. plugin.name)
-        if ok then core.log_quiet("Loaded plugin %q from %s in %.1fms", basename, plugin_dir, (system.get_time() - start)*1000) end
+        if ok then
+          core.log_quiet(
+            "Loaded plugin %q from %s in %.1fms",
+            plugin.name,
+            plugin.dir,
+            (system.get_time() - start) * 1000
+          )
+        end
         if not ok then
           no_errors = false
         end
       end
     end
   end
-  core.log_quiet("Loaded all plugins in %.1fms", (system.get_time() - load_start)*1000)
+  core.log_quiet(
+    "Loaded all plugins in %.1fms",
+    (system.get_time() - load_start) * 1000
+  )
   return no_errors, refused_list
 end