Skip to content

Commit 6bfe8b5

Browse files
author
epriestley
committedSep 19, 2014
Generate expected schemata for Calendar
Summary: Ref T1191. - There was a varchar(50) column. I changed it to `text64`, since this length is unusual. - There was an int(3) column. I changed it to `int32`, since this length is unusual. Test Plan: Ran migrations, saw warnings disappear from config tool. Reviewers: btrahan Reviewed By: btrahan Subscribers: epriestley Maniphest Tasks: T1191 Differential Revision: https://secure.phabricator.com/D10524
1 parent a42e4a8 commit 6bfe8b5

7 files changed

+34
-0
lines changed
 
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
ALTER TABLE {$NAMESPACE}_calendar.calendar_event
2+
CHANGE status status INT UNSIGNED NOT NULL;
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
ALTER TABLE {$NAMESPACE}_calendar.calendar_holiday
2+
CHANGE name name VARCHAR(64) NOT NULL COLLATE utf8_general_ci;

‎src/__phutil_library_map__.php

+2
Original file line numberDiff line numberDiff line change
@@ -1300,6 +1300,7 @@
13001300
'PhabricatorCalendarEventViewController' => 'applications/calendar/controller/PhabricatorCalendarEventViewController.php',
13011301
'PhabricatorCalendarHoliday' => 'applications/calendar/storage/PhabricatorCalendarHoliday.php',
13021302
'PhabricatorCalendarHolidayTestCase' => 'applications/calendar/storage/__tests__/PhabricatorCalendarHolidayTestCase.php',
1303+
'PhabricatorCalendarSchemaSpec' => 'applications/calendar/storage/PhabricatorCalendarSchemaSpec.php',
13031304
'PhabricatorCalendarViewController' => 'applications/calendar/controller/PhabricatorCalendarViewController.php',
13041305
'PhabricatorCampfireProtocolAdapter' => 'infrastructure/daemon/bot/adapter/PhabricatorCampfireProtocolAdapter.php',
13051306
'PhabricatorChangeParserTestCase' => 'applications/repository/worker/__tests__/PhabricatorChangeParserTestCase.php',
@@ -4208,6 +4209,7 @@
42084209
'PhabricatorCalendarEventViewController' => 'PhabricatorCalendarController',
42094210
'PhabricatorCalendarHoliday' => 'PhabricatorCalendarDAO',
42104211
'PhabricatorCalendarHolidayTestCase' => 'PhabricatorTestCase',
4212+
'PhabricatorCalendarSchemaSpec' => 'PhabricatorConfigSchemaSpec',
42114213
'PhabricatorCalendarViewController' => 'PhabricatorCalendarController',
42124214
'PhabricatorCampfireProtocolAdapter' => 'PhabricatorBotBaseStreamingProtocolAdapter',
42134215
'PhabricatorChangeParserTestCase' => 'PhabricatorWorkingCopyTestCase',

‎src/applications/calendar/storage/PhabricatorCalendarEvent.php

+6
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,12 @@ public function getHumanStatus() {
3737
public function getConfiguration() {
3838
return array(
3939
self::CONFIG_AUX_PHID => true,
40+
self::CONFIG_COLUMN_SCHEMA => array(
41+
'dateFrom' => 'epoch',
42+
'dateTo' => 'epoch',
43+
'status' => 'uint32',
44+
'description' => 'text',
45+
),
4046
) + parent::getConfiguration();
4147
}
4248

‎src/applications/calendar/storage/PhabricatorCalendarHoliday.php

+9
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,15 @@ final class PhabricatorCalendarHoliday extends PhabricatorCalendarDAO {
88
public function getConfiguration() {
99
return array(
1010
self::CONFIG_TIMESTAMPS => false,
11+
self::CONFIG_COLUMN_SCHEMA => array(
12+
'day' => 'date',
13+
'name' => 'text64',
14+
),
15+
self::CONFIG_KEY_SCHEMA => array(
16+
'day' => array(
17+
'columns' => array('day'),
18+
),
19+
),
1120
) + parent::getConfiguration();
1221
}
1322

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
<?php
2+
3+
final class PhabricatorCalendarSchemaSpec
4+
extends PhabricatorConfigSchemaSpec {
5+
6+
public function buildSchemata() {
7+
$this->buildLiskSchemata('PhabricatorCalendarDAO');
8+
}
9+
10+
}

‎src/applications/config/schema/PhabricatorConfigSchemaSpec.php

+3
Original file line numberDiff line numberDiff line change
@@ -292,6 +292,9 @@ private function getDetailsForDataType($data_type) {
292292
case 'double':
293293
$column_type = 'double';
294294
break;
295+
case 'date':
296+
$column_type = 'date';
297+
break;
295298
default:
296299
$column_type = pht('<unknown>');
297300
$charset = pht('<unknown>');

0 commit comments

Comments
 (0)
Failed to load comments.