diff options
| author | Gordon Chung <chungg@ca.ibm.com> | 2013-10-24 19:04:13 -0400 |
|---|---|---|
| committer | Gordon Chung <chungg@ca.ibm.com> | 2013-10-25 11:39:22 -0400 |
| commit | 2e436ada18e5b6d37342ff7969b189d4827df45d (patch) | |
| tree | ec89a381ab2884c5dcb5b63182cfc770f939fe3b /pycadf/attachment.py | |
| parent | 52aa7818bba4c16229d9d2b402009a36d781ea5f (diff) | |
| download | pycadf-2e436ada18e5b6d37342ff7969b189d4827df45d.tar.gz | |
improve model validation
- add isset to check "real" attribute and not descriptor
- verify only id is set in shortform
- verify either resource or resourceId value is set, not both.
blueprint improve-validation
related-bug: #1242830
Change-Id: Ie9e3f26c5d30cd36e6013a1f0b77c8fe466cb3f7
Diffstat (limited to 'pycadf/attachment.py')
| -rw-r--r-- | pycadf/attachment.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/pycadf/attachment.py b/pycadf/attachment.py index bfd9b95..0191471 100644 --- a/pycadf/attachment.py +++ b/pycadf/attachment.py @@ -57,7 +57,7 @@ class Attachment(cadftype.CADFAbstractType): def is_valid(self): # Existence test, All attributes must exist for valid Attachment type return ( - hasattr(self, ATTACHMENT_KEYNAME_TYPEURI) and - hasattr(self, ATTACHMENT_KEYNAME_NAME) and - hasattr(self, ATTACHMENT_KEYNAME_CONTENT) + self._isset(ATTACHMENT_KEYNAME_TYPEURI) and + self._isset(ATTACHMENT_KEYNAME_NAME) and + self._isset(ATTACHMENT_KEYNAME_CONTENT) ) |