summaryrefslogtreecommitdiffstats
path: root/classes
diff options
context:
space:
mode:
authornico <nico@magicbroccoli.de>2018-11-07 14:47:55 +0100
committernico <nico@magicbroccoli.de>2018-11-07 14:47:55 +0100
commitd1b8090ee8f773628cc5eb04971b575debfec249 (patch)
tree2e2bc05317362d21217dac695c101bb84eb8c76e /classes
parentde56a9315cef894a2f3c3a9b39cad4ed10f55491 (diff)
NoneType Bug fix
* fixed NoneType tag bug
Diffstat (limited to 'classes')
-rw-r--r--classes/xep.py24
1 files changed, 12 insertions, 12 deletions
diff --git a/classes/xep.py b/classes/xep.py
index fdddb22..f5fae61 100644
--- a/classes/xep.py
+++ b/classes/xep.py
@@ -66,7 +66,7 @@ class XEPRequest:
"""
# all possible subtags grouped by location
last_revision_tags = ["date", "version", "initials", "remark"]
- xep_tags = ["number", "title", "abstract", "type", "status", "approver", "shortname", "sig", "lastcall"]
+ xep_tags = ["number", "title", "abstract", "type", "status", "approver", "shortname", "sig", "lastcall", "date", "version", "initials", "remark"]
# check if xeplist is accurate
self.req_xeplist()
@@ -75,16 +75,22 @@ class XEPRequest:
# if requested number is member of acceptedxeps continue
if str(self.reqxep) in self.acceptedxeps:
searchstring = ".//*[@accepted='true']/[number='%s']" % self.reqxep
- query = None
for item in self.xeplist.findall(searchstring):
# if the opt_arg references is member of xeptag return only that tag
if self.opt_arg in xep_tags:
- query = item.find(self.opt_arg)
- # if the opt_arg references is member of last-revision_tags return only that tag
- elif self.opt_arg in last_revision_tags:
- query = item.find("last-revision").find(self.opt_arg)
+ # if the opt_arg references is member of last_revision_tags return only that subtag
+ if self.opt_arg in last_revision_tags:
+ query = item.find("last-revision").find(self.opt_arg)
+ else:
+ query = item.find(self.opt_arg)
+
+ # append opt_arg results to the result list
+ if query is not None:
+ result.append("%s : %s" % (query.tag, query.text))
+ else:
+ result.append("%s does not have a %s tag." % (self.reqxep, self.opt_arg))
# in any other case return the general answer
else:
@@ -92,12 +98,6 @@ class XEPRequest:
for tag in result_opts:
result.append(item.find(tag).text)
- # append opt_arg results to the result list
- if query is not None:
- result.append("%s : %s" % (query.tag, query.text))
- else:
- result.append("%s does not have a %s tag." % (self.reqxep, self.opt_arg))
-
# if the requested number is no member of acceptedxeps and/or not accepted return error.
else:
result.append("XEP-%s : is not available." % self.reqxep)