search

quickapps / cms

Modular CMS powered by CakePHP
GNU General Public License v3.0
164 stars 69 forks source link

content_revisions 'data' column needs to be LONGTEXT #169

Closed paulrwest closed 7 years ago

paulrwest commented 7 years ago

If a content type has more than 25 fields an afterSave error is thrown on content edit due to the revision data being incorrect format and the object not getting created correctly

To resolve this I changed the field type of content_revisions data from TEXT to TEXTLONG

I understand there is a small performance hit increasing the column type and 25+ fields is an unusual scenario but it is a scenario we have hit.

botchris commented 7 years ago

Sounds reasonable, I'll take a look

botchris commented 7 years ago

Solved, data is now of "blob" type.

https://github.com/quickapps/cms/blob/2.0/plugins/Installer/config/fixture/ContentRevisionsFixture.php#L80