From 5a7f3b8bca74f57a31633058961dbbdad8c68110 Mon Sep 17 00:00:00 2001 From: eileenmcnaugton Date: Thu, 3 Sep 2015 00:32:37 +1200 Subject: [PATCH] comment fixes jenkins seems to spit the dummy at these but the issue appears to be php5.5 style code which is fine here, rather than the change --- CRM/Utils/System/Drupal8.php | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/CRM/Utils/System/Drupal8.php b/CRM/Utils/System/Drupal8.php index 73d2882299..4592ef3593 100644 --- a/CRM/Utils/System/Drupal8.php +++ b/CRM/Utils/System/Drupal8.php @@ -29,12 +29,10 @@ * * @package CRM * @copyright CiviCRM LLC (c) 2004-2015 - * $Id$ - * */ /** - * Drupal specific stuff goes here + * Drupal specific stuff goes here. */ class CRM_Utils_System_Drupal8 extends CRM_Utils_System_DrupalBase { @@ -128,9 +126,6 @@ class CRM_Utils_System_Drupal8 extends CRM_Utils_System_DrupalBase { * Errors. * @param string $emailName * Field label for the 'email'. - * - * - * @return void */ public static function checkUserNameEmailExists(&$params, &$errors, $emailName = 'email') { // If we are given a name, let's check to see if it already exists. @@ -222,6 +217,7 @@ class CRM_Utils_System_Drupal8 extends CRM_Utils_System_DrupalBase { case 'html-header': case 'page-footer': break; + default: return FALSE; } @@ -246,6 +242,7 @@ class CRM_Utils_System_Drupal8 extends CRM_Utils_System_DrupalBase { case 'html-header': case 'page-footer': break; + default: return FALSE; } @@ -484,7 +481,8 @@ class CRM_Utils_System_Drupal8 extends CRM_Utils_System_DrupalBase { /** * Determine the location of the CMS root. - * @param null $path + * + * @param string $path * * @return NULL|string */ @@ -551,4 +549,5 @@ class CRM_Utils_System_Drupal8 extends CRM_Utils_System_DrupalBase { $cache_backend->deleteAll(); } } + } -- 2.25.1