Comments fixes
authoreileenmcnaugton <eileen@fuzion.co.nz>
Mon, 26 Oct 2015 05:36:34 +0000 (18:36 +1300)
committereileenmcnaugton <eileen@fuzion.co.nz>
Mon, 26 Oct 2015 05:36:34 +0000 (18:36 +1300)
CRM/Contact/BAO/Group.php
CRM/Contact/Import/Form/DataSource.php
CRM/Contact/Import/Form/MapField.php
CRM/Core/BAO/Mapping.php
CRM/Import/Form/DataSource.php
CRM/Import/Form/MapField.php
CRM/Import/Form/Preview.php
CRM/Import/Form/Summary.php
CRM/Import/Parser.php
CRM/Import/StateMachine.php
CRM/Utils/ReCAPTCHA.php

index 1cf5d77dad4b89accbb69b9fa48c55f4891a6acd..e8356829f531dd8127a0e083cb1d09c2bf08c145 100644 (file)
@@ -58,8 +58,6 @@ class CRM_Contact_BAO_Group extends CRM_Contact_DAO_Group {
       CRM_Core_DAO::storeValues($group, $defaults);
       return $group;
     }
-
-    return NULL;
   }
 
   /**
index 65c8cab55fe472f794fe890b6d2951d8fe071687..007b8a33232bbef6522fe9c17b48f4595e1c01d6 100644 (file)
@@ -226,17 +226,13 @@ class CRM_Contact_Import_Form_DataSource extends CRM_Core_Form {
   }
 
   /**
-   * This virtual function is used to set the default values of
-   * various form elements
+   * Set the default values of various form elements.
    *
    * access        public
    *
    * @return array
    *   reference to the array of default values
    */
