Page MenuHomePhabricator
Differential D19096

Make PhrictionContent "description" non-nullable
ClosedPublic
Actions

Authored by epriestley on Feb 15 2018, 1:50 PM.
Tags
None
Referenced Files
F14841936: D19096.diff
Sat, Feb 1, 7:40 PM
Unknown Object (File)
Tue, Jan 28, 10:58 PM
Unknown Object (File)
Tue, Jan 28, 5:24 AM
Unknown Object (File)
Sat, Jan 25, 3:21 PM
Unknown Object (File)
Sat, Jan 25, 3:21 PM
Unknown Object (File)
Sat, Jan 25, 3:21 PM
Unknown Object (File)
Thu, Jan 23, 9:11 PM
Unknown Object (File)
Tue, Jan 21, 11:07 AM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T6203: Adjust nullability of various questionable columns
T13077: Plans: Phriction modern "v3" APIs, Drafts
Commits
rPa965d8d6ae5a: Make PhrictionContent "description" non-nullable
Summary

Depends on D19095. Ref T6203. Ref T13077. This column is nullable in an inconsistent way. Make it non-nullable.

Also clean up one more content query on the history view.

Test Plan

Ran migration, then created and edited documents without providing a descriptino or hitting NULL exceptions.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 15 2018, 1:50 PM
Harbormaster completed remote builds in B19566: Diff 45762.Feb 15 2018, 1:51 PM
epriestley requested review of this revision.Feb 15 2018, 1:51 PM
epriestley added a child revision: D19097: Introduce a "phriction.content.search" API method to replace "phriction.history".Feb 15 2018, 2:29 PM
This revision was not accepted when it landed; it landed in state Needs Review.Feb 16 2018, 1:55 AM
Closed by commit rPa965d8d6ae5a: Make PhrictionContent "description" non-nullable (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
sql/
autopatches/
2 lines
2 lines
src/
applications/
phriction/
conduit/
2 lines
2 lines
controller/
17 lines
editor/
5 lines
storage/
5 lines

Diff 45776

View Options

resources/sql/autopatches/20180215.phriction.03.descempty.sql

Loading...
View Options

resources/sql/autopatches/20180215.phriction.04.descnull.sql

Loading...
View Options

src/applications/phriction/conduit/PhrictionCreateConduitAPIMethod.php

Loading...
View Options

src/applications/phriction/conduit/PhrictionEditConduitAPIMethod.php

Loading...
View Options

src/applications/phriction/controller/PhrictionHistoryController.php

Loading...
View Options

src/applications/phriction/editor/PhrictionTransactionEditor.php

Loading...
View Options

src/applications/phriction/storage/PhrictionContent.php

Loading...
Phabricator · Built by Phacility