modified student bulk upload

mwinamijr · mwinamijr · commit 91d515865a68 · 2023-10-09T09:59:21.000+03:00
diff --git a/academic/migrations/0009_remove_teacher_teacher_id_student_prem_number.py b/academic/migrations/0009_remove_teacher_teacher_id_student_prem_number.py
@@ -0,0 +1,22 @@
+# Generated by Django 4.0 on 2023-10-09 06:00
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('academic', '0008_student_gender'),
+    ]
+
+    operations = [
+        migrations.RemoveField(
+            model_name='teacher',
+            name='teacher_id',
+        ),
+        migrations.AddField(
+            model_name='student',
+            name='prem_number',
+            field=models.CharField(blank=True, max_length=50),
+        ),
+    ]
diff --git a/academic/models.py b/academic/models.py
@@ -53,7 +53,6 @@ class Teacher(models.Model):
     last_name = models.CharField(max_length=300, verbose_name="Last Name", blank=True)
     gender = models.CharField(max_length=10, choices=GENDER_CHOICE, blank=True)
     email = models.EmailField(blank=True, null=True)
-    teacher_id = models.CharField(max_length=255, unique=True, blank=True, null=True)
     empId = models.CharField(max_length=8, null=True, blank=True, unique=True)
     tin_number = models.CharField(max_length=9, null=True, blank=True)
     nssf_number = models.CharField(max_length=9, null=True, blank=True)
