From 3d2333f731133d4033a0ff494a8ca8fa587e6d41 Mon Sep 17 00:00:00 2001 From: Markus Zolliker Date: Thu, 2 Apr 2020 11:23:37 +0200 Subject: [PATCH] do not update exported parameters of unexported modules Change-Id: Ia62def1e03670be0f8a07bfc6f19731cadfbe270 Reviewed-on: https://forge.frm2.tum.de/review/c/sine2020/secop/playground/+/22825 Tested-by: JenkinsCodeReview Reviewed-by: Enrico Faulhaber Reviewed-by: Markus Zolliker --- secop/modules.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/secop/modules.py b/secop/modules.py index 0ff3bc4..f2ccb90 100644 --- a/secop/modules.py +++ b/secop/modules.py @@ -147,6 +147,8 @@ class Module(HasProperties, metaclass=ModuleMeta): if aobj.needscfg is None: aobj.properties['needscfg'] = not aobj.poll + if not self.export: # do not export parameters of a module not exported + aobj.properties['export'] = False if aobj.export: if aobj.export is True: predefined_obj = PREDEFINED_ACCESSIBLES.get(aname, None) @@ -269,7 +271,8 @@ class Module(HasProperties, metaclass=ModuleMeta): the error will be cleared when the parameter is set """ pobj = self.parameters[pname] - self.DISPATCHER.announce_update_error(self, pname, pobj, exception) + if pobj.export: + self.DISPATCHER.announce_update_error(self, pname, pobj, exception) def isBusy(self, status=None): """helper function for treating substates of BUSY correctly"""