Pylint Fixes

Author: lonelycoder1011

server/database/app.js

@@ -1,31 +1,35 @@
+/* jshint esversion: 8 */
+
 const express = require('express');
 const mongoose = require('mongoose');
 const fs = require('fs');
 const cors = require('cors');
+const bodyParser = require('body-parser');
+
 const app = express();
 const port = 3030;
 
 app.use(cors());
-app.use(require('body-parser').urlencoded({ extended: false }));
+app.use(bodyParser.urlencoded({ extended: false }));
 
-const reviews_data = JSON.parse(fs.readFileSync("reviews.json", 'utf8'));
-const dealerships_data = JSON.parse(fs.readFileSync("dealerships.json", 'utf8'));
+const reviewsData = JSON.parse(fs.readFileSync("reviews.json", 'utf8'));
+const dealershipsData = JSON.parse(fs.readFileSync("dealerships.json", 'utf8'));
 
 mongoose.connect("mongodb://mongo_db:27017/", { dbName: 'dealershipsDB' });
 
 const Reviews = require('./review');
 const Dealerships = require('./dealership');
 
-try {
-  Reviews.deleteMany({}).then(() => {
-    Reviews.insertMany(reviews_data['reviews']);
-  });
-  Dealerships.deleteMany({}).then(() => {
-    Dealerships.insertMany(dealerships_data['dealerships']);
-  });
-} catch (error) {
-  console.error('Error initializing the database:', error);
-}
+(async () => {
+  try {
+    await Reviews.deleteMany({});
+    await Reviews.insertMany(reviewsData.reviews);
+    await Dealerships.deleteMany({});
+    await Dealerships.insertMany(dealershipsData.dealerships);
+  } catch (error) {
+    console.error('Error initializing the database:', error);
+  }
+})();
 
 // Express route to home
 app.get('/', async (req, res) => {
@@ -88,23 +92,23 @@ app.get('/fetchDealer/:id', async (req, res) => {
 
 // Express route to insert review
 app.post('/insert_review', express.raw({ type: '*/*' }), async (req, res) => {
-  const data = JSON.parse(req.body);
-  const documents = await Reviews.find().sort({ id: -1 });
-  let 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'],
-  });
-
   try {
+    const data = JSON.parse(req.body);
+    const documents = await Reviews.find().sort({ id: -1 });
+    const newId = documents.length > 0 ? documents[0].id + 1 : 1;
+
+    const review = new Reviews({
+      id: newId,
+      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,
+    });
+
     const savedReview = await review.save();
     res.json(savedReview);
   } catch (error) {

server/database/dealership.js

@@ -1,43 +1,45 @@
+/* jshint esversion: 6 */
+
 const mongoose = require('mongoose');
 
 const Schema = mongoose.Schema;
 
 const dealerships = new Schema({
-	id: {
+  id: {
     type: Number,
     required: true,
-	},
-	city: {
+  },
+  city: {
     type: String,
-    required: true
+    required: true,
   },
   state: {
     type: String,
-    required: true
+    required: true,
   },
   address: {
     type: String,
-    required: true
+    required: true,
   },
   zip: {
     type: String,
-    required: true
+    required: true,
   },
   lat: {
     type: String,
-    required: true
+    required: true,
   },
   long: {
     type: String,
-    required: true
+    required: true,
   },
   short_name: {
     type: String,
   },
   full_name: {
     type: String,
-    required: true
-  }
+    required: true,
+  },
 });
 
 module.exports = mongoose.model('dealerships', dealerships);

server/database/inventory.js

@@ -1,33 +1,35 @@
+/* jshint esversion: 6 */
+
 const { Int32 } = require('mongodb');
 const mongoose = require('mongoose');
 
 const Schema = mongoose.Schema;
 
 const cars = new Schema({
-dealer_id: {
+  dealer_id: {
     type: Number,
-    required: true
-},
-make: {
+    required: true,
+  },
+  make: {
     type: String,
-    required: true
+    required: true,
   },
-model: {
+  model: {
     type: String,
-    required: true
+    required: true,
   },
-bodyType: {
+  bodyType: {
     type: String,
-    required: true
+    required: true,
   },
-year: {
+  year: {
     type: Number,
-    required: true
+    required: true,
   },
-mileage: {
+  mileage: {
     type: Number,
-    required: true
-  }
+    required: true,
+  },
 });
 
 module.exports = mongoose.model('cars', cars);

server/database/review.js

@@ -1,43 +1,45 @@
+/* jshint esversion: 6 */
+
 const mongoose = require('mongoose');
 
 const Schema = mongoose.Schema;
 
 const reviews = new Schema({
-	id: {
+  id: {
     type: Number,
     required: true,
-	},
-	name: {
+  },
+  name: {
     type: String,
-    required: true
+    required: true,
   },
   dealership: {
     type: Number,
     required: true,
   },
   review: {
     type: String,
-    required: true
+    required: true,
   },
   purchase: {
     type: Boolean,
-    required: true
+    required: true,
   },
   purchase_date: {
     type: String,
-    required: true
+    required: true,
   },
   car_make: {
     type: String,
-    required: true
+    required: true,
   },
   car_model: {
     type: String,
-    required: true
+    required: true,
   },
   car_year: {
     type: Number,
-    required: true
+    required: true,
   },
 });
 

server/djangoapp/models.py

@@ -1,33 +1,8 @@
-# Uncomment the following imports before adding the Model code
-
-# from django.db import models
-# from django.utils.timezone import now
-# from django.core.validators import MaxValueValidator, MinValueValidator
-
-
-# Create your models here.
-
-# <HINT> Create a Car Make model `class CarMake(models.Model)`:
-# - Name
-# - Description
-# - Any other fields you would like to include in car make model
-# - __str__ method to print a car make object
-
-
-# <HINT> Create a Car Model model `class CarModel(models.Model):`:
-# - Many-To-One relationship to Car Make model (One Car Make has many
-# Car Models, using ForeignKey field)
-# - Name
-# - Type (CharField with a choices argument to provide limited choices
-# such as Sedan, SUV, WAGON, etc.)
-# - Year (IntegerField) with min value 2015 and max value 2023
-# - Any other fields you would like to include in car model
-# - __str__ method to print a car make object
-
-# Uncomment the following imports before adding the Model code
 from django.db import models
 from django.utils.timezone import now
 from django.core.validators import MaxValueValidator, MinValueValidator
+from django.contrib import admin
+
 
 # Create your models here.
 
@@ -39,12 +14,14 @@ class CarMake(models.Model):
     created_at = models.DateTimeField(default=now, editable=False)  # Timestamp for creation
 
     def __str__(self):
-        return self.name  # String representation of the CarMake object
+        return self.name  # String representation
+
 
 # Car Model model
 class CarModel(models.Model):
-    # Establish a Many-to-One relationship with CarMake
-    car_make = models.ForeignKey(CarMake, on_delete=models.CASCADE, related_name='models')  
+    car_make = models.ForeignKey(
+        CarMake, on_delete=models.CASCADE, related_name='models'
+    )
     name = models.CharField(max_length=100)  # Car model name
     dealer_id = models.IntegerField()  # Reference to dealer in Cloudant database
 
@@ -55,22 +32,26 @@ class CarModel(models.Model):
         ('WAGON', 'Wagon'),
         ('TRUCK', 'Truck'),
         ('COUPE', 'Coupe'),
-        # Add more car types if required
     ]
-    type = models.CharField(max_length=10, choices=CAR_TYPES, default='SUV')  # Car type
+    type = models.CharField(
+        max_length=10,
+        choices=CAR_TYPES,
+        default='SUV'
+    )  # Car type
+
     year = models.IntegerField(
         validators=[
             MinValueValidator(2015),
             MaxValueValidator(2023)
         ],
         default=2023
     )  # Year of the car model
-    description = models.TextField(blank=True, null=True)  # Optional description of the car model
+
+    description = models.TextField(blank=True, null=True)  # Optional description
 
     def __str__(self):
-        return f"{self.car_make.name} {self.name} ({self.type})"  # String representation of the CarModel object
+        return f"{self.car_make.name} {self.name} ({self.type})"
+
 
-# Register models to the Django Admin site
-from django.contrib import admin
 admin.site.register(CarMake)
 admin.site.register(CarModel)