patch(endpoints): fix if-statement in tag post route for 24 files

Justin-f-Reeves · Justin-f-Reeves · commit de40a40f61f6 · 2023-01-11T22:43:11.000-05:00
diff --git a/docs/docs/07_sqlalchemy_many_to_many/02_one_to_many_review/README.md b/docs/docs/07_sqlalchemy_many_to_many/02_one_to_many_review/README.md
@@ -117,7 +117,7 @@ class TagsInStore(MethodView):
     @blp.arguments(TagSchema)
     @blp.response(201, TagSchema)
     def post(self, tag_data, store_id):
-        if TagModel.query.filter(TagModel.store_id == store_id).first():
+        if TagModel.query.filter(TagModel.store_id == store_id, TagModel.name == tag_data["name"]).first():
             abort(400, message="A tag with that name already exists in that store.")
 
         tag = TagModel(**tag_data, store_id=store_id)
diff --git a/docs/docs/07_sqlalchemy_many_to_many/02_one_to_many_review/end/resources/tag.py b/docs/docs/07_sqlalchemy_many_to_many/02_one_to_many_review/end/resources/tag.py
@@ -20,7 +20,7 @@ def get(self, store_id):
     @blp.arguments(TagSchema)
     @blp.response(201, TagSchema)
     def post(self, tag_data, store_id):
-        if TagModel.query.filter(TagModel.store_id == store_id).first():
+        if TagModel.query.filter(TagModel.store_id == store_id, TagModel.name == tag_data["name"]).first():
             abort(400, message="A tag with that name already exists in that store.")
 
         tag = TagModel(**tag_data, store_id=store_id)
diff --git a/docs/docs/07_sqlalchemy_many_to_many/03_many_to_many_relationships/README.md b/docs/docs/07_sqlalchemy_many_to_many/03_many_to_many_relationships/README.md
@@ -190,7 +190,7 @@ class TagsInStore(MethodView):
     @blp.arguments(TagSchema)
     @blp.response(201, TagSchema)
     def post(self, tag_data, store_id):
-        if TagModel.query.filter(TagModel.store_id == store_id).first():
+        if TagModel.query.filter(TagModel.store_id == store_id, TagModel.name == tag_data["name"]).first():
             abort(400, message="A tag with that name already exists in that store.")
 
         tag = TagModel(**tag_data, store_id=store_id)
diff --git a/docs/docs/07_sqlalchemy_many_to_many/03_many_to_many_relationships/end/resources/tag.py b/docs/docs/07_sqlalchemy_many_to_many/03_many_to_many_relationships/end/resources/tag.py
@@ -20,7 +20,7 @@ def get(self, store_id):
     @blp.arguments(TagSchema)
     @blp.response(201, TagSchema)
     def post(self, tag_data, store_id):
-        if TagModel.query.filter(TagModel.store_id == store_id).first():
+        if TagModel.query.filter(TagModel.store_id == store_id, TagModel.name == tag_data["name"]).first():
             abort(400, message="A tag with that name already exists in that store.")
 
         tag = TagModel(**tag_data, store_id=store_id)
diff --git a/docs/docs/07_sqlalchemy_many_to_many/03_many_to_many_relationships/start/resources/tag.py b/docs/docs/07_sqlalchemy_many_to_many/03_many_to_many_relationships/start/resources/tag.py
@@ -20,7 +20,7 @@ def get(self, store_id):
     @blp.arguments(TagSchema)
     @blp.response(201, TagSchema)
     def post(self, tag_data, store_id):
-        if TagModel.query.filter(TagModel.store_id == store_id).first():
+        if TagModel.query.filter(TagModel.store_id == store_id, TagModel.name == tag_data["name"]).first():
             abort(400, message="A tag with that name already exists in that store.")
 
         tag = TagModel(**tag_data, store_id=store_id)
diff --git a/docs/docs/08_flask_jwt_extended/04_flask_jwt_extended_setup/end/resources/tag.py b/docs/docs/08_flask_jwt_extended/04_flask_jwt_extended_setup/end/resources/tag.py
@@ -20,7 +20,7 @@ def get(self, store_id):
     @blp.arguments(TagSchema)
     @blp.response(201, TagSchema)
     def post(self, tag_data, store_id):
