From 7cc2e7fa72f1a2fd5a088155af44f815a383ef95 Mon Sep 17 00:00:00 2001 From: Ethan Ho Date: Tue, 12 Mar 2024 13:38:51 -0700 Subject: [PATCH 1/4] Implemented event deletion, updated schemas/databases. Minor bugfixes. --- common/utils.js | 2 +- routes/catalog.js | 20 ++++++++++++++------ server/schema/day.sql | 5 +++-- server/schema/published_schedule.sql | 2 +- 4 files changed, 19 insertions(+), 10 deletions(-) diff --git a/common/utils.js b/common/utils.js index b39ec62..ba67f5e 100644 --- a/common/utils.js +++ b/common/utils.js @@ -27,7 +27,7 @@ const isInteger = (value) => { // dependency for publishedSchedule.js const calculateYear = (eventDate, gradeLevel) => { - if (gradeLevel) { + if (gradeLevel && gradeLevel.length) { const currentDay = new Date(eventDate); // console.log('current day', currentDay.getFullYear() + (currentDay.getMonth() >= 7 ? 2 : 1)); if (gradeLevel.toLowerCase() === 'junior') { diff --git a/routes/catalog.js b/routes/catalog.js index f3c18a5..fa067f8 100644 --- a/routes/catalog.js +++ b/routes/catalog.js @@ -15,7 +15,7 @@ catalogRouter.get('/', async (req, res) => { const offset = (page - 1) * limit; - let query = 'FROM catalog WHERE 1=1'; + let query = 'FROM catalog WHERE 1=1 AND hidden = false'; // removed space at beginning here const params = []; @@ -85,7 +85,9 @@ catalogRouter.get('/', async (req, res) => { catalogRouter.get('/:id', async (req, res) => { try { const { id } = req.params; - const allUsers = await db.query(`SELECT * FROM catalog WHERE id = $1;`, [id]); + const allUsers = await db.query(`SELECT * FROM catalog WHERE id = $1 AND hidden = false;`, [ + id, + ]); res.status(200).json(keysToCamel(allUsers)); } catch (err) { res.status(500).send(err.message); @@ -97,8 +99,8 @@ catalogRouter.post('/', async (req, res) => { const { host, title, eventType, subject, description, year, season, location } = req.body; try { const returnedData = await db.query( - `INSERT INTO catalog (id, host, title, event_type, subject, description, year, season, location) - VALUES (nextval('catalog_id_seq'), $1, $2, $3, $4, $5, $6, $7, $8) + `INSERT INTO catalog (id, host, title, event_type, subject, description, year, season, location, hidden) + VALUES (nextval('catalog_id_seq'), $1, $2, $3, $4, $5, $6, $7, $8, false) RETURNING id;`, [host, title, eventType, subject, description, year, season, location], ); @@ -153,8 +155,14 @@ catalogRouter.put('/:id', async (req, res) => { catalogRouter.delete('/:id', async (req, res) => { try { const { id } = req.params; - const delUser = await db.query(`DELETE FROM catalog WHERE id = $1 RETURNING *;`, [id]); - res.status(200).send(keysToCamel(delUser)); + const inUse = await db.query(`SELECT * FROM published_schedule WHERE event_id = $1;`, [id]); + let hidden; + if (inUse && inUse.length) { + hidden = await db.query(`UPDATE catalog SET hidden = true WHERE id = $1 RETURNING *;`, [id]); + } else { + hidden = await db.query(`DELETE FROM catalog WHERE id = $1 RETURNING *;`, [id]); + } + res.status(200).send(keysToCamel(hidden)); } catch (err) { res.status(500).send(err.message); } diff --git a/server/schema/day.sql b/server/schema/day.sql index 1ac0b41..c29f5d2 100644 --- a/server/schema/day.sql +++ b/server/schema/day.sql @@ -1,9 +1,10 @@ DROP TABLE IF EXISTS day; CREATE TABLE IF NOT EXISTS day ( id serial PRIMARY KEY, - event_date DATE NOT NULL, + event_date DATE UNIQUE NOT NULL, start_time TIME NOT NULL, end_time TIME NOT NULL, location VARCHAR( 256 ) NOT NULL, - notes VARCHAR( 250 ) + notes VARCHAR( 250 ), + day_count INTEGER ); \ No newline at end of file diff --git a/server/schema/published_schedule.sql b/server/schema/published_schedule.sql index 90e6fe7..ee3feee 100644 --- a/server/schema/published_schedule.sql +++ b/server/schema/published_schedule.sql @@ -13,7 +13,7 @@ CREATE TABLE IF NOT EXISTS published_schedule ( FOREIGN KEY (event_id) REFERENCES catalog (id), FOREIGN KEY (day_id) - REFERENCES day (id) + REFERENCES day (id) ON DELETE CASCADE ); CREATE INDEX idx_day_id ON published_schedule (day_id); \ No newline at end of file From 57bfc916958aaf62e7deda5c231d06b177225e38 Mon Sep 17 00:00:00 2001 From: Ethan Ho Date: Tue, 12 Mar 2024 19:58:26 -0700 Subject: [PATCH 2/4] Bugfixes: Added comma --- routes/publishedSchedule.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/routes/publishedSchedule.js b/routes/publishedSchedule.js index a515f27..8502938 100644 --- a/routes/publishedSchedule.js +++ b/routes/publishedSchedule.js @@ -394,7 +394,7 @@ publishedScheduleRouter.put('/:id', async (req, res) => { start_time = COALESCE($5, start_time), end_time = COALESCE($6, end_time), cohort = COALESCE($7, cohort), - notes = COALESCE($8, notes) + notes = COALESCE($8, notes), created_on = COALESCE($9, created_on) WHERE id = $10 From 0d3708a4b0c3f157c00e02b6fbc0da4b0263b07e Mon Sep 17 00:00:00 2001 From: Ethan Ho Date: Wed, 13 Mar 2024 12:47:38 -0700 Subject: [PATCH 3/4] updated catalog schema --- server/schema/catalog.sql | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/server/schema/catalog.sql b/server/schema/catalog.sql index c3096a2..7302266 100644 --- a/server/schema/catalog.sql +++ b/server/schema/catalog.sql @@ -13,5 +13,6 @@ CREATE TABLE catalog ( description VARCHAR(50) NOT NULL, year year NOT NULL, season season, - location VARCHAR(256) + location VARCHAR(256), + hidden boolean NOT NULL ); From c693ca243534b0300eb2cab93c230c91d0dc0b48 Mon Sep 17 00:00:00 2001 From: Ethan Ho Date: Wed, 13 Mar 2024 18:25:30 -0700 Subject: [PATCH 4/4] fixed schema (for realsies this time) and updated some sql queries because db changed --- routes/catalog.js | 20 +++++++++----------- server/schema/catalog.sql | 5 ++--- 2 files changed, 11 insertions(+), 14 deletions(-) diff --git a/routes/catalog.js b/routes/catalog.js index fa067f8..e286012 100644 --- a/routes/catalog.js +++ b/routes/catalog.js @@ -96,13 +96,13 @@ catalogRouter.get('/:id', async (req, res) => { // -- POST - Adds a new row to the catalog table catalogRouter.post('/', async (req, res) => { - const { host, title, eventType, subject, description, year, season, location } = req.body; + const { host, title, eventType, subject, description, year, season } = req.body; try { const returnedData = await db.query( - `INSERT INTO catalog (id, host, title, event_type, subject, description, year, season, location, hidden) - VALUES (nextval('catalog_id_seq'), $1, $2, $3, $4, $5, $6, $7, $8, false) + `INSERT INTO catalog (id, host, title, event_type, subject, description, year, season, hidden) + VALUES (nextval('catalog_id_seq'), $1, $2, $3::event[], $4::subject[], $5, $6::year[], $7::season[], false) RETURNING id;`, - [host, title, eventType, subject, description, year, season, location], + [host, title, eventType, subject, description, year, season], ); res.status(201).json({ id: returnedData[0].id, status: 'Success' }); } catch (err) { @@ -118,18 +118,17 @@ catalogRouter.post('/', async (req, res) => { catalogRouter.put('/:id', async (req, res) => { try { const { id } = req.params; - const { host, title, eventType, subject, description, year, location, season } = req.body; + const { host, title, eventType, subject, description, year, season } = req.body; const updatedCatalog = await db.query( `UPDATE catalog SET ${host ? 'host = $(host), ' : ''} ${title ? 'title = $(title),' : ''} - ${eventType ? 'event_type = $(eventType), ' : ''} - ${subject ? 'subject = $(subject), ' : ''} + ${eventType ? 'event_type = $(eventType)::event[], ' : ''} + ${subject ? 'subject = $(subject)::subject[], ' : ''} ${description ? 'description = $(description), ' : ''} - ${year ? 'year = $(year), ' : ''} - ${location ? 'location = $(location), ' : ''} - ${season ? 'season = $(season), ' : ''} + ${year ? 'year = $(year)::year[], ' : ''} + ${season ? 'season = $(season)::season[], ' : ''} id = '${id}' WHERE id = '${id}' RETURNING *;`, @@ -141,7 +140,6 @@ catalogRouter.put('/:id', async (req, res) => { description, year, id, - location, season, }, ); diff --git a/server/schema/catalog.sql b/server/schema/catalog.sql index 7302266..e32df3e 100644 --- a/server/schema/catalog.sql +++ b/server/schema/catalog.sql @@ -8,11 +8,10 @@ CREATE TABLE catalog ( id SERIAL PRIMARY KEY, host VARCHAR(50) NOT NULL, title VARCHAR(50) NOT NULL, - event_type event NOT NULL, - subject subject NOT NULL, + event_type event[] NOT NULL, + subject subject[] NOT NULL, description VARCHAR(50) NOT NULL, year year NOT NULL, season season, - location VARCHAR(256), hidden boolean NOT NULL );