Clean up parentDomain issues in PhameBlog
Summary: Ref T9360. These weren't getting set properly, also make them nullable since they're optional. Test Plan: run upgrade, make a new blog with and without a parent domain. Edit a current blog. Reviewers: epriestley Reviewed By: epriestley Subscribers: Korvin Maniphest Tasks: T9360 Differential Revision: https://secure.phabricator.com/D16242
This commit is contained in:
@@ -0,0 +1,2 @@
|
|||||||
|
ALTER TABLE {$NAMESPACE}_phame.phame_blog
|
||||||
|
MODIFY parentDomain VARCHAR(128) NULL COLLATE {$COLLATE_TEXT};
|
||||||
@@ -0,0 +1,2 @@
|
|||||||
|
ALTER TABLE {$NAMESPACE}_phame.phame_blog
|
||||||
|
MODIFY parentSite VARCHAR(128) NULL COLLATE {$COLLATE_TEXT};
|
||||||
@@ -66,6 +66,7 @@ final class PhameBlogEditor
|
|||||||
case PhameBlogTransaction::TYPE_DESCRIPTION:
|
case PhameBlogTransaction::TYPE_DESCRIPTION:
|
||||||
case PhameBlogTransaction::TYPE_STATUS:
|
case PhameBlogTransaction::TYPE_STATUS:
|
||||||
case PhameBlogTransaction::TYPE_PARENTSITE:
|
case PhameBlogTransaction::TYPE_PARENTSITE:
|
||||||
|
case PhameBlogTransaction::TYPE_PARENTDOMAIN:
|
||||||
case PhameBlogTransaction::TYPE_PROFILEIMAGE:
|
case PhameBlogTransaction::TYPE_PROFILEIMAGE:
|
||||||
case PhameBlogTransaction::TYPE_HEADERIMAGE:
|
case PhameBlogTransaction::TYPE_HEADERIMAGE:
|
||||||
return $xaction->getNewValue();
|
return $xaction->getNewValue();
|
||||||
|
|||||||
@@ -48,8 +48,8 @@ final class PhameBlog extends PhameDAO
|
|||||||
'description' => 'text',
|
'description' => 'text',
|
||||||
'domain' => 'text128?',
|
'domain' => 'text128?',
|
||||||
'domainFullURI' => 'text128?',
|
'domainFullURI' => 'text128?',
|
||||||
'parentSite' => 'text128',
|
'parentSite' => 'text128?',
|
||||||
'parentDomain' => 'text128',
|
'parentDomain' => 'text128?',
|
||||||
'status' => 'text32',
|
'status' => 'text32',
|
||||||
'mailKey' => 'bytes20',
|
'mailKey' => 'bytes20',
|
||||||
'profileImagePHID' => 'phid?',
|
'profileImagePHID' => 'phid?',
|
||||||
|
|||||||
Reference in New Issue
Block a user