Page MenuHomePhabricator
Differential D21748

Make Phame blog policies non-nullable
ClosedPublic
Actions

Authored by epriestley on Apr 1 2022, 6:42 PM.
Tags
None
Referenced Files
F14832209: D21748.id51855.diff
Thu, Jan 30, 4:30 AM
Unknown Object (File)
Tue, Jan 28, 5:52 AM
Unknown Object (File)
Sun, Jan 26, 2:11 AM
Unknown Object (File)
Sun, Jan 26, 2:11 AM
Unknown Object (File)
Sun, Jan 26, 2:11 AM
Unknown Object (File)
Tue, Jan 21, 9:23 AM
Unknown Object (File)
Sat, Jan 11, 9:18 AM
Unknown Object (File)
Wed, Jan 1, 7:03 PM
View All Files
Subscribers
Restricted Owners Package

Details

Reviewers
None
Maniphest Tasks
T6203: Adjust nullability of various questionable columns
T13661: Give Phame configurable interact policies
Commits
rP28b12fb3cdaf: Make Phame blog policies non-nullable
Summary

Ref T6203. Ref T13661. These policies are incorrectly nullable, although it's likely that no pathway exists in the application to write NULL into them. Fix the schema.

Test Plan

Ran bin/storage upgrade.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Apr 1 2022, 6:42 PM
Owners added a subscriber: Restricted Owners Package.Apr 1 2022, 6:42 PM
epriestley requested review of this revision.Apr 1 2022, 6:42 PM
Harbormaster completed remote builds in B25631: Diff 51846.Apr 1 2022, 6:43 PM
This revision was not accepted when it landed; it landed in state Needs Review.Apr 1 2022, 7:53 PM
Closed by commit rP28b12fb3cdaf: Make Phame blog policies non-nullable (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rP28b12fb3cdaf: Make Phame blog policies non-nullable.

Revision Contents
Changeset List

PathSize
resources/
sql/
autopatches/
5 lines
5 lines
src/
applications/
phame/
storage/
6 lines

Diff 51855

View Options

resources/sql/autopatches/20220401.phameinteract.01.sql

Loading...
View Options

resources/sql/autopatches/20220401.phameinteract.02.sql

Loading...
View Options

src/applications/phame/storage/PhameBlog.php

Loading...
Phabricator · Built by Phacility