List does't not have attribute lower() error on requestdict, fixes #9

Author: mikeckennedy

app/ch12-forms/final/pypi_org/infrastructure/request_dict.py

@@ -1,4 +1,5 @@
 import flask
+from werkzeug.datastructures import MultiDict
 
 
 class RequestDictionary(dict):
@@ -13,10 +14,21 @@ def __getattr__(self, key):
 def create(default_val=None, **route_args) -> RequestDictionary:
     request = flask.request
 
+    # Adding this retro actively. Some folks are experiencing issues where they
+    # are getting a list rather than plain dict. I think it's from multiple
+    # entries in the multidict. This should fix it.
+    args = request.args
+    if isinstance(request.args, MultiDict):
+        args = request.args.to_dict()
+
+    form = request.form
+    if isinstance(request.args, MultiDict):
+        form = request.form.to_dict()
+
     data = {
-        **request.args,  # The key/value pairs in the URL query string
+        **args,  # The key/value pairs in the URL query string
         **request.headers,  # Header values
-        **request.form,  # The key/value pairs in the body, from a HTML post form
+        **form,  # The key/value pairs in the body, from a HTML post form
         **route_args  # And additional arguments the method access, if they want them merged.
     }
 

app/ch13-validation/final/pypi_org/bin/load_data.py

@@ -10,6 +10,7 @@
 sys.path.insert(0, os.path.abspath(os.path.join(
     os.path.dirname(__file__), "..", "..")))
 
+from pypi_org.bin.load_data import try_int
 import pypi_org.data.db_session as db_session
 from pypi_org.data.languages import ProgrammingLanguage
 from pypi_org.data.licenses import License
@@ -339,13 +340,6 @@ def make_version_num(version_text):
         return major, minor, build
 
 
-def try_int(text) -> int:
-    try:
-        return int(text)
-    except:
-        return 0
-
-
 def init_db():
     top_folder = os.path.dirname(__file__)
     rel_file = os.path.join('..', 'db', 'pypi.sqlite')

app/ch13-validation/final/pypi_org/infrastructure/request_dict.py

@@ -1,4 +1,5 @@
 import flask
+from werkzeug.datastructures import MultiDict
 
 
 class RequestDictionary(dict):
@@ -13,10 +14,21 @@ def __getattr__(self, key):
 def create(default_val=None, **route_args) -> RequestDictionary:
     request = flask.request
 
+    # Adding this retro actively. Some folks are experiencing issues where they
+    # are getting a list rather than plain dict. I think it's from multiple
+    # entries in the multidict. This should fix it.
+    args = request.args
+    if isinstance(request.args, MultiDict):
+        args = request.args.to_dict()
+
+    form = request.form
+    if isinstance(request.args, MultiDict):
+        form = request.form.to_dict()
+
     data = {
-        **request.args,  # The key/value pairs in the URL query string
+        **args,  # The key/value pairs in the URL query string
         **request.headers,  # Header values
-        **request.form,  # The key/value pairs in the body, from a HTML post form
+        **form,  # The key/value pairs in the body, from a HTML post form
         **route_args  # And additional arguments the method access, if they want them merged.
     }
 

app/ch13-validation/starter/pypi_org/infrastructure/request_dict.py

@@ -1,4 +1,5 @@
 import flask
+from werkzeug.datastructures import MultiDict
 
 
 class RequestDictionary(dict):
@@ -13,10 +14,21 @@ def __getattr__(self, key):
 def create(default_val=None, **route_args) -> RequestDictionary:
     request = flask.request
 
+    # Adding this retro actively. Some folks are experiencing issues where they
+    # are getting a list rather than plain dict. I think it's from multiple
+    # entries in the multidict. This should fix it.
+    args = request.args
+    if isinstance(request.args, MultiDict):
+        args = request.args.to_dict()
+
+    form = request.form
+    if isinstance(request.args, MultiDict):
+        form = request.form.to_dict()
+
     data = {
-        **request.args,  # The key/value pairs in the URL query string
+        **args,  # The key/value pairs in the URL query string
         **request.headers,  # Header values
-        **request.form,  # The key/value pairs in the body, from a HTML post form
+        **form,  # The key/value pairs in the body, from a HTML post form
         **route_args  # And additional arguments the method access, if they want them merged.
     }
 

app/ch14_testing/final/pypi_org/infrastructure/request_dict.py

@@ -1,4 +1,5 @@
 import flask
+from werkzeug.datastructures import MultiDict
 
 
 class RequestDictionary(dict):
@@ -13,10 +14,21 @@ def __getattr__(self, key):
 def create(default_val=None, **route_args) -> RequestDictionary:
     request = flask.request
 
+    # Adding this retro actively. Some folks are experiencing issues where they
+    # are getting a list rather than plain dict. I think it's from multiple
+    # entries in the multidict. This should fix it.
+    args = request.args
+    if isinstance(request.args, MultiDict):
+        args = request.args.to_dict()
+
+    form = request.form
+    if isinstance(request.args, MultiDict):
+        form = request.form.to_dict()
+
     data = {
-        **request.args,  # The key/value pairs in the URL query string
+        **args,  # The key/value pairs in the URL query string
         **request.headers,  # Header values
-        **request.form,  # The key/value pairs in the body, from a HTML post form
+        **form,  # The key/value pairs in the body, from a HTML post form
         **route_args  # And additional arguments the method access, if they want them merged.
     }
 

app/ch14_testing/starter/pypi_org/infrastructure/request_dict.py

@@ -1,4 +1,5 @@
 import flask
