diff --git a/scripts/setup/manage_garbage.php b/scripts/setup/manage_garbage.php --- a/scripts/setup/manage_garbage.php +++ b/scripts/setup/manage_garbage.php @@ -5,7 +5,7 @@ require_once $root.'/scripts/__init_script__.php'; $args = new PhutilArgumentParser($argv); -$args->setTagline(pht('manage garbage colletors')); +$args->setTagline(pht('manage garbage collectors')); $args->setSynopsis(<<isImported()) { diff --git a/src/applications/auth/controller/PhabricatorAuthController.php b/src/applications/auth/controller/PhabricatorAuthController.php --- a/src/applications/auth/controller/PhabricatorAuthController.php +++ b/src/applications/auth/controller/PhabricatorAuthController.php @@ -134,7 +134,7 @@ // checks later on to make sure this account is valid for the intended // operation. This requires edit permission for completeness and consistency // but it won't actually be meaningfully checked because we're using the - // ominpotent user. + // omnipotent user. $account = id(new PhabricatorExternalAccountQuery()) ->setViewer(PhabricatorUser::getOmnipotentUser()) diff --git a/src/applications/auth/controller/PhabricatorAuthRegisterController.php b/src/applications/auth/controller/PhabricatorAuthRegisterController.php --- a/src/applications/auth/controller/PhabricatorAuthRegisterController.php +++ b/src/applications/auth/controller/PhabricatorAuthRegisterController.php @@ -79,7 +79,7 @@ } if ($default_email !== null) { - // We should bypass policy here becase e.g. limiting an application use + // We should bypass policy here because e.g. limiting an application use // to a subset of users should not allow the others to overwrite // configured application emails. $application_email = id(new PhabricatorMetaMTAApplicationEmailQuery()) diff --git a/src/applications/auth/editor/PhabricatorAuthSSHKeyEditor.php b/src/applications/auth/editor/PhabricatorAuthSSHKeyEditor.php --- a/src/applications/auth/editor/PhabricatorAuthSSHKeyEditor.php +++ b/src/applications/auth/editor/PhabricatorAuthSSHKeyEditor.php @@ -226,7 +226,7 @@ ->addHeader('Thread-Topic', $phid); // The primary value of this mail is alerting users to account compromises, - // so force delivery. In particular, this mail should still be delievered + // so force delivery. In particular, this mail should still be delivered // even if "self mail" is disabled. $mail->setForceDelivery(true); diff --git a/src/applications/auth/factor/__tests__/PhabricatorAuthInviteTestCase.php b/src/applications/auth/factor/__tests__/PhabricatorAuthInviteTestCase.php --- a/src/applications/auth/factor/__tests__/PhabricatorAuthInviteTestCase.php +++ b/src/applications/auth/factor/__tests__/PhabricatorAuthInviteTestCase.php @@ -51,7 +51,7 @@ $caught = $ex; } - // This first time should accept the invite and verify the addresss. + // This first time should accept the invite and verify the address. $this->assertTrue( ($caught instanceof PhabricatorAuthInviteRegisteredException)); diff --git a/src/applications/auth/management/PhabricatorAuthManagementRevokeWorkflow.php b/src/applications/auth/management/PhabricatorAuthManagementRevokeWorkflow.php --- a/src/applications/auth/management/PhabricatorAuthManagementRevokeWorkflow.php +++ b/src/applications/auth/management/PhabricatorAuthManagementRevokeWorkflow.php @@ -74,7 +74,7 @@ if (!strlen($from) && !$is_everywhere) { throw new PhutilArgumentUsageException( pht( - 'Specify the target to revoke credentals from with "--from" or '. + 'Specify the target to revoke credentials from with "--from" or '. 'specify "--everywhere".')); } else if (strlen($from) && $is_everywhere) { throw new PhutilArgumentUsageException( diff --git a/src/applications/auth/provider/PhabricatorOAuth1AuthProvider.php b/src/applications/auth/provider/PhabricatorOAuth1AuthProvider.php --- a/src/applications/auth/provider/PhabricatorOAuth1AuthProvider.php +++ b/src/applications/auth/provider/PhabricatorOAuth1AuthProvider.php @@ -270,7 +270,7 @@ } private function getHandshakeTokenKeyFromClientCode($client_code) { - // NOTE: This is very slightly coersive since the TemporaryToken table + // NOTE: This is very slightly coercive since the TemporaryToken table // expects an "objectPHID" as an identifier, but nothing about the storage // is bound to PHIDs. diff --git a/src/applications/base/PhabricatorApplication.php b/src/applications/base/PhabricatorApplication.php --- a/src/applications/base/PhabricatorApplication.php +++ b/src/applications/base/PhabricatorApplication.php @@ -108,7 +108,7 @@ * * Launchable applications can be pinned to the home page, and show up in the * "Launcher" view of the Applications application. Making an application - * unlauncahble prevents pinning and hides it from this view. + * unlaunchable prevents pinning and hides it from this view. * * Usually, an application should be marked unlaunchable if: * diff --git a/src/applications/base/controller/PhabricatorController.php b/src/applications/base/controller/PhabricatorController.php --- a/src/applications/base/controller/PhabricatorController.php +++ b/src/applications/base/controller/PhabricatorController.php @@ -552,7 +552,7 @@ public function buildApplicationCrumbsForEditEngine() { - // TODO: This is kind of gross, I'm bascially just making this public so + // TODO: This is kind of gross, I'm basically just making this public so // I can use it in EditEngine. We could do this without making it public // by using controller delegation, or make it properly public. return $this->buildApplicationCrumbs(); diff --git a/src/applications/calendar/editor/PhabricatorCalendarImportEditEngine.php b/src/applications/calendar/editor/PhabricatorCalendarImportEditEngine.php --- a/src/applications/calendar/editor/PhabricatorCalendarImportEditEngine.php +++ b/src/applications/calendar/editor/PhabricatorCalendarImportEditEngine.php @@ -134,7 +134,7 @@ $fields[] = id(new PhabricatorSelectEditField()) ->setKey('frequency') ->setLabel(pht('Update Automatically')) - ->setDescription(pht('Configure an automatic update frequncy.')) + ->setDescription(pht('Configure an automatic update frequency.')) ->setTransactionType( PhabricatorCalendarImportFrequencyTransaction::TRANSACTIONTYPE) ->setConduitDescription(pht('Set the automatic update frequency.')) diff --git a/src/applications/calendar/icon/PhabricatorCalendarIconSet.php b/src/applications/calendar/icon/PhabricatorCalendarIconSet.php --- a/src/applications/calendar/icon/PhabricatorCalendarIconSet.php +++ b/src/applications/calendar/icon/PhabricatorCalendarIconSet.php @@ -16,7 +16,7 @@ 'fa-plane' => pht('Travel'), 'fa-plus-square' => pht('Health / Appointment'), - 'fa-rocket' => pht('Sabatical / Leave'), + 'fa-rocket' => pht('Sabbatical / Leave'), 'fa-home' => pht('Working From Home'), 'fa-tree' => pht('Holiday'), 'fa-gamepad' => pht('Staycation'), diff --git a/src/applications/calendar/util/CalendarTimeUtil.php b/src/applications/calendar/util/CalendarTimeUtil.php --- a/src/applications/calendar/util/CalendarTimeUtil.php +++ b/src/applications/calendar/util/CalendarTimeUtil.php @@ -8,7 +8,7 @@ * calendar views - one for the conpherence calendar widget and one for the * user profile calendar view. These have slight differences such as * conpherence showing both a three day "today 'til 2 days from now" *and* - * a Sunday -> Saturday list, whilest the profile view shows a more simple + * a Sunday -> Saturday list, whilst the profile view shows a more simple * seven day rolling list of events. */ final class CalendarTimeUtil extends Phobject { diff --git a/src/applications/calendar/xaction/PhabricatorCalendarEventFrequencyTransaction.php b/src/applications/calendar/xaction/PhabricatorCalendarEventFrequencyTransaction.php --- a/src/applications/calendar/xaction/PhabricatorCalendarEventFrequencyTransaction.php +++ b/src/applications/calendar/xaction/PhabricatorCalendarEventFrequencyTransaction.php @@ -23,7 +23,7 @@ // month, it means "the 30th of every month" as far as the RRULE is // concerned. Such an event will not occur on months with fewer days. - // This is surprising, and proably not what the user wants. Instead, + // This is surprising, and probably not what the user wants. Instead, // schedule these events relative to the end of the month: on the "-1st", // "-2nd" or "-3rd" day of the month. For example, a monthly event on // the 31st of a 31-day month translates to "every month, on the last @@ -66,7 +66,7 @@ if (!isset($valid[$value])) { $errors[] = $this->newInvalidError( pht( - 'Event frequency "%s" is not valid. Valid frequences are: %s.', + 'Event frequency "%s" is not valid. Valid frequencies are: %s.', $value, implode(', ', $valid)), $xaction); diff --git a/src/applications/calendar/xaction/PhabricatorCalendarEventInviteTransaction.php b/src/applications/calendar/xaction/PhabricatorCalendarEventInviteTransaction.php --- a/src/applications/calendar/xaction/PhabricatorCalendarEventInviteTransaction.php +++ b/src/applications/calendar/xaction/PhabricatorCalendarEventInviteTransaction.php @@ -132,7 +132,7 @@ $this->renderHandleList($rem)); } else { return pht( - '%s invited %s attendee(s): %s; uninvinted %s attendee(s): %s.', + '%s invited %s attendee(s): %s; uninvited %s attendee(s): %s.', $this->renderAuthor(), phutil_count($add), $this->renderHandleList($add), @@ -161,7 +161,7 @@ } else { return pht( '%s updated the invite list for %s, invited %s: %s; '. - 'uninvinted %s: %s.', + 'uninvited %s: %s.', $this->renderAuthor(), $this->renderObject(), phutil_count($add), diff --git a/src/applications/calendar/xaction/PhabricatorCalendarImportFrequencyTransaction.php b/src/applications/calendar/xaction/PhabricatorCalendarImportFrequencyTransaction.php --- a/src/applications/calendar/xaction/PhabricatorCalendarImportFrequencyTransaction.php +++ b/src/applications/calendar/xaction/PhabricatorCalendarImportFrequencyTransaction.php @@ -32,7 +32,7 @@ if (!isset($valid[$value])) { $errors[] = $this->newInvalidError( pht( - 'Import frequency "%s" is not valid. Valid frequences are: %s.', + 'Import frequency "%s" is not valid. Valid frequencies are: %s.', $value, implode(', ', $valid)), $xaction); diff --git a/src/applications/config/check/PhabricatorStorageSetupCheck.php b/src/applications/config/check/PhabricatorStorageSetupCheck.php --- a/src/applications/config/check/PhabricatorStorageSetupCheck.php +++ b/src/applications/config/check/PhabricatorStorageSetupCheck.php @@ -85,7 +85,7 @@ 'be buffered into memory before being written to permanent '. 'storage. Phabricator needs memory available to store these '. 'chunks while they are uploaded, but PHP is currently configured '. - 'to severly limit the available memory.'. + 'to severely limit the available memory.'. "\n\n". 'PHP processes currently have very little free memory available '. '(%s). To work well, processes should have at least %s.'. diff --git a/src/applications/conpherence/query/ConpherenceThreadQuery.php b/src/applications/conpherence/query/ConpherenceThreadQuery.php --- a/src/applications/conpherence/query/ConpherenceThreadQuery.php +++ b/src/applications/conpherence/query/ConpherenceThreadQuery.php @@ -290,7 +290,7 @@ ->withObjectPHIDs(array_keys($conpherences)) ->needHandles(true); - // We have to flip these for the underyling query class. The semantics of + // We have to flip these for the underlying query class. The semantics of // paging are tricky business. if ($this->afterTransactionID) { $query->setBeforeID($this->afterTransactionID); diff --git a/src/applications/dashboard/customfield/PhabricatorDashboardPanelTabsCustomField.php b/src/applications/dashboard/customfield/PhabricatorDashboardPanelTabsCustomField.php --- a/src/applications/dashboard/customfield/PhabricatorDashboardPanelTabsCustomField.php +++ b/src/applications/dashboard/customfield/PhabricatorDashboardPanelTabsCustomField.php @@ -78,7 +78,7 @@ public function renderEditControl(array $handles) { // NOTE: This includes archived panels so we don't mutate the tabs - // when saving a tab panel that includes archied panels. This whole UI is + // when saving a tab panel that includes archived panels. This whole UI is // hopefully temporary anyway. $value = $this->getFieldValue(); diff --git a/src/applications/differential/controller/DifferentialController.php b/src/applications/differential/controller/DifferentialController.php --- a/src/applications/differential/controller/DifferentialController.php +++ b/src/applications/differential/controller/DifferentialController.php @@ -217,7 +217,7 @@ // by default and let the user toggle the rest. With modern messages, // we can send the user to the Harbormaster detail page. Just show // "a lot" of messages in legacy cases to try to strike a balance - // between implementation simplicitly and compatibility. + // between implementation simplicity and compatibility. $legacy_messages = array_slice($legacy_messages, 0, 100); $messages = array(); diff --git a/src/applications/differential/customfield/DifferentialHarbormasterField.php b/src/applications/differential/customfield/DifferentialHarbormasterField.php --- a/src/applications/differential/customfield/DifferentialHarbormasterField.php +++ b/src/applications/differential/customfield/DifferentialHarbormasterField.php @@ -45,7 +45,7 @@ // by default and let the user toggle the rest. With modern messages, // we can send the user to the Harbormaster detail page. Just show // "a lot" of messages in legacy cases to try to strike a balance - // between implementation simplicitly and compatibility. + // between implementation simplicity and compatibility. $legacy_messages = array_slice($legacy_messages, 0, 100); foreach ($legacy_messages as $message) { diff --git a/src/applications/differential/editor/DifferentialDiffEditor.php b/src/applications/differential/editor/DifferentialDiffEditor.php --- a/src/applications/differential/editor/DifferentialDiffEditor.php +++ b/src/applications/differential/editor/DifferentialDiffEditor.php @@ -106,7 +106,7 @@ * We run Herald as part of transaction validation because Herald can * block diff creation for Differential diffs. Its important to do this * separately so no Herald logs are saved; these logs could expose - * information the Herald rules are inteneded to block. + * information the Herald rules are intended to block. */ protected function validateTransaction( PhabricatorLiskDAO $object, diff --git a/src/applications/differential/parser/DifferentialChangesetParser.php b/src/applications/differential/parser/DifferentialChangesetParser.php --- a/src/applications/differential/parser/DifferentialChangesetParser.php +++ b/src/applications/differential/parser/DifferentialChangesetParser.php @@ -699,7 +699,7 @@ $hunk_parser->parseHunksForLineData($changeset->getHunks()); // Depending on the whitespace mode, we may need to compute a different - // set of changes than the set of changes in the hunk data (specificaly, + // set of changes than the set of changes in the hunk data (specifically, // we might want to consider changed lines which have only whitespace // changes as unchanged). if ($ignore_all) { @@ -1192,11 +1192,11 @@ * Mask - compute the actual lines that need to be shown (because they * are near changes lines, near inline comments, or the request has * explicitly asked for them, i.e. resulting from the user clicking - * "show more"). The $mask returned is a sparesely populated dictionary + * "show more"). The $mask returned is a sparsely populated dictionary * of $visible_line_number => true. * * Depths - compute how indented any given line is. The $depths returned - * is a sparesely populated dictionary of $visible_line_number => $depth. + * is a sparsely populated dictionary of $visible_line_number => $depth. * * This function also has the side effect of modifying member variable * new such that tabs are normalized to spaces for each line of the diff. diff --git a/src/applications/differential/storage/DifferentialModernHunk.php b/src/applications/differential/storage/DifferentialModernHunk.php --- a/src/applications/differential/storage/DifferentialModernHunk.php +++ b/src/applications/differential/storage/DifferentialModernHunk.php @@ -128,7 +128,7 @@ $this->setData($file->getPHID()); // NOTE: Because hunks don't have a PHID and we just load hunk data with - // the ominipotent viewer, we do not need to attach the file to anything. + // the omnipotent viewer, we do not need to attach the file to anything. $result = $this->save(); diff --git a/src/applications/differential/storage/DifferentialRevision.php b/src/applications/differential/storage/DifferentialRevision.php --- a/src/applications/differential/storage/DifferentialRevision.php +++ b/src/applications/differential/storage/DifferentialRevision.php @@ -442,7 +442,7 @@ // For each path which the viewer owns a package for, find other packages // which that authority can be used to force-accept. Once we find a way to - // force-accept a package, we don't need to keep loooking. + // force-accept a package, we don't need to keep looking. $has_control = array(); foreach ($force_map as $path => $spec) { $path_fragments = PhabricatorOwnersPackage::splitPath($path); @@ -891,7 +891,7 @@ self::TABLE_COMMIT, $this->getID()); - // we have to do paths a little differentally as they do not have + // we have to do paths a little differently as they do not have // an id or phid column for delete() to act on $dummy_path = new DifferentialAffectedPath(); queryfx( diff --git a/src/applications/differential/typeahead/DifferentialResponsibleViewerFunctionDatasource.php b/src/applications/differential/typeahead/DifferentialResponsibleViewerFunctionDatasource.php --- a/src/applications/differential/typeahead/DifferentialResponsibleViewerFunctionDatasource.php +++ b/src/applications/differential/typeahead/DifferentialResponsibleViewerFunctionDatasource.php @@ -22,7 +22,7 @@ 'summary' => pht('Use the current viewing user.'), 'description' => pht( 'Show revisions the current viewer is responsible for. This '. - 'function inclues revisions the viewer is responsible for through '. + 'function includes revisions the viewer is responsible for through '. 'membership in projects and packages.'), ), ); 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 @@ -479,7 +479,7 @@ // chains of events). This should be rare, but does not indicate a bug // or data issue. - // NOTE: We never query push logs in SVN because the commiter is always + // NOTE: We never query push logs in SVN because the committer is always // the pusher and the commit time is always the push time; the push log // is redundant and we save a query by skipping it. diff --git a/src/applications/diffusion/editor/DiffusionRepositoryEditEngine.php b/src/applications/diffusion/editor/DiffusionRepositoryEditEngine.php --- a/src/applications/diffusion/editor/DiffusionRepositoryEditEngine.php +++ b/src/applications/diffusion/editor/DiffusionRepositoryEditEngine.php @@ -338,7 +338,7 @@ ->setIsCopyable(true) ->setDescription(pht('Track only these branches.')) ->setConduitDescription(pht('Set the tracked branches.')) - ->setConduitTypeDescription(pht('New tracked branchs.')) + ->setConduitTypeDescription(pht('New tracked branches.')) ->setValue($track_value), id(new PhabricatorTextAreaEditField()) ->setIsStringList(true) @@ -349,7 +349,7 @@ ->setIsCopyable(true) ->setDescription(pht('Autoclose commits on only these branches.')) ->setConduitDescription(pht('Set the autoclose branches.')) - ->setConduitTypeDescription(pht('New default tracked branchs.')) + ->setConduitTypeDescription(pht('New default tracked branches.')) ->setValue($autoclose_value), id(new PhabricatorTextEditField()) ->setKey('importOnly') @@ -396,7 +396,7 @@ ->setConduitDescription( pht('Change symbol languages for this repository.')) ->setConduitTypeDescription( - pht('New symbol langauges.')) + pht('New symbol languages.')) ->setValue($object->getSymbolLanguages()), id(new PhabricatorDatasourceEditField()) ->setKey('symbolRepositoryPHIDs') diff --git a/src/applications/diffusion/engine/DiffusionCommitHookEngine.php b/src/applications/diffusion/engine/DiffusionCommitHookEngine.php --- a/src/applications/diffusion/engine/DiffusionCommitHookEngine.php +++ b/src/applications/diffusion/engine/DiffusionCommitHookEngine.php @@ -1230,7 +1230,7 @@ // also look at author data (do the commits come from multiple different // authors?) and commit date data (is the oldest commit more than 48 hours // old), but we don't have immediate access to those and this simple - // heruistic might be good enough. + // heuristic might be good enough. $commit_count = 0; $type_commit = PhabricatorRepositoryPushLog::REFTYPE_COMMIT; diff --git a/src/applications/diffusion/gitlfs/DiffusionGitLFSAuthenticateWorkflow.php b/src/applications/diffusion/gitlfs/DiffusionGitLFSAuthenticateWorkflow.php --- a/src/applications/diffusion/gitlfs/DiffusionGitLFSAuthenticateWorkflow.php +++ b/src/applications/diffusion/gitlfs/DiffusionGitLFSAuthenticateWorkflow.php @@ -80,7 +80,7 @@ $lfs_uri = $repository->getGitLFSURI('info/lfs'); - // Generate a temporary token to allow the user to acces LFS over HTTP. + // Generate a temporary token to allow the user to access LFS over HTTP. // This works even if normal HTTP repository operations are not available // on this host, and does not require the user to have a VCS password. diff --git a/src/applications/diffusion/herald/HeraldCommitAdapter.php b/src/applications/diffusion/herald/HeraldCommitAdapter.php --- a/src/applications/diffusion/herald/HeraldCommitAdapter.php +++ b/src/applications/diffusion/herald/HeraldCommitAdapter.php @@ -112,7 +112,7 @@ $phids[] = $repository_phid; // NOTE: This is projects for the repository, not for the commit. When - // Herald evalutes, commits normally can not have any project tags yet. + // Herald evaluates, commits normally can not have any project tags yet. $repository_project_phids = PhabricatorEdgeQuery::loadDestinationPHIDs( $repository_phid, $project_type); diff --git a/src/applications/diffusion/protocol/DiffusionCommandEngine.php b/src/applications/diffusion/protocol/DiffusionCommandEngine.php --- a/src/applications/diffusion/protocol/DiffusionCommandEngine.php +++ b/src/applications/diffusion/protocol/DiffusionCommandEngine.php @@ -178,7 +178,7 @@ if (!$device) { throw new Exception( pht( - 'Attempting to build a reposiory command (for repository "%s") '. + 'Attempting to build a repository command (for repository "%s") '. 'as device, but this host ("%s") is not configured as a cluster '. 'device.', $repository->getDisplayName(), diff --git a/src/applications/diffusion/protocol/DiffusionMercurialWireProtocol.php b/src/applications/diffusion/protocol/DiffusionMercurialWireProtocol.php --- a/src/applications/diffusion/protocol/DiffusionMercurialWireProtocol.php +++ b/src/applications/diffusion/protocol/DiffusionMercurialWireProtocol.php @@ -101,7 +101,7 @@ /** If the server version is running 3.4+ it will respond * with 'bundle2' capability in the format of "bundle2=(url-encoding)". - * Until we maange to properly package up bundles to send back we + * Until we manage to properly package up bundles to send back we * disallow the client from knowing we speak bundle2 by removing it * from the capabilities listing. * diff --git a/src/applications/diffusion/protocol/DiffusionRepositoryClusterEngine.php b/src/applications/diffusion/protocol/DiffusionRepositoryClusterEngine.php --- a/src/applications/diffusion/protocol/DiffusionRepositoryClusterEngine.php +++ b/src/applications/diffusion/protocol/DiffusionRepositoryClusterEngine.php @@ -231,7 +231,7 @@ // no way to tell which one has the "right" data. If we pick wrong, we // might erase some or all of the data in the repository. - // Since this is dangeorus, we refuse to guess unless there is only one + // Since this is dangerous, we refuse to guess unless there is only one // device. If we're the only device in the group, we obviously must be // a leader. @@ -252,7 +252,7 @@ 'Repository "%s" exists on more than one device, but no device '. 'has any repository version information. Phabricator can not '. 'guess which copy of the existing data is authoritative. Promote '. - 'a device or see "Ambigous Leaders" in the documentation.', + 'a device or see "Ambiguous Leaders" in the documentation.', $repository->getDisplayName())); } diff --git a/src/applications/diffusion/query/DiffusionCachedResolveRefsQuery.php b/src/applications/diffusion/query/DiffusionCachedResolveRefsQuery.php --- a/src/applications/diffusion/query/DiffusionCachedResolveRefsQuery.php +++ b/src/applications/diffusion/query/DiffusionCachedResolveRefsQuery.php @@ -9,7 +9,7 @@ * low-level query can. * * This class can resolve the most common refs (commits, branches, tags) and - * can do so cheapy (by examining the database, without needing to make calls + * can do so cheaply (by examining the database, without needing to make calls * to the VCS or the service host). */ final class DiffusionCachedResolveRefsQuery diff --git a/src/applications/diffusion/query/DiffusionCommitQuery.php b/src/applications/diffusion/query/DiffusionCommitQuery.php --- a/src/applications/diffusion/query/DiffusionCommitQuery.php +++ b/src/applications/diffusion/query/DiffusionCommitQuery.php @@ -69,7 +69,7 @@ /** * Look up commits in a specific repository. Prefer - * @{method:withRepositoryIDs}; the underyling table is keyed by ID such + * @{method:withRepositoryIDs}; the underlying table is keyed by ID such * that this method requires a separate initial query to map PHID to ID. */ public function withRepositoryPHIDs(array $phids) { diff --git a/src/applications/diffusion/xaction/DiffusionCommitConcernTransaction.php b/src/applications/diffusion/xaction/DiffusionCommitConcernTransaction.php --- a/src/applications/diffusion/xaction/DiffusionCommitConcernTransaction.php +++ b/src/applications/diffusion/xaction/DiffusionCommitConcernTransaction.php @@ -53,7 +53,7 @@ } // Even if you've already raised a concern, you can raise again as long - // as the author requsted you verify. + // as the author requested you verify. $state_verify = PhabricatorAuditCommitStatusConstants::NEEDS_VERIFICATION; if ($this->isViewerFullyRejected($object, $viewer)) { diff --git a/src/applications/diffusion/xaction/DiffusionCommitVerifyTransaction.php b/src/applications/diffusion/xaction/DiffusionCommitVerifyTransaction.php --- a/src/applications/diffusion/xaction/DiffusionCommitVerifyTransaction.php +++ b/src/applications/diffusion/xaction/DiffusionCommitVerifyTransaction.php @@ -53,7 +53,7 @@ throw new Exception( pht( 'You can not request verification of this commit because no '. - 'auditors have raised conerns with it.')); + 'auditors have raised concerns with it.')); } } diff --git a/src/applications/drydock/blueprint/DrydockBlueprintImplementation.php b/src/applications/drydock/blueprint/DrydockBlueprintImplementation.php --- a/src/applications/drydock/blueprint/DrydockBlueprintImplementation.php +++ b/src/applications/drydock/blueprint/DrydockBlueprintImplementation.php @@ -64,7 +64,7 @@ * is a coarse compatibility check between a lease and a resource. * * @param DrydockBlueprint Concrete blueprint to allocate for. - * @param DrydockResource Candidiate resource to allocate the lease on. + * @param DrydockResource Candidate resource to allocate the lease on. * @param DrydockLease Pending lease that wants to allocate here. * @return bool True if the resource and lease are compatible. * @task lease @@ -76,7 +76,7 @@ /** - * Acquire a lease. Allows resources to peform setup as leases are brought + * Acquire a lease. Allows resources to perform setup as leases are brought * online. * * If acquisition fails, throw an exception. diff --git a/src/applications/drydock/storage/DrydockAuthorization.php b/src/applications/drydock/storage/DrydockAuthorization.php --- a/src/applications/drydock/storage/DrydockAuthorization.php +++ b/src/applications/drydock/storage/DrydockAuthorization.php @@ -100,7 +100,7 @@ } /** - * Apply external authorization effects after a user chagnes the value of a + * Apply external authorization effects after a user changes the value of a * blueprint selector control an object. * * @param PhabricatorUser User applying the change. diff --git a/src/applications/drydock/worker/DrydockLeaseUpdateWorker.php b/src/applications/drydock/worker/DrydockLeaseUpdateWorker.php --- a/src/applications/drydock/worker/DrydockLeaseUpdateWorker.php +++ b/src/applications/drydock/worker/DrydockLeaseUpdateWorker.php @@ -241,7 +241,7 @@ // NOTE: We have not acquired the lease yet, so it is possible that the // resource we just built will be snatched up by some other lease before // we can acquire it. This is not problematic: we'll retry a little later - // and should suceed eventually. + // and should succeed eventually. } $resources = $this->rankResources($resources, $lease); @@ -261,7 +261,7 @@ if (!$allocated) { throw new PhutilAggregateException( pht( - 'Unable to acquire lease "%s" on any resouce.', + 'Unable to acquire lease "%s" on any resource.', $lease->getPHID()), $exceptions); } @@ -725,7 +725,7 @@ // performed the read above and now, the resource might have closed, so // we may activate leases on dead resources. At least for now, this seems // fine: a resource dying right before we activate a lease on it should not - // be distinguisahble from a resource dying right after we activate a lease + // be distinguishable from a resource dying right after we activate a lease // on it. We end up with an active lease on a dead resource either way, and // can not prevent resources dying from lightning strikes. diff --git a/src/applications/drydock/worker/DrydockWorker.php b/src/applications/drydock/worker/DrydockWorker.php --- a/src/applications/drydock/worker/DrydockWorker.php +++ b/src/applications/drydock/worker/DrydockWorker.php @@ -199,7 +199,7 @@ $viewer = $this->getViewer(); // Don't reclaim a resource if it has been updated recently. If two - // leases are fighting, we don't want them to keep reclaming resources + // leases are fighting, we don't want them to keep reclaiming resources // from one another forever without making progress, so make resources // immune to reclamation for a little while after they activate or update. diff --git a/src/applications/files/controller/PhabricatorFileDataController.php b/src/applications/files/controller/PhabricatorFileDataController.php --- a/src/applications/files/controller/PhabricatorFileDataController.php +++ b/src/applications/files/controller/PhabricatorFileDataController.php @@ -119,7 +119,7 @@ // make this logic simpler and more consistent. // Beyond making the policy check itself more consistent, this also makes - // sure we're consitent about returning HTTP 404 on bad requests instead + // sure we're consistent about returning HTTP 404 on bad requests instead // of serving HTTP 200 with a login page, which can mislead some clients. $viewer = PhabricatorUser::getOmnipotentUser(); diff --git a/src/applications/files/management/PhabricatorFilesManagementCatWorkflow.php b/src/applications/files/management/PhabricatorFilesManagementCatWorkflow.php --- a/src/applications/files/management/PhabricatorFilesManagementCatWorkflow.php +++ b/src/applications/files/management/PhabricatorFilesManagementCatWorkflow.php @@ -24,7 +24,7 @@ 'help' => pht( 'DANGEROUS. Attempt to salvage file content even if the '. 'integrity check fails. If an adversary has tampered with '. - 'the file, the conent may be unsafe.'), + 'the file, the content may be unsafe.'), ), array( 'name' => 'names', diff --git a/src/applications/fund/editor/FundInitiativeEditEngine.php b/src/applications/fund/editor/FundInitiativeEditEngine.php --- a/src/applications/fund/editor/FundInitiativeEditEngine.php +++ b/src/applications/fund/editor/FundInitiativeEditEngine.php @@ -50,7 +50,7 @@ } protected function getObjectName() { - return pht('Initivative'); + return pht('Initiative'); } protected function getObjectCreateCancelURI($object) { diff --git a/src/applications/harbormaster/conduit/HarbormasterSendMessageConduitAPIMethod.php b/src/applications/harbormaster/conduit/HarbormasterSendMessageConduitAPIMethod.php --- a/src/applications/harbormaster/conduit/HarbormasterSendMessageConduitAPIMethod.php +++ b/src/applications/harbormaster/conduit/HarbormasterSendMessageConduitAPIMethod.php @@ -142,7 +142,7 @@ "=============\n". "When you send Harbormaster a message, you must include a `type`, ". "which describes the overall state of the build. For example, use ". - "`pass` to tell Harbomaster that a build completed successfully.". + "`pass` to tell Harbormaster that a build completed successfully.". "\n\n". "Supported message types are:". "\n\n". @@ -155,7 +155,7 @@ "message, but you can also send a `work` message to report intermediate ". "results.\n\n". "To provide unit test results, pass a list of results in the `unit` ". - "parameter. Each result shoud be a dictionary with these keys:". + "parameter. Each result should be a dictionary with these keys:". "\n\n". "%s". "\n\n". diff --git a/src/applications/harbormaster/storage/configuration/HarbormasterBuildStep.php b/src/applications/harbormaster/storage/configuration/HarbormasterBuildStep.php --- a/src/applications/harbormaster/storage/configuration/HarbormasterBuildStep.php +++ b/src/applications/harbormaster/storage/configuration/HarbormasterBuildStep.php @@ -38,7 +38,7 @@ // T6203/NULLABILITY // This should not be nullable. Current `null` values indicate steps // which predated editable names. These should be backfilled with - // default names, then the code for handling `null` shoudl be removed. + // default names, then the code for handling `null` should be removed. 'name' => 'text255?', 'stepAutoKey' => 'text32?', ), diff --git a/src/applications/legalpad/controller/LegalpadDocumentSignController.php b/src/applications/legalpad/controller/LegalpadDocumentSignController.php --- a/src/applications/legalpad/controller/LegalpadDocumentSignController.php +++ b/src/applications/legalpad/controller/LegalpadDocumentSignController.php @@ -236,7 +236,7 @@ // Use the last content update as the modified date. We don't want to // show that a document like a TOS was "updated" by an incidental change - // to a field like the preamble or privacy settings which does not acutally + // to a field like the preamble or privacy settings which does not actually // affect the content of the agreement. $content_updated = $document_body->getDateCreated(); diff --git a/src/applications/maniphest/controller/ManiphestReportController.php b/src/applications/maniphest/controller/ManiphestReportController.php --- a/src/applications/maniphest/controller/ManiphestReportController.php +++ b/src/applications/maniphest/controller/ManiphestReportController.php @@ -116,7 +116,7 @@ case ManiphestTaskMergedIntoTransaction::TRANSACTIONTYPE: // NOTE: Merging a task does not generate a "status" transaction. // We pretend it did. Note that this is not always accurate: it is - // possble to merge a task which was previously closed, but this + // possible to merge a task which was previously closed, but this // fake transaction always counts a merge as a closure. $oldv = $default_status; $newv = $duplicate_status; @@ -762,7 +762,7 @@ // and equal distance in the past. This is so users can type "6 days" (which // means "6 days from now") and get the behavior of "6 days ago", rather // than no results (because the window epoch is in the future). This might - // be a little confusing because it casues "tomorrow" to mean "yesterday" + // be a little confusing because it causes "tomorrow" to mean "yesterday" // and "2022" (or whatever) to mean "ten years ago", but these inputs are // nonsense anyway. diff --git a/src/applications/maniphest/editor/ManiphestTransactionEditor.php b/src/applications/maniphest/editor/ManiphestTransactionEditor.php --- a/src/applications/maniphest/editor/ManiphestTransactionEditor.php +++ b/src/applications/maniphest/editor/ManiphestTransactionEditor.php @@ -972,7 +972,7 @@ $object_phid = $object->getPHID(); - // We're doing layout with the ominpotent viewer to make sure we don't + // We're doing layout with the omnipotent viewer to make sure we don't // remove positions in columns that exist, but which the actual actor // can't see. $omnipotent_viewer = PhabricatorUser::getOmnipotentUser(); diff --git a/src/applications/metamta/management/PhabricatorMailManagementUnverifyWorkflow.php b/src/applications/metamta/management/PhabricatorMailManagementUnverifyWorkflow.php --- a/src/applications/metamta/management/PhabricatorMailManagementUnverifyWorkflow.php +++ b/src/applications/metamta/management/PhabricatorMailManagementUnverifyWorkflow.php @@ -63,7 +63,7 @@ echo tsprintf( "%s\n", pht( - 'Address "%s" (owned by "%s") is already unveriifed.', + 'Address "%s" (owned by "%s") is already unverified.', $address, $user->getUsername())); continue; diff --git a/src/applications/metamta/receiver/PhabricatorMailReceiver.php b/src/applications/metamta/receiver/PhabricatorMailReceiver.php --- a/src/applications/metamta/receiver/PhabricatorMailReceiver.php +++ b/src/applications/metamta/receiver/PhabricatorMailReceiver.php @@ -255,7 +255,7 @@ /** - * Reduce an email address to its canonical form. For example, an adddress + * Reduce an email address to its canonical form. For example, an address * like: * * "Abraham Lincoln" < ALincoln@example.com > diff --git a/src/applications/owners/storage/__tests__/PhabricatorOwnersPackageTestCase.php b/src/applications/owners/storage/__tests__/PhabricatorOwnersPackageTestCase.php --- a/src/applications/owners/storage/__tests__/PhabricatorOwnersPackageTestCase.php +++ b/src/applications/owners/storage/__tests__/PhabricatorOwnersPackageTestCase.php @@ -82,7 +82,7 @@ // Now, add a more specific path to Package #1. This tests nested ownership // in packages with weak dominion rules. This time, Package #1 should end - // up back on top, with Package #2 cedeing control to its more specific + // up back on top, with Package #2 ceding control to its more specific // path. $rows[] = array( 'id' => 1, diff --git a/src/applications/passphrase/credentialtype/PassphraseCredentialType.php b/src/applications/passphrase/credentialtype/PassphraseCredentialType.php --- a/src/applications/passphrase/credentialtype/PassphraseCredentialType.php +++ b/src/applications/passphrase/credentialtype/PassphraseCredentialType.php @@ -104,7 +104,7 @@ /** - * Return true if the provided credental requires a password to decrypt. + * Return true if the provided credential requires a password to decrypt. * * @param PhutilOpaqueEnvelope Credential secret value. * @return bool True if the credential needs a password. diff --git a/src/applications/passphrase/credentialtype/PassphraseSSHPrivateKeyTextCredentialType.php b/src/applications/passphrase/credentialtype/PassphraseSSHPrivateKeyTextCredentialType.php --- a/src/applications/passphrase/credentialtype/PassphraseSSHPrivateKeyTextCredentialType.php +++ b/src/applications/passphrase/credentialtype/PassphraseSSHPrivateKeyTextCredentialType.php @@ -47,7 +47,7 @@ pht( 'Decrypting SSH keys requires the `%s` binary, but it '. 'is not available in %s. Either make it available or strip the '. - 'password fromt his SSH key manually before uploading it.', + 'password from this SSH key manually before uploading it.', 'ssh-keygen', '$PATH')); } diff --git a/src/applications/passphrase/view/PassphraseCredentialControl.php b/src/applications/passphrase/view/PassphraseCredentialControl.php --- a/src/applications/passphrase/view/PassphraseCredentialControl.php +++ b/src/applications/passphrase/view/PassphraseCredentialControl.php @@ -67,7 +67,7 @@ $user_credential->getName()); } } catch (PhabricatorPolicyException $policy_exception) { - // Pull the credential with the ominipotent viewer so we can look up + // Pull the credential with the omnipotent viewer so we can look up // the ID and provide the monogram. $omnipotent_credential = id(new PassphraseCredentialQuery()) ->setViewer(PhabricatorUser::getOmnipotentUser()) diff --git a/src/applications/people/markup/PhabricatorMentionRemarkupRule.php b/src/applications/people/markup/PhabricatorMentionRemarkupRule.php --- a/src/applications/people/markup/PhabricatorMentionRemarkupRule.php +++ b/src/applications/people/markup/PhabricatorMentionRemarkupRule.php @@ -10,7 +10,7 @@ // NOTE: The negative lookbehind prevents matches like "mail@lists", while // allowing constructs like "@tomo/@mroch". Since we now allow periods in - // usernames, we can't resonably distinguish that "@company.com" isn't a + // usernames, we can't reasonably distinguish that "@company.com" isn't a // username, so we'll incorrectly pick it up, but there's little to be done // about that. We forbid terminal periods so that we can correctly capture // "@joe" instead of "@joe." in "Hey, @joe.". diff --git a/src/applications/people/query/PhabricatorPeopleQuery.php b/src/applications/people/query/PhabricatorPeopleQuery.php --- a/src/applications/people/query/PhabricatorPeopleQuery.php +++ b/src/applications/people/query/PhabricatorPeopleQuery.php @@ -425,7 +425,7 @@ } // If the user is set to "Available" for this event, don't consider it - // when computin their away status. + // when computing their away status. if (!$invitee->getDisplayAvailability($event)) { continue; } @@ -492,7 +492,7 @@ // valid for that long. // NOTE: This doesn't handle overlapping events with the greatest - // possible care. In theory, if you're attenting multiple events + // possible care. In theory, if you're attending multiple events // simultaneously we should accommodate that. However, it's complex // to compute, rare, and probably not confusing most of the time. diff --git a/src/applications/people/query/PhabricatorPeopleSearchEngine.php b/src/applications/people/query/PhabricatorPeopleSearchEngine.php --- a/src/applications/people/query/PhabricatorPeopleSearchEngine.php +++ b/src/applications/people/query/PhabricatorPeopleSearchEngine.php @@ -79,7 +79,7 @@ ->setOptions( pht('(Show All)'), pht('Show Only Unapproved Users'), - pht('Hide Unappproved Users')) + pht('Hide Unapproved Users')) ->setDescription( pht( 'Pass true to find only users awaiting administrative approval, '. diff --git a/src/applications/people/storage/PhabricatorUser.php b/src/applications/people/storage/PhabricatorUser.php --- a/src/applications/people/storage/PhabricatorUser.php +++ b/src/applications/people/storage/PhabricatorUser.php @@ -407,7 +407,7 @@ $token = substr($token, $breach_prelen + self::CSRF_SALT_LENGTH); // When the user posts a form, we check that it contains a valid CSRF token. - // Tokens cycle each hour (every CSRF_CYLCE_FREQUENCY seconds) and we accept + // Tokens cycle each hour (every CSRF_CYCLE_FREQUENCY seconds) and we accept // either the current token, the next token (users can submit a "future" // token if you have two web frontends that have some clock skew) or any of // the last 6 tokens. This means that pages are valid for up to 7 hours. @@ -1180,7 +1180,7 @@ /** * Get a scalar string identifying this user. * - * This is similar to using the PHID, but distinguishes between ominpotent + * This is similar to using the PHID, but distinguishes between omnipotent * and public users explicitly. This allows safe construction of cache keys * or cache buckets which do not conflate public and omnipotent users. * @@ -1459,7 +1459,7 @@ id(new PhabricatorConduitSearchFieldSpecification()) ->setKey('roles') ->setType('list') - ->setDescription(pht('List of acccount roles.')), + ->setDescription(pht('List of account roles.')), ); } diff --git a/src/applications/phid/handle/pool/PhabricatorHandleList.php b/src/applications/phid/handle/pool/PhabricatorHandleList.php --- a/src/applications/phid/handle/pool/PhabricatorHandleList.php +++ b/src/applications/phid/handle/pool/PhabricatorHandleList.php @@ -82,7 +82,7 @@ if (!isset($this[$phid])) { throw new Exception( pht( - 'Trying to create a new sublist of an existsing handle list, '. + 'Trying to create a new sublist of an existing handle list, '. 'but PHID "%s" does not appear in the parent list.', $phid)); } diff --git a/src/applications/phid/resolver/PhabricatorPHIDResolver.php b/src/applications/phid/resolver/PhabricatorPHIDResolver.php --- a/src/applications/phid/resolver/PhabricatorPHIDResolver.php +++ b/src/applications/phid/resolver/PhabricatorPHIDResolver.php @@ -3,7 +3,7 @@ /** * Resolve a list of identifiers into PHIDs. * - * This class simplifies the process of convering a list of mixed token types + * This class simplifies the process of converting a list of mixed token types * (like some PHIDs and some usernames) into a list of just PHIDs. */ abstract class PhabricatorPHIDResolver extends Phobject { diff --git a/src/applications/phortune/provider/PhortuneStripePaymentProvider.php b/src/applications/phortune/provider/PhortuneStripePaymentProvider.php --- a/src/applications/phortune/provider/PhortuneStripePaymentProvider.php +++ b/src/applications/phortune/provider/PhortuneStripePaymentProvider.php @@ -361,7 +361,7 @@ case 'error:duplicate_transaction': case 'error:processing_error': default: - // NOTE: These errors currently don't recevive a detailed message. + // NOTE: These errors currently don't receive a detailed message. // NOTE: We can also end up here with "http:nnn" messages. // TODO: At least some of these should have a better message, or be diff --git a/src/applications/phrequent/storage/PhrequentTimeBlock.php b/src/applications/phrequent/storage/PhrequentTimeBlock.php --- a/src/applications/phrequent/storage/PhrequentTimeBlock.php +++ b/src/applications/phrequent/storage/PhrequentTimeBlock.php @@ -124,7 +124,7 @@ } } else { // Here, we've prematurely ended a deeper stratum. Merge higher - // stata. This looks like this: + // strata. This looks like this: // // V // V diff --git a/src/applications/phriction/editor/PhrictionTransactionEditor.php b/src/applications/phriction/editor/PhrictionTransactionEditor.php --- a/src/applications/phriction/editor/PhrictionTransactionEditor.php +++ b/src/applications/phriction/editor/PhrictionTransactionEditor.php @@ -452,7 +452,7 @@ $verb) { $errors = array(); - // NOTE: We use the ominpotent user for these checks because policy + // NOTE: We use the omnipotent user for these checks because policy // doesn't matter; existence does. $other_doc_viewer = PhabricatorUser::getOmnipotentUser(); $ancestral_slugs = PhabricatorSlug::getAncestry($object->getSlug()); diff --git a/src/applications/policy/filter/PhabricatorPolicyFilter.php b/src/applications/policy/filter/PhabricatorPolicyFilter.php --- a/src/applications/policy/filter/PhabricatorPolicyFilter.php +++ b/src/applications/policy/filter/PhabricatorPolicyFilter.php @@ -257,7 +257,7 @@ $filtered[$key] = $object; } - // If we survied the primary checks, apply extended checks to objects + // If we survived the primary checks, apply extended checks to objects // with extended policies. $results = array(); $extended = array(); diff --git a/src/applications/project/query/PhabricatorProjectColumnQuery.php b/src/applications/project/query/PhabricatorProjectColumnQuery.php --- a/src/applications/project/query/PhabricatorProjectColumnQuery.php +++ b/src/applications/project/query/PhabricatorProjectColumnQuery.php @@ -97,7 +97,7 @@ if ($proxy_phid !== null) { $proxy = idx($proxies, $proxy_phid); - // Only attach valid proxies, so we don't end up getting surprsied if + // Only attach valid proxies, so we don't end up getting surprised if // an install somehow gets junk into their database. if (!($proxy instanceof PhabricatorColumnProxyInterface)) { $proxy = null; diff --git a/src/applications/project/remarkup/ProjectRemarkupRule.php b/src/applications/project/remarkup/ProjectRemarkupRule.php --- a/src/applications/project/remarkup/ProjectRemarkupRule.php +++ b/src/applications/project/remarkup/ProjectRemarkupRule.php @@ -24,7 +24,7 @@ protected function getObjectIDPattern() { // NOTE: The latter half of this rule matches monograms with internal // periods, like `#domain.com`, but does not match monograms with terminal - // periods, because they're probably just puncutation. + // periods, because they're probably just punctuation. // Broadly, this will not match every possible project monogram, and we // accept some false negatives -- like `#dot.` -- in order to avoid a bunch diff --git a/src/applications/releeph/differential/DifferentialReleephRequestFieldSpecification.php b/src/applications/releeph/differential/DifferentialReleephRequestFieldSpecification.php --- a/src/applications/releeph/differential/DifferentialReleephRequestFieldSpecification.php +++ b/src/applications/releeph/differential/DifferentialReleephRequestFieldSpecification.php @@ -5,7 +5,7 @@ * * 1: To parse "Releeph: picks RQ" headers in commits created by * arc-releeph so that RQs committed by arc-releeph have real - * PhabricatorRepositoryCommits associated with them (instaed of just the SHA + * PhabricatorRepositoryCommits associated with them (instead of just the SHA * of the commit, as seen by the pusher). * * 2: If requestors want to commit directly to their release branch, they can diff --git a/src/applications/releeph/field/selector/ReleephDefaultFieldSelector.php b/src/applications/releeph/field/selector/ReleephDefaultFieldSelector.php --- a/src/applications/releeph/field/selector/ReleephDefaultFieldSelector.php +++ b/src/applications/releeph/field/selector/ReleephDefaultFieldSelector.php @@ -7,7 +7,7 @@ * * TODO: This is a giant hacky mess because I am dumb and moved forward on * Releeph changes with partial information. Recover from this as gracefully - * as possible. This obivously is an abomination. -epriestley + * as possible. This obviously is an abomination. -epriestley */ public static function isFacebook() { return class_exists('ReleephFacebookKarmaFieldSpecification'); diff --git a/src/applications/repository/daemon/PhabricatorMercurialGraphStream.php b/src/applications/repository/daemon/PhabricatorMercurialGraphStream.php --- a/src/applications/repository/daemon/PhabricatorMercurialGraphStream.php +++ b/src/applications/repository/daemon/PhabricatorMercurialGraphStream.php @@ -113,7 +113,7 @@ private function parseParents($parents, $target_rev) { // The hg '{parents}' token is empty if there is one "natural" parent - // (predecessor local commit ID). Othwerwise, it may have one or two + // (predecessor local commit ID). Otherwise, it may have one or two // parents. The string looks like this: // // 151:1f6c61a60586 154:1d5f799ebe1e diff --git a/src/applications/repository/data/PhabricatorRepositoryURINormalizer.php b/src/applications/repository/data/PhabricatorRepositoryURINormalizer.php --- a/src/applications/repository/data/PhabricatorRepositoryURINormalizer.php +++ b/src/applications/repository/data/PhabricatorRepositoryURINormalizer.php @@ -27,10 +27,10 @@ * // URIs are very unlikely to be the same repository. * } * - * Because a repository can be hosted at arbitrarly many arbitrary URIs, there + * Because a repository can be hosted at arbitrarily many arbitrary URIs, there * is no way to completely prevent false negatives by only examining URIs * (that is, repositories with totally different URIs could really be the same). - * However, normalization is relatively agressive and false negatives should + * However, normalization is relatively aggressive and false negatives should * be rare: if normalization says two URIs are different repositories, they * probably are. * diff --git a/src/applications/repository/engine/PhabricatorRepositoryDiscoveryEngine.php b/src/applications/repository/engine/PhabricatorRepositoryDiscoveryEngine.php --- a/src/applications/repository/engine/PhabricatorRepositoryDiscoveryEngine.php +++ b/src/applications/repository/engine/PhabricatorRepositoryDiscoveryEngine.php @@ -703,7 +703,7 @@ // new repository for the first time is less important than any other // daemon task. - // If the repostitory has finished importing and we're just catching up + // If the repository has finished importing and we're just catching up // on recent commits, we schedule discovery at COMMIT priority, which is // slightly below the default priority. diff --git a/src/applications/repository/management/PhabricatorRepositoryManagementThawWorkflow.php b/src/applications/repository/management/PhabricatorRepositoryManagementThawWorkflow.php --- a/src/applications/repository/management/PhabricatorRepositoryManagementThawWorkflow.php +++ b/src/applications/repository/management/PhabricatorRepositoryManagementThawWorkflow.php @@ -90,7 +90,7 @@ $risk_message); $is_force = $args->getArg('force'); - $prompt = pht('Accept the possibilty of permanent data loss?'); + $prompt = pht('Accept the possibility of permanent data loss?'); if (!$is_force && !phutil_console_confirm($prompt)) { throw new PhutilArgumentUsageException( pht('User aborted the workflow.')); 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 @@ -1875,7 +1875,7 @@ /** - * Retrieve the sevice URI for the device hosting this repository. + * Retrieve the service URI for the device hosting this repository. * * See @{method:newConduitClient} for a general discussion of interacting * with repository services. This method provides lower-level resolution of @@ -2129,7 +2129,7 @@ if ($this->isGit()) { // $_ENV does not populate in CLI contexts if "E" is missing from // "variables_order" in PHP config. Currently, we do not require this - // to be configured. Since it may not be, explictitly bring expected Git + // to be configured. Since it may not be, explicitly bring expected Git // environmental variables into scope. This list is not exhaustive, but // only lists variables with a known impact on commit hook behavior. 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 @@ -362,7 +362,7 @@ return PhabricatorEnv::getEnvConfig('diffusion.ssh-port'); } - // If Phabricator is running on a nonstandard port, use that as the defualt + // If Phabricator is running on a nonstandard port, use that as the default // port for URIs with the same protocol. $is_http = ($protocol == self::BUILTIN_PROTOCOL_HTTP); diff --git a/src/applications/repository/storage/PhabricatorRepositoryWorkingCopyVersion.php b/src/applications/repository/storage/PhabricatorRepositoryWorkingCopyVersion.php --- a/src/applications/repository/storage/PhabricatorRepositoryWorkingCopyVersion.php +++ b/src/applications/repository/storage/PhabricatorRepositoryWorkingCopyVersion.php @@ -66,7 +66,7 @@ * may have committed and acknowledged a write on a node that lost the lock * partway through the write and is no longer reachable. * - * In particular, if a node loses its connection to the datbase the global + * In particular, if a node loses its connection to the database the global * lock is released by default. This is a durable lock which stays locked * by default. */ diff --git a/src/applications/repository/worker/commitchangeparser/PhabricatorRepositoryMercurialCommitChangeParserWorker.php b/src/applications/repository/worker/commitchangeparser/PhabricatorRepositoryMercurialCommitChangeParserWorker.php --- a/src/applications/repository/worker/commitchangeparser/PhabricatorRepositoryMercurialCommitChangeParserWorker.php +++ b/src/applications/repository/worker/commitchangeparser/PhabricatorRepositoryMercurialCommitChangeParserWorker.php @@ -216,7 +216,7 @@ $changes[$path]['changeType'] = DifferentialChangeType::TYPE_COPY_AWAY; } else if ($existing_type == DifferentialChangeType::TYPE_ADD) { - // This change removes a diretory and replaces it with a file. Mark + // This change removes a directory and replaces it with a file. Mark // it as "change" instead of "add". $changes[$path]['changeType'] = DifferentialChangeType::TYPE_CHANGE; } diff --git a/src/applications/search/engine/PhabricatorApplicationSearchEngine.php b/src/applications/search/engine/PhabricatorApplicationSearchEngine.php --- a/src/applications/search/engine/PhabricatorApplicationSearchEngine.php +++ b/src/applications/search/engine/PhabricatorApplicationSearchEngine.php @@ -359,7 +359,7 @@ * ); * * Any unspecified fields (including custom fields and fields added - * automatically by infrastruture) will be put in the middle. + * automatically by infrastructure) will be put in the middle. * * @return list Default ordering for field keys. */ diff --git a/src/applications/search/management/PhabricatorSearchManagementWorkflow.php b/src/applications/search/management/PhabricatorSearchManagementWorkflow.php --- a/src/applications/search/management/PhabricatorSearchManagementWorkflow.php +++ b/src/applications/search/management/PhabricatorSearchManagementWorkflow.php @@ -5,7 +5,7 @@ protected function validateClusterSearchConfig() { // Configuration is normally validated by setup self-checks on the web - // workflow, but users may reasonsably run `bin/search` commands after + // workflow, but users may reasonably run `bin/search` commands after // making manual edits to "local.json". Re-verify configuration here before // continuing. diff --git a/src/applications/search/menuitem/PhabricatorMotivatorProfileMenuItem.php b/src/applications/search/menuitem/PhabricatorMotivatorProfileMenuItem.php --- a/src/applications/search/menuitem/PhabricatorMotivatorProfileMenuItem.php +++ b/src/applications/search/menuitem/PhabricatorMotivatorProfileMenuItem.php @@ -80,7 +80,7 @@ private function getCatFacts() { return array( pht('Cats purr when they are happy, upset, or asleep.'), - pht('The first cats evolved on the savanah about 8,000 years ago.'), + pht('The first cats evolved on the savannah about 8,000 years ago.'), pht( 'Cats have a tail, two feet, between one and three ears, and two '. 'other feet.'), @@ -111,7 +111,7 @@ pht( 'Not all cats can retract their claws, but most of them can.'), pht( - 'In the wild, cats and racoons sometimes hunt together in packs.'), + 'In the wild, cats and raccoons sometimes hunt together in packs.'), pht( 'The Spanish word for cat is "cato". The biggest cat is called '. '"el cato".'), diff --git a/src/applications/settings/panel/PhabricatorSettingsPanel.php b/src/applications/settings/panel/PhabricatorSettingsPanel.php --- a/src/applications/settings/panel/PhabricatorSettingsPanel.php +++ b/src/applications/settings/panel/PhabricatorSettingsPanel.php @@ -143,7 +143,7 @@ /** * Return false to prevent this panel from being displayed or used. You can * do, e.g., configuration checks here, to determine if the feature your - * panel controls is unavailble in this install. By default, all panels are + * panel controls is unavailable in this install. By default, all panels are * enabled. * * @return bool True if the panel should be shown. diff --git a/src/applications/settings/query/PhabricatorUserPreferencesQuery.php b/src/applications/settings/query/PhabricatorUserPreferencesQuery.php --- a/src/applications/settings/query/PhabricatorUserPreferencesQuery.php +++ b/src/applications/settings/query/PhabricatorUserPreferencesQuery.php @@ -49,7 +49,7 @@ * If no settings exist for a user, a new empty settings object with * appropriate defaults is returned. * - * @param bool True to generat synthetic preferences for missing users. + * @param bool True to generate synthetic preferences for missing users. */ public function needSyntheticPreferences($synthetic) { $this->synthetic = $synthetic; diff --git a/src/applications/settings/setting/PhabricatorTimezoneSetting.php b/src/applications/settings/setting/PhabricatorTimezoneSetting.php --- a/src/applications/settings/setting/PhabricatorTimezoneSetting.php +++ b/src/applications/settings/setting/PhabricatorTimezoneSetting.php @@ -46,7 +46,7 @@ throw new Exception( pht( - 'Timezone "%s" is not a valid timezone identiifer.', + 'Timezone "%s" is not a valid timezone identifier.', $value)); } diff --git a/src/applications/subscriptions/controller/PhabricatorSubscriptionsEditController.php b/src/applications/subscriptions/controller/PhabricatorSubscriptionsEditController.php --- a/src/applications/subscriptions/controller/PhabricatorSubscriptionsEditController.php +++ b/src/applications/subscriptions/controller/PhabricatorSubscriptionsEditController.php @@ -83,7 +83,7 @@ } else { // TODO: Eventually, get rid of this once everything implements - // PhabriatorApplicationTransactionInterface. + // PhabricatorApplicationTransactionInterface. $editor = id(new PhabricatorSubscriptionsEditor()) ->setActor($viewer) diff --git a/src/applications/transactions/editengine/PhabricatorEditEngine.php b/src/applications/transactions/editengine/PhabricatorEditEngine.php --- a/src/applications/transactions/editengine/PhabricatorEditEngine.php +++ b/src/applications/transactions/editengine/PhabricatorEditEngine.php @@ -763,7 +763,7 @@ * Load an object given a configured query. * * @param PhabricatorPolicyAwareQuery Configured query. - * @param list List of required capabilitiy constants, or omit for + * @param list List of required capability constants, or omit for * defaults. * @return object|null Object, or null if no such object exists. * @task load @@ -1382,7 +1382,7 @@ * and that field is visible and editable for the user. * * For example, you can use it to test if a user is able to reassign tasks - * or not, prior to rendering dedicated UI for task reassingment. + * or not, prior to rendering dedicated UI for task reassignment. * * Note that this method does NOT test if the user can actually edit the * current object, just if they have access to the related field. 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 @@ -3701,7 +3701,7 @@ // If a later project in the list is an ancestor of this one, it will // have added itself to the map. If any ancestor of this project points - // at itself in the map, this project should be dicarded in favor of + // at itself in the map, this project should be discarded in favor of // that later ancestor. foreach ($project->getAncestorProjects() as $ancestor) { $ancestor_phid = $ancestor->getPHID(); diff --git a/src/applications/typeahead/datasource/PhabricatorTypeaheadDatasource.php b/src/applications/typeahead/datasource/PhabricatorTypeaheadDatasource.php --- a/src/applications/typeahead/datasource/PhabricatorTypeaheadDatasource.php +++ b/src/applications/typeahead/datasource/PhabricatorTypeaheadDatasource.php @@ -451,7 +451,7 @@ public static function isFunctionToken($token) { // We're looking for a "(" so that a string like "members(q" is identified // and parsed as a function call. This allows us to start generating - // results immeidately, before the user fully types out "members(quack)". + // results immediately, before the user fully types out "members(quack)". return (strpos($token, '(') !== false); } diff --git a/src/applications/uiexample/examples/PhabricatorStatusUIExample.php b/src/applications/uiexample/examples/PhabricatorStatusUIExample.php --- a/src/applications/uiexample/examples/PhabricatorStatusUIExample.php +++ b/src/applications/uiexample/examples/PhabricatorStatusUIExample.php @@ -33,7 +33,7 @@ $view->addItem( id(new PHUIStatusItemView()) ->setIcon(PHUIStatusItemView::ICON_WARNING, 'dark', pht('Caution')) - ->setTarget(pht('Pomegranite')) + ->setTarget(pht('Pomegranate')) ->setNote(pht('Lots of seeds. Watch out.'))); $view->addItem( diff --git a/src/docs/contributor/cla.diviner b/src/docs/contributor/cla.diviner --- a/src/docs/contributor/cla.diviner +++ b/src/docs/contributor/cla.diviner @@ -87,7 +87,7 @@ do not contribute to Phabricator. **Limitation of Liability**: The second benefit the CLA provides is that it -makes the terms of your contribition explicitly clear upfront, and it puts us +makes the terms of your contribution explicitly clear upfront, and it puts us in a much stronger legal position if a contributor later claims there is ambiguity about ownership of their work. We can point at the document they signed as proof that they consented to our use and understood the terms of @@ -100,7 +100,7 @@ and the litigation has now been ongoing for more than a decade. We want to avoid situations like this in the future by making the terms of -contibution explicit upfront. +contribution explicit upfront. Generally, we believe the terms of the CLA are fair and reasonable for contributors, and that the primary way contributors benefit from contributing @@ -158,7 +158,7 @@ If you are unsure, you should speak with your employer or a lawyer. If you contribute code you do not own under the individual CLA, you are exposing -yourself to liability. You may also be exposing us to liablity, but we'll have +yourself to liability. You may also be exposing us to liability, but we'll have the CLA on our side to show that we were unwilling pawns in your malicious scheme to defraud your employer. diff --git a/src/docs/contributor/describing_problems.diviner b/src/docs/contributor/describing_problems.diviner --- a/src/docs/contributor/describing_problems.diviner +++ b/src/docs/contributor/describing_problems.diviner @@ -140,7 +140,7 @@ Poor problem descriptions are a common issue in software development and extensively documented elsewhere. Here are some additional resources describing -how to describe problems and ask questions effectivey: +how to describe problems and ask questions effectively: - [[ http://www.catb.org/esr/faqs/smart-questions.html | How To Ask Questions The Smart Way ]], by Eric S. Raymond diff --git a/src/docs/contributor/internationalization.diviner b/src/docs/contributor/internationalization.diviner --- a/src/docs/contributor/internationalization.diviner +++ b/src/docs/contributor/internationalization.diviner @@ -198,7 +198,7 @@ meaning, but no native English speaker would speak or write like this. However, some languages have different subject-verb order rules or -colloquisalisms, and a word order which transliterates like this may sound more +colloquialisms, and a word order which transliterates like this may sound more natural to a native speaker. By translating fragments instead of complete sentences, you lock translators into English word order. diff --git a/src/docs/contributor/reproduction_steps.diviner b/src/docs/contributor/reproduction_steps.diviner --- a/src/docs/contributor/reproduction_steps.diviner +++ b/src/docs/contributor/reproduction_steps.diviner @@ -112,7 +112,7 @@ If you can't follow your steps because they depend on information which is not available on a clean instance (for example, a certain config setting), rewrite -them to include instrutions to create that information (for example, adjusting +them to include instructions to create that information (for example, adjusting the config to the problematic value). If you follow your instructions but the issue does not reproduce, the issue @@ -191,7 +191,7 @@ - Files like `A`, which have uppercase letters, do not work. With a simpler reproduction scenario, you can simplify your steps to be more -tailored and mimimal. This will help us pointpoint the issue more quickly and +tailored and minimal. This will help us pinpoint the issue more quickly and be more certain that we've understood and resolved it. It is more important that steps be complete than that they be simple, and it's diff --git a/src/docs/user/cluster/cluster.diviner b/src/docs/user/cluster/cluster.diviner --- a/src/docs/user/cluster/cluster.diviner +++ b/src/docs/user/cluster/cluster.diviner @@ -225,7 +225,7 @@ Configuring multiple notification hosts is simple and has no pre-requisites. With multiple notification hosts, you can survive the loss of any subset of -hosts as long as at least one host remains alive. Service may be breifly +hosts as long as at least one host remains alive. Service may be briefly disrupted directly after the incident which destroys the other hosts. Notifications are noncritical, so this normally has little practical impact diff --git a/src/docs/user/cluster/cluster_databases.diviner b/src/docs/user/cluster/cluster_databases.diviner --- a/src/docs/user/cluster/cluster_databases.diviner +++ b/src/docs/user/cluster/cluster_databases.diviner @@ -31,7 +31,7 @@ Phabricator applications //can// be partitioned across multiple database masters. This does not provide redundancy and generally does not increase -resiliance or resistance to data loss, but can help you scale and operate +resilience or resistance to data loss, but can help you scale and operate Phabricator. For details, see @{article:Cluster: Partitioning and Advanced Configuration}. diff --git a/src/docs/user/cluster/cluster_devices.diviner b/src/docs/user/cluster/cluster_devices.diviner --- a/src/docs/user/cluster/cluster_devices.diviner +++ b/src/docs/user/cluster/cluster_devices.diviner @@ -212,7 +212,7 @@ For example, you might run this command on `repo001` when using a shared key: ``` -$ ./bin/almanac register +$ ./bin/almanac register \ --device keywarden.mycompany.net \ --private-key /path/to/private-key \ --identify-as repo001.mycompany.net diff --git a/src/docs/user/cluster/cluster_partitioning.diviner b/src/docs/user/cluster/cluster_partitioning.diviner --- a/src/docs/user/cluster/cluster_partitioning.diviner +++ b/src/docs/user/cluster/cluster_partitioning.diviner @@ -21,7 +21,7 @@ it. Phabricator will normally run comfortably with a single database master even for large organizations. -Partitioning generally does not do much to increase resiliance or make it +Partitioning generally does not do much to increase resilience or make it easier to recover from disasters, and is primarily a mechanism for scaling and operational convenience. @@ -212,14 +212,14 @@ `persistent` //(bool)// Enables persistent connections. Defaults to off. -With persitent connections enabled, Phabricator will keep a pool of database +With persistent connections enabled, Phabricator will keep a pool of database connections open between web requests and reuse them when serving subsequent requests. The primary benefit of using persistent connections is that it will greatly reduce pressure on how quickly outbound TCP ports are opened and closed. After a TCP port closes, it normally can't be used again for about 60 seconds, so -rapidly cycling ports can cause resource exuastion. If you're seeing failures +rapidly cycling ports can cause resource exhaustion. If you're seeing failures because requests are unable to bind to an outbound port, enabling this option is likely to fix the issue. This option may also slightly increase performance. diff --git a/src/docs/user/cluster/cluster_repositories.diviner b/src/docs/user/cluster/cluster_repositories.diviner --- a/src/docs/user/cluster/cluster_repositories.diviner +++ b/src/docs/user/cluster/cluster_repositories.diviner @@ -440,7 +440,7 @@ When Phabricator can not tell which device in a cluster is a leader, it freezes the cluster because it is possible that some devices have less data and others -have more, and if it choses a leader arbitrarily it may destroy some data +have more, and if it chooses a leader arbitrarily it may destroy some data which you would prefer to retain. To resolve this, you need to tell Phabricator which device has the most diff --git a/src/docs/user/cluster/cluster_search.diviner b/src/docs/user/cluster/cluster_search.diviner --- a/src/docs/user/cluster/cluster_search.diviner +++ b/src/docs/user/cluster/cluster_search.diviner @@ -100,14 +100,14 @@ Service Type: Elasticsearch ====================== -The `elasticsearch` sevice type supports these options: +The `elasticsearch` service type supports these options: | Key | Description | |---|---| | `protocol` | Either `"http"` (default) or `"https"`. | `port` | Elasticsearch TCP port. | `version` | Elasticsearch version, either `2` or `5` (default). -| `path` | Path for the index. Defaults to `/phabriator`. Advanced. +| `path` | Path for the index. Defaults to `/phabricator`. Advanced. A typical `elasticsearch` service configuration looks like this: diff --git a/src/docs/user/configuration/configuration_guide.diviner b/src/docs/user/configuration/configuration_guide.diviner --- a/src/docs/user/configuration/configuration_guide.diviner +++ b/src/docs/user/configuration/configuration_guide.diviner @@ -116,7 +116,7 @@ = Webserver: Configuring lighttpd = -NOTE: Follow these instructions to use lighttpd. To use Apache or niginx, scroll +NOTE: Follow these instructions to use lighttpd. To use Apache or nginx, scroll up to their sections. For lighttpd, add a section like this to your lighttpd.conf: @@ -138,7 +138,7 @@ Finally, you should run the following commands to enable php support: - $ sudo apt-get install php5-cgi # for ubuntu; other distros should be similar + $ sudo apt-get install php5-cgi # for Ubuntu; other distros should be similar $ sudo lighty-enable-mod fastcgi-php Restart lighttpd after making your edits, then continue below. diff --git a/src/docs/user/configuration/configuring_backups.diviner b/src/docs/user/configuration/configuring_backups.diviner --- a/src/docs/user/configuration/configuring_backups.diviner +++ b/src/docs/user/configuration/configuring_backups.diviner @@ -64,7 +64,7 @@ repository. To back them up, copy them elsewhere. You can also just clone them and keep the clones up to date, or use -{nav Add Mirror} to have the mirror somewhere automatically. +{nav Add Mirror} to have them mirror somewhere automatically. Restore: Hosted Repositories diff --git a/src/docs/user/configuration/configuring_encryption.diviner b/src/docs/user/configuration/configuring_encryption.diviner --- a/src/docs/user/configuration/configuring_encryption.diviner +++ b/src/docs/user/configuration/configuring_encryption.diviner @@ -80,7 +80,7 @@ ================ The `raw` storage format is automatically selected for all newly uploaded -file data if no key is makred as the `default` key in the keyring. This is +file data if no key is marked as the `default` key in the keyring. This is the behavior of Phabricator if you haven't configured anything. This format stores raw data without modification. @@ -137,7 +137,7 @@ phabricator/ $ ./bin/files encode --as F123 [--key ] ``` -This will change the storage format of the sepcified file. +This will change the storage format of the specified file. Verifying Storage Formats diff --git a/src/docs/user/configuration/managing_garbage.diviner b/src/docs/user/configuration/managing_garbage.diviner --- a/src/docs/user/configuration/managing_garbage.diviner +++ b/src/docs/user/configuration/managing_garbage.diviner @@ -21,7 +21,7 @@ You can reconfigure the data retention policies for most collectors. -The default retention polcies should be suitable for most installs. However, +The default retention policies should be suitable for most installs. However, you might want to **decrease** retention to reduce the amount of disk space used by some high-volume log that you don't find particularly interesting, or to adhere to an organizational data retention policy. diff --git a/src/docs/user/field/conduit_changes.diviner b/src/docs/user/field/conduit_changes.diviner --- a/src/docs/user/field/conduit_changes.diviner +++ b/src/docs/user/field/conduit_changes.diviner @@ -11,7 +11,7 @@ methods and may update older methods. This document discusses API stability and how to minimize disruption when -transitionig between API versions. +transitioning between API versions. Method Statuses diff --git a/src/docs/user/field/darkconsole.diviner b/src/docs/user/field/darkconsole.diviner --- a/src/docs/user/field/darkconsole.diviner +++ b/src/docs/user/field/darkconsole.diviner @@ -131,7 +131,7 @@ - Unbatched queries which should be batched (see @{article:Performance: N+1 Query Problem}). - Opportunities to improve performance with caching. - - General goofiness in how service calls are woking. + - General goofiness in how service calls are working. If the services tab looks fine, and particularly if a page is slow but the "All Services" cost is small, that may indicate a problem in PHP. The best diff --git a/src/docs/user/field/performance.diviner b/src/docs/user/field/performance.diviner --- a/src/docs/user/field/performance.diviner +++ b/src/docs/user/field/performance.diviner @@ -34,7 +34,7 @@ to download over the network, and your browser will probably not be able to render it especially quickly. -We may be able to improve perfomance in some cases, but Phabricator is not +We may be able to improve performance in some cases, but Phabricator is not magic and can not wish away real complexity. The best solution to these problems is usually to find another way to solve your problem: for example, maybe the 100MB document can be split into several smaller documents. @@ -51,7 +51,7 @@ inherent complexity, like these: - {icon times, color=red} A 100MB wiki page takes a long time to render. - - {icon times, color=red} A turing-complete simulation of Conway's Game of + - {icon times, color=red} A Turing-complete simulation of Conway's Game of Life implemented in 958,000 Herald rules executes slowly. - {icon times, color=red} Uploading an 8GB file takes several minutes. diff --git a/src/docs/user/field/repository_imports.diviner b/src/docs/user/field/repository_imports.diviner --- a/src/docs/user/field/repository_imports.diviner +++ b/src/docs/user/field/repository_imports.diviner @@ -127,7 +127,7 @@ daemons were killed halfway through the work. These commits will retry eventually and usually succeed, but some of the retry -time limits are very conserative (up to 24 hours) and you might not want to +time limits are very conservative (up to 24 hours) and you might not want to wait that long. In the Daemon console, temporarily failures usually look like tasks in the @@ -137,7 +137,7 @@ In the daemon log, these temporary failures might have created log entries, but might also not have. For example, if the failure was rooted in a network issue -that probably will create a log entry, but if the faiulre was rooted in the +that probably will create a log entry, but if the failure was rooted in the daemons being abruptly killed that may not create a log entry. You can follow the instructions from "Handling Permanent Failures" above to @@ -167,7 +167,7 @@ ============== In rare cases, the actual tasks may be lost from the task queue. Usually, they -have been stolen by gremlins or spritied away by ghosts, or someone may have +have been stolen by gremlins or spirited away by ghosts, or someone may have been too ambitious with running manual SQL commands and deleted a bunch of extra things they shouldn't have. @@ -226,7 +226,7 @@ ============ Broadly, `bin/repository` contains several useful debugging commands which -let you figure out where failures are occuring. You can add the `--trace` flag +let you figure out where failures are occurring. You can add the `--trace` flag to any command to get more details about what it is doing. For any command, you can use `help` to learn more about what it does and which flag it takes: diff --git a/src/docs/user/field/xhprof.diviner b/src/docs/user/field/xhprof.diviner --- a/src/docs/user/field/xhprof.diviner +++ b/src/docs/user/field/xhprof.diviner @@ -88,7 +88,7 @@ to go down further to identify and understand it. Conversely, if the "Wall Time (Exclusive)" column is large, or the children -of a call are all cheap, there's probably something expesive happening in the +of a call are all cheap, there's probably something expensive happening in the call itself. The "Count" column can also sometimes tip you off that something is amiss, if diff --git a/src/docs/user/userguide/almanac.diviner b/src/docs/user/userguide/almanac.diviner --- a/src/docs/user/userguide/almanac.diviner +++ b/src/docs/user/userguide/almanac.diviner @@ -122,7 +122,7 @@ **Bindings**: Bindings are connections between services and interfaces. They tell callers which devices host a named service. -**Networks**: Networks allow Almanac to distingiush between addresses on +**Networks**: Networks allow Almanac to distinguish between addresses on different networks, like VPNs vs the public internet. If you have hosts in different VPNs or on private networks, you might have diff --git a/src/docs/user/userguide/arcanist.diviner b/src/docs/user/userguide/arcanist.diviner --- a/src/docs/user/userguide/arcanist.diviner +++ b/src/docs/user/userguide/arcanist.diviner @@ -3,7 +3,7 @@ Guide to Arcanist, a command-line interface to Phabricator. -Arcanists provides command-line access to many Phabricator tools (like +Arcanist provides command-line access to many Phabricator tools (like Differential, Files, and Paste), integrates with static analysis ("lint") and unit tests, and manages common workflows like getting changes into Differential for review. diff --git a/src/docs/user/userguide/arcanist_lint.diviner b/src/docs/user/userguide/arcanist_lint.diviner --- a/src/docs/user/userguide/arcanist_lint.diviner +++ b/src/docs/user/userguide/arcanist_lint.diviner @@ -169,7 +169,7 @@ review. In that context, the severities behave like this: - `error` When a file contains lint errors, they are always reported. These - are intended to be severe problems, like a syntax error. Unresoved lint + are intended to be severe problems, like a syntax error. Unresolved lint errors require you to confirm that you want to continue. - `warning` When a file contains warnings, they are reported by default only if they appear on lines that you have changed. They are intended to be diff --git a/src/docs/user/userguide/calendar_exports.diviner b/src/docs/user/userguide/calendar_exports.diviner --- a/src/docs/user/userguide/calendar_exports.diviner +++ b/src/docs/user/userguide/calendar_exports.diviner @@ -61,7 +61,7 @@ - Example: All events tagged `#meetup`. - Select the {nav Use Results... > Export Query as .ics} action to turn the query into an export. - - Name the export with a descritive name. + - Name the export with a descriptive name. - Select a policy mode for the export (see below for discussion). - Click {nav Create New Export} to finish the process. diff --git a/src/docs/user/userguide/calendar_imports.diviner b/src/docs/user/userguide/calendar_imports.diviner --- a/src/docs/user/userguide/calendar_imports.diviner +++ b/src/docs/user/userguide/calendar_imports.diviner @@ -10,7 +10,7 @@ @{article:User Guide: Prototype Applications}. You can import events into Phabricator to other calendar applications or from -`.ics` files. This document will guide you through how to importe event data +`.ics` files. This document will guide you through how to import event data into Phabricator. When you import events from another application, they can not be edited in diff --git a/src/docs/user/userguide/conduit_edit.diviner b/src/docs/user/userguide/conduit_edit.diviner --- a/src/docs/user/userguide/conduit_edit.diviner +++ b/src/docs/user/userguide/conduit_edit.diviner @@ -37,7 +37,7 @@ To edit objects, pass a list of transactions and use `objectIdentifier` to specify which object to apply them to. You can normally pass an ID or PHID, -and many applicaitons also allow you to pass a monogram (for example, you can +and many applications also allow you to pass a monogram (for example, you can edit a task by passing `T123`). diff --git a/src/docs/user/userguide/diffusion_api.diviner b/src/docs/user/userguide/diffusion_api.diviner --- a/src/docs/user/userguide/diffusion_api.diviner +++ b/src/docs/user/userguide/diffusion_api.diviner @@ -147,7 +147,7 @@ ======================= Now that any URIs have been configured, activate the repository with another -call to `diffusion.repository.edit`. This time, modify the existing repostitory +call to `diffusion.repository.edit`. This time, modify the existing repository instead of creating a new one: ``` diff --git a/src/docs/user/userguide/diffusion_hosting.diviner b/src/docs/user/userguide/diffusion_hosting.diviner --- a/src/docs/user/userguide/diffusion_hosting.diviner +++ b/src/docs/user/userguide/diffusion_hosting.diviner @@ -151,7 +151,7 @@ This is just a template. In the real configuration file, you need to: - - Replace `www-user`, `dameon-user` and `vcs-user` with the correct + - Replace `www-user`, `daemon-user` and `vcs-user` with the correct usernames for your system. - List every binary that these users need access to, as described above. - Make sure each binary path is the full path to the correct binary location diff --git a/src/docs/user/userguide/diffusion_managing.diviner b/src/docs/user/userguide/diffusion_managing.diviner --- a/src/docs/user/userguide/diffusion_managing.diviner +++ b/src/docs/user/userguide/diffusion_managing.diviner @@ -121,7 +121,7 @@ scalable workflows rarely rewrite repository history and it's easy to make mistakes which are expensive to correct if this protection is disabled. -If you do occasionally need to rewite published history, you can treat this +If you do occasionally need to rewrite published history, you can treat this option like a safety: disable it, perform required rewrites, then enable it again. @@ -148,7 +148,7 @@ - the repository will be hidden from view in default queries. When repositories are created for the first time, they are deactivated. This -gives you an opportuinty to customize settings, like adjusting policies or +gives you an opportunity to customize settings, like adjusting policies or configuring a URI to observe. You must activate a repository before it will start working normally. @@ -177,7 +177,7 @@ manage repository access policies. You can configure granular access policies for each repository to control who -can view, clone, administate, and push to the repository. +can view, clone, administrate, and push to the repository. Policies: View @@ -302,7 +302,7 @@ The **Actions** panel can configure notifications and publishing behavior. Normally, Phabricator publishes notifications when it discovers new commits. -You can disable publishing for a repository by turning off **Publish/Noitfy**. +You can disable publishing for a repository by turning off **Publish/Notify**. This will disable notifications, feed, and Herald (including audits and build plans) for this repository. @@ -347,7 +347,7 @@ ================== Diffusion uses repository identifiers and information about the commit itself -to generate globally unique identifers for each commit, like `rE12345`. +to generate globally unique identifiers for each commit, like `rE12345`. Each commit may have several identifiers: diff --git a/src/docs/user/userguide/diffusion_uris.diviner b/src/docs/user/userguide/diffusion_uris.diviner --- a/src/docs/user/userguide/diffusion_uris.diviner +++ b/src/docs/user/userguide/diffusion_uris.diviner @@ -20,7 +20,7 @@ repository. This will create a read-only copy of the repository in Phabricator. **Mirror Repositories**: Phabricator can publish any repository to mirrors, -overwiting them with an exact copy of the repository that stays up to date as +overwriting them with an exact copy of the repository that stays up to date as the source changes. This works with both local repositories that Phabricator is hosting and remote repositories that Phabricator is observing. @@ -293,7 +293,7 @@ human-readable URI defaults to //Always// and the others default to //Never//. **Always**: This URI will be shown to users as a clone/checkout URI. You can -add URIs in this mode to cutomize exactly what users are shown. +add URIs in this mode to customize exactly what users are shown. **Never**: This URI will not be shown to users. You can hide less-preferred URIs to guide users to the URIs they should be using to interact with the diff --git a/src/docs/user/userguide/drydock.diviner b/src/docs/user/userguide/drydock.diviner --- a/src/docs/user/userguide/drydock.diviner +++ b/src/docs/user/userguide/drydock.diviner @@ -151,7 +151,7 @@ work. Drydock is intended to solve resource management problems at very large scales -and minimzes blocking operations, locks, and artificial sequencing. Drydock is +and minimizes blocking operations, locks, and artificial sequencing. Drydock is designed to fully utilize an almost arbitrarily large pool of resources and improve performance roughly linearly with available hardware. diff --git a/src/docs/user/userguide/drydock_blueprints.diviner b/src/docs/user/userguide/drydock_blueprints.diviner --- a/src/docs/user/userguide/drydock_blueprints.diviner +++ b/src/docs/user/userguide/drydock_blueprints.diviner @@ -47,7 +47,7 @@ This also broadly prevents Drydock from surprising you by coming up with a valid but unintended solution to an allocation problem which runs some -operation on resources that are techincally suitable but not desirable. For +operation on resources that are technically suitable but not desirable. For example, you may not want your Android builds running on your iPhone build tier, even if there's no technical reason they can't. diff --git a/src/docs/user/userguide/drydock_security.diviner b/src/docs/user/userguide/drydock_security.diviner --- a/src/docs/user/userguide/drydock_security.diviner +++ b/src/docs/user/userguide/drydock_security.diviner @@ -141,7 +141,7 @@ Even if the host is not privileged, many Drydock processes have some level of privilege (enabling them to clone repositories, or report test results back to Phabricator). Be aware that tests can hijack credentials they are run with, -and potentialy hijack credentials given to other processes on the same hosts. +and potentially hijack credentials given to other processes on the same hosts. You should use credentials with a minimum set of privileges and assume all processes on a host have the highest level of access that any process on the host has. diff --git a/src/docs/user/userguide/forms.diviner b/src/docs/user/userguide/forms.diviner --- a/src/docs/user/userguide/forms.diviner +++ b/src/docs/user/userguide/forms.diviner @@ -34,7 +34,7 @@ comment form if you're working in an application which supports comments. This can streamline the edit workflow for less experienced users. -Anyone can use prefiling, but you must have permission to configure an +Anyone can use prefilling, but you must have permission to configure an application in order to modify the application's forms. By default, only administrators can configure applications. diff --git a/src/docs/user/userguide/herald.diviner b/src/docs/user/userguide/herald.diviner --- a/src/docs/user/userguide/herald.diviner +++ b/src/docs/user/userguide/herald.diviner @@ -126,7 +126,7 @@ For example, if you want to match revisions which add or remove calls to a "muffinize" function, //but only in JS files//, you can set the value to `["/\\.js$/", "/muffinize/"]` or similar. This condition is satisfied only -when the filename matches the first expression and the conent matches the +when the filename matches the first expression and the content matches the second expression. **Another Herald rule**: you can create Herald rules which depend on other diff --git a/src/docs/user/userguide/jump.diviner b/src/docs/user/userguide/jump.diviner --- a/src/docs/user/userguide/jump.diviner +++ b/src/docs/user/userguide/jump.diviner @@ -1,7 +1,7 @@ @title Search User Guide: Shortcuts @group userguide -Command reference for global search shorcuts. +Command reference for global search shortcuts. Overview ======== diff --git a/src/docs/user/userguide/owners.diviner b/src/docs/user/userguide/owners.diviner --- a/src/docs/user/userguide/owners.diviner +++ b/src/docs/user/userguide/owners.diviner @@ -24,7 +24,7 @@ Any files in those directories are considered to be part of the package, and you can now conveniently refer to them (for example, in a Herald rule) by -refering to the package instead of copy/pasting a huge regular expression +referring to the package instead of copy/pasting a huge regular expression into a bunch of places. If new source files are later added, or the scope of the package otherwise diff --git a/src/docs/user/userguide/projects.diviner b/src/docs/user/userguide/projects.diviner --- a/src/docs/user/userguide/projects.diviner +++ b/src/docs/user/userguide/projects.diviner @@ -14,7 +14,7 @@ For example, you can create projects to provide: - - **Organization**: Create a project to represent a product or initative, + - **Organization**: Create a project to represent a product or initiative, then use it to organize related work. - **Groups**: Create a project to represent a group of people (like a team), then add members of the group as project members. diff --git a/src/docs/user/userguide/remarkup.diviner b/src/docs/user/userguide/remarkup.diviner --- a/src/docs/user/userguide/remarkup.diviner +++ b/src/docs/user/userguide/remarkup.diviner @@ -366,7 +366,7 @@ T123#412 # Link to comment id #412 of task T123 -See the Phabricator configuraton setting `remarkup.ignored-object-names` to +See the Phabricator configuration setting `remarkup.ignored-object-names` to modify this behavior. = Embedding Objects diff --git a/src/docs/user/userguide/search.diviner b/src/docs/user/userguide/search.diviner --- a/src/docs/user/userguide/search.diviner +++ b/src/docs/user/userguide/search.diviner @@ -87,7 +87,7 @@ You can share queries with other users by sending them the URL. This will run the same query for them with all the parameters you've set (they may see -different results than you do, because they may not have the same permisions). +different results than you do, because they may not have the same permissions). Typeaheads diff --git a/src/docs/user/userguide/spaces.diviner b/src/docs/user/userguide/spaces.diviner --- a/src/docs/user/userguide/spaces.diviner +++ b/src/docs/user/userguide/spaces.diviner @@ -70,7 +70,7 @@ Space Policies ============== -Briefly, spacess affect policies like this: +Briefly, spaces affect policies like this: - Spaces apply their view policy to all objects inside the space. - Space policies are absolute, and stronger than all other policies. A diff --git a/src/docs/user/userguide/users.diviner b/src/docs/user/userguide/users.diviner --- a/src/docs/user/userguide/users.diviner +++ b/src/docs/user/userguide/users.diviner @@ -73,7 +73,7 @@ is created. Some options can be configured for mailing lists by browsing to the list user's -profile and clicking {nav Edit Settings}. You can change the addresss for a +profile and clicking {nav Edit Settings}. You can change the address for a list by editing "Email Addresses" here, choose the language and format for email the list receives, and customize which actions the list is notified about. diff --git a/src/infrastructure/customfield/field/PhabricatorCustomFieldList.php b/src/infrastructure/customfield/field/PhabricatorCustomFieldList.php --- a/src/infrastructure/customfield/field/PhabricatorCustomFieldList.php +++ b/src/infrastructure/customfield/field/PhabricatorCustomFieldList.php @@ -160,7 +160,7 @@ if ($value !== null) { switch ($field->getStyleForPropertyView()) { case 'header': - // We want to hide headers if the fields the're assciated with + // We want to hide headers if the fields they're associated with // don't actually produce any visible properties. For example, in a // list like this: // diff --git a/src/infrastructure/daemon/workers/query/PhabricatorWorkerTriggerQuery.php b/src/infrastructure/daemon/workers/query/PhabricatorWorkerTriggerQuery.php --- a/src/infrastructure/daemon/workers/query/PhabricatorWorkerTriggerQuery.php +++ b/src/infrastructure/daemon/workers/query/PhabricatorWorkerTriggerQuery.php @@ -5,7 +5,7 @@ // NOTE: This is a PolicyAware query so it can work with other infrastructure // like handles; triggers themselves are low-level and do not have - // meaninguful policies. + // meaningful policies. const ORDER_ID = 'id'; const ORDER_EXECUTION = 'execution'; @@ -69,7 +69,7 @@ protected function nextPage(array $page) { // NOTE: We don't implement paging because we don't currently ever need - // it and paging ORDER_EXCUTION is a hassle. + // it and paging ORDER_EXECUTION is a hassle. throw new PhutilMethodNotImplementedException(); } diff --git a/src/infrastructure/diff/view/PHUIDiffGraphView.php b/src/infrastructure/diff/view/PHUIDiffGraphView.php --- a/src/infrastructure/diff/view/PHUIDiffGraphView.php +++ b/src/infrastructure/diff/view/PHUIDiffGraphView.php @@ -160,7 +160,7 @@ $terminated[$ii] = true; - // If this thread is joinining some other node here, we don't want + // If this thread is joining some other node here, we don't want // to terminate it. if (isset($graph[$key + 1])) { $joins = $graph[$key + 1]['join']; diff --git a/src/infrastructure/diff/view/PHUIDiffInlineCommentTableScaffold.php b/src/infrastructure/diff/view/PHUIDiffInlineCommentTableScaffold.php --- a/src/infrastructure/diff/view/PHUIDiffInlineCommentTableScaffold.php +++ b/src/infrastructure/diff/view/PHUIDiffInlineCommentTableScaffold.php @@ -4,7 +4,7 @@ * Wraps an inline comment row scaffold in a table. * * This scaffold is used to ship inlines over the wire to the client, so they - * arrive in a form that's easy to mainipulate (a valid table node). + * arrive in a form that's easy to manipulate (a valid table node). */ final class PHUIDiffInlineCommentTableScaffold extends AphrontView { diff --git a/src/infrastructure/env/PhabricatorEnv.php b/src/infrastructure/env/PhabricatorEnv.php --- a/src/infrastructure/env/PhabricatorEnv.php +++ b/src/infrastructure/env/PhabricatorEnv.php @@ -670,7 +670,7 @@ * Detect if a URI identifies some valid linkable remote resource. * * @param string URI to test. - * @return bool True if a URI idenfies a remote resource with an allowed + * @return bool True if a URI identifies a remote resource with an allowed * protocol. * @task uri */ diff --git a/src/infrastructure/internationalization/translation/PhabricatorUSEnglishTranslation.php b/src/infrastructure/internationalization/translation/PhabricatorUSEnglishTranslation.php --- a/src/infrastructure/internationalization/translation/PhabricatorUSEnglishTranslation.php +++ b/src/infrastructure/internationalization/translation/PhabricatorUSEnglishTranslation.php @@ -730,7 +730,7 @@ '%s added %s reverting commit(s) for %s: %s.' => array( array( '%s added a reverting commit for %3$s: %4$s.', - '%s added reverting commitsi for %3$s: %4$s.', + '%s added reverting commits for %3$s: %4$s.', ), ), @@ -1564,7 +1564,7 @@ '%s uninvited %s attendee(s): %s.' => '%s uninvited: %3$s.', - '%s invited %s attendee(s): %s; uninvinted %s attendee(s): %s.' => + '%s invited %s attendee(s): %s; uninvited %s attendee(s): %s.' => '%s invited: %3$s; uninvited: %5$s.', '%s invited %s attendee(s) to %s: %s.' => @@ -1573,7 +1573,7 @@ '%s uninvited %s attendee(s) to %s: %s.' => '%s removed invites for %3$s: %4$s.', - '%s updated the invite list for %s, invited %s: %s; uninvinted %s: %s.' => + '%s updated the invite list for %s, invited %s: %s; uninvited %s: %s.' => '%s updated the invite list for %s, invited: %4$s; uninvited: %6$s.', 'Restart %s build(s)?' => array( diff --git a/src/infrastructure/query/policy/PhabricatorCursorPagedPolicyAwareQuery.php b/src/infrastructure/query/policy/PhabricatorCursorPagedPolicyAwareQuery.php --- a/src/infrastructure/query/policy/PhabricatorCursorPagedPolicyAwareQuery.php +++ b/src/infrastructure/query/policy/PhabricatorCursorPagedPolicyAwareQuery.php @@ -175,7 +175,7 @@ * So, generally, internal paging must bypass policy controls. * * This method returns the appropriate viewer, based on the context in which - * the paging is occuring. + * the paging is occurring. * * @return PhabricatorUser Viewer for executing paging queries. */ @@ -536,7 +536,7 @@ * * @param AphrontDatabaseConnection Connection query will execute on. * @param list Column description dictionaries. - * @param map Additional constuction options. + * @param map Additional construction options. * @return string Query clause. * @task paging */ diff --git a/src/infrastructure/storage/lisk/LiskDAO.php b/src/infrastructure/storage/lisk/LiskDAO.php --- a/src/infrastructure/storage/lisk/LiskDAO.php +++ b/src/infrastructure/storage/lisk/LiskDAO.php @@ -1121,7 +1121,7 @@ $this->willSaveObject(); $data = $this->getAllLiskPropertyValues(); - // Remove colums flagged as nonmutable from the update statement. + // Remove columns flagged as nonmutable from the update statement. $no_mutate = $this->getConfigOption(self::CONFIG_NO_MUTATE); if ($no_mutate) { foreach ($no_mutate as $column) { diff --git a/src/infrastructure/storage/lisk/__tests__/LiskFixtureTestCase.php b/src/infrastructure/storage/lisk/__tests__/LiskFixtureTestCase.php --- a/src/infrastructure/storage/lisk/__tests__/LiskFixtureTestCase.php +++ b/src/infrastructure/storage/lisk/__tests__/LiskFixtureTestCase.php @@ -98,7 +98,7 @@ $obj = new HarbormasterObject(); $conn_w = $obj->establishConnection('w'); - // Test that the counter bascially behaves as expected. + // Test that the counter basically behaves as expected. $this->assertEqual(1, LiskDAO::loadNextCounterValue($conn_w, 'a')); $this->assertEqual(2, LiskDAO::loadNextCounterValue($conn_w, 'a')); $this->assertEqual(3, LiskDAO::loadNextCounterValue($conn_w, 'a')); diff --git a/src/infrastructure/storage/management/workflow/PhabricatorStorageManagementWorkflow.php b/src/infrastructure/storage/management/workflow/PhabricatorStorageManagementWorkflow.php --- a/src/infrastructure/storage/management/workflow/PhabricatorStorageManagementWorkflow.php +++ b/src/infrastructure/storage/management/workflow/PhabricatorStorageManagementWorkflow.php @@ -181,8 +181,8 @@ if (!$this->force && !$api->isCharacterSetAvailable('utf8mb4')) { $message = pht( "You have an old version of MySQL (older than 5.5) which does not ". - "support the utf8mb4 character set. We strongly recomend upgrading to ". - "5.5 or newer.\n\n". + "support the utf8mb4 character set. We strongly recommend upgrading ". + "to 5.5 or newer.\n\n". "If you apply adjustments now and later update MySQL to 5.5 or newer, ". "you'll need to apply adjustments again (and they will take a long ". "time).\n\n". diff --git a/src/infrastructure/util/PhabricatorHash.php b/src/infrastructure/util/PhabricatorHash.php --- a/src/infrastructure/util/PhabricatorHash.php +++ b/src/infrastructure/util/PhabricatorHash.php @@ -11,7 +11,7 @@ * weak. Callers should prefer @{method:digestWithNamedKey}. * * @param string Input string. - * @return string 32-byte hexidecimal SHA1+HMAC hash. + * @return string 32-byte hexadecimal SHA1+HMAC hash. */ public static function weakDigest($string, $key = null) { if ($key === null) { diff --git a/src/infrastructure/util/password/PhabricatorPasswordHasher.php b/src/infrastructure/util/password/PhabricatorPasswordHasher.php --- a/src/infrastructure/util/password/PhabricatorPasswordHasher.php +++ b/src/infrastructure/util/password/PhabricatorPasswordHasher.php @@ -70,7 +70,7 @@ /** * Return an indicator of this hasher's strength. When choosing to hash - * new passwords, the strongest available hasher which is usuable for new + * new passwords, the strongest available hasher which is usable for new * passwords will be used, and the presence of a stronger hasher will * prompt users to update their hashes. * @@ -208,7 +208,7 @@ * Get all available password hashers. This may include hashers which can not * actually be used (for example, a required extension is missing). * - * @return list Hasher objects. + * @return list Hasher objects. * @task hashing */ public static function getAllHashers() { @@ -243,7 +243,7 @@ * Get all usable password hashers. This may include hashers which are * not desirable or advisable. * - * @return list Hasher objects. + * @return list Hasher objects. * @task hashing */ public static function getAllUsableHashers() { @@ -280,7 +280,7 @@ /** - * Get the hashser for a given stored hash. + * Get the hasher for a given stored hash. * * @return PhabricatorPasswordHasher Corresponding hasher. * @task hashing diff --git a/src/view/AphrontView.php b/src/view/AphrontView.php --- a/src/view/AphrontView.php +++ b/src/view/AphrontView.php @@ -42,7 +42,7 @@ /** - * Test if a viewer has been set on this elmeent. + * Test if a viewer has been set on this element. * * @return bool True if a viewer is available. */ diff --git a/src/view/page/PhabricatorStandardPageView.php b/src/view/page/PhabricatorStandardPageView.php --- a/src/view/page/PhabricatorStandardPageView.php +++ b/src/view/page/PhabricatorStandardPageView.php @@ -272,7 +272,7 @@ 'doc_href' => $doc_href, 'message' => pht( 'Phabricator thinks you are using %s, but your '. - 'client is conviced that it is using %s. This is a serious '. + 'client is convinced that it is using %s. This is a serious '. 'misconfiguration with subtle, but significant, consequences.', $server_protocol, $client_protocol), )); diff --git a/src/view/phui/PHUIHeaderView.php b/src/view/phui/PHUIHeaderView.php --- a/src/view/phui/PHUIHeaderView.php +++ b/src/view/phui/PHUIHeaderView.php @@ -166,7 +166,7 @@ $classes[] = 'phui-header-shell'; if ($this->noBackground) { - $classes[] = 'phui-header-no-backgound'; + $classes[] = 'phui-header-no-background'; } if ($this->bleedHeader) { diff --git a/webroot/rsrc/css/phui/phui-document.css b/webroot/rsrc/css/phui/phui-document.css --- a/webroot/rsrc/css/phui/phui-document.css +++ b/webroot/rsrc/css/phui/phui-document.css @@ -35,13 +35,13 @@ } .phui-document-content - .phui-header-shell.phui-header-no-backgound { + .phui-header-shell.phui-header-no-background { border-bottom: 1px solid {$thinblueborder}; margin: 0 0 16px 0; } .phui-document-content - .phui-header-shell.phui-header-no-backgound + .phui-header-shell.phui-header-no-background .phui-header-view { padding: 8px 0 4px; } diff --git a/webroot/rsrc/css/phui/phui-header-view.css b/webroot/rsrc/css/phui/phui-header-view.css --- a/webroot/rsrc/css/phui/phui-header-view.css +++ b/webroot/rsrc/css/phui/phui-header-view.css @@ -34,7 +34,7 @@ vertical-align: middle; } -body .phui-header-shell.phui-header-no-backgound { +body .phui-header-shell.phui-header-no-background { background-color: transparent; border: none; } diff --git a/webroot/rsrc/js/application/conpherence/behavior-menu.js b/webroot/rsrc/js/application/conpherence/behavior-menu.js --- a/webroot/rsrc/js/application/conpherence/behavior-menu.js +++ b/webroot/rsrc/js/application/conpherence/behavior-menu.js @@ -94,7 +94,7 @@ var _oldDevice = null; /** - * Initializes this behavior based on all the configuraton jonx and the + * Initializes this behavior based on all the configuration jonx and the * result of JX.Device.getDevice(); */ function init() { diff --git a/webroot/rsrc/js/application/conpherence/behavior-participant-pane.js b/webroot/rsrc/js/application/conpherence/behavior-participant-pane.js --- a/webroot/rsrc/js/application/conpherence/behavior-participant-pane.js +++ b/webroot/rsrc/js/application/conpherence/behavior-participant-pane.js @@ -69,7 +69,7 @@ // While the user is removing themselves, disable the notification // update behavior. If we don't do this, the user can get an error // when they remove themselves about permissions as the notification - // code tries to load what jist happened. + // code tries to load what just happened. var loadedPhid = threadManager.getLoadedThreadPHID(); threadManager.setLoadedThreadPHID(null); diff --git a/webroot/rsrc/js/application/diffusion/DiffusionLocateFileSource.js b/webroot/rsrc/js/application/diffusion/DiffusionLocateFileSource.js --- a/webroot/rsrc/js/application/diffusion/DiffusionLocateFileSource.js +++ b/webroot/rsrc/js/application/diffusion/DiffusionLocateFileSource.js @@ -260,7 +260,7 @@ /** * Score a matching string by finding the longest prefix of the search - * query it contains continguously. + * query it contains contiguously. */ scoreMatch: function(haystack, haypos, search) { var pos = 0; diff --git a/webroot/rsrc/js/application/pholio/behavior-pholio-mock-view.js b/webroot/rsrc/js/application/pholio/behavior-pholio-mock-view.js --- a/webroot/rsrc/js/application/pholio/behavior-pholio-mock-view.js +++ b/webroot/rsrc/js/application/pholio/behavior-pholio-mock-view.js @@ -201,7 +201,7 @@ } function render_image_header(image) { - // Render image dimensions and visible size. If we have this infomation + // Render image dimensions and visible size. If we have this information // from the server we can display some of it immediately; otherwise, we need // to wait for the image to load so we can read dimension information from // it. diff --git a/webroot/rsrc/js/core/Prefab.js b/webroot/rsrc/js/core/Prefab.js --- a/webroot/rsrc/js/core/Prefab.js +++ b/webroot/rsrc/js/core/Prefab.js @@ -184,7 +184,7 @@ var self_hits = {}; // We'll put matches where the user's input is a prefix of the name - // above mathches where that isn't true. + // above matches where that isn't true. var prefix_hits = {}; var tokens = this.tokenize(value); @@ -212,7 +212,7 @@ } // If one result is open and one is closed, show the open result - // first. The "!" tricks here are becaused closed values are display + // first. The "!" tricks here are because closed values are display // strings, so the value is either `null` or some truthy string. If // we compare the values directly, we'll apply this rule to two // objects which are both closed but for different reasons, like diff --git a/webroot/rsrc/js/core/behavior-device.js b/webroot/rsrc/js/core/behavior-device.js --- a/webroot/rsrc/js/core/behavior-device.js +++ b/webroot/rsrc/js/core/behavior-device.js @@ -27,7 +27,7 @@ var self = JX.Device; // Even when we emit a '' tag which tells - // devices to fit the conent to the screen width, we'll sometimes measure + // devices to fit the content to the screen width, we'll sometimes measure // a viewport dimension which is larger than the available screen width, // particularly if we check too early. diff --git a/webroot/rsrc/js/phuix/PHUIXAutocomplete.js b/webroot/rsrc/js/phuix/PHUIXAutocomplete.js --- a/webroot/rsrc/js/phuix/PHUIXAutocomplete.js +++ b/webroot/rsrc/js/phuix/PHUIXAutocomplete.js @@ -543,7 +543,7 @@ // If the input is terminated by a space or another word-terminating // punctuation mark, we're going to deactivate if the results can not - // be refined by addding more words. + // be refined by adding more words. // The idea is that if you type "@alan ab", you're allowed to keep // editing "ab" until you type a space, period, or other terminator, diff --git a/webroot/rsrc/js/phuix/PHUIXDropdownMenu.js b/webroot/rsrc/js/phuix/PHUIXDropdownMenu.js --- a/webroot/rsrc/js/phuix/PHUIXDropdownMenu.js +++ b/webroot/rsrc/js/phuix/PHUIXDropdownMenu.js @@ -124,7 +124,7 @@ } // If this action was built dynamically with PHUIXActionView, don't - // do anything by default. The caller is repsonsible for installing a + // do anything by default. The caller is responsible for installing a // handler if they want to react to clicks. if (e.getNode('phuix-action-view')) { return;