Christopher Johnson (WMDE) has submitted this change and it was merged.
Change subject: Created new standard directory structure
......................................................................
Created new standard directory structure
Change-Id: Ib1c5f6a02e0c23ea002654ac2fb39384ee2e3a70
---
M __phutil_library_map__.php
R src/__tests__/BurndownTestDataGenerator.php
R src/application/BurndownApplication.php
R src/constants/SprintConstants.php
R src/controller/BurndownController.php
R src/controller/BurndownDataViewController.php
R src/controller/BurndownListController.php
R src/controller/SprintReportController.php
R src/customfield/SprintEndDateField.php
R src/customfield/SprintProjectCustomField.php
R src/customfield/SprintStartDateField.php
R src/customfield/SprintTaskStoryPointsField.php
R src/events/BurndownActionMenuEventListener.php
R src/exception/BurndownException.php
R src/util/BurndownDataDate.php
R src/view/BurndownDataView.php
R src/view/SprintReportBurndownView.php
R src/view/SprintReportOpenTasksView.php
R src/view/SprintView.php
19 files changed, 40 insertions(+), 70 deletions(-)
Approvals:
Christopher Johnson (WMDE): Verified; Looks good to me, approved
diff --git a/__phutil_library_map__.php b/__phutil_library_map__.php
index fe151c1..148ffec 100644
--- a/__phutil_library_map__.php
+++ b/__phutil_library_map__.php
@@ -9,27 +9,24 @@
phutil_register_library_map(array(
'__library_version__' => 2,
'class' => array(
- 'BurndownActionMenuEventListener' =>
'src/BurndownActionMenuEventListener.php',
- 'BurndownApplication' => 'src/BurndownApplication.php',
- 'BurndownController' => 'src/BurndownController.php',
- 'BurndownDataDate' => 'src/BurndownDataDate.php',
- 'BurndownDataView' => 'src/BurndownDataView.php',
- 'BurndownDataViewController' => 'src/BurndownDataViewController.php',
- 'BurndownException' => 'src/BurndownException.php',
- 'BurndownListController' => 'src/BurndownListController.php',
- 'BurndownTestDataGenerator' => 'src/BurndownTestDataGenerator.php',
- 'SprintConstants' => 'src/SprintConstants.php',
- 'SprintEndDateField' => 'src/SprintEndDateField.php',
- 'SprintProjectCustomField' => 'src/SprintProjectCustomField.php',
- 'SprintReportBurndownView' => 'src/SprintReportBurndownView.php',
- 'SprintReportController' => 'src/SprintReportController.php',
- 'SprintStartDateField' => 'src/SprintStartDateField.php',
-
-
- 'SprintReportOpenTasksView' => 'src/SprintReportOpenTasksView.php',
- 'SprintStartDateField' => 'src/SprintStartDateField.php',
- 'SprintTaskStoryPointsField' => 'src/SprintTaskStoryPointsField.php',
- 'SprintView' => 'src/SprintView.php',
+ 'BurndownActionMenuEventListener' =>
'src/events/BurndownActionMenuEventListener.php',
+ 'BurndownApplication' => 'src/application/BurndownApplication.php',
+ 'BurndownController' => 'src/controller/BurndownController.php',
+ 'BurndownDataDate' => 'src/util/BurndownDataDate.php',
+ 'BurndownDataView' => 'src/view/BurndownDataView.php',
+ 'BurndownDataViewController' =>
'src/controller/BurndownDataViewController.php',
+ 'BurndownException' => 'src/exception/BurndownException.php',
+ 'BurndownListController' => 'src/controller/BurndownListController.php',
+ 'BurndownTestDataGenerator' =>
'src/__tests__/BurndownTestDataGenerator.php',
+ 'SprintConstants' => 'src/constants/SprintConstants.php',
+ 'SprintEndDateField' => 'src/customfield/SprintEndDateField.php',
+ 'SprintProjectCustomField' =>
'src/customfield/SprintProjectCustomField.php',
+ 'SprintReportBurndownView' => 'src/view/SprintReportBurndownView.php',
+ 'SprintReportController' => 'src/controller/SprintReportController.php',
+ 'SprintReportOpenTasksView' => 'src/view/SprintReportOpenTasksView.php',
+ 'SprintStartDateField' => 'src/customfield/SprintStartDateField.php',
+ 'SprintTaskStoryPointsField' =>
'src/customfield/SprintTaskStoryPointsField.php',
+ 'SprintView' => 'src/view/SprintView.php',
),
'function' => array(),
'xmap' => array(
@@ -43,15 +40,11 @@
'BurndownTestDataGenerator' => 'PhabricatorTestDataGenerator',
'SprintConstants' => 'ManiphestConstants',
'SprintStartDateField' => 'SprintProjectCustomField',
- 'SprintConstants' => 'ManiphestConstants',
- 'SprintStartDateField' => 'SprintProjectCustomField',
'SprintEndDateField' => 'SprintProjectCustomField',
'SprintProjectCustomField' => array(
'PhabricatorProjectCustomField',
'PhabricatorStandardCustomFieldInterface',
),
- 'SprintReportController' => 'BurndownController',
- 'SprintReportController' => 'ManiphestController',
'SprintReportBurndownView' => 'SprintView',
'SprintReportController' => 'BurndownController',
'SprintReportOpenTasksView' => 'SprintView',
diff --git a/src/BurndownTestDataGenerator.php
b/src/__tests__/BurndownTestDataGenerator.php
similarity index 100%
rename from src/BurndownTestDataGenerator.php
rename to src/__tests__/BurndownTestDataGenerator.php
diff --git a/src/BurndownApplication.php
b/src/application/BurndownApplication.php
similarity index 100%
rename from src/BurndownApplication.php
rename to src/application/BurndownApplication.php
diff --git a/src/SprintConstants.php b/src/constants/SprintConstants.php
similarity index 100%
rename from src/SprintConstants.php
rename to src/constants/SprintConstants.php
diff --git a/src/BurndownController.php b/src/controller/BurndownController.php
similarity index 76%
rename from src/BurndownController.php
rename to src/controller/BurndownController.php
index 55e2694..1893c0e 100644
--- a/src/BurndownController.php
+++ b/src/controller/BurndownController.php
@@ -6,7 +6,7 @@
abstract class BurndownController extends PhabricatorController {
- public function shouldAllowPublic() {
+ public function shouldAllowPublic() {
return true;
}
@@ -18,6 +18,20 @@
return $this->buildSideNavView(true)->getMenu();
}
+ public function buildNavMenu() {
+ $nav = new AphrontSideNavFilterView();
+ $nav->setBaseURI(new PhutilURI('/sprint/report/'));
+ $nav->addLabel(pht('Sprint Projects'));
+ $nav->addFilter('list', pht('List'));
+ $nav->addLabel(pht('Open Tasks'));
+ $nav->addFilter('project', pht('By Project'));
+ $nav->addFilter('user', pht('By User'));
+ $nav->addLabel(pht('Burndown'));
+ $nav->addFilter('burn', pht('Burndown Rate'));
+
+ return $nav;
+ }
+
public function buildSideNavView($for_app = false) {
$user = $this->getRequest()->getUser();
diff --git a/src/BurndownDataViewController.php
b/src/controller/BurndownDataViewController.php
similarity index 100%
rename from src/BurndownDataViewController.php
rename to src/controller/BurndownDataViewController.php
diff --git a/src/BurndownListController.php
b/src/controller/BurndownListController.php
similarity index 88%
rename from src/BurndownListController.php
rename to src/controller/BurndownListController.php
index 31a92d6..e112242 100644
--- a/src/BurndownListController.php
+++ b/src/controller/BurndownListController.php
@@ -15,7 +15,7 @@
$nav = $this->buildNavMenu();
$projects = $this->loadAllProjects($viewer);
-
+ $this->view = $nav->selectFilter($this->view, 'list');
$order = $request->getStr('order', 'name');
list($order, $reverse) = AphrontTableView::parseSort($order);
@@ -120,22 +120,6 @@
'title' => array(pht('Sprint List')),
'device' => true,
));
- }
-
- private function buildNavMenu() {
- $nav = new AphrontSideNavFilterView();
- $nav->setBaseURI(new PhutilURI('/sprint/report/'));
- $nav->addLabel(pht('Sprint Projects'));
- $nav->addFilter('list', pht('List'));
- $nav->addLabel(pht('Open Tasks'));
- $nav->addFilter('project', pht('By Project'));
- $nav->addFilter('user', pht('By User'));
- $nav->addLabel(pht('Burndown'));
- $nav->addFilter('burn', pht('Burndown Rate'));
-
- $this->view = $nav->selectFilter($this->view, 'list');
-
- return $nav;
}
// Load all projects with "§" in the name.
diff --git a/src/SprintReportController.php
b/src/controller/SprintReportController.php
similarity index 73%
rename from src/SprintReportController.php
rename to src/controller/SprintReportController.php
index fa01104..37e066f 100644
--- a/src/SprintReportController.php
+++ b/src/controller/SprintReportController.php
@@ -26,9 +26,7 @@
}
$nav = $this->buildNavMenu();
- $handle_factory = function(){ return id(new loadViewerHandles())
- ->execute(); };
-
+ $this->view = $nav->selectFilter($this->view, 'List');
require_celerity_resource('maniphest-report-css');
switch ($this->view) {
@@ -61,23 +59,5 @@
'device' => false,
));
}
-
- private function buildNavMenu() {
-
- $nav = new AphrontSideNavFilterView();
- $nav->setBaseURI(new PhutilURI('/sprint/report/'));
- $nav->addLabel(pht('Sprint Projects'));
- $nav->addFilter('list', pht('List'));
- $nav->addLabel(pht('Open Tasks'));
- $nav->addFilter('project', pht('By Project'));
- $nav->addFilter('user', pht('By User'));
- $nav->addLabel(pht('Burndown'));
- $nav->addFilter('burn', pht('Burndown Rate'));
-
- $this->view = $nav->selectFilter($this->view, 'List');
-
- return $nav;
- }
-
}
diff --git a/src/SprintEndDateField.php b/src/customfield/SprintEndDateField.php
similarity index 100%
rename from src/SprintEndDateField.php
rename to src/customfield/SprintEndDateField.php
diff --git a/src/SprintProjectCustomField.php
b/src/customfield/SprintProjectCustomField.php
similarity index 100%
rename from src/SprintProjectCustomField.php
rename to src/customfield/SprintProjectCustomField.php
diff --git a/src/SprintStartDateField.php
b/src/customfield/SprintStartDateField.php
similarity index 100%
rename from src/SprintStartDateField.php
rename to src/customfield/SprintStartDateField.php
diff --git a/src/SprintTaskStoryPointsField.php
b/src/customfield/SprintTaskStoryPointsField.php
similarity index 100%
rename from src/SprintTaskStoryPointsField.php
rename to src/customfield/SprintTaskStoryPointsField.php
diff --git a/src/BurndownActionMenuEventListener.php
b/src/events/BurndownActionMenuEventListener.php
similarity index 95%
rename from src/BurndownActionMenuEventListener.php
rename to src/events/BurndownActionMenuEventListener.php
index e197fa5..192c3fe 100644
--- a/src/BurndownActionMenuEventListener.php
+++ b/src/events/BurndownActionMenuEventListener.php
@@ -22,7 +22,7 @@
$object = $event->getValue('object');
$actions = null;
- if ($object instanceof PhabricatorProject &&
+ if ($object instanceof \PhabricatorProject &&
stripos($object->getName(), '§') !== false) {
$actions = $this->renderUserItems($event);
}
diff --git a/src/BurndownException.php b/src/exception/BurndownException.php
similarity index 100%
rename from src/BurndownException.php
rename to src/exception/BurndownException.php
diff --git a/src/BurndownDataDate.php b/src/util/BurndownDataDate.php
similarity index 100%
rename from src/BurndownDataDate.php
rename to src/util/BurndownDataDate.php
diff --git a/src/BurndownDataView.php b/src/view/BurndownDataView.php
similarity index 99%
rename from src/BurndownDataView.php
rename to src/view/BurndownDataView.php
index 35ba12b..c1014a1 100644
--- a/src/BurndownDataView.php
+++ b/src/view/BurndownDataView.php
@@ -648,7 +648,6 @@
private function getPointsData () {
- $handle = null;
$project_phid = $this->project->getPHID();
$table = new ManiphestTransaction();
@@ -705,7 +704,7 @@
if ($old_is_closed == $new_is_closed) {
// This was just a status change from one open status to another,
- // or from one closed status to another, so it's not an event we
+ // or from one closed status to another, so it's not an events we
// care about.
break;
}
@@ -770,7 +769,7 @@
break;
}
- // If we found some kind of event that we care about, stick it in the
+ // If we found some kind of events that we care about, stick it in the
// list of events.
if ($event_type !== null) {
$events[] = array(
diff --git a/src/SprintReportBurndownView.php
b/src/view/SprintReportBurndownView.php
similarity index 98%
rename from src/SprintReportBurndownView.php
rename to src/view/SprintReportBurndownView.php
index 2e638d0..5154c31 100644
--- a/src/SprintReportBurndownView.php
+++ b/src/view/SprintReportBurndownView.php
@@ -73,7 +73,7 @@
$data[$key]['_is_close'] = $is_close;
if (!$is_open && !$is_close) {
- // This is either some kind of bogus event, or a resolution change
+ // This is either some kind of bogus events, or a resolution change
// (e.g., resolved -> invalid). Just skip it.
continue;
}
diff --git a/src/SprintReportOpenTasksView.php
b/src/view/SprintReportOpenTasksView.php
similarity index 99%
rename from src/SprintReportOpenTasksView.php
rename to src/view/SprintReportOpenTasksView.php
index af93a68..2f6d4a1 100644
--- a/src/SprintReportOpenTasksView.php
+++ b/src/view/SprintReportOpenTasksView.php
@@ -312,7 +312,7 @@
*/
private function loadRecentlyClosedTasks() {
- list($ignored, $window_epoch) = $this->getWindow();
+ list(,, $window_epoch) = $this->getWindow();
$table = new ManiphestTask();
$xtable = new ManiphestTransaction();
diff --git a/src/SprintView.php b/src/view/SprintView.php
similarity index 100%
rename from src/SprintView.php
rename to src/view/SprintView.php
--
To view, visit https://gerrit.wikimedia.org/r/166718
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib1c5f6a02e0c23ea002654ac2fb39384ee2e3a70
Gerrit-PatchSet: 1
Gerrit-Project: phabricator/extensions/Sprint
Gerrit-Branch: master
Gerrit-Owner: Christopher Johnson (WMDE) <[email protected]>
Gerrit-Reviewer: Christopher Johnson (WMDE) <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