@@ -91,7 +90,9 @@ def save(
         #  check if the person is already a student
         # if Student.objects.filter(id=self.id).count():
         #    raise ValidationError("cannot have a someone be a student and a Teacher")
-
+        # create username
+        username = self.first_name + self.last_name
+        self.username = username
         # save model
         super(Teacher, self).save()
 
@@ -351,7 +352,9 @@ class Student(models.Model):
     gender = models.CharField(
         max_length=10, choices=GENDER_CHOICE, blank=True, null=True
     )
-    religion = models.CharField(max_length=50, choices=RELIGION_CHOICE, null=True)
+    religion = models.CharField(
+        max_length=50, choices=RELIGION_CHOICE, blank=True, null=True
+    )
     region = models.CharField(max_length=255, blank=True, null=True)
     city = models.CharField(max_length=255, blank=True, null=True)
     street = models.CharField(max_length=255, blank=True)
@@ -363,6 +366,7 @@ class Student(models.Model):
     date_of_birth = models.DateField(blank=True)
     admission_date = models.DateTimeField(auto_now_add=True)
     admission_number = models.CharField(max_length=50, blank=True, unique=True)
+    prem_number = models.CharField(max_length=50, blank=True)
     siblings = models.ManyToManyField("Student", blank=True)
     image = models.ImageField(upload_to="StudentsImages", blank=True)
 
diff --git a/academic/serializers.py b/academic/serializers.py
@@ -108,7 +108,7 @@ def create(self, request):
         student.first_name = data["first_name"]
         student.middle_name = data["middle_name"]
         student.last_name = data["last_name"]
-        student.addmission_number = data["addmission_number"]
+        student.admission_number = data["addmission_number"]
         student.parent_contact = data["parent_contact"]
         student.region = data["region"]
         student.city = data["city"]
@@ -121,23 +121,18 @@ def create(self, request):
         student.save()
         return student
 
-    def bulk_create(self, request):
-        data = request.data
+    def bulk_create(self, student):
+        data = student
+        print(data)
         student = Student()
-        student.addmission_number = data["addmission_number"]
-        student.first_name = data["first_name"]
-        student.middle_name = data["middle_name"]
-        student.last_name = data["last_name"]
-        student.class_level = ClassRoom.objects.get(name=data["class_level"])
-        student.birthday = data["birthday"]
-        print(data["birthday"][:10])
-        # student.grad_date = data['grad_date']
-        student.region = data["region"]
-        student.city = data["city"]
-        student.street = data["street"]
-        student.prems_number = data["prems_number"]
-        student.sex = data["sex"]
-        student.std_vii_number = data["std_vii_number"]
-
-        # student.save()
+        student.first_name = data["first_name"].lower()
+        student.middle_name = data["middle_name"].lower()
+        student.last_name = data["last_name"].lower()
+        student.admission_number = data["addmission_number"]
+        student.parent_contact = data["parent_contact"]
+        student.grade_level = GradeLevel.objects.get(name=data["grade_level"])
+        student.gender = data["gender"]
+        student.date_of_birth = "2000-01-01"
+        # student.class_of_year = ClassYear.objects.get(year=data['class_of_year'])
+        student.save()
         return student
diff --git a/sis/views.py b/sis/views.py
@@ -12,102 +12,120 @@
 from .serializers import FileUploadSerializer
 from academic.serializers import StudentSerializer
 
+
 class StudentListView(views.APIView):
-	"""
+    """
     List all students, or create a new student.
     """
-	#permission_classes = [IsAuthenticated]
-	def get(self, request, format=None):
-		students = Student.objects.all()
-		serializer = StudentSerializer(students, many=True)
-		return Response(serializer.data)
-
-	def post(self, request, format=None):
-		serializer = StudentSerializer(data=request.data)
-
-		print(serializer.is_valid())
-		print(request.data)
-		if serializer.is_valid():
-			student = serializer.create(request)
-			if student:
-				#serializer.save()
-				return Response(status=status.HTTP_201_CREATED)
-		return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST)
+
+    # permission_classes = [IsAuthenticated]
+    def get(self, request, format=None):
+        students = Student.objects.all()
+        serializer = StudentSerializer(students, many=True)
+        return Response(serializer.data)
+
+    def post(self, request, format=None):
+        serializer = StudentSerializer(data=request.data)
+
+        print(serializer.is_valid())
+        print(request.data)
+        if serializer.is_valid():
+            student = serializer.create(request)
+            if student:
+                # serializer.save()
+                return Response(data=student, status=status.HTTP_201_CREATED)
+        return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST)
+
 
 class StudentDetailView(views.APIView):
-	permission_classes = [IsAuthenticated]
-	def get_object(self, pk):
-		try:
-			return Student.objects.get(pk=pk)
-		except Student.DoesNotExist:
-			raise Http404
-	def get(self, request, pk, format=None):
-		student = self.get_object(pk)
-		serializer = StudentSerializer(student)
-		return Response(serializer.data)
-		
-	def put(self, request, pk, format=None):
-		student = self.get_object(pk)
-		serializer = StudentSerializer(student, data=request.data)
-		if serializer.is_valid():
-			serializer.save()
-			return Response(serializer.data)
-		return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST)
-	
-	def delete(self, request, pk, format=None):
-		student = self.get_object(pk)
-		student.delete()
-		return Response(status=status.HTTP_204_NO_CONTENT)
+    permission_classes = [IsAuthenticated]
+
+    def get_object(self, pk):
+        try:
+            return Student.objects.get(pk=pk)
+        except Student.DoesNotExist:
+            raise Http404
+
+    def get(self, request, pk, format=None):
+        student = self.get_object(pk)
+        serializer = StudentSerializer(student)
+        return Response(serializer.data)
+
+    def put(self, request, pk, format=None):
+        student = self.get_object(pk)
+        serializer = StudentSerializer(student, data=request.data)
+        if serializer.is_valid():
+            serializer.save()
+            return Response(serializer.data)
+        return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST)
+
+    def delete(self, request, pk, format=None):
+        student = self.get_object(pk)
+        student.delete()
+        return Response(status=status.HTTP_204_NO_CONTENT)
+
 
 class StudentBulkUploadView(views.APIView):
-	"""
-	This uploads bulk number of students from excel file
-	"""
-
-	parser_class = [FileUploadParser]
-	def post(self, request, filename, format="xlsx"):
-		file_obj = request.data
-		print(request.data)
-		
-		xlfile = file_obj["filename"]
-
-		wb = load_workbook(xlfile)
-		ws = wb.active
-		print(ws.title)
-		
-		studentz = []
-		for row in ws.iter_rows(min_row=2, max_col=9, max_row=6, values_only=True):
-			studentz.append(row)
-		#print(studentz)
-		
-		students = []
-		for i in range(len(studentz)):
-			student = {
-				"addmission_number": f"{studentz[i][0]}",
-				"first_name": f"{studentz[i][1]}",
-				"middle_name": f"{studentz[i][2]}",
-				"last_name": f"{studentz[i][3]}",
-				"grad_date": f"{studentz[i][4]}",
-				"sex": f"{studentz[i][5]}",
-				"birthday": f"{studentz[i][6]}",
-				"class_level": f"{studentz[i][8]}",	
-					}
-			students.append(student)
-		
-		for student in students:
-			if student in Student.objects.all():
-				print("student exists!")
-				continue
-			else: 
-				serializer = StudentSerializer(data=student)
-				if serializer.is_valid():
-					serializer.save()
-
-		return Response(status=status.HTTP_201_CREATED)
-
-
-
-'''
+    """
+    This uploads bulk number of students from excel file
+    """
+
+    parser_class = [FileUploadParser]
+
+    def post(self, request, filename, format="xlsx"):
+        file_obj = request.data
+
+        xlfile = file_obj["filename"]
+        print(xlfile)
+
+        wb = load_workbook(xlfile)
+        ws = wb.active
+        print(ws.title)
+
+        studentz = []
+        for row in ws.iter_rows(min_row=5, max_col=9, max_row=99, values_only=True):
+            studentz.append(row)
+        # print(studentz)
+
+        students = []
+        for i in range(len(studentz)):
+            student = {
+                "first_name": f"{studentz[i][1].split()[0]}",
+                "middle_name": f"{studentz[i][1].split()[1]}",
+                "last_name": f"{studentz[i][1].split()[2]}",
+                "gender": f"{studentz[i][2]}",
+                "prem number": f"{studentz[i][0]}",
+                "grade_level": f"{studentz[i][3]}",
+                "parent_contact": f"{studentz[i][4]}",
+                "addmission_number": f"{studentz[i][5]}",
+                "religion": f"{studentz[i][7]}",
+            }
+            students.append(student)
+        print(students)
+
+        created = []
+        exist = []
+        for student in students:
+            if student in Student.objects.all():
+                print("student exists!")
+                exist.append(student)
+                continue
+            else:
+                serializer = StudentSerializer(data=student)
+                print(serializer.is_valid())
+                if serializer.is_valid():
+                    student = serializer.bulk_create(student)
+                    created.append(student)
+                    # if student:
+                    # serializer.save()
+                    # serializer.save()
+
+        return Response(
+            data={"created": created, "existing": exist}, status=status.HTTP_201_CREATED
+        )
+
+
+"""
 class StudentHealthRecordViewSet(viewsets.ModelViewSet):
 	queryset = StudentHealthRecord.objects.all()
 	serializer_class = StudentHealthRecordSerializer
@@ -127,4 +145,4 @@ class SchoolYearViewSet(viewsets.ModelViewSet):
 class MessageToStudentViewSet(viewsets.ModelViewSet):
 	queryset = MessageToStudent.objects.all()
 	serializer_class = MessageToStudentSerializer
-'''
+"""
