phpcs - Fix error, "Visibility must be declared on method"

[civicrm-core.git] / CRM / Bridge / OG / Drupal.php

diff --git a/CRM/Bridge/OG/Drupal.php b/CRM/Bridge/OG/Drupal.php
index 854334e5b74a5cab1d1a225767b8087ae76f2177..86d384a082c995316f118bd9a3fc1cf51c52467e 100644 (file)
--- a/CRM/Bridge/OG/Drupal.php
+++ b/CRM/Bridge/OG/Drupal.php
@@ -1,7 +1,7 @@
 <?php
 /*
  +--------------------------------------------------------------------+
- | CiviCRM version 4.5                                                |
+ | CiviCRM version 4.6                                                |
  +--------------------------------------------------------------------+
  | Copyright CiviCRM LLC (c) 2004-2014                                |
  +--------------------------------------------------------------------+
@@ -36,7 +36,11 @@
 // d6 compatible
 class CRM_Bridge_OG_Drupal {
 
-  static function nodeapi(&$params, $op) {
+  /**
+   * @param array $params
+   * @param $op
+   */
+  public static function nodeapi(&$params, $op) {
 
     $transaction = new CRM_Core_Transaction();
 
@@ -63,7 +67,12 @@ class CRM_Bridge_OG_Drupal {
     $transaction->commit();
   }
 
-  static function updateCiviGroup(&$params, $op, $groupType = NULL) {
+  /**
+   * @param array $params
+   * @param $op
+   * @param null $groupType
+   */
+  public static function updateCiviGroup(&$params, $op, $groupType = NULL) {
     $abort             = false;
     $params['version'] = 3;
     $params['id']      = CRM_Bridge_OG_Utils::groupID($params['source'], $params['title'], $abort);
@@ -88,7 +97,11 @@ class CRM_Bridge_OG_Drupal {
     unset($params['id']);
   }
 
-  static function updateCiviACLTables($aclParams, $op) {
+  /**
+   * @param array $aclParams
+   * @param $op
+   */
+  public static function updateCiviACLTables($aclParams, $op) {
     if ($op == 'delete') {
       self::updateCiviACL($aclParams, $op);
       self::updateCiviACLEntityRole($aclParams, $op);
@@ -101,7 +114,11 @@ class CRM_Bridge_OG_Drupal {
     }
   }
 
-  static function updateCiviACLRole(&$params, $op) {
+  /**
+   * @param array $params
+   * @param $op
+   */
+  public static function updateCiviACLRole(&$params, $op) {
 
     $optionGroupID = CRM_Core_DAO::getFieldValue('CRM_Core_DAO_OptionGroup',
       'acl_role',
@@ -144,7 +161,11 @@ SELECT v.id
     $params['acl_role_id'] = $dao->value;
   }
 
-  static function updateCiviACLEntityRole(&$params, $op) {
+  /**
+   * @param array $params
+   * @param $op
+   */
+  public static function updateCiviACLEntityRole(&$params, $op) {
     $dao = new CRM_ACL_DAO_EntityRole();
 
     $dao->entity_table = 'civicrm_group';
@@ -162,7 +183,11 @@ SELECT v.id
     $params['acl_entity_role_id'] = $dao->id;
   }
 
-  static function updateCiviACL(&$params, $op) {
+  /**
+   * @param array $params
+   * @param $op
+   */
+  public static function updateCiviACL(&$params, $op) {
     $dao = new CRM_ACL_DAO_ACL();
 
     $dao->object_table = 'civicrm_saved_search';
@@ -184,7 +209,13 @@ SELECT v.id
     $params['acl_id'] = $dao->id;
   }
 
-  static function og(&$params, $op) {
+  /**
+   * @param array $params
+   * @param $op
+   *
+   * @throws Exception
+   */
+  public static function og(&$params, $op) {
 
     $contactID = CRM_Bridge_OG_Utils::contactID($params['uf_id']);
     if (!$contactID) {