Module 5 commit 9 python issues linting done

Author: neural-techno

server/database/app.js

@@ -1,3 +1,4 @@
+/*jshint esversion: 8 */
 const express = require('express');
 const mongoose = require('mongoose');
 const fs = require('fs');
@@ -18,10 +19,10 @@ const Dealerships = require('./dealership');
 
 try {
   Reviews.deleteMany({}).then(() => {
-    Reviews.insertMany(reviews_data['reviews']);
+    Reviews.insertMany(reviews_data.reviews);
   });
   Dealerships.deleteMany({}).then(() => {
-    Dealerships.insertMany(dealerships_data['dealerships']);
+    Dealerships.insertMany(dealerships_data.dealerships);
   });
 } catch (error) {
   res.status(500).json({ error: 'Error fetching documents' });
@@ -86,18 +87,18 @@ app.get('/fetchDealer/:id', async (req, res) => {
 app.post('/insert_review', express.raw({ type: '*/*' }), async (req, res) => {
   const data = JSON.parse(req.body);
   const documents = await Reviews.find().sort({ id: -1 });
-  const new_id = documents[0]['id'] + 1;
+  const new_id = documents[0].id + 1;
 
   const review = new Reviews({
     id: new_id,
-    name: data['name'],
-    dealership: data['dealership'],
-    review: data['review'],
-    purchase: data['purchase'],
-    purchase_date: data['purchase_date'],
-    car_make: data['car_make'],
-    car_model: data['car_model'],
-    car_year: data['car_year'],
+    name: data.name,
+    dealership: data.dealership,
+    review: data.review,
+    purchase: data.purchase,
+    purchase_date: data.purchase_date,
+    car_make: data.car_make,
+    car_model: data.car_model,
+    car_year: data.car_year,
   });
 
   try {

server/database/dealership.js

@@ -1,3 +1,4 @@
+/*jshint esversion: 8 */
 const mongoose = require('mongoose');
 
 const Schema = mongoose.Schema;

server/database/inventory.js

@@ -1,3 +1,4 @@
+/*jshint esversion: 8 */
 const { Int32 } = require('mongodb');
 const mongoose = require('mongoose');
 

server/database/review.js

@@ -1,3 +1,4 @@
+/*jshint esversion: 8 */
 const mongoose = require('mongoose');
 
 const Schema = mongoose.Schema;

server/djangoapp/models.py

@@ -38,17 +38,17 @@ class CarModel(models.Model):
     mileage, horsepower, and transmission.
 
     Attributes:
-        car_make (ForeignKey): 
+        car_make (ForeignKey):
         name (str): The name of the car model (e.g., Camry, A-Class).
         type (str): The type of the car (e.g., Sedan, SUV, Wagon).
         year (int): The year the car model was manufactured.
         engine_type (str, optional): The type of engine in the car.
-        fuel_type (str): 
+        fuel_type (str):
         color (str, optional): The color of the car.
         price (decimal, optional): The price of the car.
         mileage (float, optional): The mileage of the car.
         horsepower (int, optional): The horsepower of the car's engine.
-        transmission (str): 
+        transmission (str):
     """
     car_make = models.ForeignKey(CarMake, on_delete=models.CASCADE)
     name = models.CharField(max_length=100)
@@ -59,11 +59,12 @@ class CarModel(models.Model):
         # Add more choices as required
     ]
     type = models.CharField(max_length=10, choices=CAR_TYPES, default='SUV')
-    year = models.IntegerField(default=2023,
-    validators = [
-        MaxValueValidator(2023),
-        MinValueValidator(2015)
-    ])
+    year = models.IntegerField(
+        default=2023,
+        validators = [
+            MaxValueValidator(2023),
+            MinValueValidator(2015)
+        ])
     engine_type = models.CharField(max_length=50, blank=True, null=True)
     fuel_type = models.CharField(
         max_length=20,
@@ -87,5 +88,6 @@ class CarModel(models.Model):
         choices=[('AUTOMATIC', 'Automatic'), ('MANUAL', 'Manual')],
         default='AUTOMATIC'
     )
+
     def __str__(self):
         return self.name

server/djangoapp/populate.py

@@ -11,8 +11,8 @@ def initiate():
 
     car_make_data = [
         {
-            "name": "NISSAN", 
-            "description": "Great cars. Japanese technology", 
+            "name": "NISSAN",
+            "description": "Great cars. Japanese technology",
             "country_of_origin": "Japan",
             "established_date": "1933-12-26"
         },

server/djangoapp/views.py

@@ -24,9 +24,16 @@ def login_user(request):
             user = authenticate(username=username, password=password)
             if user is not None:
                 login(request, user)
-                return JsonResponse({"userName": username, "status": "Authenticated"})
+                return JsonResponse(
+                    {
+                        "userName": username,
+                        "status": "Authenticated"
+                    })
 
-        return JsonResponse({"userName": username, "status": "Invalid credentials"})
+        return JsonResponse({
+            "userName": username,
+            "status": "Invalid credentials"
+        })
 
     except json.JSONDecodeError:
         logger.exception("JSON decode error")
@@ -64,11 +71,19 @@ def registration(request):
             return JsonResponse({"userName": username, "error": "Already Registered"})
 
         # Create user in auth_user table
-        user = User.objects.create_user(username=username, first_name=first_name,
-                                        last_name=last_name, password=password, email=email)
+        user = User.objects.create_user(
+            username=username,
+            first_name=first_name,
+            last_name=last_name,
+            password=password,
+            email=email
+        )
         login(request, user)
 
-        return JsonResponse({"userName": username, "status": "Authenticated"})
+        return JsonResponse({
+            "userName": username,
+            "status": "Authenticated"
+        })
 
     except json.JSONDecodeError:
         logger.exception("JSON decode error")
@@ -84,7 +99,10 @@ def get_dealerships(request, state="All"):
     try:
         endpoint = f"/fetchDealers/{state}" if state != "All" else "/fetchDealers"
         dealerships = get_request(endpoint)
-        return JsonResponse({"status": 200, "dealers": dealerships})
+        return JsonResponse({
+            "status": 200,
+            "dealers": dealerships
+        })
 
     except Exception as e:
         logger.exception("An error occurred while fetching dealerships")
@@ -133,7 +151,7 @@ def add_review(request):
     try:
         if not request.user.is_anonymous:
             data = json.loads(request.body)
-            post_review_response = post_review(data)
+            post_review(data)
             return JsonResponse({"status": 200})
 
         return JsonResponse({"status": 403, "message": "Unauthorized"})

server/djangoproj/settings.py

@@ -33,9 +33,14 @@
 
 ALLOWED_HOSTS = [
     'localhost',
-    'https://psoldev9-8000.theiadockernext-1-labs-prod-theiak8s-4-tor01.proxy.cognitiveclass.ai/'
+    'https://psoldev9-8000.theiadockernext-1-labs-prod-theiak8s-4-tor01.proxy.'
+    'cognitiveclass.ai/'
+]
+
+CSRF_TRUSTED_ORIGINS = [
+    'https://psoldev9-8000.theiadockernext-1-labs-prod-theiak8s-4-tor01.proxy.'
+    'cognitiveclass.ai'
 ]
-CSRF_TRUSTED_ORIGINS = ['https://psoldev9-8000.theiadockernext-1-labs-prod-theiak8s-4-tor01.proxy.cognitiveclass.ai']
 
 REST_FRAMEWORK = {
     'DEFAULT_AUTHENTICATION_CLASSES': [],
@@ -99,16 +104,20 @@
 
 AUTH_PASSWORD_VALIDATORS = [
     {
-        'NAME': 'django.contrib.auth.password_validation.UserAttributeSimilarityValidator',
+        'NAME': 'django.contrib.auth.'
+        'password_validation.UserAttributeSimilarityValidator',
     },
     {
-        'NAME': 'django.contrib.auth.password_validation.MinimumLengthValidator',
+        'NAME': 'django.contrib.auth.password_validation.'
+        'MinimumLengthValidator',
     },
     {
-        'NAME': 'django.contrib.auth.password_validation.CommonPasswordValidator',
+        'NAME': 'django.contrib.auth.password_validation.'
+        'CommonPasswordValidator',
     },
     {
-        'NAME': 'django.contrib.auth.password_validation.NumericPasswordValidator',
+        'NAME': 'django.contrib.auth.password_validation.'
+        'NumericPasswordValidator',
     },
 ]