Pull Request

.github/workflows/main.yml

@@ -0,0 +1,59 @@
+name: 'Lint Code'
+
+on:
+  push:
+    branches: [master, main]
+  pull_request:
+    branches: [master, main]
+
+jobs:
+  lint_python:
+    name: Lint Python Files
+    runs-on: ubuntu-latest
+
+    steps:
+
+    - name: Checkout Repository
+      uses: actions/checkout@v3
+
+    - name: Set up Python
+      uses: actions/setup-python@v4
+      with:
+        python-version: 3.12
+
+    - name: Install dependencies
+      run: |
+        python -m pip install --upgrade pip
+        pip install flake8
+
+    - name: Print working directory
+      run: pwd
+
+    - name: Run Linter
+      run: |
+        pwd
+        # This command finds all Python files recursively and runs flake8 on them
+        find . -name "*.py" -exec flake8 {} +
+        echo "Linted all the python files successfully"
+
+  lint_js:
+      name: Lint JavaScript Files
+      runs-on: ubuntu-latest
+
+      steps:
+      - name: Checkout Repository
+        uses: actions/checkout@v3
+
+      - name: Install Node.js
+        uses: actions/setup-node@v3
+        with:
+          node-version: 14
+
+      - name: Install JSHint
+        run: npm install jshint --global
+
+      - name: Run Linter
+        run: |
+          # This command finds all JavaScript files recursively and runs JSHint on them
+          find ./server/database -name "*.js" -exec jshint {} +
+          echo "Linted all the js files successfully"

server/.jshintrc

@@ -0,0 +1,13 @@
+{
+  "esversion": 8,
+  "node": true,
+  "strict": false,
+  "undef": true,
+  "unused": true,
+  "globals": {
+    "describe": false, 
+    "it": false,
+    "before": false,
+    "after": false
+  }
+}

server/database/app.js

@@ -1,39 +1,45 @@
 const express = require('express');
 const mongoose = require('mongoose');
 const fs = require('fs');
-const  cors = require('cors')
-const app = express()
+const cors = require('cors');
+const app = express();
 const port = 3030;
 
-app.use(cors())
+app.use(cors());
 app.use(require('body-parser').urlencoded({ extended: false }));
 
-const reviews_data = JSON.parse(fs.readFileSync("reviews.json", 'utf8'));
-const dealerships_data = JSON.parse(fs.readFileSync("dealerships.json", 'utf8'));
+const reviews_data = JSON.parse(
+	fs.readFileSync("reviews.json", 'utf8')
+);
+
+const dealerships_data = JSON.parse(
+	fs.readFileSync("dealerships.json", 'utf8')
+);
 
 mongoose.connect("mongodb://mongo_db:27017/",{'dbName':'dealershipsDB'});
+initializeDatabase();
 
 
 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) {
-  res.status(500).json({ error: 'Error fetching documents' });
+async function initializeDatabase() {
+	try {
+		await Reviews.deleteMany({});
+		await Reviews.insertMany(reviews_data.reviews);
+		await Dealerships.deleteMany({});
+		await Dealerships.insertMany(dealerships_data.dealerships);
+		console.log("Database initialized");
+	} catch (error) {
+		console.error("Error initializing database:", error);
+	}
 }
 
 
 // Express route to home
 app.get('/', async (req, res) => {
-    res.send("Welcome to the Mongoose API")
+    res.send("Welcome to the Mongoose API");
 });
 
 // Express route to fetch all reviews
@@ -48,45 +54,69 @@ app.get('/fetchReviews', async (req, res) => {
 
 // Express route to fetch reviews by a particular dealer
 app.get('/fetchReviews/dealer/:id', async (req, res) => {
-  try {
-    const documents = await Reviews.find({dealership: req.params.id});
-    res.json(documents);
-  } catch (error) {
-    res.status(500).json({ error: 'Error fetching documents' });
-  }
+	try {
+		const dealerId = Number(req.params.id);
+		if (isNaN(dealerId)) {
+			return res.status(400).json({error: 'Ungültige Dealer-ID'});
+		}
+		const documents = await Reviews.find({dealership: dealerId});
+		if (documents.length === 0) {
+			return res.status(404).json(
+				{error: 'Keine Reviews für diesen Dealer gefunden'});
+		}
+		res.json(documents);
+	} catch (error) {
+		res.status(500).json({error: 'Fehler beim Abrufen der Reviews'});
+	}
 });
 
 // Express route to fetch all dealerships
 app.get('/fetchDealers', async (req, res) => {
-//Write your code here
+	try {
+		const documents = await Dealerships.find();
+		res.json(documents);
+	} catch (error) {
+		res.status(500).json({ error: 'Error fetching documents' });
+	}
 });
 
 // Express route to fetch Dealers by a particular state
 app.get('/fetchDealers/:state', async (req, res) => {
-//Write your code here
+	try {
+		const documents = await Dealerships.find({state: req.params.state});
+		res.json(documents);
+	} catch (error) {
+		res.status(500).json({ error: 'Error fetching documents' });
+	}
 });
 
 // Express route to fetch dealer by a particular id
 app.get('/fetchDealer/:id', async (req, res) => {
-//Write your code here
+	try {
+		const dealerId = Number(req.params.id);
+		const documents = await Dealerships.find({id: dealerId});
+		res.json(documents);
+	} catch (error) {
+		res.status(500).json({error: 'Error fetching documents'});
+	}
 });
 
 //Express route to insert review
 app.post('/insert_review', express.raw({ type: '*/*' }), async (req, res) => {
-  data = JSON.parse(req.body);
-  const documents = await Reviews.find().sort( { id: -1 } )
-  let new_id = documents[0]['id']+1
+  const data = JSON.parse(req.body);
+  const documents = await Reviews.find().sort( { id: -1 } );
+	let new_id = documents.length > 0 ? documents[0].id + 1 : 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/inventory.js

@@ -1,4 +1,3 @@
-const { Int32 } = require('mongodb');
 const mongoose = require('mongoose');
 
 const Schema = mongoose.Schema;

server/djangoapp/.env

@@ -1,2 +1,2 @@
-backend_url =your backend url
-sentiment_analyzer_url=your code engine deployment url
+backend_url = http://localhost:3030
+sentiment_analyzer_url = http://127.0.0.1:5000

server/djangoapp/admin.py

@@ -1,5 +1,5 @@
-# from django.contrib import admin
-# from .models import related models
+from django.contrib import admin
+from .models import CarMake, CarModel
 
 
 # Register your models here.
@@ -11,3 +11,5 @@
 # CarMakeAdmin class with CarModelInline
 
 # Register models here
+admin.site.register(CarMake)
+admin.site.register(CarModel)