Merge pull request #475 from seamuslee001/civitheme_lint

Lint Civitheme module
civicrm · Sep 18, 2017 · ba50ea8 · ba50ea8
2 parents ba7a36e + 9896b36
commit ba50ea8
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 12 deletions.
diff --git a/modules/civicrmtheme/civicrmtheme.install b/modules/civicrmtheme/civicrmtheme.install
@@ -23,7 +23,7 @@
  | GNU Affero General Public License or the licensing of CiviCRM,     |
  | see the CiviCRM license FAQ at http://civicrm.org/licensing        |
  +--------------------------------------------------------------------+
-*/
+ */
 
 /**
  *
@@ -33,28 +33,28 @@
  *
  */
 
-/*
- * Implemenation of hook_enable()
+/**
+ * Implements hook_enable().
  */
 function civicrmtheme_enable() {
   // update module weight to 110, to make it run after system and civicrm
   db_query("UPDATE {system} SET weight = 110 WHERE name = 'civicrmtheme'");
 }
 
 /**
- * Implementation of hook_install().
+ * Implements hook_install().
  */
 function civicrmtheme_install() {
-  $link = l(t('Administer > Appearance > Administration theme'), 'admin/appearance');
-  drupal_set_message(t("CiviCRM theme configuration setting is available under !link", array('!link' => $link)));
+  $t = get_t();
+  $link = l($t('Administer > Appearance > Administration theme'), 'admin/appearance');
+  drupal_set_message($t("CiviCRM theme configuration setting is available under !link", array('!link' => $link)));
 }
 
 /**
- * Implementation of hook_uninstall().
+ * Implements hook_uninstall().
  */
 function civicrmtheme_uninstall() {
   // cleaning variables
   variable_del('civicrmtheme_theme_admin');
   variable_del('civicrmtheme_theme_public');
 }
-
diff --git a/modules/civicrmtheme/civicrmtheme.module b/modules/civicrmtheme/civicrmtheme.module
@@ -23,7 +23,7 @@
  | GNU Affero General Public License or the licensing of CiviCRM,     |
  | see the CiviCRM license FAQ at http://civicrm.org/licensing        |
  +--------------------------------------------------------------------+
-*/
+ */
 
 /**
  *
@@ -53,7 +53,7 @@ function civicrmtheme_list_themes() {
 }
 
 /**
- * Implementation of hook_form_alter().
+ * Implements hook_form_alter().
  */
 function civicrmtheme_form_system_themes_admin_form_alter(&$form, $form_state) {
   // Define a fieldset for civicrm theme setting section
@@ -98,7 +98,7 @@ function civicrmtheme_system_themes_admin_form_submit($form, &$form_state) {
 }
 
 /**
- * Implementation of hook_custom_theme().
+ * Implements hook_custom_theme().
  */
 function civicrmtheme_custom_theme() {
   if (arg(0) != 'civicrm') {
@@ -135,4 +135,3 @@ function civicrmtheme_custom_theme() {
     return $admin_theme;
   }
 }
-