Permalink
Browse files

Move setUser() to AphrontView

Summary: This is used in every other view.

Test Plan: Browsed around.

Reviewers: epriestley

Reviewed By: epriestley

CC: aran, Korvin

Differential Revision: https://secure.phabricator.com/D4248
  • Loading branch information...
1 parent 45b66be commit ef214e94cebd27e2a775fbf60a4c2b1c3e2524af vrana committed Dec 20, 2012
Showing with 11 additions and 352 deletions.
  1. +0 −6 src/applications/audit/view/PhabricatorAuditCommitListView.php
  2. +0 −6 src/applications/audit/view/PhabricatorAuditListView.php
  3. +0 −6 src/applications/calendar/view/AphrontCalendarMonthView.php
  4. +0 −6 src/applications/daemon/view/PhabricatorDaemonLogEventsView.php
  5. +0 −6 src/applications/daemon/view/PhabricatorDaemonLogListView.php
  6. +0 −6 src/applications/differential/view/DifferentialAddCommentView.php
  7. +1 −7 src/applications/differential/view/DifferentialChangesetListView.php
  8. +0 −6 src/applications/differential/view/DifferentialDiffTableOfContentsView.php
  9. +0 −6 src/applications/differential/view/DifferentialInlineCommentEditView.php
  10. +0 −6 src/applications/differential/view/DifferentialLocalCommitsView.php
  11. +0 −6 src/applications/differential/view/DifferentialRevisionCommentListView.php
  12. +0 −6 src/applications/differential/view/DifferentialRevisionCommentView.php
  13. +0 −9 src/applications/differential/view/DifferentialRevisionDetailView.php
  14. +0 −6 src/applications/differential/view/DifferentialRevisionListView.php
  15. +0 −6 src/applications/differential/view/DifferentialRevisionStatsView.php
  16. +0 −10 src/applications/differential/view/DifferentialRevisionUpdateHistoryView.php
  17. +0 −6 src/applications/diffusion/view/DiffusionBranchTableView.php
  18. +0 −6 src/applications/diffusion/view/DiffusionBrowseTableView.php
  19. +0 −6 src/applications/diffusion/view/DiffusionCommentListView.php
  20. +0 −6 src/applications/diffusion/view/DiffusionCommentView.php
  21. +0 −6 src/applications/diffusion/view/DiffusionTagListView.php
  22. +0 −6 src/applications/flag/view/PhabricatorFlagListView.php
  23. +0 −6 src/applications/herald/view/HeraldRuleEditHistoryView.php
  24. +0 −6 src/applications/herald/view/HeraldRuleListView.php
  25. +0 −6 src/applications/maniphest/view/ManiphestTaskListView.php
  26. +0 −6 src/applications/maniphest/view/ManiphestTaskSummaryView.php
  27. +0 −6 src/applications/maniphest/view/ManiphestTransactionDetailView.php
  28. +0 −6 src/applications/maniphest/view/ManiphestTransactionListView.php
  29. +0 −6 src/applications/meta/view/PhabricatorApplicationLaunchView.php
  30. +0 −7 src/applications/metamta/contentsource/PhabricatorContentSourceView.php
  31. +0 −9 src/applications/phortune/control/PhortuneMonthYearExpiryControl.php
  32. +0 −9 src/applications/phortune/stripe/view/PhortuneStripePaymentFormView.php
  33. +0 −6 src/applications/ponder/view/PonderAddAnswerView.php
  34. +0 −6 src/applications/ponder/view/PonderAddCommentView.php
  35. +0 −6 src/applications/ponder/view/PonderAnswerListView.php
  36. +0 −6 src/applications/ponder/view/PonderCommentListView.php
  37. +0 −6 src/applications/ponder/view/PonderPostBodyView.php
  38. +0 −6 src/applications/ponder/view/PonderQuestionDetailView.php
  39. +0 −6 src/applications/ponder/view/PonderQuestionSummaryView.php
  40. +0 −6 src/applications/ponder/view/PonderUserProfileView.php
  41. +0 −6 src/applications/xhprof/view/PhabricatorXHProfSampleListView.php
  42. +0 −6 src/view/AphrontDialogView.php
  43. +10 −0 src/view/AphrontView.php
  44. +0 −6 src/view/form/AphrontFormView.php
  45. +0 −6 src/view/form/control/AphrontFormDateControl.php
  46. +0 −10 src/view/form/control/AphrontFormPolicyControl.php
  47. +0 −6 src/view/form/control/AphrontFormTokenizerControl.php
  48. +0 −11 src/view/form/control/PhabricatorRemarkupControl.php
  49. +0 −6 src/view/layout/AphrontSideNavFilterView.php
  50. +0 −6 src/view/layout/PhabricatorActionListView.php
  51. +0 −6 src/view/layout/PhabricatorActionView.php
  52. +0 −6 src/view/layout/PhabricatorTransactionView.php
  53. +0 −6 src/view/layout/headsup/AphrontHeadsupActionView.php
  54. +0 −6 src/view/page/menu/PhabricatorMainMenuSearchView.php
  55. +0 −10 src/view/page/menu/PhabricatorMainMenuView.php
