feat: enhance CourseClass pagination and UI by adding academic term and semester number details
This commit is contained in:
parent
c57787eb20
commit
1b68cf6125
@ -19,13 +19,17 @@ public function getAllForSelect(): Collection
|
|||||||
public function paginated(int $perPage = 25, string $search = '', ?int $academicTermId = null, ?string $method = null): LengthAwarePaginator
|
public function paginated(int $perPage = 25, string $search = '', ?int $academicTermId = null, ?string $method = null): LengthAwarePaginator
|
||||||
{
|
{
|
||||||
return CourseClass::query()
|
return CourseClass::query()
|
||||||
->select(['id', 'course_id', 'lecturer_id', 'academic_term_id', 'method'])
|
->select(['course_classes.id', 'course_classes.course_id', 'course_classes.lecturer_id', 'course_classes.academic_term_id', 'course_classes.method'])
|
||||||
|
->join('academic_terms', 'academic_terms.id', '=', 'course_classes.academic_term_id')
|
||||||
|
->join('courses', 'courses.id', '=', 'course_classes.course_id')
|
||||||
->withCount('enrollments')
|
->withCount('enrollments')
|
||||||
->with(['course:id,code,name,department_id', 'lecturer.user.profile', 'academicTerm:id,academic_year,semester,start_date,end_date'])
|
->with(['course:id,code,name,department_id,semester_number', 'lecturer.user.profile', 'academicTerm:id,academic_year,semester,start_date,end_date'])
|
||||||
->when($search, fn ($q) => $q->whereHas('course', fn ($q) => $q->where('name', 'like', "%{$search}%")->orWhere('code', 'like', "%{$search}%")))
|
->when($search, fn ($q) => $q->whereHas('course', fn ($q) => $q->where('name', 'like', "%{$search}%")->orWhere('code', 'like', "%{$search}%")))
|
||||||
->when($academicTermId, fn ($q) => $q->where('academic_term_id', $academicTermId))
|
->when($academicTermId, fn ($q) => $q->where('course_classes.academic_term_id', $academicTermId))
|
||||||
->when($method, fn ($q) => $q->where('method', $method))
|
->when($method, fn ($q) => $q->where('course_classes.method', $method))
|
||||||
->latest()
|
->orderByDesc('academic_terms.start_date')
|
||||||
|
->orderBy('courses.semester_number')
|
||||||
|
->orderBy('courses.name')
|
||||||
->paginate($perPage);
|
->paginate($perPage);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@ -3,7 +3,6 @@ import { Pencil, Trash2, Users } from 'lucide-react';
|
|||||||
import { RowActions } from '@/components/row-actions';
|
import { RowActions } from '@/components/row-actions';
|
||||||
import { Badge } from '@/components/ui/badge';
|
import { Badge } from '@/components/ui/badge';
|
||||||
import { index as enrollmentsIndex } from '@/routes/admin/manage/course-classes/enrollments';
|
import { index as enrollmentsIndex } from '@/routes/admin/manage/course-classes/enrollments';
|
||||||
import { formatAcademicTermLabel } from '@/types/academic-term';
|
|
||||||
import { ClassMethodLabels } from '@/types/course-class';
|
import { ClassMethodLabels } from '@/types/course-class';
|
||||||
import type { CourseClass } from '@/types/course-class';
|
import type { CourseClass } from '@/types/course-class';
|
||||||
|
|
||||||
@ -44,15 +43,6 @@ export function createCourseClassColumns(
|
|||||||
cell: ({ row }) =>
|
cell: ({ row }) =>
|
||||||
row.original.lecturer?.user?.profile?.full_name ?? '-',
|
row.original.lecturer?.user?.profile?.full_name ?? '-',
|
||||||
},
|
},
|
||||||
{
|
|
||||||
accessorKey: 'academic_term.academic_year',
|
|
||||||
header: () => <span>Periode</span>,
|
|
||||||
cell: ({ row }) => {
|
|
||||||
const term = row.original.academic_term;
|
|
||||||
|
|
||||||
return term ? formatAcademicTermLabel(term) : '-';
|
|
||||||
},
|
|
||||||
},
|
|
||||||
{
|
{
|
||||||
accessorKey: 'method',
|
accessorKey: 'method',
|
||||||
header: () => <span className="block text-center">Metode</span>,
|
header: () => <span className="block text-center">Metode</span>,
|
||||||
|
|||||||
@ -290,6 +290,9 @@ export default function CourseClassIndex({
|
|||||||
onPerPageChange={handlePerPageChange}
|
onPerPageChange={handlePerPageChange}
|
||||||
onSearchChange={handleSearchChange}
|
onSearchChange={handleSearchChange}
|
||||||
searchValue={search}
|
searchValue={search}
|
||||||
|
groupBy={(courseClass) =>
|
||||||
|
`${courseClass.academic_term ? formatAcademicTermLabel(courseClass.academic_term) : 'Tanpa Periode'} — Semester ${courseClass.course?.semester_number ?? 'Tidak ditentukan'}`
|
||||||
|
}
|
||||||
toolbar={
|
toolbar={
|
||||||
<FilterDialog
|
<FilterDialog
|
||||||
fields={filterFields}
|
fields={filterFields}
|
||||||
|
|||||||
@ -16,6 +16,7 @@ export type CourseClass = {
|
|||||||
code: string;
|
code: string;
|
||||||
name: string;
|
name: string;
|
||||||
department_id: number;
|
department_id: number;
|
||||||
|
semester_number: number | null;
|
||||||
} | null;
|
} | null;
|
||||||
lecturer_id: number;
|
lecturer_id: number;
|
||||||
lecturer: {
|
lecturer: {
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user