# HG changeset patch
# User Matthew Wild <mwild1@gmail.com>
# Date 1701351803 0
# Node ID 80a1ce9974e5b755dd0cc06540f97723b87c2adf
# Parent  82513890a1d8bf19c735d1a863396e0801bf22d5
moduleapi: Log error message when ambiguous period spec is found in config

diff -r 82513890a1d8 -r 80a1ce9974e5 core/moduleapi.lua
--- a/core/moduleapi.lua	Thu Nov 30 13:42:44 2023 +0000
+++ b/core/moduleapi.lua	Thu Nov 30 13:43:23 2023 +0000
@@ -281,7 +281,15 @@
 	elseif type(value) == "string" then
 		ret = human_io.parse_duration(value);
 		if value ~= nil and ret == nil then
-			self:log("error", "Config option '%s' not understood, expecting a period (e.g. \"2 days\")", name);
+			ret = human_io.parse_duration_lax(value);
+			if ret then
+				local num = value:match("%d+");
+				self:log("error", "Config option '%s' is set to ambiguous period '%s' - use full syntax e.g. '%s months' or '%s minutes'", name, value, num, num);
+				-- COMPAT: w/more relaxed behaviour in post-0.12 trunk. Return nil for this case too, eventually.
+			else
+				self:log("error", "Config option '%s' not understood, expecting a period (e.g. \"2 days\")", name);
+				return nil;
+			end
 		end
 	elseif value ~= nil then
 		self:log("error", "Config option '%s' expects a number or a period description string (e.g. \"3 hours\"), not %s", name, type(value));