diff --git a/app/Controllers/Http/Api/v1/LessonsController.ts b/app/Controllers/Http/Api/v1/LessonsController.ts index 3ac125c..90a9101 100644 --- a/app/Controllers/Http/Api/v1/LessonsController.ts +++ b/app/Controllers/Http/Api/v1/LessonsController.ts @@ -1,6 +1,7 @@ import Application from '@ioc:Adonis/Core/Application'; import { Exception, inject, Ioc } from '@adonisjs/core/build/standalone'; import { HttpContextContract } from '@ioc:Adonis/Core/HttpContext'; +import { schema } from '@ioc:Adonis/Core/Validator'; /** * Services @@ -137,6 +138,25 @@ export default class LessonsController extends BaseController { return this.sendResponse(ctx, data, message, status); } + + /** + * Save order of lessons + * POST /lessons/save-order + */ + // eslint-disable-next-line class-methods-use-this + public async saveOrder(ctx: HttpContextContract) { + const saveOrderSchema = schema.create({ + ids: schema.array().members(schema.string()), + }); + const payload = await ctx.request.validate({ schema: saveOrderSchema }); + const { data, message, status, success, error } = await this.lessonService.updateOrder(payload.ids); + + if (!success && error) { + throw new Exception(message, status, error.code); + } + + return this.sendResponse(ctx, data, message, status); + } } new Ioc().make(LessonsController); diff --git a/app/Datatypes/Enums/ColorEnum.ts b/app/Datatypes/Enums/ColorEnum.ts index a71f6e7..4ee5616 100644 --- a/app/Datatypes/Enums/ColorEnum.ts +++ b/app/Datatypes/Enums/ColorEnum.ts @@ -11,15 +11,6 @@ enum ColorEnum { Purple = '#800080', Indigo = '#4B0082', Teal = '#008080', - LightGray = '#D3D3D3', - LightSlateGray = '#778899', - LightGreen = '#90EE90', - LightSeaGreen = '#20B2AA', - LightCoral = '#FFA07A', - LightCyan = '#E0FFFF', - LightSteelBlue = '#B0C4DE', - LightBlue = '#ADD8E6', - LightSkyBlue = '#87CEFA', MediumVioletRed = '#C71585', MediumOrchid = '#BA55D3', MediumPurple = '#9370DB', diff --git a/app/Models/Lesson.ts b/app/Models/Lesson.ts index 8c1cd5f..30e79b2 100644 --- a/app/Models/Lesson.ts +++ b/app/Models/Lesson.ts @@ -2,6 +2,7 @@ import { DateTime } from 'luxon'; import { nanoid } from 'nanoid'; import { BaseModel, beforeCreate, BelongsTo, belongsTo, column, HasOne, hasOne } from '@ioc:Adonis/Lucid/Orm'; +import ColorHelper from 'App/Helpers/ColorHelper'; import Course from './Course'; // eslint-disable-next-line import/no-cycle import LessonContent from './LessonContent'; @@ -17,6 +18,9 @@ export default class Lesson extends BaseModel { @column() public title: string; + @column({ serializeAs: null }) + public display_order: number; + @column() public description: string; @@ -55,4 +59,13 @@ export default class Lesson extends BaseModel { public static assignLessonId(lesson: Lesson) { lesson.id = nanoid(); } + + @beforeCreate() + public static async assignRandomColor(course: Course) { + const color = await ColorHelper.generateRandomColor(); + + if (color) { + course.color_id = color.id; + } + } } diff --git a/app/Repositories/CourseRepository.ts b/app/Repositories/CourseRepository.ts index a9fa990..28fb993 100644 --- a/app/Repositories/CourseRepository.ts +++ b/app/Repositories/CourseRepository.ts @@ -82,7 +82,7 @@ export default class CourseRepository { .preload('teacher') .preload('color') .preload('category') - .preload('lessons', q => q.preload('color')) + .preload('lessons', q => q.preload('color').orderBy('display_order', 'asc')) .preload('students') .where('id', id) .first(); diff --git a/app/Repositories/LessonRepository.ts b/app/Repositories/LessonRepository.ts index 9eff7fc..437144e 100644 --- a/app/Repositories/LessonRepository.ts +++ b/app/Repositories/LessonRepository.ts @@ -72,10 +72,13 @@ export default class LessonRepository { * @returns Lesson */ public async create(course: Course, data: CreateLessonValidator['schema']['props']): Promise { + await course.loadCount('lessons'); + const lesson = await this.Lesson.create({ title: data.title, description: data.description, duration: data.duration.toFormat('HH:mm:ss'), + display_order: Number.parseInt(course.$extras.lessons_count, 10) + 1, }); /** diff --git a/app/Services/LessonService.ts b/app/Services/LessonService.ts index b3401cd..7a39a07 100644 --- a/app/Services/LessonService.ts +++ b/app/Services/LessonService.ts @@ -1,5 +1,6 @@ import { inject, Ioc } from '@adonisjs/core/build/standalone'; import { HttpContextContract } from '@ioc:Adonis/Core/HttpContext'; +import Database from '@ioc:Adonis/Lucid/Database'; /** * Datatypes @@ -264,6 +265,32 @@ export default class LessonService { data: lesson.content, }; } + + // eslint-disable-next-line class-methods-use-this + public async updateOrder(ids: string[]): Promise> { + /** + * Update order of lessons + */ + const params = ids + .map((id, i) => ({ id, i: i + 1 })) + .reduce((result: Array, { id, i }) => result.concat(id, i), []); + + const query = ` + UPDATE lessons + SET display_order = t.display_order + FROM (VALUES ${ids.map(() => `(?, ??)`).join(',')}) as t(id, display_order) + WHERE t.id = lessons.id + `; + + await Database.rawQuery(query, params); + + return { + success: true, + status: HttpStatusEnum.OK, + message: 'Order updated.', + data: {}, + }; + } } new Ioc().make(LessonService); diff --git a/database/factories/index.ts b/database/factories/index.ts index 18f2dc0..b1c2c90 100644 --- a/database/factories/index.ts +++ b/database/factories/index.ts @@ -64,12 +64,13 @@ export const LessonFactory = Factory.define(Lesson, ({ faker }) => { return { title: faker.lorem.sentence(6), description: faker.lorem.sentence(7), + display_order: 1, duration: DateTime.fromObject({ hour: faker.datatype.number(2), minute: faker.datatype.number({ min: 30, max: 59 }), second: 0, }).toFormat('HH:mm:ss'), - color_id: faker.datatype.number({ min: 1, max: 25 }), + color_id: faker.datatype.number({ min: 1, max: 15 }), }; }) .relation('content', () => LessonContentFactory) diff --git a/database/migrations/1617351321869_colors.ts b/database/migrations/1617351321869_colors.ts index b531583..f26702f 100644 --- a/database/migrations/1617351321869_colors.ts +++ b/database/migrations/1617351321869_colors.ts @@ -11,7 +11,7 @@ export default class Colors extends BaseSchema { .enu('name', Object.keys(ColorEnum), { useNative: true, enumName: 'color_name_enum', - existingType: true, + existingType: false, schemaName: 'public', }) .notNullable(); @@ -20,6 +20,7 @@ export default class Colors extends BaseSchema { } public async down() { + await this.schema.raw('DROP TYPE IF EXISTS "color_name_enum" CASCADE'); this.schema.dropTable(this.tableName); } } diff --git a/database/migrations/1627300211260_courses.ts b/database/migrations/1627300211260_courses.ts index be8408e..9657c28 100644 --- a/database/migrations/1627300211260_courses.ts +++ b/database/migrations/1627300211260_courses.ts @@ -21,7 +21,7 @@ export default class Courses extends BaseSchema { .enu('status', Object.values(CourseStatusEnum), { useNative: true, enumName: 'course_status_enum', - existingType: true, + existingType: false, schemaName: 'public', }) .notNullable(); @@ -31,7 +31,7 @@ export default class Courses extends BaseSchema { } public async down() { - await this.schema.raw('DROP TYPE IF EXISTS "course_status" CASCADE'); + await this.schema.raw('DROP TYPE IF EXISTS "course_status_enum" CASCADE'); this.schema.dropTable(this.tableName); } } diff --git a/database/migrations/1628016919501_lessons.ts b/database/migrations/1628016919501_lessons.ts index 7545313..6c65853 100644 --- a/database/migrations/1628016919501_lessons.ts +++ b/database/migrations/1628016919501_lessons.ts @@ -8,6 +8,7 @@ export default class Lessons extends BaseSchema { table.string('id', 21).primary(); table.string('course_id', 21).unsigned().references('courses.id').onDelete('CASCADE'); table.string('title').notNullable(); + table.integer('display_order').notNullable(); table.integer('color_id').unsigned().references('colors.id'); table.string('description'); table.time('duration').notNullable(); diff --git a/database/seeders/Course.ts b/database/seeders/Course.ts index 4a21c77..78ca18d 100644 --- a/database/seeders/Course.ts +++ b/database/seeders/Course.ts @@ -7,7 +7,7 @@ export default class CourseSeeder extends BaseSeeder { public async run() { this.CourseFactory = CourseFactory; - await this.CourseFactory.with('lessons', 2, lessonFactory => lessonFactory.with('content')) + await this.CourseFactory.with('lessons', 10, lessonFactory => lessonFactory.with('content')) .with('category') .with('teacher') .createMany(3); diff --git a/routes/apis/v1/lessons.ts b/routes/apis/v1/lessons.ts index e8d14ab..f2fa3f4 100644 --- a/routes/apis/v1/lessons.ts +++ b/routes/apis/v1/lessons.ts @@ -3,6 +3,9 @@ import Route from '@ioc:Adonis/Core/Route'; Route.group(() => { Route.get('/', 'Api/v1/LessonsController.list').middleware('role:admin,teacher,student').as('lessons.list'); Route.post('/', 'Api/v1/LessonsController.create').middleware('role:admin,teacher').as('lessons.create'); + Route.post('/save-order', 'Api/v1/LessonsController.saveOrder') + .middleware('role:admin,teacher') + .as('lessons.update-order'); Route.get('/:id', 'Api/v1/LessonsController.show').middleware('role:admin,teacher,student').as('lessons.show'); Route.delete('/:id', 'Api/v1/LessonsController.delete').middleware('role:admin,teacher').as('lessons.delete'); Route.put('/:id', 'Api/v1/LessonsController.update').middleware('role:admin,teacher').as('lessons.update');