Page MenuHomePhabricator
Files F15458820

D21864.id52131.diff
No OneTemporary
Actions

D21864.id52131.diff
View Options

diff --git a/scripts/ssh/ssh-auth.php b/scripts/ssh/ssh-auth.php
--- a/scripts/ssh/ssh-auth.php
+++ b/scripts/ssh/ssh-auth.php
@@ -36,7 +36,7 @@
$authstruct = null;
-if (strlen($authstruct_raw)) {
+if ($authstruct_raw !== null && strlen($authstruct_raw)) {
try {
$authstruct = phutil_json_decode($authstruct_raw);
} catch (Exception $ex) {
@@ -135,7 +135,7 @@
$cmd = csprintf('%s %Ls', $bin, $key_argv);
- if (strlen($instance)) {
+ if ($instance !== null && strlen($instance)) {
$cmd = csprintf('PHABRICATOR_INSTANCE=%s %C', $instance, $cmd);
}
diff --git a/src/applications/almanac/util/AlmanacKeys.php b/src/applications/almanac/util/AlmanacKeys.php
--- a/src/applications/almanac/util/AlmanacKeys.php
+++ b/src/applications/almanac/util/AlmanacKeys.php
@@ -58,7 +58,7 @@
public static function getClusterSSHUser() {
$username = PhabricatorEnv::getEnvConfig('diffusion.ssh-user');
- if (strlen($username)) {
+ if ($username !== null && strlen($username)) {
return $username;
}
diff --git a/src/applications/config/controller/module/PhabricatorConfigModuleController.php b/src/applications/config/controller/module/PhabricatorConfigModuleController.php
--- a/src/applications/config/controller/module/PhabricatorConfigModuleController.php
+++ b/src/applications/config/controller/module/PhabricatorConfigModuleController.php
@@ -9,7 +9,7 @@
$all_modules = PhabricatorConfigModule::getAllModules();
- if (!strlen($key)) {
+ if ($key === null || !strlen($key)) {
$key = head_key($all_modules);
}
diff --git a/src/applications/differential/storage/DifferentialChangeset.php b/src/applications/differential/storage/DifferentialChangeset.php
--- a/src/applications/differential/storage/DifferentialChangeset.php
+++ b/src/applications/differential/storage/DifferentialChangeset.php
@@ -325,7 +325,7 @@
public function getOldStatePathVector() {
$path = $this->getOldFile();
- if (!strlen($path)) {
+ if ($path === null || !strlen($path)) {
$path = $this->getFilename();
}
diff --git a/src/applications/diffusion/conduit/DiffusionBranchQueryConduitAPIMethod.php b/src/applications/diffusion/conduit/DiffusionBranchQueryConduitAPIMethod.php
--- a/src/applications/diffusion/conduit/DiffusionBranchQueryConduitAPIMethod.php
+++ b/src/applications/diffusion/conduit/DiffusionBranchQueryConduitAPIMethod.php
@@ -30,7 +30,7 @@
$repository = $drequest->getRepository();
$contains = $request->getValue('contains');
- if (strlen($contains)) {
+ if ($contains !== null && strlen($contains)) {
// See PHI958 (and, earlier, PHI720). If "patterns" are provided, pass
// them to "git branch ..." to let callers test for reachability from
@@ -80,7 +80,7 @@
->setRepository($repository);
$contains = $request->getValue('contains');
- if (strlen($contains)) {
+ if ($contains !== null && strlen($contains)) {
$query->withContainsCommit($contains);
}
diff --git a/src/applications/diffusion/conduit/DiffusionBrowseQueryConduitAPIMethod.php b/src/applications/diffusion/conduit/DiffusionBrowseQueryConduitAPIMethod.php
--- a/src/applications/diffusion/conduit/DiffusionBrowseQueryConduitAPIMethod.php
+++ b/src/applications/diffusion/conduit/DiffusionBrowseQueryConduitAPIMethod.php
@@ -282,8 +282,13 @@
$results = array();
- $match_against = trim($path, '/');
- $match_len = strlen($match_against);
+ if ($path !== null) {
+ $match_against = trim($path, '/');
+ $match_len = strlen($match_against);
+ } else {
+ $match_against = '';
+ $match_len = 0;
+ }
// For the root, don't trim. For other paths, trim the "/" after we match.
// We need this because Mercurial's canonical paths have no leading "/",
@@ -295,7 +300,7 @@
if (strncmp($path, $match_against, $match_len)) {
continue;
}
- if (!strlen($path)) {
+ if ($path === null || !strlen($path)) {
continue;
}
$remainder = substr($path, $trim_len);
diff --git a/src/applications/diffusion/conduit/DiffusionHistoryQueryConduitAPIMethod.php b/src/applications/diffusion/conduit/DiffusionHistoryQueryConduitAPIMethod.php
--- a/src/applications/diffusion/conduit/DiffusionHistoryQueryConduitAPIMethod.php
+++ b/src/applications/diffusion/conduit/DiffusionHistoryQueryConduitAPIMethod.php
@@ -47,14 +47,21 @@
$against_hash = $request->getValue('against');
$path = $request->getValue('path');
- if (!strlen($path)) {
+ if ($path === null || !strlen($path)) {
$path = null;
}
$offset = $request->getValue('offset');
+ if ($offset === null) {
+ $offset = 0;
+ }
+
$limit = $request->getValue('limit');
+ if ($limit === null) {
+ $limit = 0;
+ }
- if (strlen($against_hash)) {
+ if ($against_hash !== null && strlen($against_hash)) {
$commit_range = "{$against_hash}..{$commit_hash}";
} else {
$commit_range = $commit_hash;
@@ -115,6 +122,13 @@
$path = DiffusionPathIDQuery::normalizePath($path);
$path = ltrim($path, '/');
+ if ($offset === null) {
+ $offset = 0;
+ }
+ if ($limit === null) {
+ $limit = 0;
+ }
+
// NOTE: Older versions of Mercurial give different results for these
// commands (see T1268):
//
diff --git a/src/applications/diffusion/controller/DiffusionBranchTableController.php b/src/applications/diffusion/controller/DiffusionBranchTableController.php
--- a/src/applications/diffusion/controller/DiffusionBranchTableController.php
+++ b/src/applications/diffusion/controller/DiffusionBranchTableController.php
@@ -26,7 +26,7 @@
);
$contains = $drequest->getSymbolicCommit();
- if (strlen($contains)) {
+ if ($contains !== null && strlen($contains)) {
$params['contains'] = $contains;
}
diff --git a/src/applications/diffusion/controller/DiffusionBrowseController.php b/src/applications/diffusion/controller/DiffusionBrowseController.php
--- a/src/applications/diffusion/controller/DiffusionBrowseController.php
+++ b/src/applications/diffusion/controller/DiffusionBrowseController.php
@@ -22,7 +22,7 @@
// list.
$grep = $request->getStr('grep');
- if (strlen($grep)) {
+ if (phutil_nonempty_string($grep)) {
return $this->browseSearch();
}
@@ -290,6 +290,11 @@
$header = $this->buildHeaderView($drequest);
$header->setHeaderIcon('fa-folder-open');
+ $title = '/';
+ if ($drequest->getPath() !== null) {
+ $title = nonempty(basename($drequest->getPath()), '/');
+ }
+
$empty_result = null;
$browse_panel = null;
if (!$results->isValidResults()) {
@@ -303,7 +308,6 @@
->setPaths($results->getPaths())
->setUser($request->getUser());
- $title = nonempty(basename($drequest->getPath()), '/');
$icon = 'fa-folder-open';
$browse_header = $this->buildPanelHeaderView($title, $icon);
@@ -351,7 +355,7 @@
return $this->newPage()
->setTitle(array(
- nonempty(basename($drequest->getPath()), '/'),
+ $title,
$repository->getDisplayName(),
))
->setCrumbs($crumbs)
diff --git a/src/applications/diffusion/controller/DiffusionCommitController.php b/src/applications/diffusion/controller/DiffusionCommitController.php
--- a/src/applications/diffusion/controller/DiffusionCommitController.php
+++ b/src/applications/diffusion/controller/DiffusionCommitController.php
@@ -27,8 +27,11 @@
// If this page is being accessed via "/source/xyz/commit/...", redirect
// to the canonical URI.
- $has_callsign = strlen($request->getURIData('repositoryCallsign'));
- $has_id = strlen($request->getURIData('repositoryID'));
+ $repo_callsign = $request->getURIData('repositoryCallsign');
+ $has_callsign = $repo_callsign !== null && strlen($repo_callsign);
+ $repo_id = $request->getURIData('repositoryID');
+ $has_id = $repo_id !== null && strlen($repo_id);
+
if (!$has_callsign && !$has_id) {
$canonical_uri = $repository->getCommitURI($commit_identifier);
return id(new AphrontRedirectResponse())
@@ -922,7 +925,7 @@
private function linkBugtraq($corpus) {
$url = PhabricatorEnv::getEnvConfig('bugtraq.url');
- if (!strlen($url)) {
+ if ($url !== null && !strlen($url)) {
return $corpus;
}
diff --git a/src/applications/diffusion/controller/DiffusionController.php b/src/applications/diffusion/controller/DiffusionController.php
--- a/src/applications/diffusion/controller/DiffusionController.php
+++ b/src/applications/diffusion/controller/DiffusionController.php
@@ -265,7 +265,10 @@
protected function renderPathLinks(DiffusionRequest $drequest, $action) {
$path = $drequest->getPath();
- $path_parts = array_filter(explode('/', trim($path, '/')));
+ $path_parts = array();
+ if ($path !== null && strlen($path)) {
+ $path_parts = array_filter(explode('/', trim($path, '/')));
+ }
$divider = phutil_tag(
'span',
diff --git a/src/applications/diffusion/controller/DiffusionHistoryController.php b/src/applications/diffusion/controller/DiffusionHistoryController.php
--- a/src/applications/diffusion/controller/DiffusionHistoryController.php
+++ b/src/applications/diffusion/controller/DiffusionHistoryController.php
@@ -50,7 +50,7 @@
// ancestors appropriately, but this would currrently be prohibitively
// expensive in the general case.
- $show_graph = !strlen($drequest->getPath());
+ $show_graph = $drequest->getPath() === null || !strlen($drequest->getPath());
if ($show_graph) {
$history_list
->setParents($history_results['parents'])
@@ -98,7 +98,7 @@
$viewer = $this->getViewer();
$repository = $drequest->getRepository();
- $no_path = !strlen($drequest->getPath());
+ $no_path = $drequest->getPath() === null || !strlen($drequest->getPath());
if ($no_path) {
$header_text = pht('History');
} else {
diff --git a/src/applications/diffusion/controller/DiffusionRepositoryController.php b/src/applications/diffusion/controller/DiffusionRepositoryController.php
--- a/src/applications/diffusion/controller/DiffusionRepositoryController.php
+++ b/src/applications/diffusion/controller/DiffusionRepositoryController.php
@@ -490,7 +490,7 @@
} else {
return $this->renderStatusMessage(
pht('Unable to Retrieve Paths'),
- $browse_exception->getMessage());
+ $browse_exception->getTraceAsString());
}
}
diff --git a/src/applications/diffusion/controller/DiffusionRepositoryManagePanelsController.php b/src/applications/diffusion/controller/DiffusionRepositoryManagePanelsController.php
--- a/src/applications/diffusion/controller/DiffusionRepositoryManagePanelsController.php
+++ b/src/applications/diffusion/controller/DiffusionRepositoryManagePanelsController.php
@@ -40,7 +40,7 @@
}
$selected = $request->getURIData('panel');
- if (!strlen($selected)) {
+ if ($selected === null || !strlen($selected)) {
$selected = head_key($panels);
}
diff --git a/src/applications/diffusion/controller/DiffusionTagListController.php b/src/applications/diffusion/controller/DiffusionTagListController.php
--- a/src/applications/diffusion/controller/DiffusionTagListController.php
+++ b/src/applications/diffusion/controller/DiffusionTagListController.php
@@ -25,7 +25,9 @@
'offset' => $pager->getOffset(),
);
- if (strlen($drequest->getSymbolicCommit())) {
+ if ($drequest->getSymbolicCommit() !== null
+ && strlen($drequest->getSymbolicCommit())) {
+
$is_commit = true;
$params['commit'] = $drequest->getSymbolicCommit();
} else {
diff --git a/src/applications/diffusion/data/DiffusionCommitRef.php b/src/applications/diffusion/data/DiffusionCommitRef.php
--- a/src/applications/diffusion/data/DiffusionCommitRef.php
+++ b/src/applications/diffusion/data/DiffusionCommitRef.php
@@ -131,6 +131,13 @@
}
private function formatUser($name, $email) {
+ if ($name === null) {
+ $name = '';
+ }
+ if ($email === null) {
+ $email = '';
+ }
+
if (strlen($name) && strlen($email)) {
return "{$name} <{$email}>";
} else if (strlen($email)) {
diff --git a/src/applications/diffusion/document/DiffusionDocumentRenderingEngine.php b/src/applications/diffusion/document/DiffusionDocumentRenderingEngine.php
--- a/src/applications/diffusion/document/DiffusionDocumentRenderingEngine.php
+++ b/src/applications/diffusion/document/DiffusionDocumentRenderingEngine.php
@@ -87,7 +87,7 @@
$ref->setSymbolMetadata($this->getSymbolMetadata());
$coverage = $drequest->loadCoverage();
- if (strlen($coverage)) {
+ if ($coverage !== null && strlen($coverage)) {
$ref->addCoverage($coverage);
}
}
diff --git a/src/applications/diffusion/query/DiffusionFileFutureQuery.php b/src/applications/diffusion/query/DiffusionFileFutureQuery.php
--- a/src/applications/diffusion/query/DiffusionFileFutureQuery.php
+++ b/src/applications/diffusion/query/DiffusionFileFutureQuery.php
@@ -92,7 +92,10 @@
$drequest = $this->getRequest();
- $name = basename($drequest->getPath());
+ $name = '';
+ if ($drequest->getPath() !== null) {
+ $name = basename($drequest->getPath());
+ }
$relative_ttl = phutil_units('48 hours in seconds');
try {
diff --git a/src/applications/diffusion/query/lowlevel/DiffusionLowLevelMercurialPathsQuery.php b/src/applications/diffusion/query/lowlevel/DiffusionLowLevelMercurialPathsQuery.php
--- a/src/applications/diffusion/query/lowlevel/DiffusionLowLevelMercurialPathsQuery.php
+++ b/src/applications/diffusion/query/lowlevel/DiffusionLowLevelMercurialPathsQuery.php
@@ -32,8 +32,12 @@
$hg_paths_command = 'locate --print0 --rev %s -I %s';
}
- $match_against = trim($path, '/');
- $prefix = trim('./'.$match_against, '/');
+ if ($path !== null) {
+ $match_against = trim($path, '/');
+ $prefix = trim('./'.$match_against, '/');
+ } else {
+ $prefix = '.';
+ }
list($entire_manifest) = $repository->execxLocalCommand(
$hg_paths_command,
hgsprintf('%s', $commit),
diff --git a/src/applications/diffusion/query/pathid/DiffusionPathIDQuery.php b/src/applications/diffusion/query/pathid/DiffusionPathIDQuery.php
--- a/src/applications/diffusion/query/pathid/DiffusionPathIDQuery.php
+++ b/src/applications/diffusion/query/pathid/DiffusionPathIDQuery.php
@@ -48,6 +48,10 @@
*/
public static function normalizePath($path) {
+ if ($path === null) {
+ return '/';
+ }
+
// Normalize to single slashes, e.g. "///" => "/".
$path = preg_replace('@[/]{2,}@', '/', $path);
diff --git a/src/applications/diffusion/query/rawdiff/DiffusionGitRawDiffQuery.php b/src/applications/diffusion/query/rawdiff/DiffusionGitRawDiffQuery.php
--- a/src/applications/diffusion/query/rawdiff/DiffusionGitRawDiffQuery.php
+++ b/src/applications/diffusion/query/rawdiff/DiffusionGitRawDiffQuery.php
@@ -35,7 +35,7 @@
}
$path = $drequest->getPath();
- if (!strlen($path)) {
+ if ($path === null || !strlen($path)) {
$path = '.';
}
diff --git a/src/applications/diffusion/request/DiffusionGitRequest.php b/src/applications/diffusion/request/DiffusionGitRequest.php
--- a/src/applications/diffusion/request/DiffusionGitRequest.php
+++ b/src/applications/diffusion/request/DiffusionGitRequest.php
@@ -3,7 +3,7 @@
final class DiffusionGitRequest extends DiffusionRequest {
protected function isStableCommit($symbol) {
- return preg_match('/^[a-f0-9]{40}\z/', $symbol);
+ return $symbol !== null && preg_match('/^[a-f0-9]{40}\z/', $symbol);
}
public function getBranch() {
diff --git a/src/applications/diffusion/request/DiffusionMercurialRequest.php b/src/applications/diffusion/request/DiffusionMercurialRequest.php
--- a/src/applications/diffusion/request/DiffusionMercurialRequest.php
+++ b/src/applications/diffusion/request/DiffusionMercurialRequest.php
@@ -3,7 +3,7 @@
final class DiffusionMercurialRequest extends DiffusionRequest {
protected function isStableCommit($symbol) {
- return preg_match('/^[a-f0-9]{40}\z/', $symbol);
+ return $symbol !== null && preg_match('/^[a-f0-9]{40}\z/', $symbol);
}
public function getBranch() {
diff --git a/src/applications/diffusion/request/DiffusionSvnRequest.php b/src/applications/diffusion/request/DiffusionSvnRequest.php
--- a/src/applications/diffusion/request/DiffusionSvnRequest.php
+++ b/src/applications/diffusion/request/DiffusionSvnRequest.php
@@ -3,7 +3,7 @@
final class DiffusionSvnRequest extends DiffusionRequest {
protected function isStableCommit($symbol) {
- return preg_match('/^[1-9]\d*\z/', $symbol);
+ return $symbol !== null && preg_match('/^[1-9]\d*\z/', $symbol);
}
protected function didInitialize() {
diff --git a/src/applications/diffusion/view/DiffusionBrowseTableView.php b/src/applications/diffusion/view/DiffusionBrowseTableView.php
--- a/src/applications/diffusion/view/DiffusionBrowseTableView.php
+++ b/src/applications/diffusion/view/DiffusionBrowseTableView.php
@@ -15,9 +15,13 @@
$repository = $request->getRepository();
require_celerity_resource('diffusion-css');
- $base_path = trim($request->getPath(), '/');
- if ($base_path) {
- $base_path = $base_path.'/';
+ if ($request->getPath() !== null) {
+ $base_path = trim($request->getPath(), '/');
+ if ($base_path) {
+ $base_path = $base_path.'/';
+ }
+ } else {
+ $base_path = '';
}
$need_pull = array();
diff --git a/src/applications/diffusion/view/DiffusionView.php b/src/applications/diffusion/view/DiffusionView.php
--- a/src/applications/diffusion/view/DiffusionView.php
+++ b/src/applications/diffusion/view/DiffusionView.php
@@ -71,7 +71,7 @@
$display_name = idx($details, 'name');
unset($details['name']);
- if (strlen($display_name)) {
+ if ($display_name !== null && strlen($display_name)) {
$display_name = phutil_tag(
'span',
array(
diff --git a/src/applications/repository/editor/PhabricatorRepositoryEditor.php b/src/applications/repository/editor/PhabricatorRepositoryEditor.php
--- a/src/applications/repository/editor/PhabricatorRepositoryEditor.php
+++ b/src/applications/repository/editor/PhabricatorRepositoryEditor.php
@@ -50,7 +50,7 @@
// If the repository does not have a local path yet, assign it one based
// on its ID. We can't do this earlier because we won't have an ID yet.
$local_path = $object->getLocalPath();
- if (!strlen($local_path)) {
+ if ($local_path === null || !strlen($local_path)) {
$local_key = 'repository.default-local-path';
$local_root = PhabricatorEnv::getEnvConfig($local_key);
diff --git a/src/applications/repository/engine/PhabricatorRepositoryPullEngine.php b/src/applications/repository/engine/PhabricatorRepositoryPullEngine.php
--- a/src/applications/repository/engine/PhabricatorRepositoryPullEngine.php
+++ b/src/applications/repository/engine/PhabricatorRepositoryPullEngine.php
@@ -238,7 +238,7 @@
$identifier = $repository->getPHID();
$instance = PhabricatorEnv::getEnvConfig('cluster.instance');
- if (strlen($instance)) {
+ if ($instance !== null && strlen($instance)) {
$identifier = "{$identifier}:{$instance}";
}
@@ -575,7 +575,7 @@
$ref_rules);
// Empty repositories don't have any refs.
- if (!strlen(rtrim($stdout))) {
+ if ($stdout === null || !strlen(rtrim($stdout))) {
return array();
}
diff --git a/src/applications/repository/management/PhabricatorRepositoryManagementRebuildIdentitiesWorkflow.php b/src/applications/repository/management/PhabricatorRepositoryManagementRebuildIdentitiesWorkflow.php
--- a/src/applications/repository/management/PhabricatorRepositoryManagementRebuildIdentitiesWorkflow.php
+++ b/src/applications/repository/management/PhabricatorRepositoryManagementRebuildIdentitiesWorkflow.php
@@ -220,7 +220,7 @@
$committer_name = $data->getCommitterString();
$committer_phid = $commit->getCommitterIdentityPHID();
- if (strlen($committer_name)) {
+ if (phutil_nonempty_string($committer_name)) {
$committer_identity = $this->getIdentityForCommit(
$commit,
$committer_name);
diff --git a/src/applications/repository/query/PhabricatorRepositoryRefCursorQuery.php b/src/applications/repository/query/PhabricatorRepositoryRefCursorQuery.php
--- a/src/applications/repository/query/PhabricatorRepositoryRefCursorQuery.php
+++ b/src/applications/repository/query/PhabricatorRepositoryRefCursorQuery.php
@@ -132,7 +132,7 @@
$name_hashes);
}
- if (strlen($this->datasourceQuery)) {
+ if ($this->datasourceQuery !== null && strlen($this->datasourceQuery)) {
$where[] = qsprintf(
$conn,
'refNameRaw LIKE %>',
diff --git a/src/applications/repository/storage/PhabricatorRepository.php b/src/applications/repository/storage/PhabricatorRepository.php
--- a/src/applications/repository/storage/PhabricatorRepository.php
+++ b/src/applications/repository/storage/PhabricatorRepository.php
@@ -345,7 +345,7 @@
// Make some reasonable effort to produce reasonable default directory
// names from repository names.
- if (!strlen($name)) {
+ if ($name === null || !strlen($name)) {
$name = $this->getName();
$name = phutil_utf8_strtolower($name);
$name = preg_replace('@[ -/:->]+@', '-', $name);
@@ -721,14 +721,14 @@
$head = idx($params, 'head');
$against = idx($params, 'against');
- if ($req_commit && !strlen($commit)) {
+ if ($req_commit && ($commit === null || !strlen($commit))) {
throw new Exception(
pht(
'Diffusion URI action "%s" requires commit!',
$action));
}
- if ($req_branch && !strlen($branch)) {
+ if ($req_branch && ($branch === null || !strlen($branch))) {
throw new Exception(
pht(
'Diffusion URI action "%s" requires branch!',
@@ -779,20 +779,20 @@
break;
case 'compare':
$uri = $this->getPathURI("/{$action}/");
- if (strlen($head)) {
+ if ($head !== null && strlen($head)) {
$query['head'] = $head;
- } else if (strlen($raw_commit)) {
+ } else if ($raw_commit !== null && strlen($raw_commit)) {
$query['commit'] = $raw_commit;
- } else if (strlen($raw_branch)) {
+ } else if ($raw_branch !== null && strlen($raw_branch)) {
$query['head'] = $raw_branch;
}
- if (strlen($against)) {
+ if ($against !== null && strlen($against)) {
$query['against'] = $against;
}
break;
case 'branch':
- if (strlen($path)) {
+ if ($path != null && strlen($path)) {
$uri = $this->getPathURI("/repository/{$path}");
} else {
$uri = $this->getPathURI('/');
@@ -1160,7 +1160,7 @@
*/
public function getRemoteURIObject() {
$raw_uri = $this->getDetail('remote-uri');
- if (!strlen($raw_uri)) {
+ if ($raw_uri === null || !strlen($raw_uri)) {
return new PhutilURI('');
}
@@ -2818,7 +2818,7 @@
$permanent_rules = $this->getStringListForConduit($permanent_rules);
$default_branch = $this->getDefaultBranch();
- if (!strlen($default_branch)) {
+ if ($default_branch === null || !strlen($default_branch)) {
$default_branch = null;
}
diff --git a/src/applications/repository/storage/PhabricatorRepositoryCommit.php b/src/applications/repository/storage/PhabricatorRepositoryCommit.php
--- a/src/applications/repository/storage/PhabricatorRepositoryCommit.php
+++ b/src/applications/repository/storage/PhabricatorRepositoryCommit.php
@@ -478,7 +478,7 @@
}
$author = $this->getRawAuthorStringForDisplay();
- if (strlen($author)) {
+ if ($author !== null && strlen($author)) {
return DiffusionView::renderName($author);
}
@@ -493,7 +493,7 @@
}
$committer = $this->getRawCommitterStringForDisplay();
- if (strlen($committer)) {
+ if ($committer !== null && strlen($committer)) {
return DiffusionView::renderName($committer);
}
diff --git a/src/applications/repository/storage/PhabricatorRepositoryCommitData.php b/src/applications/repository/storage/PhabricatorRepositoryCommitData.php
--- a/src/applications/repository/storage/PhabricatorRepositoryCommitData.php
+++ b/src/applications/repository/storage/PhabricatorRepositoryCommitData.php
@@ -97,7 +97,7 @@
$ref = $this->getCommitRef();
$author = $ref->getAuthor();
- if (strlen($author)) {
+ if ($author !== null && strlen($author)) {
return $author;
}
@@ -131,7 +131,7 @@
$ref = $this->getCommitRef();
$committer = $ref->getCommitter();
- if (strlen($committer)) {
+ if ($committer !== null && strlen($committer)) {
return $committer;
}
diff --git a/src/applications/repository/storage/PhabricatorRepositoryPushLog.php b/src/applications/repository/storage/PhabricatorRepositoryPushLog.php
--- a/src/applications/repository/storage/PhabricatorRepositoryPushLog.php
+++ b/src/applications/repository/storage/PhabricatorRepositoryPushLog.php
@@ -158,6 +158,9 @@
}
public function getRefName() {
+ if ($this->getRefNameRaw() === null) {
+ return null;
+ }
return $this->getUTF8StringFromStorage(
$this->getRefNameRaw(),
$this->getRefNameEncoding());
@@ -175,6 +178,9 @@
if ($this->getRepository()->isSVN()) {
return $this->getRefOld();
}
+ if ($this->getRefOld() === null) {
+ return null;
+ }
return substr($this->getRefOld(), 0, 12);
}
diff --git a/src/applications/repository/storage/PhabricatorRepositoryURI.php b/src/applications/repository/storage/PhabricatorRepositoryURI.php
--- a/src/applications/repository/storage/PhabricatorRepositoryURI.php
+++ b/src/applications/repository/storage/PhabricatorRepositoryURI.php
@@ -752,7 +752,7 @@
// without requiring an index rebuild.
$ssh_host = PhabricatorEnv::getEnvConfig('diffusion.ssh-host');
- if (strlen($ssh_host)) {
+ if ($ssh_host !== null && strlen($ssh_host)) {
$domain_map['<ssh-host>'] = $ssh_host;
}
diff --git a/src/applications/repository/worker/commitmessageparser/PhabricatorRepositoryCommitMessageParserWorker.php b/src/applications/repository/worker/commitmessageparser/PhabricatorRepositoryCommitMessageParserWorker.php
--- a/src/applications/repository/worker/commitmessageparser/PhabricatorRepositoryCommitMessageParserWorker.php
+++ b/src/applications/repository/worker/commitmessageparser/PhabricatorRepositoryCommitMessageParserWorker.php
@@ -36,7 +36,7 @@
$author = $ref->getAuthor();
$committer = $ref->getCommitter();
- $has_committer = (bool)strlen($committer);
+ $has_committer = $committer !== null && (bool)strlen($committer);
$identity_engine = id(new DiffusionRepositoryIdentityEngine())
->setViewer($viewer)
@@ -50,7 +50,12 @@
// NULL to the empty string and merge the flows.
$author = phutil_string_cast($author);
- $author_identity = $identity_engine->newResolvedIdentity($author);
+ try {
+ $author_identity = $identity_engine->newResolvedIdentity($author);
+ } catch (Exception $e) {
+ phlog('Failed identity for '.$author);
+ throw $e;
+ }
if ($has_committer) {
$committer_identity = $identity_engine->newResolvedIdentity($committer);
diff --git a/src/applications/settings/panel/PhabricatorExternalEditorSettingsPanel.php b/src/applications/settings/panel/PhabricatorExternalEditorSettingsPanel.php
--- a/src/applications/settings/panel/PhabricatorExternalEditorSettingsPanel.php
+++ b/src/applications/settings/panel/PhabricatorExternalEditorSettingsPanel.php
@@ -39,7 +39,7 @@
$viewer = $this->getViewer();
$pattern = $viewer->getUserSetting(PhabricatorEditorSetting::SETTINGKEY);
- if (!strlen($pattern)) {
+ if ($pattern === null || !strlen($pattern)) {
return null;
}
diff --git a/src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php b/src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php
--- a/src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php
+++ b/src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php
@@ -2363,7 +2363,9 @@
// Here, we don't care about processing only new mentions after an edit
// because there is no way for an object to ever "unmention" itself on
// another object, so we can ignore the old value.
- $engine->markupText($change->getNewValue());
+ if ($change->getNewValue() !== null) {
+ $engine->markupText($change->getNewValue());
+ }
$mentioned_phids += $engine->getTextMetadata(
PhabricatorObjectRemarkupRule::KEY_MENTIONED_OBJECTS,
diff --git a/src/infrastructure/contentsource/PhabricatorUnknownContentSource.php b/src/infrastructure/contentsource/PhabricatorUnknownContentSource.php
--- a/src/infrastructure/contentsource/PhabricatorUnknownContentSource.php
+++ b/src/infrastructure/contentsource/PhabricatorUnknownContentSource.php
@@ -7,7 +7,7 @@
public function getSourceName() {
$source = $this->getSource();
- if (strlen($source)) {
+ if ($source !== null && strlen($source)) {
return pht('Unknown ("%s")', $source);
} else {
return pht('Unknown');
diff --git a/src/infrastructure/customfield/standard/PhabricatorStandardCustomFieldRemarkup.php b/src/infrastructure/customfield/standard/PhabricatorStandardCustomFieldRemarkup.php
--- a/src/infrastructure/customfield/standard/PhabricatorStandardCustomFieldRemarkup.php
+++ b/src/infrastructure/customfield/standard/PhabricatorStandardCustomFieldRemarkup.php
@@ -30,7 +30,7 @@
public function renderPropertyViewValue(array $handles) {
$value = $this->getFieldValue();
- if (!strlen($value)) {
+ if ($value === null || !strlen($value)) {
return null;
}
diff --git a/src/infrastructure/diff/view/PHUIDiffTableOfContentsItemView.php b/src/infrastructure/diff/view/PHUIDiffTableOfContentsItemView.php
--- a/src/infrastructure/diff/view/PHUIDiffTableOfContentsItemView.php
+++ b/src/infrastructure/diff/view/PHUIDiffTableOfContentsItemView.php
@@ -139,7 +139,7 @@
$not_applicable = '-';
$coverage = $this->getCoverage();
- if (!strlen($coverage)) {
+ if ($coverage === null || !strlen($coverage)) {
return $not_applicable;
}
@@ -157,7 +157,7 @@
$not_applicable = '-';
$coverage = $this->getCoverage();
- if (!strlen($coverage)) {
+ if ($coverage === null || !strlen($coverage)) {
return $not_applicable;
}
diff --git a/src/infrastructure/editor/PhabricatorEditorURIEngine.php b/src/infrastructure/editor/PhabricatorEditorURIEngine.php
--- a/src/infrastructure/editor/PhabricatorEditorURIEngine.php
+++ b/src/infrastructure/editor/PhabricatorEditorURIEngine.php
@@ -16,7 +16,7 @@
$pattern = $viewer->getUserSetting(PhabricatorEditorSetting::SETTINGKEY);
- if (!strlen(trim($pattern))) {
+ if ($pattern === null || !strlen(trim($pattern))) {
return null;
}
diff --git a/src/infrastructure/storage/lisk/PhabricatorLiskDAO.php b/src/infrastructure/storage/lisk/PhabricatorLiskDAO.php
--- a/src/infrastructure/storage/lisk/PhabricatorLiskDAO.php
+++ b/src/infrastructure/storage/lisk/PhabricatorLiskDAO.php
@@ -42,7 +42,7 @@
if (!strlen($namespace)) {
$namespace = self::getDefaultStorageNamespace();
}
- if (!strlen($namespace)) {
+ if ($namespace === null || !strlen($namespace)) {
throw new Exception(pht('No storage namespace configured!'));
}
return $namespace;
@@ -295,7 +295,7 @@
}
if (function_exists('mb_detect_encoding')) {
- if (strlen($encoding)) {
+ if ($encoding !== null && strlen($encoding)) {
$try_encodings = array(
$encoding,
);

File Metadata

Mime Type
text/plain
Expires
Apr 1 2025, 7:41 AM (5 w, 1 d ago)
Storage Engine
amazon-s3
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
phabricator/secure/js/gi/wjlcougmzkeriik7
Default Alt Text
D21864.id52131.diff (31 KB)

Event Timeline

Phabricator · Built by Phacility