Fix bug which didn't (or always) transforms the form dictionary in request dict. Fixes talkpython#4

Author: mikeckennedy

code/ch4_app/ch4_final_video_collector/infrastructure/request_dict.py

@@ -14,16 +14,16 @@ 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
+    # Adding this retroactively. 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()
+    if isinstance(args, MultiDict):
+        args = args.to_dict()
 
     form = request.form
-    if isinstance(request.args, MultiDict):
-        form = request.form.to_dict()
+    if isinstance(form, MultiDict):
+        form = form.to_dict()
 
     data = {
         **args,  # The key/value pairs in the URL query string

code/ch4_app/ch4_starter_video_collector/infrastructure/request_dict.py

@@ -14,16 +14,16 @@ 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
+    # Adding this retroactively. 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()
+    if isinstance(args, MultiDict):
+        args = args.to_dict()
 
     form = request.form
-    if isinstance(request.args, MultiDict):
-        form = request.form.to_dict()
+    if isinstance(form, MultiDict):
+        form = form.to_dict()
 
     data = {
         **args,  # The key/value pairs in the URL query string

code/ch5_partials/ch5_final_video_collector/infrastructure/request_dict.py

@@ -14,16 +14,16 @@ 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
+    # Adding this retroactively. 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()
+    if isinstance(args, MultiDict):
+        args = args.to_dict()
 
     form = request.form
-    if isinstance(request.args, MultiDict):
-        form = request.form.to_dict()
+    if isinstance(form, MultiDict):
+        form = form.to_dict()
 
     data = {
         **args,  # The key/value pairs in the URL query string

code/ch5_partials/ch5_starter_video_collector/infrastructure/request_dict.py

@@ -14,16 +14,16 @@ 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
+    # Adding this retroactively. 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()
+    if isinstance(args, MultiDict):
+        args = args.to_dict()
 
     form = request.form
-    if isinstance(request.args, MultiDict):
-        form = request.form.to_dict()
+    if isinstance(form, MultiDict):
+        form = form.to_dict()
 
     data = {
         **args,  # The key/value pairs in the URL query string

code/ch6_active_search/ch6_final_video_collector/infrastructure/request_dict.py

@@ -14,16 +14,16 @@ 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
+    # Adding this retroactively. 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()
+    if isinstance(args, MultiDict):
+        args = args.to_dict()
 
     form = request.form
-    if isinstance(request.args, MultiDict):
-        form = request.form.to_dict()
+    if isinstance(form, MultiDict):
+        form = form.to_dict()
 
     data = {
         **args,  # The key/value pairs in the URL query string

code/ch6_active_search/ch6_starter_video_collector/infrastructure/request_dict.py

@@ -14,16 +14,16 @@ 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
+    # Adding this retroactively. 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()
+    if isinstance(args, MultiDict):
+        args = args.to_dict()
 
     form = request.form
-    if isinstance(request.args, MultiDict):
-        form = request.form.to_dict()
+    if isinstance(form, MultiDict):
+        form = form.to_dict()
 
     data = {
         **args,  # The key/value pairs in the URL query string

code/ch7_infinite_scroll/ch7_final_video_collector/infrastructure/request_dict.py

@@ -14,16 +14,16 @@ 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
+    # Adding this retroactively. 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()
+    if isinstance(args, MultiDict):
+        args = args.to_dict()
 
     form = request.form
-    if isinstance(request.args, MultiDict):
-        form = request.form.to_dict()
+    if isinstance(form, MultiDict):
+        form = form.to_dict()
 
     data = {
         **args,  # The key/value pairs in the URL query string

code/ch7_infinite_scroll/ch7_final_video_collector/requirements.txt

@@ -1,8 +1,8 @@
 #
-# This file is autogenerated by pip-compile with Python 3.11
-# by the following command:
+# This file is autogenerated by pip-compile with python 3.10
+# To update, run:
 #
-#    pip-compile
+#    pip-compile requirements.in
 #
 click==8.1.3
     # via flask
@@ -24,7 +24,7 @@ markupsafe==2.1.1
     #   werkzeug
 more-itertools==9.0.0
     # via -r requirements.in
-pydantic==1.10.2
+pydantic==1.10.4
     # via -r requirements.in
 typing-extensions==4.4.0
     # via pydantic

code/ch7_infinite_scroll/ch7_starter_video_collector/infrastructure/request_dict.py

@@ -14,16 +14,16 @@ 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
+    # Adding this retroactively. 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()
+    if isinstance(args, MultiDict):
+        args = args.to_dict()
 
     form = request.form
-    if isinstance(request.args, MultiDict):
-        form = request.form.to_dict()
+    if isinstance(form, MultiDict):
+        form = form.to_dict()
 
     data = {
         **args,  # The key/value pairs in the URL query string

code/starter_video_collector/infrastructure/request_dict.py

@@ -14,16 +14,16 @@ 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
+    # Adding this retroactively. 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()
+    if isinstance(args, MultiDict):
+        args = args.to_dict()
 
     form = request.form
-    if isinstance(request.args, MultiDict):
-        form = request.form.to_dict()
+    if isinstance(form, MultiDict):
+        form = form.to_dict()
 
     data = {
         **args,  # The key/value pairs in the URL query string