-        if TagModel.query.filter(TagModel.store_id == store_id).first():
+        if TagModel.query.filter(TagModel.store_id == store_id, TagModel.name == tag_data["name"]).first():
             abort(400, message="A tag with that name already exists in that store.")
 
         tag = TagModel(**tag_data, store_id=store_id)
diff --git a/docs/docs/08_flask_jwt_extended/04_flask_jwt_extended_setup/start/resources/tag.py b/docs/docs/08_flask_jwt_extended/04_flask_jwt_extended_setup/start/resources/tag.py
@@ -20,7 +20,7 @@ def get(self, store_id):
     @blp.arguments(TagSchema)
     @blp.response(201, TagSchema)
     def post(self, tag_data, store_id):
-        if TagModel.query.filter(TagModel.store_id == store_id).first():
+        if TagModel.query.filter(TagModel.store_id == store_id, TagModel.name == tag_data["name"]).first():
             abort(400, message="A tag with that name already exists in that store.")
 
         tag = TagModel(**tag_data, store_id=store_id)
diff --git a/docs/docs/08_flask_jwt_extended/05_user_model_and_schema/end/resources/tag.py b/docs/docs/08_flask_jwt_extended/05_user_model_and_schema/end/resources/tag.py
@@ -20,7 +20,7 @@ def get(self, store_id):
     @blp.arguments(TagSchema)
     @blp.response(201, TagSchema)
     def post(self, tag_data, store_id):
-        if TagModel.query.filter(TagModel.store_id == store_id).first():
+        if TagModel.query.filter(TagModel.store_id == store_id, TagModel.name == tag_data["name"]).first():
             abort(400, message="A tag with that name already exists in that store.")
 
         tag = TagModel(**tag_data, store_id=store_id)
diff --git a/docs/docs/08_flask_jwt_extended/05_user_model_and_schema/start/resources/tag.py b/docs/docs/08_flask_jwt_extended/05_user_model_and_schema/start/resources/tag.py
@@ -20,7 +20,7 @@ def get(self, store_id):
     @blp.arguments(TagSchema)
     @blp.response(201, TagSchema)
     def post(self, tag_data, store_id):
-        if TagModel.query.filter(TagModel.store_id == store_id).first():
+        if TagModel.query.filter(TagModel.store_id == store_id, TagModel.name == tag_data["name"]).first():
             abort(400, message="A tag with that name already exists in that store.")
 
         tag = TagModel(**tag_data, store_id=store_id)
diff --git a/docs/docs/08_flask_jwt_extended/06_registering_users_rest_api/end/resources/tag.py b/docs/docs/08_flask_jwt_extended/06_registering_users_rest_api/end/resources/tag.py
@@ -20,7 +20,7 @@ def get(self, store_id):
     @blp.arguments(TagSchema)
     @blp.response(201, TagSchema)
     def post(self, tag_data, store_id):
-        if TagModel.query.filter(TagModel.store_id == store_id).first():
+        if TagModel.query.filter(TagModel.store_id == store_id, TagModel.name == tag_data["name"]).first():
             abort(400, message="A tag with that name already exists in that store.")
 
         tag = TagModel(**tag_data, store_id=store_id)
diff --git a/docs/docs/08_flask_jwt_extended/06_registering_users_rest_api/start/resources/tag.py b/docs/docs/08_flask_jwt_extended/06_registering_users_rest_api/start/resources/tag.py
@@ -20,7 +20,7 @@ def get(self, store_id):
     @blp.arguments(TagSchema)
     @blp.response(201, TagSchema)
     def post(self, tag_data, store_id):
-        if TagModel.query.filter(TagModel.store_id == store_id).first():
+        if TagModel.query.filter(TagModel.store_id == store_id, TagModel.name == tag_data["name"]).first():
             abort(400, message="A tag with that name already exists in that store.")
 
         tag = TagModel(**tag_data, store_id=store_id)