-  /**
-   * @return array
-   */
   public function setDefaultValues() {
     $config = CRM_Core_Config::singleton();
     $defaults = array(
index 2b14e69d429fd0c2ff25ce2d9b51bd23aa7951c0..0265d5cce8d7796bbdc6b81d428c69a33334d223 100644 (file)
@@ -58,10 +58,11 @@ class CRM_Contact_Import_Form_MapField extends CRM_Import_Form_MapField {
 
   /**
    * Attempt to match header labels with our mapper fields.
+   *
    * FIXME: This is essentially the same function as parent::defaultFromHeader
    *
-   * @param $columnName name of column header
-   * @param $patterns pattern to match for the column
+   * @param string $columnName name of column header
+   * @param array $patterns pattern to match for the column
    *
    * @return string
    */
index ba5e33dd5c7859484c23c2b9de0d53b0c121a5ce..acbefd978b2a0304cb821d70bcc94b4271fd6589 100644 (file)
@@ -212,7 +212,7 @@ class CRM_Core_BAO_Mapping extends CRM_Core_DAO_Mapping {
   }
 
   /**
-   * Function returns associated array of elements, that will be passed for search
+   * Function returns associated array of elements, that will be passed for search.
    *
    * @param int $smartGroupId
    *   Smart group id.
index 616d6d4d4099c7c4df77adf82dc766b949dac487..6bb5bb125ad106b1d2555cb63acc1de2191cd014 100644 (file)
  */
 
 /**
- *
  * @package CRM
  * @copyright CiviCRM LLC (c) 2004-2015
- * $Id$
- *
  */
 
 /**
- * Base class for upload-only import forms (all but Contact import)
+ * Base class for upload-only import forms (all but Contact import).
  */
 abstract class CRM_Import_Form_DataSource extends CRM_Core_Form {
 
   /**
    * Set variables up before form is built.
-   *
-   * @return void
    */
   public function preProcess() {
     $this->_id = CRM_Utils_Request::retrieve('id', 'Positive', $this, FALSE);
@@ -57,8 +52,6 @@ abstract class CRM_Import_Form_DataSource extends CRM_Core_Form {
 
   /**
    * Common form elements.
-   *
-   * @return void
    */
   public function buildQuickForm() {
     $config = CRM_Core_Config::singleton();
@@ -114,7 +107,7 @@ abstract class CRM_Import_Form_DataSource extends CRM_Core_Form {
   }
 
   /**
-   * A long-winded way to add one radio element to the form
+   * A long-winded way to add one radio element to the form.
    */
   protected function addContactTypeSelector() {
     //contact types option
@@ -145,6 +138,8 @@ abstract class CRM_Import_Form_DataSource extends CRM_Core_Form {
   }
 
   /**
+   * Store form values.
+   *
    * @param array $names
    */
   protected function storeFormValues($names) {
@@ -154,7 +149,7 @@ abstract class CRM_Import_Form_DataSource extends CRM_Core_Form {
   }
 
   /**
-   * Common form postProcess
+   * Common form postProcess.
    *
    * @param string $parserClassName
    */
@@ -186,7 +181,7 @@ abstract class CRM_Import_Form_DataSource extends CRM_Core_Form {
   }
 
   /**
-   * Return a descriptive name for the page, used in wizard header
+   * Return a descriptive name for the page, used in wizard header.
    *
    * @return string
    */
index 452f4dfbd9d79affe963839df93861d21c75a222..0d7f99d754f490686ee9ec405caee1380418a215 100644 (file)
  */
 
 /**
- *
  * @package CRM
  * @copyright CiviCRM LLC (c) 2004-2015
- * $Id$
- *
  */
 
 /**
- * This class gets the name of the file to upload
+ * This class gets the name of the file to upload.
+ *
  * TODO: CRM-11254 - There's still a lot of duplicate code in the 5 child classes that should be moved here
  */
 abstract class CRM_Import_Form_MapField extends CRM_Core_Form {
@@ -83,7 +81,7 @@ abstract class CRM_Import_Form_MapField extends CRM_Core_Form {
   protected $_fieldUsed;
 
   /**
-   * Return a descriptive name for the page, used in wizard header
+   * Return a descriptive name for the page, used in wizard header.
    *
    * @return string
    */
index 80ce4473cd6eb6023536f5fb8051a0cfd265c82b..fab8a99ea8fc8d98613276efdb6e4ae5428dd32f 100644 (file)
  *
  * @package CRM
  * @copyright CiviCRM LLC (c) 2004-2015
- * $Id$
- *
  */
 
 /**
- * This class previews the uploaded file and returns summary statistics
+ * This class previews the uploaded file and returns summary statistics.
+ *
  * TODO: CRM-11254 - if preProcess and postProcess functions can be reconciled between the 5 child classes,
  * those classes can be removed entirely and this class will not need to be abstract
  */
 abstract class CRM_Import_Form_Preview extends CRM_Core_Form {
   /**
-   * Return a descriptive name for the page, used in wizard header
+   * Return a descriptive name for the page, used in wizard header.
    *
    * @return string
    */
@@ -50,8 +49,6 @@ abstract class CRM_Import_Form_Preview extends CRM_Core_Form {
 
   /**
    * Build the form object.
-   *
-   * @return void
    */
   public function buildQuickForm() {
 
index 2d60ae4732d9aa2d0c299d7d0a81664992ba9a60..c8a564085a71a6f323b1869373cbcec7e26ce969 100644 (file)
  *
  * @package CRM
  * @copyright CiviCRM LLC (c) 2004-2015
- * $Id$
- *
  */
 
 /**
- * This class summarizes the import results
+ * This class summarizes the import results.
+ *
  * TODO: CRM-11254 - if preProcess and postProcess functions can be reconciled between the 5 child classes,
  * those classes can be removed entirely and this class will not need to be abstract
  */
@@ -42,8 +41,6 @@ abstract class CRM_Import_Form_Summary extends CRM_Core_Form {
 
   /**
    * Build the form object.
-   *
-   * @return void
    */
   public function buildQuickForm() {
     $this->addButtons(array(
@@ -57,7 +54,7 @@ abstract class CRM_Import_Form_Summary extends CRM_Core_Form {
   }
 
   /**
-   * Return a descriptive name for the page, used in wizard header
+   * Return a descriptive name for the page, used in wizard header.
    *
    * @return string
    */
index d29bc12160ab845d7761c246dd2ce3cd719d7163..d0946b26659d04d247defa02a4cb78db0689ff4a 100644 (file)
@@ -29,8 +29,6 @@
  *
  * @package CRM
  * @copyright CiviCRM LLC (c) 2004-2015
- * $Id$
- *
  */
 abstract class CRM_Import_Parser {
   /**
@@ -208,14 +206,18 @@ abstract class CRM_Import_Parser {
   abstract protected function fini();
 
   /**
-   * @param $values
+   * Map field.
+   *
+   * @param array $values
    *
    * @return mixed
    */
   abstract protected function mapField(&$values);
 
   /**
-   * @param $values
+   * Preview.
+   *
+   * @param array $values
    *
    * @return mixed
    */
@@ -339,7 +341,7 @@ abstract class CRM_Import_Parser {
   }
 
   /**
-   * Remove single-quote enclosures from a value array (row)
+   * Remove single-quote enclosures from a value array (row).
    *
    * @param array $values
    * @param string $enclosure
index d86386caf1ddc4db599f6d83b041b4524629fa95..a418e10af3aef4b667b8aa6ed0e8c356a69fe319 100644 (file)
@@ -29,8 +29,6 @@
  *
  * @package CRM
  * @copyright CiviCRM LLC (c) 2004-2015
- * $Id$
- *
  */
 
 /**
index 00bc33e6057a2e2afe2a05e4ddcbd2f60768f7b0..ea6f66d309119439b50725a4708dacd48dd39f50 100644 (file)
@@ -62,10 +62,6 @@ class CRM_Utils_ReCAPTCHA {
     return self::$_singleton;
   }
 
-  public function __construct() {
-    // Empty function?
-  }
-
   /**
    * Add element to form.
    *