From 6600ec63fdd5593b1f4d7219811d6b06d298dd80 Mon Sep 17 00:00:00 2001 From: Alexander Zaft Date: Mon, 18 Aug 2025 12:20:18 +0200 Subject: [PATCH] module: apply datatype first if the default value is given before the datatype, the module cannot be created. Also fixes the remaining errors in issue 4913. Fixes: 4913 Change-Id: I613576388bc3b9e7a5cdc7aa4d5ea73e549ae167 Reviewed-on: https://forge.frm2.tum.de/review/c/secop/frappy/+/37182 Tested-by: Jenkins Automated Tests Reviewed-by: Alexander Zaft Reviewed-by: Markus Zolliker --- frappy/modulebase.py | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/frappy/modulebase.py b/frappy/modulebase.py index 27277b28..fcf83bcb 100644 --- a/frappy/modulebase.py +++ b/frappy/modulebase.py @@ -449,6 +449,10 @@ class Module(HasAccessibles): self.commands[name] = accessible if cfg is not None: try: + # apply datatype first + datatype = cfg.pop('datatype', None) + if datatype is not None: + accessible.setProperty('datatype', datatype) for propname, propvalue in cfg.items(): if propname in {'value', 'default', 'constant'}: # these properties have ValueType(), but should be checked for datatype