judgej
commented
5 years ago I'm looking at the database structure and there are a number of issues that should be fixed to maintain data integrity:
- Use of big integers for all keys. This is a Laravel 5.8 standard out of the box, but TBH most of us should be using bit ints anyway.
- There is an index on
taggable_taggables.taggable_type. The cardinality of this table is normally going to be tiny (maybe just one or two), so an index will be less help than none at all. - A unique index on
taggable_taggablescoveringtag_id,taggable_idandtaggable_typewill ensure no model gets the same tag more than once. A non-unique index already covers the last two columns, so thetaggable_taggablescan be added to the end of that without affecting its current indexing use. - There is no referential integrity constraints between the
taggable_taggablesandtaggable_tagstables. This should be strictly enforced to help avoid unexpected problems when links become invalid. The relationship could be set to cascade, so when a tag is deleted, its use across all the models would go too.
Just allowing the migration file to be published would be a great first move here, then we can add and modify the indexes we like.
On a side note, the naming conventions for the primary key and foreign keys don't follow Laravel conventions, and this means workarounds and more time needed to understand what the database structure is.
Apart from that - it's the best tagging plugin by far :-)
mateusjunges
cviebrock
As far as I can tell, I can create multiple tags using the tag model with the same normalized name. It would be nice to disallow this at the db level (in the migration?) or at least on the model when it is saved.
Can be done? should be done?
Thanks you!