diff --git a/docs/docs/08_flask_jwt_extended/07_login_users_rest_api/end/resources/tag.py b/docs/docs/08_flask_jwt_extended/07_login_users_rest_api/end/resources/tag.py
@@ -20,7 +20,7 @@ def get(self, store_id):
     @blp.arguments(TagSchema)
     @blp.response(201, TagSchema)
     def post(self, tag_data, store_id):
-        if TagModel.query.filter(TagModel.store_id == store_id).first():
+        if TagModel.query.filter(TagModel.store_id == store_id, TagModel.name == tag_data["name"]).first():
             abort(400, message="A tag with that name already exists in that store.")
 
         tag = TagModel(**tag_data, store_id=store_id)
diff --git a/docs/docs/08_flask_jwt_extended/07_login_users_rest_api/start/resources/tag.py b/docs/docs/08_flask_jwt_extended/07_login_users_rest_api/start/resources/tag.py
@@ -20,7 +20,7 @@ def get(self, store_id):
     @blp.arguments(TagSchema)
     @blp.response(201, TagSchema)
     def post(self, tag_data, store_id):
-        if TagModel.query.filter(TagModel.store_id == store_id).first():
+        if TagModel.query.filter(TagModel.store_id == store_id, TagModel.name == tag_data["name"]).first():
             abort(400, message="A tag with that name already exists in that store.")
 
         tag = TagModel(**tag_data, store_id=store_id)
diff --git a/docs/docs/08_flask_jwt_extended/08_protect_resources_with_jwt_required/end/resources/tag.py b/docs/docs/08_flask_jwt_extended/08_protect_resources_with_jwt_required/end/resources/tag.py
@@ -20,9 +20,8 @@ def get(self, store_id):
     @blp.arguments(TagSchema)
     @blp.response(201, TagSchema)
     def post(self, tag_data, store_id):
-        if TagModel.query.filter(TagModel.store_id == store_id).first():
+        if TagModel.query.filter(TagModel.store_id == store_id, TagModel.name == tag_data["name"]).first():
             abort(400, message="A tag with that name already exists in that store.")
-
         tag = TagModel(**tag_data, store_id=store_id)
 
         try:
diff --git a/docs/docs/08_flask_jwt_extended/08_protect_resources_with_jwt_required/start/resources/tag.py b/docs/docs/08_flask_jwt_extended/08_protect_resources_with_jwt_required/start/resources/tag.py
@@ -20,7 +20,7 @@ def get(self, store_id):
     @blp.arguments(TagSchema)
     @blp.response(201, TagSchema)
     def post(self, tag_data, store_id):
-        if TagModel.query.filter(TagModel.store_id == store_id).first():
+        if TagModel.query.filter(TagModel.store_id == store_id, TagModel.name == tag_data["name"]).first():
             abort(400, message="A tag with that name already exists in that store.")
 
         tag = TagModel(**tag_data, store_id=store_id)
diff --git a/docs/docs/08_flask_jwt_extended/09_jwt_claims_and_authorization/end/resources/tag.py b/docs/docs/08_flask_jwt_extended/09_jwt_claims_and_authorization/end/resources/tag.py
@@ -20,7 +20,7 @@ def get(self, store_id):
     @blp.arguments(TagSchema)
     @blp.response(201, TagSchema)
     def post(self, tag_data, store_id):
-        if TagModel.query.filter(TagModel.store_id == store_id).first():
+        if TagModel.query.filter(TagModel.store_id == store_id, TagModel.name == tag_data["name"]).first():
             abort(400, message="A tag with that name already exists in that store.")
 
         tag = TagModel(**tag_data, store_id=store_id)
diff --git a/docs/docs/08_flask_jwt_extended/09_jwt_claims_and_authorization/start/resources/tag.py b/docs/docs/08_flask_jwt_extended/09_jwt_claims_and_authorization/start/resources/tag.py
@@ -20,7 +20,7 @@ def get(self, store_id):
     @blp.arguments(TagSchema)
     @blp.response(201, TagSchema)
     def post(self, tag_data, store_id):
-        if TagModel.query.filter(TagModel.store_id == store_id).first():
+        if TagModel.query.filter(TagModel.store_id == store_id, TagModel.name == tag_data["name"]).first():
             abort(400, message="A tag with that name already exists in that store.")
 
         tag = TagModel(**tag_data, store_id=store_id)
