Skip to content

Commit 2f106ce

Browse files
KaiPexgoKaiPexgo
committed
python lint
1 parent 3567f33 commit 2f106ce

File tree

6 files changed

+75
-56
lines changed

6 files changed

+75
-56
lines changed

‎server/djangoapp/models.py‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,4 +52,3 @@ class CarModel(models.Model):
5252

5353
def __str__(self):
5454
return self.name # Return the name as the string representation
55-

‎server/djangoapp/populate.py‎

Lines changed: 44 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -13,117 +13,125 @@ def initiate():
1313
},
1414
{
1515
"name": "Audi",
16-
"description":"Great cars. German technology"
16+
"description": "Great cars. German technology"
1717
},
1818
{
1919
"name": "Kia",
20-
"description":"Great cars. Korean technology"
20+
"description": "Great cars. Korean technology"
2121
},
2222
{
2323
"name": "Toyota",
24-
"description":"Great cars. Japanese technology"
24+
"description": "Great cars. Japanese technology"
2525
},
2626
]
2727

2828
car_make_instances = []
2929
for data in car_make_data:
30-
car_make_instances.append(CarMake.objects.create(name=data['name'], description=data['description']))
31-
30+
car_make_instances.append(
31+
CarMake.objects.create(
32+
name=data['name'],
33+
description=data['description']
34+
)
35+
)
3236

3337
# Create CarModel instances with the corresponding CarMake instances
3438
car_model_data = [
3539
{
3640
"name": "Pathfinder",
37-
"type": "SUV",
38-
"year": 2023,
41+
"type": "SUV",
42+
"year": 2023,
3943
"car_make": car_make_instances[0]
4044
},
4145
{
4246
"name": "Qashqai",
43-
"type": "SUV",
44-
"year": 2023,
47+
"type": "SUV",
48+
"year": 2023,
4549
"car_make": car_make_instances[0]
4650
},
4751
{
4852
"name": "XTRAIL",
49-
"type": "SUV",
50-
"year": 2023,
53+
"type": "SUV",
54+
"year": 2023,
5155
"car_make": car_make_instances[0]
5256
},
5357
{
5458
"name": "A-Class",
55-
"type": "SUV",
56-
"year": 2023,
59+
"type": "SUV",
60+
"year": 2023,
5761
"car_make": car_make_instances[1]
5862
},
5963
{
6064
"name": "C-Class",
61-
"type": "SUV",
62-
"year": 2023,
65+
"type": "SUV",
66+
"year": 2023,
6367
"car_make": car_make_instances[1]
6468
},
6569
{
6670
"name": "E-Class",
67-
"type": "SUV",
68-
"year": 2023,
71+
"type": "SUV",
72+
"year": 2023,
6973
"car_make": car_make_instances[1]
7074
},
7175
{
7276
"name": "A4",
73-
"type": "SUV",
74-
"year": 2023,
77+
"type": "SUV",
78+
"year": 2023,
7579
"car_make": car_make_instances[2]
7680
},
7781
{
7882
"name": "A5",
79-
"type": "SUV",
80-
"year": 2023,
83+
"type": "SUV",
84+
"year": 2023,
8185
"car_make": car_make_instances[2]
8286
},
8387
{
8488
"name": "A6",
85-
"type": "SUV",
86-
"year": 2023,
89+
"type": "SUV",
90+
"year": 2023,
8791
"car_make": car_make_instances[2]
8892
},
8993
{
9094
"name": "Sorrento",
91-
"type": "SUV",
92-
"year": 2023,
95+
"type": "SUV",
96+
"year": 2023,
9397
"car_make": car_make_instances[3]
9498
},
9599
{
96100
"name": "Carnival",
97-
"type": "SUV",
98-
"year": 2023,
101+
"type": "SUV",
102+
"year": 2023,
99103
"car_make": car_make_instances[3]
100104
},
101105
{
102106
"name": "Cerato",
103-
"type": "Sedan",
104-
"year": 2023,
107+
"type": "Sedan",
108+
"year": 2023,
105109
"car_make": car_make_instances[3]
106110
},
107111
{
108112
"name": "Corolla",
109-
"type": "Sedan",
110-
"year": 2023,
113+
"type": "Sedan",
114+
"year": 2023,
111115
"car_make": car_make_instances[4]
112116
},
113117
{
114118
"name": "Camry",
115-
"type": "Sedan",
116-
"year": 2023,
119+
"type": "Sedan",
120+
"year": 2023,
117121
"car_make": car_make_instances[4]
118122
},
119123
{
120124
"name": "Kluger",
121-
"type": "SUV",
122-
"year": 2023,
125+
"type": "SUV",
126+
"year": 2023,
123127
"car_make": car_make_instances[4]
124128
},
125129
# Add more CarModel instances as needed
126130
]
127131

128132
for data in car_model_data:
129-
CarModel.objects.create(name=data['name'], car_make=data['car_make'], type=data['type'], year=data['year'])
133+
CarModel.objects.create(
134+
name=data['name'],
135+
car_make=data['car_make'],
136+
type=data['type'], year=data['year']
137+
)

