From 82c2b91090e792fc5c39ae9fbfc28b8640bec27b Mon Sep 17 00:00:00 2001
From: Gregory Ballantine <gregory.w.ballantine@nasa.gov>
Date: Tue, 21 May 2024 09:35:39 -0400
Subject: [PATCH] Removed activities table; this will be handled after users
 are implemented

---
 migrations/0003_add_activities_table.js | 30 -------------------------
 src/models/activity.js                  | 26 ---------------------
 src/models/index.js                     |  8 -------
 src/routes/item.js                      | 10 ---------
 views/item/view.twig                    | 15 -------------
 5 files changed, 89 deletions(-)
 delete mode 100644 migrations/0003_add_activities_table.js
 delete mode 100644 src/models/activity.js

diff --git a/migrations/0003_add_activities_table.js b/migrations/0003_add_activities_table.js
deleted file mode 100644
index e85c9f9..0000000
--- a/migrations/0003_add_activities_table.js
+++ /dev/null
@@ -1,30 +0,0 @@
-module.exports = {
-  up: (queryInterface, Sequelize) => {
-    return queryInterface.createTable('activities', {
-      id: {
-        allowNull: false,
-        autoIncrement: true,
-        primaryKey: true,
-        type: Sequelize.INTEGER
-      },
-      user: {
-        type: Sequelize.STRING,
-        allowNull: false,
-      },
-      action: {
-        type: Sequelize.STRING,
-        allowNull: false,
-      },
-      itemId: {
-        type: Sequelize.NUMBER,
-        allowNull: false,
-      },
-      createdAt: Sequelize.DataTypes.DATE,
-      updatedAt: Sequelize.DataTypes.DATE,
-    });
-  },
-
-  down: (queryInterface, Sequelize) => {
-    return queryInterface.dropTable('activities');
-  }
-};
diff --git a/src/models/activity.js b/src/models/activity.js
deleted file mode 100644
index 37a8b75..0000000
--- a/src/models/activity.js
+++ /dev/null
@@ -1,26 +0,0 @@
-module.exports = (sequelize, Sequelize) => {
-  const Activity = sequelize.define('activity', {
-
-    user: {
-      type: Sequelize.STRING,
-      allowNull: false,
-    },
-
-    action: {
-      type: Sequelize.STRING,
-      allowNull: false,
-    },
-
-    itemId: {
-      type: Sequelize.NUMBER,
-      allowNull: false,
-    },
-
-  });
-
-  Activity.render = function() {
-    return this.user + ' has ' + this.action + ' this item at ' + this.createdAt + '.';
-  };
-
-  return Activity;
-};
diff --git a/src/models/index.js b/src/models/index.js
index d6da015..c5736ea 100644
--- a/src/models/index.js
+++ b/src/models/index.js
@@ -10,14 +10,6 @@ db.sequelize = sequelize;
 
 db.items = require('./item.js')(sequelize, Sequelize);
 db.licenses = require('./license.js')(sequelize, Sequelize);
-db.activities = require('./activity.js')(sequelize, Sequelize);
-
-// Model associations
-db.items.hasMany(db.activities, {
-  onDelete: 'CASCADE',
-  onUpdate: 'CASCADE',
-});
-db.activities.belongsTo(db.items);
 
 module.exports = db;
 
diff --git a/src/routes/item.js b/src/routes/item.js
index 4dba76c..e27960f 100644
--- a/src/routes/item.js
+++ b/src/routes/item.js
@@ -1,6 +1,5 @@
 const db = require('../models');
 const Item = db.items;
-const Activity = db.activities;
 
 // GET - /item/add
 exports.getAdd = async function(req, res) {
@@ -21,12 +20,6 @@ exports.postAdd = async function(req, res) {
 
   console.log(`Saved item ${item.name} to the database.`);
 
-  const activity = await Activity.create({
-    user: 'Admin',
-    action: 'create',
-    itemId: item.id,
-  });
-
   res.redirect('/');
 };
 
@@ -36,11 +29,8 @@ exports.getItem = async function(req, res) {
     where: {
       id: req.params.id,
     },
-    include: Activity,
   });
 
-  console.log(item[0].activities[0].render());
-
   res.render('item/view.twig', {
     item: item[0],
   });
diff --git a/views/item/view.twig b/views/item/view.twig
index 043c701..f32c269 100644
--- a/views/item/view.twig
+++ b/views/item/view.twig
@@ -44,19 +44,4 @@
   </table>
 </section>
 
-<hr>
-
-<section id="item-activity" class="row">
-  <h2>Recorded activity</h2>
-  {% if item.activities | length > 0 %}
-    <ul class="twelve columns">
-      {% for a in item.activities %}
-        <li><p>{{ a.render() }}</p></li>
-      {% endfor %}
-    </ul>
-  {% else %}
-    <p>There hasn't been any activity recorded for this item.</p>
-  {% endif %}
-</section>
-
 {% endblock %}