Skip to content

Commit

Permalink
PHPCS2-Auto Fixers Applied
Browse files Browse the repository at this point in the history
- Please consider an empty line before the return statement;
- Please consider an empty line before the try statement;
- block comments start and end with `/*` and `*/` and no text
  • Loading branch information
photodude committed Dec 26, 2017
1 parent f96a132 commit 17128b5
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 2 deletions.
1 change: 1 addition & 0 deletions plugins/fields/sql/sql.php
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,7 @@ public function onContentBeforeSave($context, $item, $isNew, $data = array())
if (!JAccess::getAssetRules(1)->allow('core.admin', JFactory::getUser()->getAuthorisedGroups()))
{
$item->setError(JText::_('PLG_FIELDS_SQL_CREATE_NOT_POSSIBLE'));

return false;
}

Expand Down
1 change: 1 addition & 0 deletions plugins/system/log/log.php
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,7 @@ public function onUserLoginFailure($response)
}

JLog::addLogger(array(), JLog::INFO);

try
{
JLog::add($errorlog['comment'], JLog::INFO, $errorlog['status']);
Expand Down
6 changes: 4 additions & 2 deletions plugins/system/updatenotification/updatenotification.php
Original file line number Diff line number Diff line change
Expand Up @@ -192,11 +192,13 @@ public function onAfterRender()
return;
}

/* Load the appropriate language. We try to load English (UK), the current user's language and the forced
/*
* Load the appropriate language. We try to load English (UK), the current user's language and the forced
* language preference, in this order. This ensures that we'll never end up with untranslated strings in the
* update email which would make Joomla! seem bad. So, please, if you don't fully understand what the
* following code does DO NOT TOUCH IT. It makes the difference between a hobbyist CMS and a professional
* solution! */
* solution!
*/
$jLanguage = JFactory::getLanguage();
$jLanguage->load('plg_system_updatenotification', JPATH_ADMINISTRATOR, 'en-GB', true, true);
$jLanguage->load('plg_system_updatenotification', JPATH_ADMINISTRATOR, null, true, false);
Expand Down
1 change: 1 addition & 0 deletions plugins/user/joomla/joomla.php
Original file line number Diff line number Diff line change
Expand Up @@ -117,6 +117,7 @@ public function onUserAfterSave($user, $isnew, $success, $msg)
if (strpos($this->app->get('mailfrom'), '@') === false)
{
$this->app->enqueueMessage(JText::_('JERROR_SENDING_EMAIL'), 'warning');

return;
}

Expand Down

0 comments on commit 17128b5

Please sign in to comment.