Success

User tests: Successful: Unsuccessful:

avatar mbabker
mbabker
3 Aug 2013

This is a first cut at cleaning up the database schema some. Overall changes:

  • Remove unused #__languages and #__session tables
  • Adding FKs as appropriate
  • Reordering the table order in the installer to work with the FKs

The last unused table is the #__categories table which was being used for field data in the item submit/edit form in the Platform version of the app. Left to decide is whether we still use this data or if we just drop it in favor of the pure Markdown interface in place now.

Image below shows the altered schema (as compared to the one I posted to G+ the other day)

jtracker_cleanup

avatar mbabker mbabker - open - 3 Aug 2013
avatar mbabker mbabker - open - 3 Aug 2013
avatar elkuku
elkuku - comment - 3 Aug 2013

Umm, I somewhat missed your post on G+...
Yes I think we should scratch the custom fields idea for now and go with "markdown". The mayor goal with this approach was reusing com_categories, which is gone now (RIP).
Along with the #__categories table, this would also make #__tracker_fields_values. obsolete

Another thing that might need review is the "issue relations" (#76) thing, which is currently not enabled but the code is still present (most of it I think).
This would affect #__issues_relations_types and the fields rel_id and rel_type in #__issues

avatar mbabker
mbabker - comment - 5 Aug 2013

Alright, both of those tables are gone now, which leaves the relations code/schema.

Looking at the relations stuff, it looks like the only reference remaining to that code is in IssueModel where the table is joined to but the data is unused. So we could either (re-)implement that stuff (I'm in favor of that personally) or just drop that little bit out and revisit the subject down the road.

avatar elkuku
elkuku - comment - 5 Aug 2013

Fine with me, let me see if I find some time to re-implement the relations thingy.

avatar - close - 5 Aug 2013
avatar elkuku elkuku - reference | b9e015b - 5 Aug 13
avatar elkuku elkuku - merge - 5 Aug 2013
avatar elkuku elkuku - close - 5 Aug 2013
avatar mbabker mbabker - head_ref_deleted - 5 Aug 2013

Add a Comment

Login with GitHub to post a comment