Skip to content

Properly catch unset preferences#16

Open
w-biggs wants to merge 1 commit intoCommReteris:mainfrom
w-biggs:bad-argument-fix
Open

Properly catch unset preferences#16
w-biggs wants to merge 1 commit intoCommReteris:mainfrom
w-biggs:bad-argument-fix

Conversation

@w-biggs
Copy link

@w-biggs w-biggs commented Mar 11, 2026

With Darktable 5.4.1, when trying to load this script, this error fires:

     5.4318 LUA ERROR: dtutils.lua: prequire: 223: Error loading contrib/nind_denoise_rl 
     5.4319 LUA ERROR: dtutils.lua: prequire: 224: Error returned is bad argument #3 to '?' (number expected, got nil) 
     5.4319 LUA ERROR: script_manager.lua: activate: 541: error loading contrib/nind_denoise_rl 
     5.4319 LUA ERROR: script_manager.lua: activate: 542: error message: bad argument #3 to '?' (number expected, got nil) 

It seems like some API change made it so dt.preferences.read returns empty strings for unset preferences instead of erroring? I'm not 100% sure, but this prevents that from propagating down and causing errors

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant