diff --git a/resources/sql/patches/098.heraldruletypemigration.php b/resources/sql/patches/098.heraldruletypemigration.php --- a/resources/sql/patches/098.heraldruletypemigration.php +++ b/resources/sql/patches/098.heraldruletypemigration.php @@ -41,7 +41,7 @@ $rule->getRuleType(), $rule->getID()); - echo "Setting rule '" . $rule->getName() . "' to personal. "; + echo "Setting rule '".$rule->getName()."' to personal. "; } } } diff --git a/scripts/aphront/aphrontpath.php b/scripts/aphront/aphrontpath.php --- a/scripts/aphront/aphrontpath.php +++ b/scripts/aphront/aphrontpath.php @@ -22,5 +22,5 @@ list($controller) = $application->buildControllerForPath($path.'/'); } if ($controller) { - echo get_class($controller) . "\n"; + echo get_class($controller)."\n"; } diff --git a/scripts/symbols/generate_ctags_symbols.php b/scripts/symbols/generate_ctags_symbols.php --- a/scripts/symbols/generate_ctags_symbols.php +++ b/scripts/symbols/generate_ctags_symbols.php @@ -110,7 +110,7 @@ if ($file[0] == '.' || $file[0] == '..') { array_shift($file); } - $file = '/' . implode('/', $file); + $file = '/'.implode('/', $file); $parts = array( $context, diff --git a/src/aphront/AphrontRequest.php b/src/aphront/AphrontRequest.php --- a/src/aphront/AphrontRequest.php +++ b/src/aphront/AphrontRequest.php @@ -231,19 +231,19 @@ if (PhabricatorEnv::getEnvConfig('phabricator.developer-mode')) { // TODO: Clean this up, see T1921. $more_info[] = - "To avoid this error, use phabricator_form() to construct forms. " . - "If you are already using phabricator_form(), make sure the form " . - "'action' uses a relative URI (i.e., begins with a '/'). Forms " . - "using absolute URIs do not include CSRF tokens, to prevent " . - "leaking tokens to external sites.\n\n" . - "If this page performs writes which do not require CSRF " . - "protection (usually, filling caches or logging), you can use " . - "AphrontWriteGuard::beginScopedUnguardedWrites() to temporarily " . - "bypass CSRF protection while writing. You should use this only " . - "for writes which can not be protected with normal CSRF " . - "mechanisms.\n\n" . - "Some UI elements (like PhabricatorActionListView) also have " . - "methods which will allow you to render links as forms (like " . + "To avoid this error, use phabricator_form() to construct forms. ". + "If you are already using phabricator_form(), make sure the form ". + "'action' uses a relative URI (i.e., begins with a '/'). Forms ". + "using absolute URIs do not include CSRF tokens, to prevent ". + "leaking tokens to external sites.\n\n". + "If this page performs writes which do not require CSRF ". + "protection (usually, filling caches or logging), you can use ". + "AphrontWriteGuard::beginScopedUnguardedWrites() to temporarily ". + "bypass CSRF protection while writing. You should use this only ". + "for writes which can not be protected with normal CSRF ". + "mechanisms.\n\n". + "Some UI elements (like PhabricatorActionListView) also have ". + "methods which will allow you to render links as forms (like ". "setRenderAsForm(true))."; } diff --git a/src/applications/audit/editor/PhabricatorAuditCommentEditor.php b/src/applications/audit/editor/PhabricatorAuditCommentEditor.php --- a/src/applications/audit/editor/PhabricatorAuditCommentEditor.php +++ b/src/applications/audit/editor/PhabricatorAuditCommentEditor.php @@ -261,7 +261,7 @@ ->setAuditorPHID($auditor_phid) ->setAuditStatus($audit_requested) ->setAuditReasons( - array('Added by ' . $actor->getUsername())) + array('Added by '.$actor->getUsername())) ->save(); } } @@ -274,7 +274,7 @@ ->setAuditorPHID($cc_phid) ->setAuditStatus($audit_cc) ->setAuditReasons( - array('Added by ' . $actor->getUsername())) + array('Added by '.$actor->getUsername())) ->save(); } } diff --git a/src/applications/auth/provider/PhabricatorAuthProviderOAuth.php b/src/applications/auth/provider/PhabricatorAuthProviderOAuth.php --- a/src/applications/auth/provider/PhabricatorAuthProviderOAuth.php +++ b/src/applications/auth/provider/PhabricatorAuthProviderOAuth.php @@ -81,7 +81,7 @@ public function getConfigurationHelp() { $help = $this->getProviderConfigurationHelp(); - return $help . "\n\n" . + return $help."\n\n". pht('Use the **OAuth App Notes** field to record details about which '. 'account the external application is registered under.'); } diff --git a/src/applications/chatlog/controller/PhabricatorChatLogChannelLogController.php b/src/applications/chatlog/controller/PhabricatorChatLogChannelLogController.php --- a/src/applications/chatlog/controller/PhabricatorChatLogChannelLogController.php +++ b/src/applications/chatlog/controller/PhabricatorChatLogChannelLogController.php @@ -152,7 +152,7 @@ array( 'href' => $first_uri, ), - "\xC2\xAB ". pht('Newest')); + "\xC2\xAB ".pht('Newest')); } $prev_uri = $pager->getPrevPageURI(); @@ -162,7 +162,7 @@ array( 'href' => $prev_uri, ), - "\xE2\x80\xB9 " . pht('Newer')); + "\xE2\x80\xB9 ".pht('Newer')); } $next_uri = $pager->getNextPageURI(); @@ -172,7 +172,7 @@ array( 'href' => $next_uri, ), - pht('Older') . " \xE2\x80\xBA"); + pht('Older')." \xE2\x80\xBA"); } $pager_top = phutil_tag( @@ -224,7 +224,7 @@ array( 'href' => '#latest' ), - pht('Jump to Bottom') . " \xE2\x96\xBE"); + pht('Jump to Bottom')." \xE2\x96\xBE"); $jump = phutil_tag( 'div', diff --git a/src/applications/conduit/call/ConduitCall.php b/src/applications/conduit/call/ConduitCall.php --- a/src/applications/conduit/call/ConduitCall.php +++ b/src/applications/conduit/call/ConduitCall.php @@ -27,8 +27,8 @@ $this->handler->defineParamTypes()); if ($invalid_params) { throw new ConduitException( - "Method '{$method}' doesn't define these parameters: '" . - implode("', '", array_keys($invalid_params)) . "'."); + "Method '{$method}' doesn't define these parameters: '". + implode("', '", array_keys($invalid_params))."'."); } if ($this->servers) { diff --git a/src/applications/config/controller/PhabricatorConfigIgnoreController.php b/src/applications/config/controller/PhabricatorConfigIgnoreController.php --- a/src/applications/config/controller/PhabricatorConfigIgnoreController.php +++ b/src/applications/config/controller/PhabricatorConfigIgnoreController.php @@ -34,7 +34,7 @@ 'This issue will no longer be suppressed, and will return to its '. 'rightful place as a global setup warning.'); } else { - throw new Exception('Unrecognized verb: ' . $this->verb); + throw new Exception('Unrecognized verb: '.$this->verb); } $dialog = id(new AphrontDialogView()) diff --git a/src/applications/conpherence/controller/ConpherenceWidgetController.php b/src/applications/conpherence/controller/ConpherenceWidgetController.php --- a/src/applications/conpherence/controller/ConpherenceWidgetController.php +++ b/src/applications/conpherence/controller/ConpherenceWidgetController.php @@ -274,8 +274,8 @@ phabricator_format_local_time( $status->getDateFrom(), $user, - $time_str) . - ' - ' . + $time_str). + ' - '. phabricator_format_local_time( $status->getDateTo(), $user, diff --git a/src/applications/conpherence/view/ConpherenceLayoutView.php b/src/applications/conpherence/view/ConpherenceLayoutView.php --- a/src/applications/conpherence/view/ConpherenceLayoutView.php +++ b/src/applications/conpherence/view/ConpherenceLayoutView.php @@ -59,7 +59,7 @@ $selected_id = null; $selected_thread_id = null; if ($this->thread) { - $selected_id = $this->thread->getPHID() . '-nav-item'; + $selected_id = $this->thread->getPHID().'-nav-item'; $selected_thread_id = $this->thread->getID(); } $this->initBehavior('conpherence-menu', @@ -77,7 +77,7 @@ $this->initBehavior( 'conpherence-widget-pane', array( - 'widgetBaseUpdateURI' => $this->baseURI . 'update/', + 'widgetBaseUpdateURI' => $this->baseURI.'update/', 'widgetRegistry' => array( 'conpherence-message-pane' => array( 'name' => pht('Thread'), diff --git a/src/applications/dashboard/controller/PhabricatorDashboardViewController.php b/src/applications/dashboard/controller/PhabricatorDashboardViewController.php --- a/src/applications/dashboard/controller/PhabricatorDashboardViewController.php +++ b/src/applications/dashboard/controller/PhabricatorDashboardViewController.php @@ -70,7 +70,7 @@ 'yet. Use %s to add panels.', phutil_tag( 'a', - array('href'=>$manage_uri), + array('href' => $manage_uri), pht('Manage Dashboard')))); } diff --git a/src/applications/differential/DifferentialGetWorkingCopy.php b/src/applications/differential/DifferentialGetWorkingCopy.php --- a/src/applications/differential/DifferentialGetWorkingCopy.php +++ b/src/applications/differential/DifferentialGetWorkingCopy.php @@ -17,7 +17,7 @@ $origin_path = $repo->getLocalPath(); $path = rtrim($origin_path, '/'); - $path = $path . '__workspace'; + $path = $path.'__workspace'; if (!Filesystem::pathExists($path)) { $repo->execxLocalCommand( @@ -53,7 +53,7 @@ $origin_path = $repo->getLocalPath(); $path = rtrim($origin_path, '/'); - $path = $path . '__workspace'; + $path = $path.'__workspace'; if (!Filesystem::pathExists($path)) { $repo->execxLocalCommand( diff --git a/src/applications/differential/editor/DifferentialTransactionEditor.php b/src/applications/differential/editor/DifferentialTransactionEditor.php --- a/src/applications/differential/editor/DifferentialTransactionEditor.php +++ b/src/applications/differential/editor/DifferentialTransactionEditor.php @@ -1322,7 +1322,7 @@ $result[] = "{$file}:{$range} {$inline_content}"; } else { $result[] = '================'; - $result[] = 'Comment at: ' . $file . ':' . $range; + $result[] = 'Comment at: '.$file.':'.$range; $result[] = $hunk_parser->makeContextDiff( $changeset->getHunks(), $comment->getIsNewFile(), diff --git a/src/applications/differential/landing/DifferentialLandingToHostedGit.php b/src/applications/differential/landing/DifferentialLandingToHostedGit.php --- a/src/applications/differential/landing/DifferentialLandingToHostedGit.php +++ b/src/applications/differential/landing/DifferentialLandingToHostedGit.php @@ -53,8 +53,8 @@ $missing_binary = "\nindex " - . "0000000000000000000000000000000000000000.." - . "0000000000000000000000000000000000000000\n"; + ."0000000000000000000000000000000000000000.." + ."0000000000000000000000000000000000000000\n"; if (strpos($raw_diff, $missing_binary) !== false) { throw new Exception('Patch is missing content for a binary file'); } @@ -85,7 +85,7 @@ $author_date = $revision->getDateCreated(); $workspace->execxLocal( - '-c user.name=%s -c user.email=%s ' . + '-c user.name=%s -c user.email=%s '. 'commit --date=%s --author=%s '. '--message=%s', // -c will set the 'committer' diff --git a/src/applications/differential/mail/DifferentialReplyHandler.php b/src/applications/differential/mail/DifferentialReplyHandler.php --- a/src/applications/differential/mail/DifferentialReplyHandler.php +++ b/src/applications/differential/mail/DifferentialReplyHandler.php @@ -105,11 +105,11 @@ // unrecognized commands will be parsed as part of the comment. $command = DifferentialAction::ACTION_COMMENT; $supported_commands = $this->getSupportedCommands(); - $regex = "/\A\n*!(" . implode('|', $supported_commands) . ")\n*/"; + $regex = "/\A\n*!(".implode('|', $supported_commands).")\n*/"; $matches = array(); if (preg_match($regex, $body, $matches)) { $command = $matches[1]; - $body = trim(str_replace('!' . $command, '', $body)); + $body = trim(str_replace('!'.$command, '', $body)); } $actor = $this->getActor(); diff --git a/src/applications/differential/parser/DifferentialHunkParser.php b/src/applications/differential/parser/DifferentialHunkParser.php --- a/src/applications/differential/parser/DifferentialHunkParser.php +++ b/src/applications/differential/parser/DifferentialHunkParser.php @@ -638,12 +638,12 @@ if ($hunk_offset['-'] !== null || $hunk_offset['+'] !== null) { $header = '@@'; if ($hunk_offset['-'] !== null) { - $header .= ' -' . ($hunk->getOldOffset() + $hunk_offset['-']) . - ',' . ($hunk_last['-'] - $hunk_offset['-'] + 1); + $header .= ' -'.($hunk->getOldOffset() + $hunk_offset['-']). + ','.($hunk_last['-'] - $hunk_offset['-'] + 1); } if ($hunk_offset['+'] !== null) { - $header .= ' +' . ($hunk->getNewOffset() + $hunk_offset['+']) . - ',' . ($hunk_last['+'] - $hunk_offset['+'] + 1); + $header .= ' +'.($hunk->getNewOffset() + $hunk_offset['+']). + ','.($hunk_last['+'] - $hunk_offset['+'] + 1); } $header .= ' @@'; $context[] = $header; diff --git a/src/applications/differential/parser/__tests__/data/missing_context.diff b/src/applications/differential/parser/__tests__/data/missing_context.diff --- a/src/applications/differential/parser/__tests__/data/missing_context.diff +++ b/src/applications/differential/parser/__tests__/data/missing_context.diff @@ -8,4 +8,3 @@ fig -grape honeydew - diff --git a/src/applications/differential/parser/__tests__/data/missing_context_2.diff b/src/applications/differential/parser/__tests__/data/missing_context_2.diff --- a/src/applications/differential/parser/__tests__/data/missing_context_2.diff +++ b/src/applications/differential/parser/__tests__/data/missing_context_2.diff @@ -9,4 +9,3 @@ +guava +gooseberry honeydew - diff --git a/src/applications/differential/parser/__tests__/data/missing_context_3.diff b/src/applications/differential/parser/__tests__/data/missing_context_3.diff --- a/src/applications/differential/parser/__tests__/data/missing_context_3.diff +++ b/src/applications/differential/parser/__tests__/data/missing_context_3.diff @@ -15,4 +15,3 @@ banana grape -honeydew - diff --git a/src/applications/differential/view/DifferentialInlineCommentView.php b/src/applications/differential/view/DifferentialInlineCommentView.php --- a/src/applications/differential/view/DifferentialInlineCommentView.php +++ b/src/applications/differential/view/DifferentialInlineCommentView.php @@ -75,7 +75,7 @@ $sigil = 'differential-inline-comment'; if ($this->preview) { - $sigil = $sigil . ' differential-inline-comment-preview'; + $sigil = $sigil.' differential-inline-comment-preview'; } $content = $inline->getContent(); diff --git a/src/applications/diffusion/protocol/__tests__/DiffusionSubversionWireProtocolTestCase.php b/src/applications/diffusion/protocol/__tests__/DiffusionSubversionWireProtocolTestCase.php --- a/src/applications/diffusion/protocol/__tests__/DiffusionSubversionWireProtocolTestCase.php +++ b/src/applications/diffusion/protocol/__tests__/DiffusionSubversionWireProtocolTestCase.php @@ -78,7 +78,7 @@ 'value' => 'duck', ), array( - 'type'=> 'string', + 'type' => 'string', 'value' => 'quack', ), ), diff --git a/src/applications/diffusion/request/DiffusionRequest.php b/src/applications/diffusion/request/DiffusionRequest.php --- a/src/applications/diffusion/request/DiffusionRequest.php +++ b/src/applications/diffusion/request/DiffusionRequest.php @@ -691,8 +691,8 @@ $host = php_uname('n'); $callsign = $this->getRepository()->getCallsign(); throw new DiffusionSetupException( - "The clone of this repository ('{$callsign}') on the local machine " . - "('{$host}') could not be read. Ensure that the repository is in a " . + "The clone of this repository ('{$callsign}') on the local machine ". + "('{$host}') could not be read. Ensure that the repository is in a ". "location where the web server has read permissions."); } diff --git a/src/applications/diffusion/view/DiffusionView.php b/src/applications/diffusion/view/DiffusionView.php --- a/src/applications/diffusion/view/DiffusionView.php +++ b/src/applications/diffusion/view/DiffusionView.php @@ -120,7 +120,7 @@ $callsign = $repository->getCallsign(); if (strlen($summary)) { - $commit_name .= ': ' . $summary; + $commit_name .= ': '.$summary; } return phutil_tag( diff --git a/src/applications/feed/conduit/ConduitAPI_feed_query_Method.php b/src/applications/feed/conduit/ConduitAPI_feed_query_Method.php --- a/src/applications/feed/conduit/ConduitAPI_feed_query_Method.php +++ b/src/applications/feed/conduit/ConduitAPI_feed_query_Method.php @@ -44,7 +44,7 @@ return array( 'ERR-UNKNOWN-TYPE' => - 'Unsupported view type, possibles are: ' . $view_types + 'Unsupported view type, possibles are: '.$view_types ); } diff --git a/src/applications/herald/adapter/HeraldAdapter.php b/src/applications/herald/adapter/HeraldAdapter.php --- a/src/applications/herald/adapter/HeraldAdapter.php +++ b/src/applications/herald/adapter/HeraldAdapter.php @@ -483,7 +483,7 @@ // We add the 'S' flag because we use the regexp multiple times. // It shouldn't cause any troubles if the flag is already there // - /.*/S is evaluated same as /.*/SS. - $result = @preg_match($condition_value . 'S', $value); + $result = @preg_match($condition_value.'S', $value); if ($result === false) { throw new HeraldInvalidConditionException( 'Regular expression is not valid!'); diff --git a/src/applications/herald/controller/HeraldRuleController.php b/src/applications/herald/controller/HeraldRuleController.php --- a/src/applications/herald/controller/HeraldRuleController.php +++ b/src/applications/herald/controller/HeraldRuleController.php @@ -643,7 +643,7 @@ foreach ($all_rules as $current_rule) { if ($current_rule->getIsDisabled()) { $current_rule->makeEphemeral(); - $current_rule->setName($rule->getName(). ' '.pht('(Disabled)')); + $current_rule->setName($rule->getName().' '.pht('(Disabled)')); } } diff --git a/src/applications/meta/controller/PhabricatorApplicationUninstallController.php b/src/applications/meta/controller/PhabricatorApplicationUninstallController.php --- a/src/applications/meta/controller/PhabricatorApplicationUninstallController.php +++ b/src/applications/meta/controller/PhabricatorApplicationUninstallController.php @@ -54,7 +54,7 @@ if ($selected->canUninstall()) { $dialog->setTitle('Confirmation') ->appendChild( - 'Install '. $selected->getName(). ' application?') + 'Install '.$selected->getName().' application?') ->addSubmitButton('Install'); } else { @@ -65,7 +65,7 @@ if ($selected->canUninstall()) { $dialog->setTitle('Confirmation') ->appendChild( - 'Really Uninstall '. $selected->getName(). ' application?') + 'Really Uninstall '.$selected->getName().' application?') ->addSubmitButton('Uninstall'); } else { $dialog->setTitle('Information') diff --git a/src/applications/metamta/parser/__tests__/PhabricatorMetaMTAEmailBodyParserTestCase.php b/src/applications/metamta/parser/__tests__/PhabricatorMetaMTAEmailBodyParserTestCase.php --- a/src/applications/metamta/parser/__tests__/PhabricatorMetaMTAEmailBodyParserTestCase.php +++ b/src/applications/metamta/parser/__tests__/PhabricatorMetaMTAEmailBodyParserTestCase.php @@ -49,7 +49,7 @@ $bodies = $this->getEmailBodies(); $with_commands = array(); foreach ($bodies as $body) { - $with_commands[] = "!whatevs dude\n" . $body; + $with_commands[] = "!whatevs dude\n".$body; } return $with_commands; } @@ -58,7 +58,7 @@ $bodies = $this->getEmailBodies(); $with_commands = array(); foreach ($bodies as $body) { - $with_commands[] = "!whatevs\n" . $body; + $with_commands[] = "!whatevs\n".$body; } return $with_commands; } diff --git a/src/applications/metamta/replyhandler/PhabricatorMailReplyHandler.php b/src/applications/metamta/replyhandler/PhabricatorMailReplyHandler.php --- a/src/applications/metamta/replyhandler/PhabricatorMailReplyHandler.php +++ b/src/applications/metamta/replyhandler/PhabricatorMailReplyHandler.php @@ -239,7 +239,7 @@ $single_handle_prefix = PhabricatorEnv::getEnvConfig( 'metamta.single-reply-handler-prefix'); return ($single_handle_prefix) - ? $single_handle_prefix . '+' . $address + ? $single_handle_prefix.'+'.$address : $address; } diff --git a/src/applications/owners/conduit/ConduitAPI_owners_query_Method.php b/src/applications/owners/conduit/ConduitAPI_owners_query_Method.php --- a/src/applications/owners/conduit/ConduitAPI_owners_query_Method.php +++ b/src/applications/owners/conduit/ConduitAPI_owners_query_Method.php @@ -7,9 +7,9 @@ extends ConduitAPI_owners_Method { public function getMethodDescription() { - return 'Query for packages by one of the following: repository/path, ' . - 'packages with a given user or project owner, or packages affiliated ' . - 'with a user (owned by either the user or a project they are a member ' . + return 'Query for packages by one of the following: repository/path, '. + 'packages with a given user or project owner, or packages affiliated '. + 'with a user (owned by either the user or a project they are a member '. 'of.) You should only provide at most one search query.'; } @@ -31,7 +31,7 @@ public function defineErrorTypes() { return array( 'ERR-INVALID-USAGE' => - 'Provide one of a single owner phid (user/project), a single ' . + 'Provide one of a single owner phid (user/project), a single '. 'affiliated user phid (user), or a repository/path.', 'ERR-INVALID-PARAMETER' => 'parameter should be a phid', 'ERR_REP_NOT_FOUND' => 'The repository callsign is not recognized', diff --git a/src/applications/owners/mail/PackageMail.php b/src/applications/owners/mail/PackageMail.php --- a/src/applications/owners/mail/PackageMail.php +++ b/src/applications/owners/mail/PackageMail.php @@ -44,7 +44,7 @@ $handles = $this->getHandles(); $section = array(); $section[] = ' In repository '.$handles[$repository_phid]->getName(). - ' - '. PhabricatorEnv::getProductionURI($handles[$repository_phid] + ' - '.PhabricatorEnv::getProductionURI($handles[$repository_phid] ->getURI()); foreach ($paths as $path => $excluded) { $section[] = ' '.($excluded ? 'Excluded' : 'Included').' '.$path; diff --git a/src/applications/owners/mail/PackageModifyMail.php b/src/applications/owners/mail/PackageModifyMail.php --- a/src/applications/owners/mail/PackageModifyMail.php +++ b/src/applications/owners/mail/PackageModifyMail.php @@ -81,9 +81,9 @@ $section = array(); $section[] = 'PRIMARY OWNER CHANGE'; - $section[] = ' Old owner: ' . + $section[] = ' Old owner: '. $handles[$old_primary_owner_phid]->getName(); - $section[] = ' New owner: ' . + $section[] = ' New owner: '. $handles[$primary_owner_phid]->getName(); return implode("\n", $section); diff --git a/src/applications/owners/query/PhabricatorOwnerPathQuery.php b/src/applications/owners/query/PhabricatorOwnerPathQuery.php --- a/src/applications/owners/query/PhabricatorOwnerPathQuery.php +++ b/src/applications/owners/query/PhabricatorOwnerPathQuery.php @@ -20,9 +20,9 @@ $result = array(); foreach ($paths as $path) { - $basic_path = '/' . $path->getPath(); + $basic_path = '/'.$path->getPath(); if ($path->getFileType() == DifferentialChangeType::FILE_DIRECTORY) { - $basic_path = rtrim($basic_path, '/') . '/'; + $basic_path = rtrim($basic_path, '/').'/'; } $result[] = $basic_path; } diff --git a/src/applications/pholio/query/PholioMockSearchEngine.php b/src/applications/pholio/query/PholioMockSearchEngine.php --- a/src/applications/pholio/query/PholioMockSearchEngine.php +++ b/src/applications/pholio/query/PholioMockSearchEngine.php @@ -41,9 +41,9 @@ ->execute(); $statuses = array( - ''=>pht('Any Status'), - 'closed'=>pht('Closed'), - 'open'=>pht('Open')); + '' => pht('Any Status'), + 'closed' => pht('Closed'), + 'open' => pht('Open')); $status = $saved_query->getParameter('statuses', array()); $status = head($status); diff --git a/src/applications/phrequent/controller/PhrequentTrackController.php b/src/applications/phrequent/controller/PhrequentTrackController.php --- a/src/applications/phrequent/controller/PhrequentTrackController.php +++ b/src/applications/phrequent/controller/PhrequentTrackController.php @@ -23,7 +23,7 @@ if (!$this->isStartingTracking() && !$this->isStoppingTracking()) { - throw new Exception('Unrecognized verb: ' . $this->verb); + throw new Exception('Unrecognized verb: '.$this->verb); } switch ($this->verb) { diff --git a/src/applications/project/controller/PhabricatorProjectEditIconController.php b/src/applications/project/controller/PhabricatorProjectEditIconController.php --- a/src/applications/project/controller/PhabricatorProjectEditIconController.php +++ b/src/applications/project/controller/PhabricatorProjectEditIconController.php @@ -66,7 +66,7 @@ if ($icon == $project->getIcon()) { $class_extra = ' selected'; - $tip = $label . pht(' - selected'); + $tip = $label.pht(' - selected'); } else { $class_extra = null; $tip = $label; diff --git a/src/applications/releeph/controller/product/ReleephProductEditController.php b/src/applications/releeph/controller/product/ReleephProductEditController.php --- a/src/applications/releeph/controller/product/ReleephProductEditController.php +++ b/src/applications/releeph/controller/product/ReleephProductEditController.php @@ -162,7 +162,7 @@ ->setValue($pick_failure_instructions) ->setName('pickFailureInstructions') ->setCaption( - pht('Instructions for pick failures, which will be used ' . + pht('Instructions for pick failures, which will be used '. 'in emails generated by failed picks'))) ->appendChild( id(new AphrontFormTextAreaControl()) diff --git a/src/applications/releeph/controller/request/ReleephRequestEditController.php b/src/applications/releeph/controller/request/ReleephRequestEditController.php --- a/src/applications/releeph/controller/request/ReleephRequestEditController.php +++ b/src/applications/releeph/controller/request/ReleephRequestEditController.php @@ -196,7 +196,7 @@ $age_string = ''; if ($is_edit) { $age_string = phabricator_format_relative_time( - time() - $pull->getDateCreated()) . ' ago'; + time() - $pull->getDateCreated()).' ago'; } // Warn the user if we've been redirected here because we tried to diff --git a/src/applications/settings/panel/PhabricatorSettingsPanelDisplayPreferences.php b/src/applications/settings/panel/PhabricatorSettingsPanelDisplayPreferences.php --- a/src/applications/settings/panel/PhabricatorSettingsPanelDisplayPreferences.php +++ b/src/applications/settings/panel/PhabricatorSettingsPanelDisplayPreferences.php @@ -113,10 +113,10 @@ ->setOptions( array( 'glyph' => - pht("In page titles, show Tool names as unicode glyphs: " . + pht("In page titles, show Tool names as unicode glyphs: ". "\xE2\x9A\x99"), 'text' => - pht('In page titles, show Tool names as plain text: ' . + pht('In page titles, show Tool names as plain text: '. '[Differential]'), ))) ->appendChild( diff --git a/src/infrastructure/daemon/bot/handler/PhabricatorBotFeedNotificationHandler.php b/src/infrastructure/daemon/bot/handler/PhabricatorBotFeedNotificationHandler.php --- a/src/infrastructure/daemon/bot/handler/PhabricatorBotFeedNotificationHandler.php +++ b/src/infrastructure/daemon/bot/handler/PhabricatorBotFeedNotificationHandler.php @@ -104,7 +104,7 @@ $latest = $this->getConduit()->callMethodSynchronous( 'feed.query', array( - 'limit'=>1 + 'limit' => 1, )); foreach ($latest as $story) { @@ -127,9 +127,9 @@ $stories = $this->getConduit()->callMethodSynchronous( 'feed.query', array( - 'limit'=>$config_page_size, - 'after'=>$chrono_key_cursor, - 'view'=>'text' + 'limit' => $config_page_size, + 'after' => $chrono_key_cursor, + 'view' => 'text', )); foreach ($stories as $story) { diff --git a/src/infrastructure/sms/adapter/PhabricatorSMSImplementationTwilioAdapter.php b/src/infrastructure/sms/adapter/PhabricatorSMSImplementationTwilioAdapter.php --- a/src/infrastructure/sms/adapter/PhabricatorSMSImplementationTwilioAdapter.php +++ b/src/infrastructure/sms/adapter/PhabricatorSMSImplementationTwilioAdapter.php @@ -93,7 +93,7 @@ $prepend = '+1'; break; } - return $prepend . $number; + return $prepend.$number; } } diff --git a/src/view/control/AphrontCursorPagerView.php b/src/view/control/AphrontCursorPagerView.php --- a/src/view/control/AphrontCursorPagerView.php +++ b/src/view/control/AphrontCursorPagerView.php @@ -146,7 +146,7 @@ array( 'href' => $first_uri, ), - "\xC2\xAB ". pht('First')); + "\xC2\xAB ".pht('First')); } $prev_uri = $this->getPrevPageURI(); @@ -156,7 +156,7 @@ array( 'href' => $prev_uri, ), - "\xE2\x80\xB9 " . pht('Prev')); + "\xE2\x80\xB9 ".pht('Prev')); } $next_uri = $this->getNextPageURI(); @@ -166,7 +166,7 @@ array( 'href' => $next_uri, ), - pht('Next') . " \xE2\x80\xBA"); + pht('Next')." \xE2\x80\xBA"); } return phutil_tag( diff --git a/src/view/layout/PhabricatorSourceCodeView.php b/src/view/layout/PhabricatorSourceCodeView.php --- a/src/view/layout/PhabricatorSourceCodeView.php +++ b/src/view/layout/PhabricatorSourceCodeView.php @@ -71,7 +71,7 @@ } if ($this->canClickHighlight) { - $line_uri = $this->uri . '$' . $line_number; + $line_uri = $this->uri.'$'.$line_number; $line_href = (string) new PhutilURI($line_uri); $tag_number = javelin_tag(