diff --git a/docs/docs/08_flask_jwt_extended/10_logout_users_rest_api/end/resources/tag.py b/docs/docs/08_flask_jwt_extended/10_logout_users_rest_api/end/resources/tag.py
@@ -20,7 +20,7 @@ def get(self, store_id):
     @blp.arguments(TagSchema)
     @blp.response(201, TagSchema)
     def post(self, tag_data, store_id):
-        if TagModel.query.filter(TagModel.store_id == store_id).first():
+        if TagModel.query.filter(TagModel.store_id == store_id, TagModel.name == tag_data["name"]).first():
             abort(400, message="A tag with that name already exists in that store.")
 
         tag = TagModel(**tag_data, store_id=store_id)
diff --git a/docs/docs/08_flask_jwt_extended/10_logout_users_rest_api/start/resources/tag.py b/docs/docs/08_flask_jwt_extended/10_logout_users_rest_api/start/resources/tag.py
@@ -20,7 +20,7 @@ def get(self, store_id):
     @blp.arguments(TagSchema)
     @blp.response(201, TagSchema)
     def post(self, tag_data, store_id):
-        if TagModel.query.filter(TagModel.store_id == store_id).first():
+        if TagModel.query.filter(TagModel.store_id == store_id, TagModel.name == tag_data["name"]).first():
             abort(400, message="A tag with that name already exists in that store.")
 
         tag = TagModel(**tag_data, store_id=store_id)
diff --git a/docs/docs/08_flask_jwt_extended/12_token_refreshing_flask_jwt_extended/end/resources/tag.py b/docs/docs/08_flask_jwt_extended/12_token_refreshing_flask_jwt_extended/end/resources/tag.py
@@ -20,7 +20,7 @@ def get(self, store_id):
     @blp.arguments(TagSchema)
     @blp.response(201, TagSchema)
     def post(self, tag_data, store_id):
-        if TagModel.query.filter(TagModel.store_id == store_id).first():
+        if TagModel.query.filter(TagModel.store_id == store_id, TagModel.name == tag_data["name"]).first():
             abort(400, message="A tag with that name already exists in that store.")
 
         tag = TagModel(**tag_data, store_id=store_id)
diff --git a/docs/docs/08_flask_jwt_extended/12_token_refreshing_flask_jwt_extended/start/resources/tag.py b/docs/docs/08_flask_jwt_extended/12_token_refreshing_flask_jwt_extended/start/resources/tag.py
@@ -20,7 +20,7 @@ def get(self, store_id):
     @blp.arguments(TagSchema)
     @blp.response(201, TagSchema)
     def post(self, tag_data, store_id):
-        if TagModel.query.filter(TagModel.store_id == store_id).first():
+        if TagModel.query.filter(TagModel.store_id == store_id, TagModel.name == tag_data["name"]).first():
             abort(400, message="A tag with that name already exists in that store.")
 
         tag = TagModel(**tag_data, store_id=store_id)
diff --git a/project/05-add-many-to-many/resources/tag.py b/project/05-add-many-to-many/resources/tag.py
@@ -20,7 +20,7 @@ def get(self, store_id):
     @blp.arguments(TagSchema)
     @blp.response(201, TagSchema)
     def post(self, tag_data, store_id):
-        if TagModel.query.filter(TagModel.store_id == store_id).first():
+        if TagModel.query.filter(TagModel.store_id == store_id, TagModel.name == tag_data["name"]).first():
             abort(400, message="A tag with that name already exists in that store.")
 
         tag = TagModel(**tag_data, store_id=store_id)
diff --git a/project/06-add-db-migrations/resources/tag.py b/project/06-add-db-migrations/resources/tag.py
@@ -20,7 +20,7 @@ def get(self, store_id):
     @blp.arguments(TagSchema)
     @blp.response(201, TagSchema)
     def post(self, tag_data, store_id):
-        if TagModel.query.filter(TagModel.store_id == store_id).first():
+        if TagModel.query.filter(TagModel.store_id == store_id, TagModel.name == tag_data["name"]).first():
             abort(400, message="A tag with that name already exists in that store.")
 
         tag = TagModel(**tag_data, store_id=store_id)
