Merge pull request #16 from mborho/master

Optimizing charset check.
psolbach · Jun 4, 2018 · 567f487 · 567f487
2 parents 8ef4ffb + c23e20a
commit 567f487
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/metadoc/__init__.py b/metadoc/__init__.py
@@ -187,8 +187,9 @@ def _request_url(self):
           raise Exception('Requesting article body failed with {} status code.'.format(req.status_code))
 
         # check for encoding conflicts (e.g. t3n.de)
-        if req.encoding.lower() != req.apparent_encoding.lower() and \
-           req.apparent_encoding.lower() != "windows-1254":
-            logger.info("Switching html encoding: {} -> {}".format(req.encoding, req.apparent_encoding))
-            req.encoding = req.apparent_encoding
+        enc_apparent = req.apparent_encoding.lower()
+        if req.encoding.lower() != enc_apparent and \
+           enc_apparent != "windows-1254":
+            logger.info("Switching html encoding: {} -> {}".format(req.encoding, enc_apparent))
+            req.encoding = enc_apparent
         return req.text