‎server/djangoapp/restapis.py‎

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313

1414
# def get_request(endpoint, **kwargs):
1515
# Add code for get requests to back end
16+
1617
def get_request(endpoint, **kwargs):
1718
params = ""
1819
if (kwargs):
@@ -30,8 +31,6 @@ def get_request(endpoint, **kwargs):
3031
# If any error occurs
3132
print("Network exception occurred")
3233

33-
# def analyze_review_sentiments(text):
34-
# request_url = sentiment_analyzer_url+"analyze/"+text
3534
# Add code for retrieving sentiments
3635

3736
def analyze_review_sentiments(text):
@@ -44,7 +43,6 @@ def analyze_review_sentiments(text):
4443
print(f"Unexpected {err=}, {type(err)=}")
4544
print("Network exception occurred")
4645

47-
# def post_review(data_dict):
4846
# Add code for posting review
4947

5048
def post_review(data_dict):

‎server/djangoapp/urls.py‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,5 +26,5 @@
2626
name='dealer_details'
2727
),
2828
path(route='add_review', view=views.add_review, name='add_review'),
29-
path(route='get_cars', view=views.get_cars, name ='getcars'),
29+
path(route='get_cars', view=views.get_cars, name='getcars'),
3030
] + static(settings.MEDIA_URL, document_root=settings.MEDIA_ROOT)

‎server/djangoapp/views.py‎

Lines changed: 23 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -98,26 +98,27 @@ def registration(request):
9898
login(request, user)
9999
data = {"userName": username, "status": "Authenticated"}
100100
return JsonResponse(data)
101-
else :
102-
data = {"userName": username,"error": "Already Registered"}
101+
else:
102+
data = {"userName": username, "error": "Already Registered"}
103103
return JsonResponse(data)
104104

105-
#Update the `get_dealerships` render list of dealerships all by default, particular state if state is passed
105+
# Update the `get_dealerships` render list of dealerships all by default,
106+
# particular state if state is passed
106107

107108
def get_dealerships(request, state="All"):
108-
if(state == "All"):
109+
if (state == "All"):
109110
endpoint = "/fetchDealers"
110111
else:
111112
endpoint = "/fetchDealers/"+state
112113
dealerships = get_request(endpoint)
113-
return JsonResponse({"status":200,"dealers":dealerships})
114+
return JsonResponse({"status": 200, "dealers": dealerships})
114115

115116
# Create a `get_dealer_reviews` view to render the reviews of a dealer
116117

117118
def get_dealer_reviews(request, dealer_id):
118119
# if dealer id has been provided
119120
# print(dealer_id)
120-
if(dealer_id):
121+
if (dealer_id):
121122
endpoint = "/fetchReviews/dealer/"+str(dealer_id)
122123
reviews = get_request(endpoint)
123124
for review_detail in reviews:
@@ -131,7 +132,7 @@ def get_dealer_reviews(request, dealer_id):
131132
# Create a `get_dealer_details` view to render the dealer details
132133

133134
def get_dealer_details(request, dealer_id):
134-
if(dealer_id):
135+
if (dealer_id):
135136
endpoint = "/fetchDealer/"+str(dealer_id)
136137
dealership = get_request(endpoint)
137138
return JsonResponse({"status": 200, "dealer": dealership})
@@ -141,13 +142,22 @@ def get_dealer_details(request, dealer_id):
141142
# Create a `add_review` view to submit a review
142143

143144
def add_review(request):
144-
if(request.user.is_anonymous == False):
145+
if (request.user.is_anonymous is False):
145146
data = json.loads(request.body)
146-
147147
try:
148-
response = post_review(data)
148+
post_review(data)
149149
return JsonResponse({"status": 200})
150-
except:
151-
return JsonResponse({"status": 401, "message": "Error in posting review"})
150+
except Exception:
151+
return JsonResponse(
152+
{
153+
"status": 401,
154+
"message": "Error in posting review"
155+
}
156+
)
152157
else:
153-
return JsonResponse({"status": 403, "message": "Unauthorized"})
158+
return JsonResponse(
159+
{
160+
"status": 403,
161+
"message": "Unauthorized"
162+
}
163+
)

‎server/djangoproj/settings.py‎

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,10 +30,14 @@
3030

3131
ALLOWED_HOSTS = [
3232
'localhost',
33-
'https://kyleaguilar2-8000.theiadockernext-0-labs-prod-theiak8s-4-tor01.proxy.cognitiveclass.ai'
33+
"""https://kyleaguilar2-8000
34+
.theiadockernext-0-labs-prod-theiak8s-4-tor01
35+
.proxy.cognitiveclass.ai"""
3436
]
3537
CSRF_TRUSTED_ORIGINS = [
36-
'https://kyleaguilar2-8000.theiadockernext-0-labs-prod-theiak8s-4-tor01.proxy.cognitiveclass.ai'
38+
"""https://kyleaguilar2-8000
39+
.theiadockernext-0-labs-prod-theiak8s-4-tor01
40+
.proxy.cognitiveclass.ai"""
3741
]
3842

3943
REST_FRAMEWORK = {

0 commit comments

Comments
 (0)