Fix a number of incorrect escapes

Pointed out by newer versions of pep8, but they were never correct.

diff --git a/pgweb/account/forms.py b/pgweb/account/forms.py
index f16d30eb963bc77fd5fb4dc1f1c9d433729edc5f..a9f322ee750b2c10464194f8120a07fffab9b51e 100644 (file)
--- a/pgweb/account/forms.py
+++ b/pgweb/account/forms.py
@@ -16,7 +16,7 @@ log = logging.getLogger(__name__)
 def _clean_username(username):
     username = username.lower()
 
-    if not re.match('^[a-z0-9\.-]+$', username):
+    if not re.match(r'^[a-z0-9\.-]+$', username):
         # XXX: Note! Should we ever allow @ signs in usernames again, we need to also
         #      update util/auth.py and the code for identifying email addresses.
         raise forms.ValidationError("Invalid character in user name. Only a-z, 0-9, . and - allowed for compatibility with third party software.")

diff --git a/pgweb/docs/views.py b/pgweb/docs/views.py
index 9d0f55fd3d146c07221844051009d8b5ad1912bd..056612951a8ddb91d84e3f0f213291aa7e8c9bc2 100644 (file)
--- a/pgweb/docs/views.py
+++ b/pgweb/docs/views.py
@@ -244,9 +244,9 @@ def release_notes(request, major_version=None, minor_version=None):
             SELECT
                 {content}
                 file,
-                string_to_array(regexp_replace(file, 'release-(.*)\.htm.*', '\\1'), '-') AS v
+                string_to_array(regexp_replace(file, 'release-(.*)\\.htm.*', '\\1'), '-') AS v
             FROM docs
-            WHERE file ~ '^release-\d+' AND version >= 9.3
+            WHERE file ~ '^release-\\d+' AND version >= 9.3
         ) r
     ) rr
     """

diff --git a/pgweb/security/models.py b/pgweb/security/models.py
index 18418561bec00f0a7d689fccc6c334eb9ec6b860..6242e628d7352385973ab27c9e104bdbcb44f0d9 100644 (file)
--- a/pgweb/security/models.py
+++ b/pgweb/security/models.py
@@ -20,7 +20,7 @@ component_choices = (
 )
 
 
-re_cve = re.compile('^(\d{4})-(\d{4,5})$')
+re_cve = re.compile(r'^(\d{4})-(\d{4,5})$')
 
 
 def cve_validator(val):

diff --git a/pgweb/util/misc.py b/pgweb/util/misc.py
index 331ac90f526c308bb60310f1ce75a80c500dc2f4..5d45b7a073539fab2fcb6625cc2f86b6e59214cb 100644 (file)
--- a/pgweb/util/misc.py
+++ b/pgweb/util/misc.py
@@ -69,10 +69,10 @@ def version_sort(l):
     generally don't have that.
     """
     mkey = l['link']
-    m = re.match('v?([0-9]+)\.([0-9]+)\.([0-9]+)$', l['url'])
+    m = re.match(r'v?([0-9]+)\.([0-9]+)\.([0-9]+)$', l['url'])
     if m:
         mkey = m.group(1) + '%02d' % int(m.group(2)) + '%02d' % int(m.group(3))
-    m = re.match('v?([0-9]+)\.([0-9]+)$', l['url'])
+    m = re.match(r'v?([0-9]+)\.([0-9]+)$', l['url'])
     if m:
         mkey = m.group(1) + '%02d' % int(m.group(2))
         # SOOO ugly. But if it's v10 and up, just prefix it to get it higher

diff --git a/tools/docs/docload.py b/tools/docs/docload.py
index 094d965141cf0f5b96e8a4ab13842ff8ac7c16e8..362da79b1f9a6fa49fe6b5a535dacd3bcade364f 100755 (executable)
--- a/tools/docs/docload.py
+++ b/tools/docs/docload.py
@@ -23,7 +23,7 @@ pagecount = 0
 quiet = False
 # regular expression used to search and extract the title on a given piece of
 # documentation, for further use in the application
-re_titlematch = re.compile('<title\s*>([^<]+)</title\s*>', re.IGNORECASE)
+re_titlematch = re.compile(r'<title\s*>([^<]+)</title\s*>', re.IGNORECASE)
 # regular expression used to find any images that are in the HTML and apply
 # additional bootstrap classes
 re_figure_match = re.compile('<div([^<>]+)class="figure"', re.IGNORECASE)

diff --git a/tools/ftp/spider_yum.py b/tools/ftp/spider_yum.py
index 4dfd6191991a83967d87361f5e5565185fdd7814..2dfa543150908bdef06b4a53e0e2e13043933cd6 100755 (executable)
--- a/tools/ftp/spider_yum.py
+++ b/tools/ftp/spider_yum.py
@@ -8,7 +8,7 @@ import requests
 from collections import defaultdict
 from tempfile import NamedTemporaryFile
 
-re_platformdir = re.compile('^(\w+)-(\d+)-([^-]+)$')
+re_platformdir = re.compile(r'^(\w+)-(\d+)-([^-]+)$')
 
 if __name__ == "__main__":
     parser = argparse.ArgumentParser(description="Spider repo RPMs")

diff --git a/tools/localhtmlvalidate/localhtmlvalidate.py b/tools/localhtmlvalidate/localhtmlvalidate.py
index 4332350bf78f18e45cfeda5e944a05accd917580..7160b52bcc613ff16165f421fde2c5cb66ff24b2 100755 (executable)
--- a/tools/localhtmlvalidate/localhtmlvalidate.py
+++ b/tools/localhtmlvalidate/localhtmlvalidate.py
@@ -61,7 +61,7 @@ if __name__ == "__main__":
         print("Warnings: %s" % resp.headers['x-w3c-validator-warnings'])
         hp = html.parser.HTMLParser()
         for m in re.findall('<li class="msg_err">.*?</li>', resp.text, re.DOTALL):
-            r = re.search('<em>Line <a href="[^"]+">(\d+)</a>.*<span class="msg">(.*?)</span>', m, re.DOTALL)
+            r = re.search(r'<em>Line <a href="[^"]+">(\d+)</a>.*<span class="msg">(.*?)</span>', m, re.DOTALL)
             if r:
                 print("Line %s (should be around %s): %s" % (r.group(1), int(r.group(1)) - firstline, hp.unescape(r.group(2))))
             r2 = re.search('<code class="input">(.*?)<strong title=".*?">(.*?)</strong>(.*?)</code>', m, re.DOTALL)