diff options
author | Jelle van der Waa <jelle@vdwaa.nl> | 2017-06-07 21:56:50 +0200 |
---|---|---|
committer | Jelle van der Waa <jelle@vdwaa.nl> | 2017-06-07 21:56:50 +0200 |
commit | 6951bb59e7ca6484d61b99558716f37b70f118dc (patch) | |
tree | 009f7c3b5bef961663877e8f3f7ad94082f3910b | |
parent | c5a4172113392abbcc0058f3256dccd030efbd6e (diff) | |
download | archweb-6951bb59e7ca6484d61b99558716f37b70f118dc.tar.gz archweb-6951bb59e7ca6484d61b99558716f37b70f118dc.zip |
syncisos: reduce nesting in handle_starttag
Reduce nesting for readability.
-rw-r--r-- | releng/management/commands/syncisos.py | 14 |
1 files changed, 9 insertions, 5 deletions
diff --git a/releng/management/commands/syncisos.py b/releng/management/commands/syncisos.py index 68da87a7..0a082911 100644 --- a/releng/management/commands/syncisos.py +++ b/releng/management/commands/syncisos.py @@ -17,11 +17,15 @@ class IsoListParser(HTMLParser): self.url_re = re.compile('(?!\.{2})/$') def handle_starttag(self, tag, attrs): - if tag == 'a': - for name, value in attrs: - if name == "href": - if value != '../' and self.url_re.search(value) is not None: - self.hyperlinks.append(value[:-1]) + if tag != 'a': + return + + for name, value in attrs: + if name != "href": + continue + + if value != '../' and self.url_re.search(value) is not None: + self.hyperlinks.append(value[:-1]) def parse(self, url): try: |