+from werkzeug.datastructures import MultiDict
 
 
 class RequestDictionary(dict):
@@ -13,10 +14,21 @@ def __getattr__(self, key):
 def create(default_val=None, **route_args) -> RequestDictionary:
     request = flask.request
 
+    # Adding this retro actively. Some folks are experiencing issues where they
+    # are getting a list rather than plain dict. I think it's from multiple
+    # entries in the multidict. This should fix it.
+    args = request.args
+    if isinstance(request.args, MultiDict):
+        args = request.args.to_dict()
+
+    form = request.form
+    if isinstance(request.args, MultiDict):
+        form = request.form.to_dict()
+
     data = {
-        **request.args,  # The key/value pairs in the URL query string
+        **args,  # The key/value pairs in the URL query string
         **request.headers,  # Header values
-        **request.form,  # The key/value pairs in the body, from a HTML post form
+        **form,  # The key/value pairs in the body, from a HTML post form
         **route_args  # And additional arguments the method access, if they want them merged.
     }
 

app/ch15_deploy/final/pypi_org/infrastructure/request_dict.py

@@ -1,4 +1,5 @@
 import flask
+from werkzeug.datastructures import MultiDict
 
 
 class RequestDictionary(dict):
@@ -13,10 +14,21 @@ def __getattr__(self, key):
 def create(default_val=None, **route_args) -> RequestDictionary:
     request = flask.request
 
+    # Adding this retro actively. Some folks are experiencing issues where they
+    # are getting a list rather than plain dict. I think it's from multiple
+    # entries in the multidict. This should fix it.
+    args = request.args
+    if isinstance(request.args, MultiDict):
+        args = request.args.to_dict()
+
+    form = request.form
+    if isinstance(request.args, MultiDict):
+        form = request.form.to_dict()
+
     data = {
-        **request.args,  # The key/value pairs in the URL query string
+        **args,  # The key/value pairs in the URL query string
         **request.headers,  # Header values
-        **request.form,  # The key/value pairs in the body, from a HTML post form
+        **form,  # The key/value pairs in the body, from a HTML post form
         **route_args  # And additional arguments the method access, if they want them merged.
     }
 

app/ch15_deploy/starter/pypi_org/infrastructure/request_dict.py

@@ -1,4 +1,5 @@
 import flask
+from werkzeug.datastructures import MultiDict
 
 
 class RequestDictionary(dict):
@@ -13,10 +14,21 @@ def __getattr__(self, key):
 def create(default_val=None, **route_args) -> RequestDictionary:
     request = flask.request
 
+    # Adding this retro actively. Some folks are experiencing issues where they
+    # are getting a list rather than plain dict. I think it's from multiple
+    # entries in the multidict. This should fix it.
+    args = request.args
+    if isinstance(request.args, MultiDict):
+        args = request.args.to_dict()
+
+    form = request.form
+    if isinstance(request.args, MultiDict):
+        form = request.form.to_dict()
+
     data = {
-        **request.args,  # The key/value pairs in the URL query string
+        **args,  # The key/value pairs in the URL query string
         **request.headers,  # Header values
-        **request.form,  # The key/value pairs in the body, from a HTML post form
+        **form,  # The key/value pairs in the body, from a HTML post form
         **route_args  # And additional arguments the method access, if they want them merged.
     }
 

app/ch16_mongodb/final/pypi_org/infrastructure/request_dict.py

@@ -1,4 +1,5 @@
 import flask
+from werkzeug.datastructures import MultiDict
 
 
 class RequestDictionary(dict):
@@ -13,10 +14,21 @@ def __getattr__(self, key):
 def create(default_val=None, **route_args) -> RequestDictionary:
     request = flask.request
 
+    # Adding this retro actively. Some folks are experiencing issues where they
+    # are getting a list rather than plain dict. I think it's from multiple
+    # entries in the multidict. This should fix it.
+    args = request.args
+    if isinstance(request.args, MultiDict):
+        args = request.args.to_dict()
+
+    form = request.form
+    if isinstance(request.args, MultiDict):
+        form = request.form.to_dict()
+
     data = {
-        **request.args,  # The key/value pairs in the URL query string
+        **args,  # The key/value pairs in the URL query string
         **request.headers,  # Header values
-        **request.form,  # The key/value pairs in the body, from a HTML post form
+        **form,  # The key/value pairs in the body, from a HTML post form
         **route_args  # And additional arguments the method access, if they want them merged.
     }
 

app/ch16_mongodb/starter/pypi_org/infrastructure/request_dict.py

@@ -1,4 +1,5 @@
 import flask
+from werkzeug.datastructures import MultiDict
 
 
 class RequestDictionary(dict):
@@ -13,10 +14,21 @@ def __getattr__(self, key):
 def create(default_val=None, **route_args) -> RequestDictionary:
     request = flask.request
 
+    # Adding this retro actively. Some folks are experiencing issues where they
+    # are getting a list rather than plain dict. I think it's from multiple
+    # entries in the multidict. This should fix it.
+    args = request.args
+    if isinstance(request.args, MultiDict):
+        args = request.args.to_dict()
+
+    form = request.form
+    if isinstance(request.args, MultiDict):
+        form = request.form.to_dict()
+
     data = {
-        **request.args,  # The key/value pairs in the URL query string
+        **args,  # The key/value pairs in the URL query string
         **request.headers,  # Header values
-        **request.form,  # The key/value pairs in the body, from a HTML post form
+        **form,  # The key/value pairs in the body, from a HTML post form
         **route_args  # And additional arguments the method access, if they want them merged.
     }