← Back to team overview

mahara-contributors team mailing list archive

[Bug 1774103] A change has been merged

 

Reviewed:  https://reviews.mahara.org/8907
Committed: https://git.mahara.org/mahara/mahara/commit/ffa516591dfd22d8b7265d1f239cfa05023011e6
Submitter: Robert Lyon (robertl@xxxxxxxxxxxxxxx)
Branch:    master

commit ffa516591dfd22d8b7265d1f239cfa05023011e6
Author: Robert Lyon <robertl@xxxxxxxxxxxxxxx>
Date:   Tue May 29 15:21:59 2018 +1200

Bug 1774103: New structure for the tags table

Looking at consolidating all the differnet *_tag tables into one table

Also fixing up older upgrade where we add a column and then delete it
again. It slows down the upgrade for no gain.

behatnotneeded

Change-Id: I715a076dab74bca2897c56b171cc0536d6c65ff4
Signed-off-by: Robert Lyon <robertl@xxxxxxxxxxxxxxx>

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1774103

Title:
  Combine all the *_tag tables into one table

Status in Mahara:
  Fix Committed

Bug description:
  Currently we have

  - artefact_tag
  - collection_tag
  - view_tag
  - usr_tag

  tables and if we want to have new tagging, say for groups we would
  need to add another table

  It might be easier to combine all these tables so that we can do
  easier queries, like what are all the tags for an owner. Rather than
  having to do a bunch of table joins and checking what things are owned
  by what view/artefact etc we can just query the one table.

  I'm thinking of having one table with the fields

   id, tag, resourcetype, resourceid, ownertype, ownerid, editedby,
  ctime, mtime

  where resourcetype is either 'view', 'artefact' etc
  and resourceid is the id of that type

  Where ownertype can be 'user', 'group', institution'
  and ownerid can be the id number or shortname

  And editedby can be the ID of the current user interacting with the
  tag - useful for group/institution tags

  ctime = created time
  mtime = modified time

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1774103/+subscriptions


References