2011-03-11 18:34:22 +01:00
|
|
|
<?php
|
|
|
|
|
2012-03-10 00:46:25 +01:00
|
|
|
final class DiffusionCommitController extends DiffusionController {
|
2011-03-11 18:34:22 +01:00
|
|
|
|
2011-04-07 23:55:06 +02:00
|
|
|
const CHANGES_LIMIT = 100;
|
|
|
|
|
2012-03-26 18:44:06 +02:00
|
|
|
private $auditAuthorityPHIDs;
|
2012-07-17 00:00:32 +02:00
|
|
|
private $highlightedAudits;
|
2012-03-26 18:44:06 +02:00
|
|
|
|
Fix many encoding and architecture problems in Diffusion request and URI handling
Summary:
Diffusion request/uri handling is currently a big, hastily ported mess. In particular, it has:
- Tons and tons of duplicated code.
- Bugs with handling unusual branch and file names.
- An excessively large (and yet insufficiently expressive) API on DiffusionRequest, including a nonsensical concrete base class.
- Other tools were doing hacky things like passing ":" branch names.
This diff attempts to fix these issues.
- Make the base class abstract (it was concrete ONLY for "/diffusion/").
- Move all URI generation to DiffusionRequest. Make the core static. Add unit tests.
- Delete the 300 copies of URI generation code throughout Diffusion.
- Move all URI parsing to DiffusionRequest. Make the core static. Add unit tests.
- Add an appropriate static initializer for other callers.
- Convert all code calling `newFromAphrontRequestDictionary` outside of Diffusion to the new `newFromDictionary` API.
- Refactor static initializers to be sensibly-sized.
- Refactor derived DiffusionRequest classes to remove duplicated code.
- Properly encode branch names (fixes branches with "/", see <https://github.com/facebook/phabricator/issues/100>).
- Properly encode path names (fixes issues in D1742).
- Properly escape delimiter characters ";" and "$" in path names so files like "$100" are not interpreted as "line 100".
- Fix a couple warnings.
- Fix a couple lint issues.
- Fix a bug where we would not parse filenames with spaces in them correctly in the Git browse query.
- Fix a bug where Git change queries would fail unnecessarily.
- Provide or improve some documentation.
This thing is pretty gigantic but also kind of hard to split up. If it's unreasonably difficult to review, let me know and I can take a stab at it though.
This supplants D1742.
Test Plan:
- Used home, repository, branch, browse, change, history, diff (ajax), lastmodified (ajax) views of Diffusion.
- Used Owners typeaheads and search.
- Used diffusion.getrecentcommitsbypath method.
- Pushed a change to an absurdly-named file on an absurdly-named branch, everything worked properly.
{F9185}
Reviewers: nh, vrana, btrahan
Reviewed By: btrahan
CC: aran, epriestley
Differential Revision: https://secure.phabricator.com/D1921
2012-03-20 03:52:14 +01:00
|
|
|
public function willProcessRequest(array $data) {
|
|
|
|
// This controller doesn't use blob/path stuff, just pass the dictionary
|
|
|
|
// in directly instead of using the AphrontRequest parsing mechanism.
|
|
|
|
$drequest = DiffusionRequest::newFromDictionary($data);
|
|
|
|
$this->diffusionRequest = $drequest;
|
|
|
|
}
|
|
|
|
|
2011-03-11 18:34:22 +01:00
|
|
|
public function processRequest() {
|
2011-03-14 00:19:39 +01:00
|
|
|
$drequest = $this->getDiffusionRequest();
|
2011-04-07 23:55:06 +02:00
|
|
|
$request = $this->getRequest();
|
2011-06-18 22:07:02 +02:00
|
|
|
$user = $request->getUser();
|
2011-03-11 18:34:22 +01:00
|
|
|
|
2012-05-02 22:43:45 +02:00
|
|
|
if ($request->getStr('diff')) {
|
|
|
|
return $this->buildRawDiffResponse($drequest);
|
|
|
|
}
|
|
|
|
|
2011-03-31 04:22:11 +02:00
|
|
|
$callsign = $drequest->getRepository()->getCallsign();
|
|
|
|
|
2011-03-14 00:19:39 +01:00
|
|
|
$content = array();
|
2011-03-14 04:12:17 +01:00
|
|
|
$repository = $drequest->getRepository();
|
|
|
|
$commit = $drequest->loadCommit();
|
2011-03-27 08:52:09 +02:00
|
|
|
|
|
|
|
if (!$commit) {
|
2012-08-09 18:27:45 +02:00
|
|
|
$query = DiffusionExistsQuery::newFromDiffusionRequest($drequest);
|
|
|
|
$exists = $query->loadExistentialData();
|
|
|
|
if (!$exists) {
|
|
|
|
return new Aphront404Response();
|
|
|
|
}
|
2012-08-08 19:03:41 +02:00
|
|
|
return $this->buildStandardPageResponse(
|
|
|
|
id(new AphrontErrorView())
|
|
|
|
->setTitle('Error displaying commit.')
|
2012-08-09 18:27:45 +02:00
|
|
|
->appendChild('Failed to load the commit because the commit has not '.
|
|
|
|
'been parsed yet.'),
|
2012-08-08 19:03:41 +02:00
|
|
|
array('title' => 'Commit Still Parsing')
|
|
|
|
);
|
2011-03-27 08:52:09 +02:00
|
|
|
}
|
|
|
|
|
2011-03-14 04:12:17 +01:00
|
|
|
$commit_data = $drequest->loadCommitData();
|
2012-03-26 18:44:06 +02:00
|
|
|
$commit->attachCommitData($commit_data);
|
2011-03-14 04:12:17 +01:00
|
|
|
|
upgrade diffusion to use modern header UI and fix a few quirks
Summary:
upgrades are CrumbsView, HeaderView, PropertyListView, and ActionListView. I had to modify CrumbsView stuff a bit to handle the "advanced" diffusion crumbs.
Quirks fixed include making file tree view show up in diffusion, the page not have extra space when the file tree is hidden, links no longer breaking once you visit files (since without the change the files always got "/" appended and thus 404'd), and a differential quirk where it read "next step:" and that colon is a no no,
Test Plan: played around in diffusion and differential
Reviewers: epriestley
Reviewed By: epriestley
CC: aran, Korvin, chad
Maniphest Tasks: T2048, T2178
Differential Revision: https://secure.phabricator.com/D4169
2012-12-13 02:50:42 +01:00
|
|
|
$top_anchor = id(new PhabricatorAnchorView())
|
|
|
|
->setAnchorName('top')
|
|
|
|
->setNavigationMarker(true);
|
|
|
|
|
Improve several Diffusion UI error states
Summary:
Give users better errors and UI:
- For subpath SVN repositories, default the path to the subdirectory, not to
"/". This makes the home screen useful and things generally less confusing.
- For unparsed commits, show a more descriptive error message without the
"blah blah" silliness.
- For paths outside of the subpath parse tree, short circuit into an
appropriate error message.
- For foreign SVN stub commits (see D892), show an explicit message.
Test Plan: Looked at unparsed commits, subpath repositories, foreign stub
commits, and paths outside of the subpath parse tree. Received sensible error
messages.
Reviewers: jungejason, nh, tuomaspelkonen, aran
Reviewed By: jungejason
CC: aran, jungejason
Differential Revision: 894
2011-09-04 23:39:52 +02:00
|
|
|
$is_foreign = $commit_data->getCommitDetail('foreign-svn-stub');
|
2013-01-21 16:45:42 +01:00
|
|
|
$changesets = null;
|
Improve several Diffusion UI error states
Summary:
Give users better errors and UI:
- For subpath SVN repositories, default the path to the subdirectory, not to
"/". This makes the home screen useful and things generally less confusing.
- For unparsed commits, show a more descriptive error message without the
"blah blah" silliness.
- For paths outside of the subpath parse tree, short circuit into an
appropriate error message.
- For foreign SVN stub commits (see D892), show an explicit message.
Test Plan: Looked at unparsed commits, subpath repositories, foreign stub
commits, and paths outside of the subpath parse tree. Received sensible error
messages.
Reviewers: jungejason, nh, tuomaspelkonen, aran
Reviewed By: jungejason
CC: aran, jungejason
Differential Revision: 894
2011-09-04 23:39:52 +02:00
|
|
|
if ($is_foreign) {
|
|
|
|
$subpath = $commit_data->getCommitDetail('svn-subpath');
|
|
|
|
|
|
|
|
$error_panel = new AphrontErrorView();
|
|
|
|
$error_panel->setTitle('Commit Not Tracked');
|
|
|
|
$error_panel->setSeverity(AphrontErrorView::SEVERITY_WARNING);
|
|
|
|
$error_panel->appendChild(
|
|
|
|
"This Diffusion repository is configured to track only one ".
|
|
|
|
"subdirectory of the entire Subversion repository, and this commit ".
|
2013-02-07 01:53:49 +01:00
|
|
|
"didn't affect the tracked subdirectory ('".$subpath."'), so no ".
|
|
|
|
"information is available.");
|
Improve several Diffusion UI error states
Summary:
Give users better errors and UI:
- For subpath SVN repositories, default the path to the subdirectory, not to
"/". This makes the home screen useful and things generally less confusing.
- For unparsed commits, show a more descriptive error message without the
"blah blah" silliness.
- For paths outside of the subpath parse tree, short circuit into an
appropriate error message.
- For foreign SVN stub commits (see D892), show an explicit message.
Test Plan: Looked at unparsed commits, subpath repositories, foreign stub
commits, and paths outside of the subpath parse tree. Received sensible error
messages.
Reviewers: jungejason, nh, tuomaspelkonen, aran
Reviewed By: jungejason
CC: aran, jungejason
Differential Revision: 894
2011-09-04 23:39:52 +02:00
|
|
|
$content[] = $error_panel;
|
upgrade diffusion to use modern header UI and fix a few quirks
Summary:
upgrades are CrumbsView, HeaderView, PropertyListView, and ActionListView. I had to modify CrumbsView stuff a bit to handle the "advanced" diffusion crumbs.
Quirks fixed include making file tree view show up in diffusion, the page not have extra space when the file tree is hidden, links no longer breaking once you visit files (since without the change the files always got "/" appended and thus 404'd), and a differential quirk where it read "next step:" and that colon is a no no,
Test Plan: played around in diffusion and differential
Reviewers: epriestley
Reviewed By: epriestley
CC: aran, Korvin, chad
Maniphest Tasks: T2048, T2178
Differential Revision: https://secure.phabricator.com/D4169
2012-12-13 02:50:42 +01:00
|
|
|
$content[] = $top_anchor;
|
Improve several Diffusion UI error states
Summary:
Give users better errors and UI:
- For subpath SVN repositories, default the path to the subdirectory, not to
"/". This makes the home screen useful and things generally less confusing.
- For unparsed commits, show a more descriptive error message without the
"blah blah" silliness.
- For paths outside of the subpath parse tree, short circuit into an
appropriate error message.
- For foreign SVN stub commits (see D892), show an explicit message.
Test Plan: Looked at unparsed commits, subpath repositories, foreign stub
commits, and paths outside of the subpath parse tree. Received sensible error
messages.
Reviewers: jungejason, nh, tuomaspelkonen, aran
Reviewed By: jungejason
CC: aran, jungejason
Differential Revision: 894
2011-09-04 23:39:52 +02:00
|
|
|
} else {
|
|
|
|
$engine = PhabricatorMarkupEngine::newDifferentialMarkupEngine();
|
|
|
|
|
|
|
|
require_celerity_resource('diffusion-commit-view-css');
|
|
|
|
require_celerity_resource('phabricator-remarkup-css');
|
|
|
|
|
2012-03-26 21:21:48 +02:00
|
|
|
$parent_query = DiffusionCommitParentsQuery::newFromDiffusionRequest(
|
|
|
|
$drequest);
|
Improve several Diffusion UI error states
Summary:
Give users better errors and UI:
- For subpath SVN repositories, default the path to the subdirectory, not to
"/". This makes the home screen useful and things generally less confusing.
- For unparsed commits, show a more descriptive error message without the
"blah blah" silliness.
- For paths outside of the subpath parse tree, short circuit into an
appropriate error message.
- For foreign SVN stub commits (see D892), show an explicit message.
Test Plan: Looked at unparsed commits, subpath repositories, foreign stub
commits, and paths outside of the subpath parse tree. Received sensible error
messages.
Reviewers: jungejason, nh, tuomaspelkonen, aran
Reviewed By: jungejason
CC: aran, jungejason
Differential Revision: 894
2011-09-04 23:39:52 +02:00
|
|
|
|
upgrade diffusion to use modern header UI and fix a few quirks
Summary:
upgrades are CrumbsView, HeaderView, PropertyListView, and ActionListView. I had to modify CrumbsView stuff a bit to handle the "advanced" diffusion crumbs.
Quirks fixed include making file tree view show up in diffusion, the page not have extra space when the file tree is hidden, links no longer breaking once you visit files (since without the change the files always got "/" appended and thus 404'd), and a differential quirk where it read "next step:" and that colon is a no no,
Test Plan: played around in diffusion and differential
Reviewers: epriestley
Reviewed By: epriestley
CC: aran, Korvin, chad
Maniphest Tasks: T2048, T2178
Differential Revision: https://secure.phabricator.com/D4169
2012-12-13 02:50:42 +01:00
|
|
|
$headsup_view = id(new PhabricatorHeaderView())
|
|
|
|
->setHeader('Commit Detail');
|
|
|
|
|
|
|
|
$headsup_actions = $this->renderHeadsupActionList($commit, $repository);
|
|
|
|
|
|
|
|
$commit_properties = $this->loadCommitProperties(
|
|
|
|
$commit,
|
|
|
|
$commit_data,
|
|
|
|
$parent_query->loadParents()
|
|
|
|
);
|
|
|
|
$property_list = id(new PhabricatorPropertyListView())
|
|
|
|
->setHasKeyboardShortcuts(true);
|
|
|
|
foreach ($commit_properties as $key => $value) {
|
|
|
|
$property_list->addProperty($key, $value);
|
|
|
|
}
|
|
|
|
|
|
|
|
$property_list->addTextContent(
|
2013-01-29 20:01:47 +01:00
|
|
|
phutil_tag(
|
|
|
|
'div',
|
|
|
|
array(
|
|
|
|
'class' => 'diffusion-commit-message phabricator-remarkup',
|
|
|
|
),
|
2013-02-13 23:08:57 +01:00
|
|
|
phutil_safe_html(
|
|
|
|
$engine->markupText($commit_data->getCommitMessage()))));
|
2011-03-11 18:34:22 +01:00
|
|
|
|
upgrade diffusion to use modern header UI and fix a few quirks
Summary:
upgrades are CrumbsView, HeaderView, PropertyListView, and ActionListView. I had to modify CrumbsView stuff a bit to handle the "advanced" diffusion crumbs.
Quirks fixed include making file tree view show up in diffusion, the page not have extra space when the file tree is hidden, links no longer breaking once you visit files (since without the change the files always got "/" appended and thus 404'd), and a differential quirk where it read "next step:" and that colon is a no no,
Test Plan: played around in diffusion and differential
Reviewers: epriestley
Reviewed By: epriestley
CC: aran, Korvin, chad
Maniphest Tasks: T2048, T2178
Differential Revision: https://secure.phabricator.com/D4169
2012-12-13 02:50:42 +01:00
|
|
|
$content[] = $top_anchor;
|
|
|
|
$content[] = $headsup_view;
|
|
|
|
$content[] = $headsup_actions;
|
|
|
|
$content[] = $property_list;
|
Improve several Diffusion UI error states
Summary:
Give users better errors and UI:
- For subpath SVN repositories, default the path to the subdirectory, not to
"/". This makes the home screen useful and things generally less confusing.
- For unparsed commits, show a more descriptive error message without the
"blah blah" silliness.
- For paths outside of the subpath parse tree, short circuit into an
appropriate error message.
- For foreign SVN stub commits (see D892), show an explicit message.
Test Plan: Looked at unparsed commits, subpath repositories, foreign stub
commits, and paths outside of the subpath parse tree. Received sensible error
messages.
Reviewers: jungejason, nh, tuomaspelkonen, aran
Reviewed By: jungejason
CC: aran, jungejason
Differential Revision: 894
2011-09-04 23:39:52 +02:00
|
|
|
}
|
2011-03-14 00:19:39 +01:00
|
|
|
|
2012-03-26 18:44:06 +02:00
|
|
|
$query = new PhabricatorAuditQuery();
|
|
|
|
$query->withCommitPHIDs(array($commit->getPHID()));
|
|
|
|
$audit_requests = $query->execute();
|
|
|
|
|
|
|
|
$this->auditAuthorityPHIDs =
|
|
|
|
PhabricatorAuditCommentEditor::loadAuditPHIDsForUser($user);
|
|
|
|
|
|
|
|
$content[] = $this->buildAuditTable($commit, $audit_requests);
|
2012-02-24 23:14:39 +01:00
|
|
|
$content[] = $this->buildComments($commit);
|
2012-02-24 22:02:14 +01:00
|
|
|
|
2013-01-30 21:01:49 +01:00
|
|
|
$hard_limit = 1000;
|
|
|
|
|
2011-03-14 00:19:39 +01:00
|
|
|
$change_query = DiffusionPathChangeQuery::newFromDiffusionRequest(
|
|
|
|
$drequest);
|
2013-01-30 21:01:49 +01:00
|
|
|
$change_query->setLimit($hard_limit + 1);
|
2011-03-14 00:19:39 +01:00
|
|
|
$changes = $change_query->loadChanges();
|
|
|
|
|
2013-01-30 21:01:49 +01:00
|
|
|
$was_limited = (count($changes) > $hard_limit);
|
|
|
|
if ($was_limited) {
|
|
|
|
$changes = array_slice($changes, 0, $hard_limit);
|
|
|
|
}
|
|
|
|
|
2012-03-23 23:32:26 +01:00
|
|
|
$content[] = $this->buildMergesTable($commit);
|
|
|
|
|
2012-07-17 00:00:32 +02:00
|
|
|
$owners_paths = array();
|
|
|
|
if ($this->highlightedAudits) {
|
|
|
|
$packages = id(new PhabricatorOwnersPackage())->loadAllWhere(
|
|
|
|
'phid IN (%Ls)',
|
|
|
|
mpull($this->highlightedAudits, 'getAuditorPHID'));
|
|
|
|
if ($packages) {
|
|
|
|
$owners_paths = id(new PhabricatorOwnersPath())->loadAllWhere(
|
|
|
|
'repositoryPHID = %s AND packageID IN (%Ld)',
|
|
|
|
$repository->getPHID(),
|
|
|
|
mpull($packages, 'getID'));
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2011-03-11 18:34:22 +01:00
|
|
|
$change_table = new DiffusionCommitChangeTableView();
|
|
|
|
$change_table->setDiffusionRequest($drequest);
|
2011-03-14 00:19:39 +01:00
|
|
|
$change_table->setPathChanges($changes);
|
2012-07-17 00:00:32 +02:00
|
|
|
$change_table->setOwnersPaths($owners_paths);
|
2011-03-11 18:34:22 +01:00
|
|
|
|
2011-04-07 23:55:06 +02:00
|
|
|
$count = count($changes);
|
2011-03-19 22:38:50 +01:00
|
|
|
|
2011-03-27 08:52:09 +02:00
|
|
|
$bad_commit = null;
|
|
|
|
if ($count == 0) {
|
|
|
|
$bad_commit = queryfx_one(
|
|
|
|
id(new PhabricatorRepository())->establishConnection('r'),
|
|
|
|
'SELECT * FROM %T WHERE fullCommitName = %s',
|
|
|
|
PhabricatorRepository::TABLE_BADCOMMIT,
|
2011-03-31 04:22:11 +02:00
|
|
|
'r'.$callsign.$commit->getCommitIdentifier());
|
2011-03-27 08:52:09 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
if ($bad_commit) {
|
|
|
|
$error_panel = new AphrontErrorView();
|
|
|
|
$error_panel->setTitle('Bad Commit');
|
2013-02-07 01:53:49 +01:00
|
|
|
$error_panel->appendChild($bad_commit['description']);
|
2011-03-27 08:52:09 +02:00
|
|
|
|
|
|
|
$content[] = $error_panel;
|
Improve several Diffusion UI error states
Summary:
Give users better errors and UI:
- For subpath SVN repositories, default the path to the subdirectory, not to
"/". This makes the home screen useful and things generally less confusing.
- For unparsed commits, show a more descriptive error message without the
"blah blah" silliness.
- For paths outside of the subpath parse tree, short circuit into an
appropriate error message.
- For foreign SVN stub commits (see D892), show an explicit message.
Test Plan: Looked at unparsed commits, subpath repositories, foreign stub
commits, and paths outside of the subpath parse tree. Received sensible error
messages.
Reviewers: jungejason, nh, tuomaspelkonen, aran
Reviewed By: jungejason
CC: aran, jungejason
Differential Revision: 894
2011-09-04 23:39:52 +02:00
|
|
|
} else if ($is_foreign) {
|
|
|
|
// Don't render anything else.
|
|
|
|
} else if (!count($changes)) {
|
|
|
|
$no_changes = new AphrontErrorView();
|
|
|
|
$no_changes->setSeverity(AphrontErrorView::SEVERITY_WARNING);
|
|
|
|
$no_changes->setTitle('Not Yet Parsed');
|
|
|
|
// TODO: This can also happen with weird SVN changes that don't do
|
|
|
|
// anything (or only alter properties?), although the real no-changes case
|
|
|
|
// is extremely rare and might be impossible to produce organically. We
|
|
|
|
// should probably write some kind of "Nothing Happened!" change into the
|
|
|
|
// DB once we parse these changes so we can distinguish between
|
|
|
|
// "not parsed yet" and "no changes".
|
|
|
|
$no_changes->appendChild(
|
|
|
|
"This commit hasn't been fully parsed yet (or doesn't affect any ".
|
|
|
|
"paths).");
|
|
|
|
$content[] = $no_changes;
|
2013-01-30 21:01:49 +01:00
|
|
|
} else if ($was_limited) {
|
|
|
|
$huge_commit = new AphrontErrorView();
|
|
|
|
$huge_commit->setSeverity(AphrontErrorView::SEVERITY_WARNING);
|
|
|
|
$huge_commit->setTitle(pht('Enormous Commit'));
|
|
|
|
$huge_commit->appendChild(
|
|
|
|
pht(
|
|
|
|
'This commit is enormous, and affects more than %d files. '.
|
|
|
|
'Changes are not shown.',
|
|
|
|
$hard_limit));
|
|
|
|
$content[] = $huge_commit;
|
2011-03-27 08:52:09 +02:00
|
|
|
} else {
|
|
|
|
$change_panel = new AphrontPanelView();
|
2011-04-07 23:55:06 +02:00
|
|
|
$change_panel->setHeader("Changes (".number_format($count).")");
|
2012-08-22 00:01:20 +02:00
|
|
|
$change_panel->setID('toc');
|
2012-08-07 08:01:23 +02:00
|
|
|
if ($count > self::CHANGES_LIMIT) {
|
2013-01-18 03:43:35 +01:00
|
|
|
$show_all_button = phutil_tag(
|
2011-04-07 23:55:06 +02:00
|
|
|
'a',
|
|
|
|
array(
|
|
|
|
'class' => 'button green',
|
|
|
|
'href' => '?show_all=true',
|
|
|
|
),
|
2013-01-18 03:43:35 +01:00
|
|
|
'Show All Changes');
|
2011-04-07 23:55:06 +02:00
|
|
|
$warning_view = id(new AphrontErrorView())
|
|
|
|
->setSeverity(AphrontErrorView::SEVERITY_WARNING)
|
2012-08-07 08:01:23 +02:00
|
|
|
->setTitle('Very Large Commit')
|
2013-02-07 01:53:49 +01:00
|
|
|
->appendChild(phutil_tag(
|
|
|
|
'p',
|
|
|
|
array(),
|
|
|
|
"This commit is very large. Load each file individually."));
|
2011-04-07 23:55:06 +02:00
|
|
|
|
|
|
|
$change_panel->appendChild($warning_view);
|
|
|
|
$change_panel->addButton($show_all_button);
|
|
|
|
}
|
|
|
|
|
2011-03-27 08:52:09 +02:00
|
|
|
$change_panel->appendChild($change_table);
|
2013-01-16 00:05:15 +01:00
|
|
|
$change_panel->setNoBackground();
|
2011-03-27 08:52:09 +02:00
|
|
|
|
|
|
|
$content[] = $change_panel;
|
|
|
|
|
Improve several Diffusion UI error states
Summary:
Give users better errors and UI:
- For subpath SVN repositories, default the path to the subdirectory, not to
"/". This makes the home screen useful and things generally less confusing.
- For unparsed commits, show a more descriptive error message without the
"blah blah" silliness.
- For paths outside of the subpath parse tree, short circuit into an
appropriate error message.
- For foreign SVN stub commits (see D892), show an explicit message.
Test Plan: Looked at unparsed commits, subpath repositories, foreign stub
commits, and paths outside of the subpath parse tree. Received sensible error
messages.
Reviewers: jungejason, nh, tuomaspelkonen, aran
Reviewed By: jungejason
CC: aran, jungejason
Differential Revision: 894
2011-09-04 23:39:52 +02:00
|
|
|
$changesets = DiffusionPathChange::convertToDifferentialChangesets(
|
|
|
|
$changes);
|
|
|
|
|
|
|
|
$vcs = $repository->getVersionControlSystem();
|
|
|
|
switch ($vcs) {
|
|
|
|
case PhabricatorRepositoryType::REPOSITORY_TYPE_SVN:
|
|
|
|
$vcs_supports_directory_changes = true;
|
|
|
|
break;
|
|
|
|
case PhabricatorRepositoryType::REPOSITORY_TYPE_GIT:
|
Basic support for Mercurial in Diffusion
Summary: Change import script plus almost all the view stuff. Still some rough
edges but this seems to mostly work. Blame is currently unsupported but I think
everything else works properly.
Test Plan:
Imported the hg repository itself. It doesn't immediately seem completely
broken. Here are some screens:
https://secure.phabricator.com/file/view/PHID-FILE-1438b71cc7c4a2eb4569/
https://secure.phabricator.com/file/view/PHID-FILE-3cec4f72f39e7de2d041/
https://secure.phabricator.com/file/view/PHID-FILE-2ea4883f160e8e5098f9/
https://secure.phabricator.com/file/view/PHID-FILE-35f751a36ebf65399ade/
All the parsers were able to churn through it without errors.
Ran the new "reparse.php" script in various one-commit and repository modes.
Browsed/imported some git repos for good measure.
NOTE: The hg repository is only 15,000 commits and around 1,000 files.
Performance is okay but hg doesn't provide performant, native APIs to get some
data efficiently so we have to do some dumb stuff. If some of these interfaces
are cripplingly slow or whatever, let me know and we can start bundling some
Mercurial extensions with Arcanist.
Reviewers: Makinde, jungejason, nh, tuomaspelkonen, aran
Reviewed By: Makinde
CC: aran, Makinde, epriestley
Differential Revision: 960
2011-09-26 20:07:38 +02:00
|
|
|
case PhabricatorRepositoryType::REPOSITORY_TYPE_MERCURIAL:
|
Improve several Diffusion UI error states
Summary:
Give users better errors and UI:
- For subpath SVN repositories, default the path to the subdirectory, not to
"/". This makes the home screen useful and things generally less confusing.
- For unparsed commits, show a more descriptive error message without the
"blah blah" silliness.
- For paths outside of the subpath parse tree, short circuit into an
appropriate error message.
- For foreign SVN stub commits (see D892), show an explicit message.
Test Plan: Looked at unparsed commits, subpath repositories, foreign stub
commits, and paths outside of the subpath parse tree. Received sensible error
messages.
Reviewers: jungejason, nh, tuomaspelkonen, aran
Reviewed By: jungejason
CC: aran, jungejason
Differential Revision: 894
2011-09-04 23:39:52 +02:00
|
|
|
$vcs_supports_directory_changes = false;
|
|
|
|
break;
|
|
|
|
default:
|
|
|
|
throw new Exception("Unknown VCS.");
|
|
|
|
}
|
2011-03-31 07:08:41 +02:00
|
|
|
|
Improve several Diffusion UI error states
Summary:
Give users better errors and UI:
- For subpath SVN repositories, default the path to the subdirectory, not to
"/". This makes the home screen useful and things generally less confusing.
- For unparsed commits, show a more descriptive error message without the
"blah blah" silliness.
- For paths outside of the subpath parse tree, short circuit into an
appropriate error message.
- For foreign SVN stub commits (see D892), show an explicit message.
Test Plan: Looked at unparsed commits, subpath repositories, foreign stub
commits, and paths outside of the subpath parse tree. Received sensible error
messages.
Reviewers: jungejason, nh, tuomaspelkonen, aran
Reviewed By: jungejason
CC: aran, jungejason
Differential Revision: 894
2011-09-04 23:39:52 +02:00
|
|
|
$references = array();
|
|
|
|
foreach ($changesets as $key => $changeset) {
|
|
|
|
$file_type = $changeset->getFileType();
|
|
|
|
if ($file_type == DifferentialChangeType::FILE_DIRECTORY) {
|
|
|
|
if (!$vcs_supports_directory_changes) {
|
|
|
|
unset($changesets[$key]);
|
|
|
|
continue;
|
2011-03-31 07:08:41 +02:00
|
|
|
}
|
2011-03-31 04:22:11 +02:00
|
|
|
}
|
|
|
|
|
Fix many encoding and architecture problems in Diffusion request and URI handling
Summary:
Diffusion request/uri handling is currently a big, hastily ported mess. In particular, it has:
- Tons and tons of duplicated code.
- Bugs with handling unusual branch and file names.
- An excessively large (and yet insufficiently expressive) API on DiffusionRequest, including a nonsensical concrete base class.
- Other tools were doing hacky things like passing ":" branch names.
This diff attempts to fix these issues.
- Make the base class abstract (it was concrete ONLY for "/diffusion/").
- Move all URI generation to DiffusionRequest. Make the core static. Add unit tests.
- Delete the 300 copies of URI generation code throughout Diffusion.
- Move all URI parsing to DiffusionRequest. Make the core static. Add unit tests.
- Add an appropriate static initializer for other callers.
- Convert all code calling `newFromAphrontRequestDictionary` outside of Diffusion to the new `newFromDictionary` API.
- Refactor static initializers to be sensibly-sized.
- Refactor derived DiffusionRequest classes to remove duplicated code.
- Properly encode branch names (fixes branches with "/", see <https://github.com/facebook/phabricator/issues/100>).
- Properly encode path names (fixes issues in D1742).
- Properly escape delimiter characters ";" and "$" in path names so files like "$100" are not interpreted as "line 100".
- Fix a couple warnings.
- Fix a couple lint issues.
- Fix a bug where we would not parse filenames with spaces in them correctly in the Git browse query.
- Fix a bug where Git change queries would fail unnecessarily.
- Provide or improve some documentation.
This thing is pretty gigantic but also kind of hard to split up. If it's unreasonably difficult to review, let me know and I can take a stab at it though.
This supplants D1742.
Test Plan:
- Used home, repository, branch, browse, change, history, diff (ajax), lastmodified (ajax) views of Diffusion.
- Used Owners typeaheads and search.
- Used diffusion.getrecentcommitsbypath method.
- Pushed a change to an absurdly-named file on an absurdly-named branch, everything worked properly.
{F9185}
Reviewers: nh, vrana, btrahan
Reviewed By: btrahan
CC: aran, epriestley
Differential Revision: https://secure.phabricator.com/D1921
2012-03-20 03:52:14 +01:00
|
|
|
$references[$key] = $drequest->generateURI(
|
|
|
|
array(
|
|
|
|
'action' => 'rendering-ref',
|
|
|
|
'path' => $changeset->getFilename(),
|
|
|
|
));
|
2011-03-31 04:22:11 +02:00
|
|
|
}
|
2011-03-27 08:52:09 +02:00
|
|
|
|
2012-04-07 20:45:31 +02:00
|
|
|
// TODO: Some parts of the views still rely on properties of the
|
Add inline comments to Diffusion/Audit
Summary:
- Add inline comments to Audits, like Differential.
- Creates new storage for the comments in the Audits database.
- Creates a new PhabricatorAuditInlineComment class, similar to DifferentialInlineComment.
- Defines an Interface which Differential and Audit comments conform to.
- Makes consumers of DifferentialInlineComments consume objects which implement that interface instead.
- Adds save
NOTE: Some features are still missing! Wanted to cut this off before it got crazy:
- Inline comments aren't shown in the main comment list.
- Inline comments aren't shown in the emails.
- Inline comments aren't previewed.
I'll followup with those but this was getting pretty big.
@vrana, does the SQL change look correct?
Test Plan:
- Created, edited, deleted, replied to, reloaded and saved inline comments in Diffusion, on the left and right side of diffs.
- Created, edited, deleted, replied to, reloaded and saved inline comments in Differentila, on the left and right side of primary and diff-versus-diff diffs.
Reviewers: btrahan, vrana
Reviewed By: btrahan
CC: aran, epriestley
Maniphest Tasks: T904
Differential Revision: https://secure.phabricator.com/D1898
2012-03-14 20:56:01 +01:00
|
|
|
// DifferentialChangeset. Make the objects ephemeral to make sure we don't
|
|
|
|
// accidentally save them, and then set their ID to the appropriate ID for
|
|
|
|
// this application (the path IDs).
|
2012-08-07 08:01:23 +02:00
|
|
|
$path_ids = array_flip(mpull($changes, 'getPath'));
|
Add inline comments to Diffusion/Audit
Summary:
- Add inline comments to Audits, like Differential.
- Creates new storage for the comments in the Audits database.
- Creates a new PhabricatorAuditInlineComment class, similar to DifferentialInlineComment.
- Defines an Interface which Differential and Audit comments conform to.
- Makes consumers of DifferentialInlineComments consume objects which implement that interface instead.
- Adds save
NOTE: Some features are still missing! Wanted to cut this off before it got crazy:
- Inline comments aren't shown in the main comment list.
- Inline comments aren't shown in the emails.
- Inline comments aren't previewed.
I'll followup with those but this was getting pretty big.
@vrana, does the SQL change look correct?
Test Plan:
- Created, edited, deleted, replied to, reloaded and saved inline comments in Diffusion, on the left and right side of diffs.
- Created, edited, deleted, replied to, reloaded and saved inline comments in Differentila, on the left and right side of primary and diff-versus-diff diffs.
Reviewers: btrahan, vrana
Reviewed By: btrahan
CC: aran, epriestley
Maniphest Tasks: T904
Differential Revision: https://secure.phabricator.com/D1898
2012-03-14 20:56:01 +01:00
|
|
|
foreach ($changesets as $changeset) {
|
|
|
|
$changeset->makeEphemeral();
|
|
|
|
$changeset->setID($path_ids[$changeset->getFilename()]);
|
|
|
|
}
|
|
|
|
|
2012-08-07 08:01:23 +02:00
|
|
|
if ($count <= self::CHANGES_LIMIT) {
|
|
|
|
$visible_changesets = $changesets;
|
|
|
|
} else {
|
|
|
|
$visible_changesets = array();
|
|
|
|
$inlines = id(new PhabricatorAuditInlineComment())->loadAllWhere(
|
|
|
|
'commitPHID = %s AND (auditCommentID IS NOT NULL OR authorPHID = %s)',
|
|
|
|
$commit->getPHID(),
|
|
|
|
$user->getPHID());
|
|
|
|
$path_ids = mpull($inlines, null, 'getPathID');
|
|
|
|
foreach ($changesets as $key => $changeset) {
|
|
|
|
if (array_key_exists($changeset->getID(), $path_ids)) {
|
|
|
|
$visible_changesets[$key] = $changeset;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2012-12-13 06:21:56 +01:00
|
|
|
$change_list_title = DiffusionView::nameCommit(
|
|
|
|
$repository,
|
|
|
|
$commit->getCommitIdentifier()
|
|
|
|
);
|
Improve several Diffusion UI error states
Summary:
Give users better errors and UI:
- For subpath SVN repositories, default the path to the subdirectory, not to
"/". This makes the home screen useful and things generally less confusing.
- For unparsed commits, show a more descriptive error message without the
"blah blah" silliness.
- For paths outside of the subpath parse tree, short circuit into an
appropriate error message.
- For foreign SVN stub commits (see D892), show an explicit message.
Test Plan: Looked at unparsed commits, subpath repositories, foreign stub
commits, and paths outside of the subpath parse tree. Received sensible error
messages.
Reviewers: jungejason, nh, tuomaspelkonen, aran
Reviewed By: jungejason
CC: aran, jungejason
Differential Revision: 894
2011-09-04 23:39:52 +02:00
|
|
|
$change_list = new DifferentialChangesetListView();
|
2012-12-13 06:21:56 +01:00
|
|
|
$change_list->setTitle($change_list_title);
|
Improve several Diffusion UI error states
Summary:
Give users better errors and UI:
- For subpath SVN repositories, default the path to the subdirectory, not to
"/". This makes the home screen useful and things generally less confusing.
- For unparsed commits, show a more descriptive error message without the
"blah blah" silliness.
- For paths outside of the subpath parse tree, short circuit into an
appropriate error message.
- For foreign SVN stub commits (see D892), show an explicit message.
Test Plan: Looked at unparsed commits, subpath repositories, foreign stub
commits, and paths outside of the subpath parse tree. Received sensible error
messages.
Reviewers: jungejason, nh, tuomaspelkonen, aran
Reviewed By: jungejason
CC: aran, jungejason
Differential Revision: 894
2011-09-04 23:39:52 +02:00
|
|
|
$change_list->setChangesets($changesets);
|
2012-08-07 08:01:23 +02:00
|
|
|
$change_list->setVisibleChangesets($visible_changesets);
|
Improve several Diffusion UI error states
Summary:
Give users better errors and UI:
- For subpath SVN repositories, default the path to the subdirectory, not to
"/". This makes the home screen useful and things generally less confusing.
- For unparsed commits, show a more descriptive error message without the
"blah blah" silliness.
- For paths outside of the subpath parse tree, short circuit into an
appropriate error message.
- For foreign SVN stub commits (see D892), show an explicit message.
Test Plan: Looked at unparsed commits, subpath repositories, foreign stub
commits, and paths outside of the subpath parse tree. Received sensible error
messages.
Reviewers: jungejason, nh, tuomaspelkonen, aran
Reviewed By: jungejason
CC: aran, jungejason
Differential Revision: 894
2011-09-04 23:39:52 +02:00
|
|
|
$change_list->setRenderingReferences($references);
|
|
|
|
$change_list->setRenderURI('/diffusion/'.$callsign.'/diff/');
|
2012-04-10 08:42:12 +02:00
|
|
|
$change_list->setRepository($repository);
|
2012-01-16 20:08:54 +01:00
|
|
|
$change_list->setUser($user);
|
2012-10-31 22:07:25 +01:00
|
|
|
// pick the first branch for "Browse in Diffusion" View Option
|
2012-12-03 18:27:47 +01:00
|
|
|
$branches = $commit_data->getCommitDetail('seenOnBranches', array());
|
2012-10-31 22:07:25 +01:00
|
|
|
$first_branch = reset($branches);
|
|
|
|
$change_list->setBranch($first_branch);
|
Improve several Diffusion UI error states
Summary:
Give users better errors and UI:
- For subpath SVN repositories, default the path to the subdirectory, not to
"/". This makes the home screen useful and things generally less confusing.
- For unparsed commits, show a more descriptive error message without the
"blah blah" silliness.
- For paths outside of the subpath parse tree, short circuit into an
appropriate error message.
- For foreign SVN stub commits (see D892), show an explicit message.
Test Plan: Looked at unparsed commits, subpath repositories, foreign stub
commits, and paths outside of the subpath parse tree. Received sensible error
messages.
Reviewers: jungejason, nh, tuomaspelkonen, aran
Reviewed By: jungejason
CC: aran, jungejason
Differential Revision: 894
2011-09-04 23:39:52 +02:00
|
|
|
|
2012-03-20 03:57:41 +01:00
|
|
|
$change_list->setStandaloneURI(
|
|
|
|
'/diffusion/'.$callsign.'/diff/');
|
|
|
|
$change_list->setRawFileURIs(
|
|
|
|
// TODO: Implement this, somewhat tricky if there's an octopus merge
|
|
|
|
// or whatever?
|
|
|
|
null,
|
|
|
|
'/diffusion/'.$callsign.'/diff/?view=r');
|
|
|
|
|
Add inline comments to Diffusion/Audit
Summary:
- Add inline comments to Audits, like Differential.
- Creates new storage for the comments in the Audits database.
- Creates a new PhabricatorAuditInlineComment class, similar to DifferentialInlineComment.
- Defines an Interface which Differential and Audit comments conform to.
- Makes consumers of DifferentialInlineComments consume objects which implement that interface instead.
- Adds save
NOTE: Some features are still missing! Wanted to cut this off before it got crazy:
- Inline comments aren't shown in the main comment list.
- Inline comments aren't shown in the emails.
- Inline comments aren't previewed.
I'll followup with those but this was getting pretty big.
@vrana, does the SQL change look correct?
Test Plan:
- Created, edited, deleted, replied to, reloaded and saved inline comments in Diffusion, on the left and right side of diffs.
- Created, edited, deleted, replied to, reloaded and saved inline comments in Differentila, on the left and right side of primary and diff-versus-diff diffs.
Reviewers: btrahan, vrana
Reviewed By: btrahan
CC: aran, epriestley
Maniphest Tasks: T904
Differential Revision: https://secure.phabricator.com/D1898
2012-03-14 20:56:01 +01:00
|
|
|
$change_list->setInlineCommentControllerURI(
|
2012-07-23 20:01:28 +02:00
|
|
|
'/diffusion/inline/edit/'.phutil_escape_uri($commit->getPHID()).'/');
|
Add inline comments to Diffusion/Audit
Summary:
- Add inline comments to Audits, like Differential.
- Creates new storage for the comments in the Audits database.
- Creates a new PhabricatorAuditInlineComment class, similar to DifferentialInlineComment.
- Defines an Interface which Differential and Audit comments conform to.
- Makes consumers of DifferentialInlineComments consume objects which implement that interface instead.
- Adds save
NOTE: Some features are still missing! Wanted to cut this off before it got crazy:
- Inline comments aren't shown in the main comment list.
- Inline comments aren't shown in the emails.
- Inline comments aren't previewed.
I'll followup with those but this was getting pretty big.
@vrana, does the SQL change look correct?
Test Plan:
- Created, edited, deleted, replied to, reloaded and saved inline comments in Diffusion, on the left and right side of diffs.
- Created, edited, deleted, replied to, reloaded and saved inline comments in Differentila, on the left and right side of primary and diff-versus-diff diffs.
Reviewers: btrahan, vrana
Reviewed By: btrahan
CC: aran, epriestley
Maniphest Tasks: T904
Differential Revision: https://secure.phabricator.com/D1898
2012-03-14 20:56:01 +01:00
|
|
|
|
2012-08-07 08:01:23 +02:00
|
|
|
$change_references = array();
|
|
|
|
foreach ($changesets as $key => $changeset) {
|
|
|
|
$change_references[$changeset->getID()] = $references[$key];
|
|
|
|
}
|
|
|
|
$change_table->setRenderingReferences($change_references);
|
|
|
|
|
2013-01-30 21:01:07 +01:00
|
|
|
$content[] = $change_list->render();
|
2011-03-27 08:52:09 +02:00
|
|
|
}
|
2011-03-14 00:19:39 +01:00
|
|
|
|
2013-01-30 21:01:07 +01:00
|
|
|
$content[] = $this->renderAddCommentPanel($commit, $audit_requests);
|
|
|
|
|
upgrade diffusion to use modern header UI and fix a few quirks
Summary:
upgrades are CrumbsView, HeaderView, PropertyListView, and ActionListView. I had to modify CrumbsView stuff a bit to handle the "advanced" diffusion crumbs.
Quirks fixed include making file tree view show up in diffusion, the page not have extra space when the file tree is hidden, links no longer breaking once you visit files (since without the change the files always got "/" appended and thus 404'd), and a differential quirk where it read "next step:" and that colon is a no no,
Test Plan: played around in diffusion and differential
Reviewers: epriestley
Reviewed By: epriestley
CC: aran, Korvin, chad
Maniphest Tasks: T2048, T2178
Differential Revision: https://secure.phabricator.com/D4169
2012-12-13 02:50:42 +01:00
|
|
|
$commit_id = 'r'.$callsign.$commit->getCommitIdentifier();
|
|
|
|
$short_name = DiffusionView::nameCommit(
|
|
|
|
$repository,
|
|
|
|
$commit->getCommitIdentifier()
|
|
|
|
);
|
|
|
|
|
|
|
|
$crumbs = $this->buildCrumbs(array(
|
|
|
|
'commit' => true,
|
|
|
|
));
|
2013-01-21 16:45:42 +01:00
|
|
|
|
2013-02-05 02:00:27 +01:00
|
|
|
$prefs = $user->loadPreferences();
|
|
|
|
$pref_filetree = PhabricatorUserPreferences::PREFERENCE_DIFF_FILETREE;
|
|
|
|
$pref_collapse = PhabricatorUserPreferences::PREFERENCE_NAV_COLLAPSED;
|
|
|
|
$show_filetree = $prefs->getPreference($pref_filetree);
|
|
|
|
$collapsed = $prefs->getPreference($pref_collapse);
|
|
|
|
|
|
|
|
if ($changesets && $show_filetree) {
|
2013-01-21 16:45:42 +01:00
|
|
|
$nav = id(new DifferentialChangesetFileTreeSideNavBuilder())
|
|
|
|
->setAnchorName('top')
|
|
|
|
->setTitle($short_name)
|
|
|
|
->setBaseURI(new PhutilURI('/'.$commit_id))
|
|
|
|
->build($changesets)
|
|
|
|
->setCrumbs($crumbs)
|
2013-02-05 02:00:27 +01:00
|
|
|
->setCollapsed((bool)$collapsed)
|
2013-01-21 16:45:42 +01:00
|
|
|
->appendChild($content);
|
|
|
|
$content = $nav;
|
|
|
|
} else {
|
|
|
|
$content = array($crumbs, $content);
|
|
|
|
}
|
upgrade diffusion to use modern header UI and fix a few quirks
Summary:
upgrades are CrumbsView, HeaderView, PropertyListView, and ActionListView. I had to modify CrumbsView stuff a bit to handle the "advanced" diffusion crumbs.
Quirks fixed include making file tree view show up in diffusion, the page not have extra space when the file tree is hidden, links no longer breaking once you visit files (since without the change the files always got "/" appended and thus 404'd), and a differential quirk where it read "next step:" and that colon is a no no,
Test Plan: played around in diffusion and differential
Reviewers: epriestley
Reviewed By: epriestley
CC: aran, Korvin, chad
Maniphest Tasks: T2048, T2178
Differential Revision: https://secure.phabricator.com/D4169
2012-12-13 02:50:42 +01:00
|
|
|
|
|
|
|
return $this->buildApplicationPage(
|
2013-01-21 16:45:42 +01:00
|
|
|
$content,
|
2011-03-11 18:34:22 +01:00
|
|
|
array(
|
upgrade diffusion to use modern header UI and fix a few quirks
Summary:
upgrades are CrumbsView, HeaderView, PropertyListView, and ActionListView. I had to modify CrumbsView stuff a bit to handle the "advanced" diffusion crumbs.
Quirks fixed include making file tree view show up in diffusion, the page not have extra space when the file tree is hidden, links no longer breaking once you visit files (since without the change the files always got "/" appended and thus 404'd), and a differential quirk where it read "next step:" and that colon is a no no,
Test Plan: played around in diffusion and differential
Reviewers: epriestley
Reviewed By: epriestley
CC: aran, Korvin, chad
Maniphest Tasks: T2048, T2178
Differential Revision: https://secure.phabricator.com/D4169
2012-12-13 02:50:42 +01:00
|
|
|
'title' => $commit_id
|
|
|
|
)
|
|
|
|
);
|
2011-03-11 18:34:22 +01:00
|
|
|
}
|
|
|
|
|
upgrade diffusion to use modern header UI and fix a few quirks
Summary:
upgrades are CrumbsView, HeaderView, PropertyListView, and ActionListView. I had to modify CrumbsView stuff a bit to handle the "advanced" diffusion crumbs.
Quirks fixed include making file tree view show up in diffusion, the page not have extra space when the file tree is hidden, links no longer breaking once you visit files (since without the change the files always got "/" appended and thus 404'd), and a differential quirk where it read "next step:" and that colon is a no no,
Test Plan: played around in diffusion and differential
Reviewers: epriestley
Reviewed By: epriestley
CC: aran, Korvin, chad
Maniphest Tasks: T2048, T2178
Differential Revision: https://secure.phabricator.com/D4169
2012-12-13 02:50:42 +01:00
|
|
|
private function loadCommitProperties(
|
2011-04-02 02:11:05 +02:00
|
|
|
PhabricatorRepositoryCommit $commit,
|
2012-03-26 21:21:48 +02:00
|
|
|
PhabricatorRepositoryCommitData $data,
|
|
|
|
array $parents) {
|
2012-08-08 19:03:41 +02:00
|
|
|
|
2012-04-04 01:22:31 +02:00
|
|
|
assert_instances_of($parents, 'PhabricatorRepositoryCommit');
|
2012-03-30 23:12:10 +02:00
|
|
|
$user = $this->getRequest()->getUser();
|
2012-08-08 19:03:41 +02:00
|
|
|
$commit_phid = $commit->getPHID();
|
|
|
|
|
|
|
|
$edges = id(new PhabricatorEdgeQuery())
|
|
|
|
->withSourcePHIDs(array($commit_phid))
|
|
|
|
->withEdgeTypes(array(
|
|
|
|
PhabricatorEdgeConfig::TYPE_COMMIT_HAS_TASK,
|
|
|
|
PhabricatorEdgeConfig::TYPE_COMMIT_HAS_PROJECT
|
|
|
|
))
|
|
|
|
->execute();
|
|
|
|
|
|
|
|
$task_phids = array_keys(
|
|
|
|
$edges[$commit_phid][PhabricatorEdgeConfig::TYPE_COMMIT_HAS_TASK]
|
|
|
|
);
|
|
|
|
$proj_phids = array_keys(
|
|
|
|
$edges[$commit_phid][PhabricatorEdgeConfig::TYPE_COMMIT_HAS_PROJECT]
|
|
|
|
);
|
|
|
|
|
|
|
|
$phids = array_merge($task_phids, $proj_phids);
|
2011-04-02 02:11:05 +02:00
|
|
|
if ($data->getCommitDetail('authorPHID')) {
|
|
|
|
$phids[] = $data->getCommitDetail('authorPHID');
|
|
|
|
}
|
|
|
|
if ($data->getCommitDetail('reviewerPHID')) {
|
|
|
|
$phids[] = $data->getCommitDetail('reviewerPHID');
|
|
|
|
}
|
2012-05-23 17:34:36 +02:00
|
|
|
if ($data->getCommitDetail('committerPHID')) {
|
|
|
|
$phids[] = $data->getCommitDetail('committerPHID');
|
|
|
|
}
|
2011-04-02 02:11:05 +02:00
|
|
|
if ($data->getCommitDetail('differential.revisionPHID')) {
|
|
|
|
$phids[] = $data->getCommitDetail('differential.revisionPHID');
|
|
|
|
}
|
2012-03-26 21:21:48 +02:00
|
|
|
if ($parents) {
|
|
|
|
foreach ($parents as $parent) {
|
|
|
|
$phids[] = $parent->getPHID();
|
|
|
|
}
|
|
|
|
}
|
2011-04-02 02:11:05 +02:00
|
|
|
|
|
|
|
$handles = array();
|
|
|
|
if ($phids) {
|
2012-09-05 04:02:56 +02:00
|
|
|
$handles = $this->loadViewerHandles($phids);
|
2011-04-02 02:11:05 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
$props = array();
|
|
|
|
|
2012-03-30 23:12:10 +02:00
|
|
|
if ($commit->getAuditStatus()) {
|
|
|
|
$status = PhabricatorAuditCommitStatusConstants::getStatusName(
|
|
|
|
$commit->getAuditStatus());
|
2013-01-18 03:43:35 +01:00
|
|
|
$props['Status'] = phutil_tag(
|
2012-03-30 23:12:10 +02:00
|
|
|
'strong',
|
|
|
|
array(),
|
2013-01-18 03:43:35 +01:00
|
|
|
$status);
|
2012-03-30 23:12:10 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
$props['Committed'] = phabricator_datetime($commit->getEpoch(), $user);
|
|
|
|
|
2011-04-02 02:11:05 +02:00
|
|
|
$author_phid = $data->getCommitDetail('authorPHID');
|
|
|
|
if ($data->getCommitDetail('authorPHID')) {
|
|
|
|
$props['Author'] = $handles[$author_phid]->renderLink();
|
|
|
|
} else {
|
2013-01-29 20:01:47 +01:00
|
|
|
$props['Author'] = $data->getAuthorName();
|
2011-04-02 02:11:05 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
$reviewer_phid = $data->getCommitDetail('reviewerPHID');
|
|
|
|
if ($reviewer_phid) {
|
|
|
|
$props['Reviewer'] = $handles[$reviewer_phid]->renderLink();
|
|
|
|
}
|
|
|
|
|
2012-05-23 17:34:36 +02:00
|
|
|
$committer = $data->getCommitDetail('committer');
|
|
|
|
if ($committer) {
|
|
|
|
$committer_phid = $data->getCommitDetail('committerPHID');
|
|
|
|
if ($data->getCommitDetail('committerPHID')) {
|
|
|
|
$props['Committer'] = $handles[$committer_phid]->renderLink();
|
|
|
|
} else {
|
2013-01-29 20:01:47 +01:00
|
|
|
$props['Committer'] = $committer;
|
2012-05-23 17:34:36 +02:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2011-04-02 02:11:05 +02:00
|
|
|
$revision_phid = $data->getCommitDetail('differential.revisionPHID');
|
|
|
|
if ($revision_phid) {
|
|
|
|
$props['Differential Revision'] = $handles[$revision_phid]->renderLink();
|
|
|
|
}
|
|
|
|
|
2012-03-26 21:21:48 +02:00
|
|
|
if ($parents) {
|
|
|
|
$parent_links = array();
|
|
|
|
foreach ($parents as $parent) {
|
|
|
|
$parent_links[] = $handles[$parent->getPHID()]->renderLink();
|
|
|
|
}
|
2013-02-13 23:08:57 +01:00
|
|
|
$props['Parents'] = array_interleave(
|
|
|
|
" \xC2\xB7 ",
|
|
|
|
$parent_links);
|
2012-03-26 21:21:48 +02:00
|
|
|
}
|
|
|
|
|
2012-01-19 20:49:51 +01:00
|
|
|
$request = $this->getDiffusionRequest();
|
|
|
|
|
2013-01-31 18:08:02 +01:00
|
|
|
$props['Branches'] = phutil_tag(
|
|
|
|
'span',
|
|
|
|
array(
|
|
|
|
'id' => 'commit-branches',
|
|
|
|
),
|
|
|
|
'Unknown');
|
|
|
|
$props['Tags'] = phutil_tag(
|
|
|
|
'span',
|
|
|
|
array(
|
|
|
|
'id' => 'commit-tags',
|
|
|
|
),
|
|
|
|
'Unknown');
|
2012-01-19 20:49:51 +01:00
|
|
|
|
2012-08-01 01:27:48 +02:00
|
|
|
$callsign = $request->getRepository()->getCallsign();
|
|
|
|
$root = '/diffusion/'.$callsign.'/commit/'.$commit->getCommitIdentifier();
|
|
|
|
Javelin::initBehavior(
|
|
|
|
'diffusion-commit-branches',
|
|
|
|
array(
|
|
|
|
$root.'/branches/' => 'commit-branches',
|
|
|
|
$root.'/tags/' => 'commit-tags',
|
|
|
|
));
|
2012-04-24 03:36:25 +02:00
|
|
|
|
2012-04-25 16:21:03 +02:00
|
|
|
$refs = $this->buildRefs($request);
|
|
|
|
if ($refs) {
|
2012-06-29 23:14:02 +02:00
|
|
|
$props['References'] = $refs;
|
2012-04-25 16:21:03 +02:00
|
|
|
}
|
|
|
|
|
2012-04-05 02:34:25 +02:00
|
|
|
if ($task_phids) {
|
|
|
|
$task_list = array();
|
|
|
|
foreach ($task_phids as $phid) {
|
|
|
|
$task_list[] = $handles[$phid]->renderLink();
|
|
|
|
}
|
2013-02-13 23:08:57 +01:00
|
|
|
$task_list = array_interleave(phutil_tag('br'), $task_list);
|
2012-04-05 02:34:25 +02:00
|
|
|
$props['Tasks'] = $task_list;
|
|
|
|
}
|
|
|
|
|
2012-08-08 19:03:41 +02:00
|
|
|
if ($proj_phids) {
|
|
|
|
$proj_list = array();
|
|
|
|
foreach ($proj_phids as $phid) {
|
|
|
|
$proj_list[] = $handles[$phid]->renderLink();
|
|
|
|
}
|
2013-02-13 23:08:57 +01:00
|
|
|
$proj_list = array_interleave(phutil_tag('br'), $proj_list);
|
2012-08-08 19:03:41 +02:00
|
|
|
$props['Projects'] = $proj_list;
|
|
|
|
}
|
|
|
|
|
2012-03-30 23:12:10 +02:00
|
|
|
return $props;
|
2011-04-02 02:11:05 +02:00
|
|
|
}
|
|
|
|
|
2012-04-04 01:22:31 +02:00
|
|
|
private function buildAuditTable(
|
|
|
|
PhabricatorRepositoryCommit $commit,
|
|
|
|
array $audits) {
|
|
|
|
assert_instances_of($audits, 'PhabricatorRepositoryAuditRequest');
|
Allow Herald to trigger audits for users or projects
Summary:
Allows you to write a commit rule that triggers an audit by a user (personal
rules) or a project (global rules).
Mostly this is trying to make auditing more lightweight and accessible in
environments where setting up Owners packages doesn't make sense.
For instance, Disqus wants a rule like "trigger an audit for everything that
didn't have a Differential revision". While not necessarily scalable, this is a
perfectly reasonable rule for a small company, but a lot of work to implement
with Owners (and you'll get a lot of collateral damage if you don't make every
committer a project owner).
Instead, they can create a project called 'Unreviewed Commits' and write a rule
like:
- When: Differential revision does not exist
- Action: Trigger an Audit for project: "Unreviewed Commits"
Then whoever cares can join that project and they'll see those audits in their
queue, and when they approve/raise on commits their actions will affect the
project audit.
Similarly, if I want to look at all commits that match some other rule (say,
XSS) but only want to do it like once a month, I can just set up an audit rule
and go through the queue when I feel like it.
NOTE: This abuses the 'packagePHID' field to also store user and project PHIDs.
Through the magic of handles, this (apparently) works fine for now; I'll do a
big schema patch soon but have several other edits I want to make at the same
time.
Also:
- Adds an "active" fiew for /audit/, eventually this will be like the
Differential "active" view (stuff that is relevant to you right now).
- On commits, highlight triggered audits you are responsible for.
Test Plan: Added personal and global audit triggers to Herald, reparsed some
commits with --herald, got audits. Browsed all audit interfaces to make sure
nothing exploded. Viewed a commit where I was responsible for only some audits.
Performed audits and made sure the triggers I am supposed to be responsible for
updated properly.
Reviewers: btrahan, jungejason
Reviewed By: jungejason
CC: aran, epriestley
Maniphest Tasks: T904
Differential Revision: https://secure.phabricator.com/D1690
2012-02-27 18:36:30 +01:00
|
|
|
$user = $this->getRequest()->getUser();
|
|
|
|
|
2012-02-24 22:02:14 +01:00
|
|
|
$view = new PhabricatorAuditListView();
|
|
|
|
$view->setAudits($audits);
|
2012-03-26 18:44:06 +02:00
|
|
|
$view->setCommits(array($commit));
|
|
|
|
$view->setUser($user);
|
|
|
|
$view->setShowDescriptions(false);
|
2012-02-24 22:02:14 +01:00
|
|
|
|
|
|
|
$phids = $view->getRequiredHandlePHIDs();
|
2012-09-05 04:02:56 +02:00
|
|
|
$handles = $this->loadViewerHandles($phids);
|
2012-02-24 22:02:14 +01:00
|
|
|
$view->setHandles($handles);
|
2012-03-26 18:44:06 +02:00
|
|
|
$view->setAuthorityPHIDs($this->auditAuthorityPHIDs);
|
2012-07-17 00:00:32 +02:00
|
|
|
$this->highlightedAudits = $view->getHighlightedAudits();
|
2012-02-24 22:02:14 +01:00
|
|
|
|
|
|
|
$panel = new AphrontPanelView();
|
|
|
|
$panel->setHeader('Audits');
|
2012-03-30 23:12:10 +02:00
|
|
|
$panel->setCaption('Audits you are responsible for are highlighted.');
|
2012-02-24 22:02:14 +01:00
|
|
|
$panel->appendChild($view);
|
2013-01-16 00:05:15 +01:00
|
|
|
$panel->setNoBackground();
|
2012-02-24 22:02:14 +01:00
|
|
|
|
|
|
|
return $panel;
|
|
|
|
}
|
|
|
|
|
2012-04-04 01:22:31 +02:00
|
|
|
private function buildComments(PhabricatorRepositoryCommit $commit) {
|
2012-02-24 23:14:39 +01:00
|
|
|
$user = $this->getRequest()->getUser();
|
|
|
|
$comments = id(new PhabricatorAuditComment())->loadAllWhere(
|
|
|
|
'targetPHID = %s ORDER BY dateCreated ASC',
|
|
|
|
$commit->getPHID());
|
|
|
|
|
2012-03-20 03:56:06 +01:00
|
|
|
$inlines = id(new PhabricatorAuditInlineComment())->loadAllWhere(
|
|
|
|
'commitPHID = %s AND auditCommentID IS NOT NULL',
|
|
|
|
$commit->getPHID());
|
|
|
|
|
|
|
|
$path_ids = mpull($inlines, 'getPathID');
|
|
|
|
|
|
|
|
$path_map = array();
|
|
|
|
if ($path_ids) {
|
|
|
|
$path_map = id(new DiffusionPathQuery())
|
|
|
|
->withPathIDs($path_ids)
|
|
|
|
->execute();
|
|
|
|
$path_map = ipull($path_map, 'path', 'id');
|
|
|
|
}
|
|
|
|
|
2012-10-24 02:46:44 +02:00
|
|
|
$engine = new PhabricatorMarkupEngine();
|
|
|
|
$engine->setViewer($user);
|
|
|
|
|
|
|
|
foreach ($comments as $comment) {
|
|
|
|
$engine->addObject(
|
|
|
|
$comment,
|
|
|
|
PhabricatorAuditComment::MARKUP_FIELD_BODY);
|
|
|
|
}
|
|
|
|
|
|
|
|
foreach ($inlines as $inline) {
|
|
|
|
$engine->addObject(
|
|
|
|
$inline,
|
|
|
|
PhabricatorInlineCommentInterface::MARKUP_FIELD_BODY);
|
|
|
|
}
|
|
|
|
|
|
|
|
$engine->process();
|
|
|
|
|
2012-02-24 23:14:39 +01:00
|
|
|
$view = new DiffusionCommentListView();
|
2012-10-24 02:46:44 +02:00
|
|
|
$view->setMarkupEngine($engine);
|
2012-02-24 23:14:39 +01:00
|
|
|
$view->setUser($user);
|
|
|
|
$view->setComments($comments);
|
2012-03-20 03:56:06 +01:00
|
|
|
$view->setInlineComments($inlines);
|
|
|
|
$view->setPathMap($path_map);
|
2012-02-24 23:14:39 +01:00
|
|
|
|
|
|
|
$phids = $view->getRequiredHandlePHIDs();
|
2012-09-05 04:02:56 +02:00
|
|
|
$handles = $this->loadViewerHandles($phids);
|
2012-02-24 23:14:39 +01:00
|
|
|
$view->setHandles($handles);
|
|
|
|
|
|
|
|
return $view;
|
|
|
|
}
|
|
|
|
|
2012-07-23 18:15:10 +02:00
|
|
|
private function renderAddCommentPanel(
|
2012-04-04 01:22:31 +02:00
|
|
|
PhabricatorRepositoryCommit $commit,
|
2013-01-30 21:01:07 +01:00
|
|
|
array $audit_requests) {
|
2012-04-04 01:22:31 +02:00
|
|
|
assert_instances_of($audit_requests, 'PhabricatorRepositoryAuditRequest');
|
2012-02-25 00:04:53 +01:00
|
|
|
$user = $this->getRequest()->getUser();
|
|
|
|
|
|
|
|
$is_serious = PhabricatorEnv::getEnvConfig('phabricator.serious-business');
|
|
|
|
|
2013-01-30 21:01:07 +01:00
|
|
|
$pane_id = celerity_generate_unique_node_id();
|
2012-03-19 03:44:28 +01:00
|
|
|
Javelin::initBehavior(
|
|
|
|
'differential-keyboard-navigation',
|
|
|
|
array(
|
2012-07-23 18:15:10 +02:00
|
|
|
'haunt' => $pane_id,
|
2012-03-19 03:44:28 +01:00
|
|
|
));
|
|
|
|
|
2012-02-27 22:00:23 +01:00
|
|
|
$draft = id(new PhabricatorDraft())->loadOneWhere(
|
|
|
|
'authorPHID = %s AND draftKey = %s',
|
|
|
|
$user->getPHID(),
|
|
|
|
'diffusion-audit-'.$commit->getID());
|
|
|
|
if ($draft) {
|
|
|
|
$draft = $draft->getDraft();
|
|
|
|
} else {
|
|
|
|
$draft = null;
|
|
|
|
}
|
|
|
|
|
2012-03-26 18:44:06 +02:00
|
|
|
$actions = $this->getAuditActions($commit, $audit_requests);
|
|
|
|
|
2012-02-25 00:04:53 +01:00
|
|
|
$form = id(new AphrontFormView())
|
|
|
|
->setUser($user)
|
|
|
|
->setAction('/audit/addcomment/')
|
|
|
|
->addHiddenInput('commit', $commit->getPHID())
|
|
|
|
->appendChild(
|
|
|
|
id(new AphrontFormSelectControl())
|
|
|
|
->setLabel('Action')
|
|
|
|
->setName('action')
|
2012-02-27 22:00:23 +01:00
|
|
|
->setID('audit-action')
|
2012-03-26 18:44:06 +02:00
|
|
|
->setOptions($actions))
|
2012-04-23 22:50:04 +02:00
|
|
|
->appendChild(
|
|
|
|
id(new AphrontFormTokenizerControl())
|
|
|
|
->setLabel('Add Auditors')
|
|
|
|
->setName('auditors')
|
|
|
|
->setControlID('add-auditors')
|
|
|
|
->setControlStyle('display: none')
|
|
|
|
->setID('add-auditors-tokenizer')
|
|
|
|
->setDisableBehavior(true))
|
|
|
|
->appendChild(
|
|
|
|
id(new AphrontFormTokenizerControl())
|
|
|
|
->setLabel('Add CCs')
|
|
|
|
->setName('ccs')
|
|
|
|
->setControlID('add-ccs')
|
|
|
|
->setControlStyle('display: none')
|
|
|
|
->setID('add-ccs-tokenizer')
|
|
|
|
->setDisableBehavior(true))
|
2012-02-25 00:04:53 +01:00
|
|
|
->appendChild(
|
2012-09-19 21:27:28 +02:00
|
|
|
id(new PhabricatorRemarkupControl())
|
2012-02-25 00:04:53 +01:00
|
|
|
->setLabel('Comments')
|
|
|
|
->setName('content')
|
2012-02-27 22:00:23 +01:00
|
|
|
->setValue($draft)
|
2012-11-27 23:06:31 +01:00
|
|
|
->setID('audit-content')
|
|
|
|
->setUser($user))
|
2012-02-25 00:04:53 +01:00
|
|
|
->appendChild(
|
|
|
|
id(new AphrontFormSubmitControl())
|
|
|
|
->setValue($is_serious ? 'Submit' : 'Cook the Books'));
|
|
|
|
|
|
|
|
$panel = new AphrontPanelView();
|
|
|
|
$panel->setHeader($is_serious ? 'Audit Commit' : 'Creative Accounting');
|
|
|
|
$panel->appendChild($form);
|
2012-07-23 18:15:10 +02:00
|
|
|
$panel->addClass('aphront-panel-accent');
|
|
|
|
$panel->addClass('aphront-panel-flush');
|
2012-02-25 00:04:53 +01:00
|
|
|
|
2012-02-27 22:00:23 +01:00
|
|
|
require_celerity_resource('phabricator-transaction-view-css');
|
|
|
|
|
2012-04-23 22:50:04 +02:00
|
|
|
Javelin::initBehavior(
|
|
|
|
'differential-add-reviewers-and-ccs',
|
|
|
|
array(
|
|
|
|
'dynamic' => array(
|
|
|
|
'add-auditors-tokenizer' => array(
|
|
|
|
'actions' => array('add_auditors' => 1),
|
|
|
|
'src' => '/typeahead/common/users/',
|
|
|
|
'row' => 'add-auditors',
|
|
|
|
'ondemand' => PhabricatorEnv::getEnvConfig('tokenizer.ondemand'),
|
|
|
|
'placeholder' => 'Type a user name...',
|
|
|
|
),
|
|
|
|
'add-ccs-tokenizer' => array(
|
|
|
|
'actions' => array('add_ccs' => 1),
|
|
|
|
'src' => '/typeahead/common/mailable/',
|
|
|
|
'row' => 'add-ccs',
|
|
|
|
'ondemand' => PhabricatorEnv::getEnvConfig('tokenizer.ondemand'),
|
|
|
|
'placeholder' => 'Type a user or mailing list...',
|
|
|
|
),
|
|
|
|
),
|
|
|
|
'select' => 'audit-action',
|
|
|
|
));
|
|
|
|
|
|
|
|
Javelin::initBehavior('differential-feedback-preview', array(
|
2012-02-27 22:00:23 +01:00
|
|
|
'uri' => '/audit/preview/'.$commit->getID().'/',
|
|
|
|
'preview' => 'audit-preview',
|
|
|
|
'content' => 'audit-content',
|
|
|
|
'action' => 'audit-action',
|
2012-04-23 22:50:04 +02:00
|
|
|
'previewTokenizers' => array(
|
|
|
|
'auditors' => 'add-auditors-tokenizer',
|
|
|
|
'ccs' => 'add-ccs-tokenizer',
|
|
|
|
),
|
2012-07-23 20:01:28 +02:00
|
|
|
'inline' => 'inline-comment-preview',
|
|
|
|
'inlineuri' => '/diffusion/inline/preview/'.$commit->getPHID().'/',
|
2012-02-27 22:00:23 +01:00
|
|
|
));
|
|
|
|
|
2013-02-13 23:08:57 +01:00
|
|
|
$preview_panel =
|
2012-07-23 18:15:10 +02:00
|
|
|
'<div class="aphront-panel-preview aphront-panel-flush">
|
2012-02-27 22:00:23 +01:00
|
|
|
<div id="audit-preview">
|
|
|
|
<div class="aphront-panel-preview-loading-text">
|
|
|
|
Loading preview...
|
|
|
|
</div>
|
|
|
|
</div>
|
2012-07-23 20:01:28 +02:00
|
|
|
<div id="inline-comment-preview">
|
|
|
|
</div>
|
2013-02-13 23:08:57 +01:00
|
|
|
</div>';
|
2012-02-27 22:00:23 +01:00
|
|
|
|
2013-01-30 21:01:07 +01:00
|
|
|
// TODO: This is pretty awkward, unify the CSS between Diffusion and
|
|
|
|
// Differential better.
|
|
|
|
require_celerity_resource('differential-core-view-css');
|
|
|
|
|
2013-02-13 23:08:57 +01:00
|
|
|
return phutil_render_tag(
|
2013-01-30 21:01:07 +01:00
|
|
|
'div',
|
|
|
|
array(
|
|
|
|
'id' => $pane_id,
|
|
|
|
),
|
2013-02-13 23:08:57 +01:00
|
|
|
phutil_render_tag(
|
|
|
|
'div',
|
|
|
|
array(
|
|
|
|
'class' => 'differential-add-comment-panel',
|
|
|
|
),
|
2013-01-30 21:01:07 +01:00
|
|
|
id(new PhabricatorAnchorView())
|
|
|
|
->setAnchorName('comment')
|
|
|
|
->setNavigationMarker(true)
|
2013-02-13 23:08:57 +01:00
|
|
|
->render().
|
|
|
|
$panel->render().
|
2013-01-30 21:01:07 +01:00
|
|
|
$preview_panel));
|
2012-02-25 00:04:53 +01:00
|
|
|
}
|
|
|
|
|
2012-03-26 18:44:06 +02:00
|
|
|
/**
|
|
|
|
* Return a map of available audit actions for rendering into a <select />.
|
|
|
|
* This shows the user valid actions, and does not show nonsense/invalid
|
|
|
|
* actions (like closing an already-closed commit, or resigning from a commit
|
|
|
|
* you have no association with).
|
|
|
|
*/
|
|
|
|
private function getAuditActions(
|
|
|
|
PhabricatorRepositoryCommit $commit,
|
|
|
|
array $audit_requests) {
|
2012-04-04 01:22:31 +02:00
|
|
|
assert_instances_of($audit_requests, 'PhabricatorRepositoryAuditRequest');
|
2012-03-26 18:44:06 +02:00
|
|
|
$user = $this->getRequest()->getUser();
|
|
|
|
|
|
|
|
$user_is_author = ($commit->getAuthorPHID() == $user->getPHID());
|
|
|
|
|
|
|
|
$user_request = null;
|
|
|
|
foreach ($audit_requests as $audit_request) {
|
|
|
|
if ($audit_request->getAuditorPHID() == $user->getPHID()) {
|
|
|
|
$user_request = $audit_request;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
$actions = array();
|
|
|
|
$actions[PhabricatorAuditActionConstants::COMMENT] = true;
|
2012-04-23 22:50:04 +02:00
|
|
|
$actions[PhabricatorAuditActionConstants::ADD_CCS] = true;
|
|
|
|
$actions[PhabricatorAuditActionConstants::ADD_AUDITORS] = true;
|
2012-03-26 18:44:06 +02:00
|
|
|
|
|
|
|
// We allow you to accept your own commits. A use case here is that you
|
|
|
|
// notice an issue with your own commit and "Raise Concern" as an indicator
|
|
|
|
// to other auditors that you're on top of the issue, then later resolve it
|
|
|
|
// and "Accept". You can not accept on behalf of projects or packages,
|
|
|
|
// however.
|
|
|
|
$actions[PhabricatorAuditActionConstants::ACCEPT] = true;
|
|
|
|
$actions[PhabricatorAuditActionConstants::CONCERN] = true;
|
|
|
|
|
|
|
|
|
|
|
|
// To resign, a user must have authority on some request and not be the
|
|
|
|
// commit's author.
|
|
|
|
if (!$user_is_author) {
|
|
|
|
$may_resign = false;
|
2012-05-07 22:37:41 +02:00
|
|
|
|
|
|
|
$authority_map = array_fill_keys($this->auditAuthorityPHIDs, true);
|
2012-03-26 18:44:06 +02:00
|
|
|
foreach ($audit_requests as $request) {
|
2012-05-07 22:37:41 +02:00
|
|
|
if (empty($authority_map[$request->getAuditorPHID()])) {
|
2012-03-26 18:44:06 +02:00
|
|
|
continue;
|
|
|
|
}
|
|
|
|
$may_resign = true;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
|
|
|
|
// If the user has already resigned, don't show "Resign...".
|
|
|
|
$status_resigned = PhabricatorAuditStatusConstants::RESIGNED;
|
|
|
|
if ($user_request) {
|
|
|
|
if ($user_request->getAuditStatus() == $status_resigned) {
|
|
|
|
$may_resign = false;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
if ($may_resign) {
|
|
|
|
$actions[PhabricatorAuditActionConstants::RESIGN] = true;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
$status_concern = PhabricatorAuditCommitStatusConstants::CONCERN_RAISED;
|
|
|
|
$concern_raised = ($commit->getAuditStatus() == $status_concern);
|
2013-01-19 02:53:52 +01:00
|
|
|
$can_close_option = PhabricatorEnv::getEnvConfig(
|
|
|
|
'audit.can-author-close-audit');
|
|
|
|
if ($can_close_option && $user_is_author && $concern_raised) {
|
2012-03-26 18:44:06 +02:00
|
|
|
$actions[PhabricatorAuditActionConstants::CLOSE] = true;
|
|
|
|
}
|
|
|
|
|
|
|
|
foreach ($actions as $constant => $ignored) {
|
|
|
|
$actions[$constant] =
|
|
|
|
PhabricatorAuditActionConstants::getActionName($constant);
|
|
|
|
}
|
|
|
|
|
|
|
|
return $actions;
|
|
|
|
}
|
|
|
|
|
2012-03-23 23:32:26 +01:00
|
|
|
private function buildMergesTable(PhabricatorRepositoryCommit $commit) {
|
|
|
|
$drequest = $this->getDiffusionRequest();
|
|
|
|
|
|
|
|
$limit = 50;
|
|
|
|
|
|
|
|
$merge_query = DiffusionMergedCommitsQuery::newFromDiffusionRequest(
|
|
|
|
$drequest);
|
|
|
|
$merge_query->setLimit($limit + 1);
|
|
|
|
$merges = $merge_query->loadMergedCommits();
|
|
|
|
|
|
|
|
if (!$merges) {
|
|
|
|
return null;
|
|
|
|
}
|
|
|
|
|
|
|
|
$caption = null;
|
|
|
|
if (count($merges) > $limit) {
|
|
|
|
$merges = array_slice($merges, 0, $limit);
|
|
|
|
$caption =
|
|
|
|
"This commit merges more than {$limit} changes. Only the first ".
|
|
|
|
"{$limit} are shown.";
|
|
|
|
}
|
|
|
|
|
|
|
|
$history_table = new DiffusionHistoryTableView();
|
2012-12-21 00:04:20 +01:00
|
|
|
$history_table->setUser($this->getRequest()->getUser());
|
2012-03-23 23:32:26 +01:00
|
|
|
$history_table->setDiffusionRequest($drequest);
|
|
|
|
$history_table->setHistory($merges);
|
2012-06-23 01:52:08 +02:00
|
|
|
$history_table->loadRevisions();
|
2012-03-23 23:32:26 +01:00
|
|
|
|
|
|
|
$phids = $history_table->getRequiredHandlePHIDs();
|
2012-09-05 04:02:56 +02:00
|
|
|
$handles = $this->loadViewerHandles($phids);
|
2012-03-23 23:32:26 +01:00
|
|
|
$history_table->setHandles($handles);
|
|
|
|
|
|
|
|
$panel = new AphrontPanelView();
|
|
|
|
$panel->setHeader('Merged Changes');
|
|
|
|
$panel->setCaption($caption);
|
|
|
|
$panel->appendChild($history_table);
|
2013-01-16 00:05:15 +01:00
|
|
|
$panel->setNoBackground();
|
2012-03-23 23:32:26 +01:00
|
|
|
|
|
|
|
return $panel;
|
|
|
|
}
|
|
|
|
|
2012-03-30 23:12:10 +02:00
|
|
|
private function renderHeadsupActionList(
|
2012-08-08 19:03:41 +02:00
|
|
|
PhabricatorRepositoryCommit $commit,
|
|
|
|
PhabricatorRepository $repository) {
|
2012-03-30 23:12:10 +02:00
|
|
|
|
2012-05-02 22:43:45 +02:00
|
|
|
$request = $this->getRequest();
|
|
|
|
$user = $request->getUser();
|
2012-03-30 23:12:10 +02:00
|
|
|
|
upgrade diffusion to use modern header UI and fix a few quirks
Summary:
upgrades are CrumbsView, HeaderView, PropertyListView, and ActionListView. I had to modify CrumbsView stuff a bit to handle the "advanced" diffusion crumbs.
Quirks fixed include making file tree view show up in diffusion, the page not have extra space when the file tree is hidden, links no longer breaking once you visit files (since without the change the files always got "/" appended and thus 404'd), and a differential quirk where it read "next step:" and that colon is a no no,
Test Plan: played around in diffusion and differential
Reviewers: epriestley
Reviewed By: epriestley
CC: aran, Korvin, chad
Maniphest Tasks: T2048, T2178
Differential Revision: https://secure.phabricator.com/D4169
2012-12-13 02:50:42 +01:00
|
|
|
$actions = id(new PhabricatorActionListView())
|
|
|
|
->setUser($user)
|
|
|
|
->setObject($commit);
|
2012-03-30 23:12:10 +02:00
|
|
|
|
2012-08-08 19:03:41 +02:00
|
|
|
// TODO -- integrate permissions into whether or not this action is shown
|
|
|
|
$uri = '/diffusion/'.$repository->getCallSign().'/commit/'.
|
|
|
|
$commit->getCommitIdentifier().'/edit/';
|
upgrade diffusion to use modern header UI and fix a few quirks
Summary:
upgrades are CrumbsView, HeaderView, PropertyListView, and ActionListView. I had to modify CrumbsView stuff a bit to handle the "advanced" diffusion crumbs.
Quirks fixed include making file tree view show up in diffusion, the page not have extra space when the file tree is hidden, links no longer breaking once you visit files (since without the change the files always got "/" appended and thus 404'd), and a differential quirk where it read "next step:" and that colon is a no no,
Test Plan: played around in diffusion and differential
Reviewers: epriestley
Reviewed By: epriestley
CC: aran, Korvin, chad
Maniphest Tasks: T2048, T2178
Differential Revision: https://secure.phabricator.com/D4169
2012-12-13 02:50:42 +01:00
|
|
|
|
|
|
|
$action = id(new PhabricatorActionView())
|
|
|
|
->setName('Edit Commit')
|
|
|
|
->setHref($uri)
|
|
|
|
->setIcon('edit');
|
|
|
|
$actions->addAction($action);
|
2012-03-30 23:12:10 +02:00
|
|
|
|
|
|
|
require_celerity_resource('phabricator-object-selector-css');
|
|
|
|
require_celerity_resource('javelin-behavior-phabricator-object-selector');
|
|
|
|
|
2012-11-02 01:36:04 +01:00
|
|
|
if (PhabricatorEnv::getEnvConfig('maniphest.enabled')) {
|
upgrade diffusion to use modern header UI and fix a few quirks
Summary:
upgrades are CrumbsView, HeaderView, PropertyListView, and ActionListView. I had to modify CrumbsView stuff a bit to handle the "advanced" diffusion crumbs.
Quirks fixed include making file tree view show up in diffusion, the page not have extra space when the file tree is hidden, links no longer breaking once you visit files (since without the change the files always got "/" appended and thus 404'd), and a differential quirk where it read "next step:" and that colon is a no no,
Test Plan: played around in diffusion and differential
Reviewers: epriestley
Reviewed By: epriestley
CC: aran, Korvin, chad
Maniphest Tasks: T2048, T2178
Differential Revision: https://secure.phabricator.com/D4169
2012-12-13 02:50:42 +01:00
|
|
|
$action = id(new PhabricatorActionView())
|
|
|
|
->setName('Edit Maniphest Tasks')
|
|
|
|
->setIcon('attach')
|
|
|
|
->setHref('/search/attach/'.$commit->getPHID().'/TASK/edge/')
|
|
|
|
->setWorkflow(true);
|
|
|
|
$actions->addAction($action);
|
2012-11-02 01:36:04 +01:00
|
|
|
}
|
2012-03-30 23:12:10 +02:00
|
|
|
|
|
|
|
if ($user->getIsAdmin()) {
|
upgrade diffusion to use modern header UI and fix a few quirks
Summary:
upgrades are CrumbsView, HeaderView, PropertyListView, and ActionListView. I had to modify CrumbsView stuff a bit to handle the "advanced" diffusion crumbs.
Quirks fixed include making file tree view show up in diffusion, the page not have extra space when the file tree is hidden, links no longer breaking once you visit files (since without the change the files always got "/" appended and thus 404'd), and a differential quirk where it read "next step:" and that colon is a no no,
Test Plan: played around in diffusion and differential
Reviewers: epriestley
Reviewed By: epriestley
CC: aran, Korvin, chad
Maniphest Tasks: T2048, T2178
Differential Revision: https://secure.phabricator.com/D4169
2012-12-13 02:50:42 +01:00
|
|
|
$action = id(new PhabricatorActionView())
|
|
|
|
->setName('MetaMTA Transcripts')
|
|
|
|
->setIcon('file')
|
|
|
|
->setHref('/mail/?phid='.$commit->getPHID());
|
|
|
|
$actions->addAction($action);
|
|
|
|
}
|
|
|
|
|
|
|
|
$action = id(new PhabricatorActionView())
|
|
|
|
->setName('Herald Transcripts')
|
|
|
|
->setIcon('file')
|
|
|
|
->setHref('/herald/transcript/?phid='.$commit->getPHID())
|
|
|
|
->setWorkflow(true);
|
|
|
|
$actions->addAction($action);
|
|
|
|
|
|
|
|
$action = id(new PhabricatorActionView())
|
|
|
|
->setName('Download Raw Diff')
|
|
|
|
->setHref($request->getRequestURI()->alter('diff', true))
|
|
|
|
->setIcon('download');
|
|
|
|
$actions->addAction($action);
|
2012-03-30 23:12:10 +02:00
|
|
|
|
upgrade diffusion to use modern header UI and fix a few quirks
Summary:
upgrades are CrumbsView, HeaderView, PropertyListView, and ActionListView. I had to modify CrumbsView stuff a bit to handle the "advanced" diffusion crumbs.
Quirks fixed include making file tree view show up in diffusion, the page not have extra space when the file tree is hidden, links no longer breaking once you visit files (since without the change the files always got "/" appended and thus 404'd), and a differential quirk where it read "next step:" and that colon is a no no,
Test Plan: played around in diffusion and differential
Reviewers: epriestley
Reviewed By: epriestley
CC: aran, Korvin, chad
Maniphest Tasks: T2048, T2178
Differential Revision: https://secure.phabricator.com/D4169
2012-12-13 02:50:42 +01:00
|
|
|
return $actions;
|
2012-03-30 23:12:10 +02:00
|
|
|
}
|
|
|
|
|
2012-04-25 16:21:03 +02:00
|
|
|
private function buildRefs(DiffusionRequest $request) {
|
|
|
|
// Not turning this into a proper Query class since it's pretty simple,
|
|
|
|
// one-off, and Git-specific.
|
|
|
|
|
|
|
|
$type_git = PhabricatorRepositoryType::REPOSITORY_TYPE_GIT;
|
|
|
|
|
|
|
|
$repository = $request->getRepository();
|
|
|
|
if ($repository->getVersionControlSystem() != $type_git) {
|
|
|
|
return null;
|
|
|
|
}
|
|
|
|
|
|
|
|
list($stdout) = $repository->execxLocalCommand(
|
|
|
|
'log --format=%s -n 1 %s --',
|
|
|
|
'%d',
|
|
|
|
$request->getCommit());
|
|
|
|
|
2012-06-29 23:14:02 +02:00
|
|
|
// %d, gives a weird output format
|
|
|
|
// similar to (remote/one, remote/two, remote/three)
|
|
|
|
$refs = trim($stdout, "() \n");
|
2012-06-30 15:26:23 +02:00
|
|
|
if (!$refs) {
|
|
|
|
return null;
|
|
|
|
}
|
2012-06-30 15:43:27 +02:00
|
|
|
$refs = explode(',', $refs);
|
|
|
|
$refs = array_map('trim', $refs);
|
2012-06-30 15:26:23 +02:00
|
|
|
|
2012-06-29 23:14:02 +02:00
|
|
|
$ref_links = array();
|
|
|
|
foreach ($refs as $ref) {
|
2013-01-18 03:43:35 +01:00
|
|
|
$ref_links[] = phutil_tag(
|
2012-06-29 23:14:02 +02:00
|
|
|
'a',
|
|
|
|
array(
|
|
|
|
'href' => $request->generateURI(
|
|
|
|
array(
|
|
|
|
'action' => 'browse',
|
|
|
|
'branch' => $ref,
|
|
|
|
)),
|
|
|
|
),
|
2013-01-18 03:43:35 +01:00
|
|
|
$ref);
|
2012-06-29 23:14:02 +02:00
|
|
|
}
|
2013-01-31 18:08:02 +01:00
|
|
|
|
2013-02-13 23:08:57 +01:00
|
|
|
return array_interleave(', ', $ref_links);
|
2012-04-25 16:21:03 +02:00
|
|
|
}
|
|
|
|
|
2012-05-02 22:43:45 +02:00
|
|
|
private function buildRawDiffResponse(DiffusionRequest $drequest) {
|
|
|
|
$raw_query = DiffusionRawDiffQuery::newFromDiffusionRequest($drequest);
|
|
|
|
$raw_diff = $raw_query->loadRawDiff();
|
|
|
|
|
2012-07-09 19:38:25 +02:00
|
|
|
$file = PhabricatorFile::buildFromFileDataOrHash(
|
|
|
|
$raw_diff,
|
|
|
|
array(
|
|
|
|
'name' => $drequest->getCommit().'.diff',
|
|
|
|
));
|
2012-05-02 22:43:45 +02:00
|
|
|
|
|
|
|
return id(new AphrontRedirectResponse())->setURI($file->getBestURI());
|
|
|
|
}
|
|
|
|
|
2011-03-11 18:34:22 +01:00
|
|
|
}
|