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
F18626069: D19096.diff
Tue, Sep 16, 12:16 AM
F18611960: D19096.id45776.diff
Sun, Sep 14, 8:12 AM
F18583696: D19096.diff
Thu, Sep 11, 10:49 AM
F18582465: D19096.id.diff
Thu, Sep 11, 8:29 AM
F18581539: D19096.id45762.diff
Thu, Sep 11, 6:23 AM
F18088256: D19096.diff
Aug 6 2025, 7:25 AM
F18002455: D19096.id.diff
Aug 2 2025, 10:08 AM
F17961061: D19096.diff
Aug 1 2025, 9:22 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