Page MenuHomePhabricator

Different data types for *_content_* in tables.sql and patches
Closed, ResolvedPublic

Description

In maintenance/tables.sql, *_content_* are defined as "int unsigned", in maintenance/archives/*_content_*.sql as "varbinary(32)" or "varbinary(64)".


Version: 1.21.x
Severity: blocker

Details

Reference
bz40960

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 1:13 AM
bzimport set Reference to bz40960.
bzimport added a subscriber: Unknown Object (MLST).

sumanah wrote:

I see this is marked as a blocker so I'm bringing it to Tim's, RobLa's, and Asher's attention.

sumanah wrote:

I believe this blocks deployment so I'm adding it to that tracking bug.

Fixed by I44a92d18, pending review

To add some possible search terms: This seems to trigger failures on fresh installs in RevisionStorageTest.php (testGetContentModel, testGetContentFormat) and WikiPageTest.php (testGetContentModel, testGetContentHandler).

(In reply to comment #4)

Fixed by I44a92d18, pending review

Merged.