diff --git a/CHANGES.rst b/CHANGES.rst index 6695468be9..9c72f89313 100644 --- a/CHANGES.rst +++ b/CHANGES.rst @@ -14,7 +14,7 @@ Changelog **Fixed** - +- #390 Remove log verbosity of UIDReference.get when value is None or empty **Security** diff --git a/bika/lims/browser/fields/uidreferencefield.py b/bika/lims/browser/fields/uidreferencefield.py index 58b8f04687..9d4fd3c29f 100644 --- a/bika/lims/browser/fields/uidreferencefield.py +++ b/bika/lims/browser/fields/uidreferencefield.py @@ -101,6 +101,8 @@ def get(self, context, **kwargs): :rtype: BaseContent | list[BaseContent] """ value = StringField.get(self, context, **kwargs) + if not value: + return [] if self.multiValued else None if self.multiValued: # Only return objects which actually exist; this is necessary here # because there are no HoldingReferences. This opens the @@ -125,6 +127,8 @@ def getRaw(self, context, aslist=False, **kwargs): :rtype: string | list[string] """ value = StringField.get(self, context, **kwargs) + if not value: + return [] if self.multiValued else None if self.multiValued: ret = value else: