See motivation in the parent issue, but in short we want to be able to query the group content data table easier. Denormalizing the group content type's data onto group content will help us with that.

CommentFileSizeAuthor
#3 group-3254311-3.patch16.39 KBkristiaanvandeneynde
#2 group-3254311-2.patch15.97 KBkristiaanvandeneynde

Comments

kristiaanvandeneynde created an issue. See original summary.

kristiaanvandeneynde’s picture

kristiaanvandeneynde
Primary language Dutch
Location Antwerp, Belgium
commented
Status: Active » Needs review
StatusFileSize
new group-3254311-2.patch15.97 KB

This adjusts tests, improves performance slightly and obviously does what the issue is all about. Haven't tested the update hook yet nor reviewed the test cases to see if we need to expand any.

kristiaanvandeneynde’s picture

kristiaanvandeneynde
Primary language Dutch
Location Antwerp, Belgium
commented
StatusFileSize
new group-3254311-3.patch16.39 KB
  • I changed the plugin_id column to 64 characters as 255 was overkill
  • Update ran great on my end
  • Could not find any tests that are affected
kristiaanvandeneynde’s picture

kristiaanvandeneynde
Primary language Dutch
Location Antwerp, Belgium
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.