@@ -2,16 +2,10 @@
final class PhabricatorAuditCommitListView extends AphrontView {
- private $user;
private $commits;
private $handles;
private $noDataString;
- public function setUser(PhabricatorUser $user) {
- $this->user = $user;
- return $this;
- }
-
public function setNoDataString($no_data_string) {
$this->noDataString = $no_data_string;
return $this;
@@ -7,7 +7,6 @@
private $authorityPHIDs = array();
private $noDataString;
private $commits;
- private $user;
private $showDescriptions = true;
private $highlightedAudits;
@@ -44,11 +43,6 @@ public function setCommits(array $commits) {
return $this;
}
- public function setUser(PhabricatorUser $user) {
- $this->user = $user;
- return $this;
- }
-
public function setShowDescriptions($show_descriptions) {
$this->showDescriptions = $show_descriptions;
return $this;
@@ -2,7 +2,6 @@
final class AphrontCalendarMonthView extends AphrontView {
- private $user;
private $month;
private $year;
private $holidays = array();
@@ -17,11 +16,6 @@ private function getBrowseURI() {
return $this->browseURI;
}
- public function setUser(PhabricatorUser $user) {
- $this->user = $user;
- return $this;
- }
-
public function addEvent(AphrontCalendarEventView $event) {
$this->events[] = $event;
return $this;
@@ -4,7 +4,6 @@
private $events;
private $combinedLog;
- private $user;
public function setEvents(array $events) {
assert_instances_of($events, 'PhabricatorDaemonLogEvent');
@@ -17,11 +16,6 @@ public function setCombinedLog($is_combined) {
return $this;
}
- public function setUser(PhabricatorUser $user) {
- $this->user = $user;
- return $this;
- }
-
public function render() {
$rows = array();
@@ -3,19 +3,13 @@
final class PhabricatorDaemonLogListView extends AphrontView {
private $daemonLogs;
- private $user;
public function setDaemonLogs(array $daemon_logs) {
assert_instances_of($daemon_logs, 'PhabricatorDaemonLog');
$this->daemonLogs = $daemon_logs;
return $this;
}
- public function setUser(PhabricatorUser $user) {
- $this->user = $user;
- return $this;
- }
-
public function render() {
$rows = array();
@@ -5,7 +5,6 @@
private $revision;
private $actions;
private $actionURI;
- private $user;
private $draft;
private $auxFields;
private $reviewers = array();
@@ -32,11 +31,6 @@ public function setActionURI($uri) {
return $this;
}
- public function setUser(PhabricatorUser $user) {
- $this->user = $user;
- return $this;
- }
-
public function setDraft(PhabricatorDraft $draft = null) {
$this->draft = $draft;
return $this;
@@ -13,15 +13,14 @@
private $leftRawFileURI;
private $rightRawFileURI;
- private $user;
private $symbolIndexes = array();
private $repository;
private $branch;
private $diff;
private $vsMap = array();
private $title;
-
+
public function setTitle($title) {
$this->title = $title;
return $this;
@@ -53,11 +52,6 @@ public function setInlineCommentControllerURI($uri) {
return $this;
}
- public function setUser(PhabricatorUser $user) {
- $this->user = $user;
- return $this;
- }
-
public function setRepository(PhabricatorRepository $repository) {
$this->repository = $repository;
return $this;
@@ -7,7 +7,6 @@
private $references = array();
private $repository;
private $diff;
- private $user;
private $renderURI = '/differential/changeset/';
private $revisionID;
private $whitespace;
@@ -43,11 +42,6 @@ public function setUnitTestData($unit_test_data) {
return $this;
}
- public function setUser(PhabricatorUser $user) {
- $this->user = $user;
- return $this;
- }
-
public function setRevisionID($revision_id) {
$this->revisionID = $revision_id;
return $this;
@@ -2,7 +2,6 @@
final class DifferentialInlineCommentEditView extends AphrontView {
- private $user;
private $inputs = array();
private $uri;
private $title;
@@ -15,11 +14,6 @@ public function addHiddenInput($key, $value) {
return $this;
}
- public function setUser(PhabricatorUser $user) {
- $this->user = $user;
- return $this;
- }
-
public function setSubmitURI($uri) {
$this->uri = $uri;
return $this;
@@ -3,18 +3,12 @@
final class DifferentialLocalCommitsView extends AphrontView {
private $localCommits;
- private $user;
public function setLocalCommits($local_commits) {
$this->localCommits = $local_commits;
return $this;
}
- public function setUser(PhabricatorUser $user) {
- $this->user = $user;
- return $this;
- }
-
public function render() {
$user = $this->user;
if (!$user) {
@@ -6,7 +6,6 @@
private $handles;
private $inlines;
private $changesets;
- private $user;
private $target;
private $versusDiffID;
private $id;
@@ -35,11 +34,6 @@ public function setChangesets(array $changesets) {
return $this;
}
- public function setUser(PhabricatorUser $user) {
- $this->user = $user;
- return $this;
- }
-
public function setTargetDiff(DifferentialDiff $target) {
$this->target = $target;
return $this;
@@ -10,7 +10,6 @@
private $changesets;
private $target;
private $anchorName;
- private $user;
private $versusDiffID;
public function setComment($comment) {
@@ -62,11 +61,6 @@ public function setAnchorName($anchor_name) {
return $this;
}
- public function setUser(PhabricatorUser $user) {
- $this->user = $user;
- return $this;
- }
-
public function render() {
if (!$this->user) {
@@ -4,7 +4,6 @@
private $revision;
private $actions;
- private $user;
private $auxiliaryFields = array();
private $diff;
@@ -29,14 +28,6 @@ private function getActions() {
return $this->actions;
}
- public function setUser(PhabricatorUser $user) {
- $this->user = $user;
- return $this;
- }
- private function getUser() {
- return $this->user;
- }
-
public function setAuxiliaryFields(array $fields) {
assert_instances_of($fields, 'DifferentialFieldSpecification');
$this->auxiliaryFields = $fields;
@@ -9,7 +9,6 @@
private $flags = array();
private $drafts = array();
private $handles;
- private $user;
private $fields;
private $highlightAge;
const NO_DATA_STRING = 'No revisions found.';
@@ -47,11 +46,6 @@ public function setHandles(array $handles) {
return $this;
}
- public function setUser(PhabricatorUser $user) {
- $this->user = $user;
- return $this;
- }
-
public function loadAssets() {
$user = $this->user;
if (!$user) {
@@ -7,7 +7,6 @@
private $comments;
private $revisions;
private $diffs;
- private $user;
private $filter;
public function setRevisions(array $revisions) {
@@ -33,11 +32,6 @@ public function setFilter($filter) {
return $this;
}
- public function setUser($user) {
- $this->user = $user;
- return $this;
- }
-
public function render() {
$user = $this->user;
if (!$user) {
@@ -6,7 +6,6 @@
private $selectedVersusDiffID;
private $selectedDiffID;
private $selectedWhitespace;
- private $user;
public function setDiffs(array $diffs) {
assert_instances_of($diffs, 'DifferentialDiff');
@@ -29,15 +28,6 @@ public function setSelectedWhitespace($whitespace) {
return $this;
}
- public function setUser($user) {
- $this->user = $user;
- return $this;
- }
-
- public function getUser() {
- return $this->user;
- }
-
public function render() {
require_celerity_resource('differential-core-view-css');
@@ -3,7 +3,6 @@
final class DiffusionBranchTableView extends DiffusionView {
private $branches;
- private $user;
private $commits = array();
public function setBranches(array $branches) {
@@ -17,11 +16,6 @@ public function setCommits(array $commits) {
return $this;
}
- public function setUser(PhabricatorUser $user) {
- $this->user = $user;
- return $this;
- }
-
public function render() {
$drequest = $this->getDiffusionRequest();
$current_branch = $drequest->getBranch();
@@ -4,7 +4,6 @@
private $paths;
private $handles = array();
- private $user;
public function setPaths(array $paths) {
assert_instances_of($paths, 'DiffusionRepositoryPath');
@@ -18,11 +17,6 @@ public function setHandles(array $handles) {
return $this;
}
- public function setUser(PhabricatorUser $user) {
- $this->user = $user;
- return $this;
- }
-
public static function renderLastModifiedColumns(
DiffusionRequest $drequest,
array $handles,
@@ -2,18 +2,12 @@
final class DiffusionCommentListView extends AphrontView {
- private $user;
private $comments;
private $inlineComments = array();
private $pathMap = array();
private $handles = array();
private $markupEngine;
- public function setUser(PhabricatorUser $user) {
- $this->user = $user;
- return $this;
- }
-
public function setComments(array $comments) {
assert_instances_of($comments, 'PhabricatorAuditComment');
$this->comments = $comments;
Oops, something went wrong.

0 comments on commit ef214e9

Please sign in to comment.