Fix variable types in DAO doc blocks
authoreileen <emcnaughton@wikimedia.org>
Wed, 22 May 2019 02:58:34 +0000 (14:58 +1200)
committereileen <emcnaughton@wikimedia.org>
Wed, 22 May 2019 02:58:44 +0000 (14:58 +1200)
Phpcs (stricter) reports

  79 | ERROR   | [x] Expected "intunsigned" but found "int unsigned" for @var tag in member variable comment
 156 | ERROR   | [x] Expected "bool" but found "boolean" for @var tag in member variable comment

We are declaring int unsigned instead of int & boolean instead of bool

Note this commit only contains sample of resulting change - will add more complete set after

CRM/ACL/DAO/ACL.php
CRM/Core/CodeGen/Specification.php

index 3e59f9d2f9da206c0c96a6f84e07f27fcfb53844..b2acb3e2415567620b5cfaab186b4bdd0bff0e8a 100644 (file)
@@ -6,7 +6,7 @@
  *
  * Generated from xml/schema/CRM/ACL/ACL.xml
  * DO NOT EDIT.  Generated by CRM_Core_CodeGen
- * (GenCodeChecksum:de12a8f0949faa12a7ffc89b7abfd7b7)
+ * (GenCodeChecksum:810da5f19a7ead8c949065156674c087)
  */
 
 /**
@@ -31,7 +31,7 @@ class CRM_ACL_DAO_ACL extends CRM_Core_DAO {
   /**
    * Unique table ID
    *
-   * @var int unsigned
+   * @var int
    */
   public $id;
 
@@ -45,7 +45,7 @@ class CRM_ACL_DAO_ACL extends CRM_Core_DAO {
   /**
    * Is this ACL entry Allow  (0) or Deny (1) ?
    *
-   * @var boolean
+   * @var bool
    */
   public $deny;
 
index 45e0a5e5348d3246af9ad669b3c9f28170613c76..101b3426b412d85fe2489bffab91f96f19092905 100644 (file)
@@ -318,7 +318,7 @@ class CRM_Core_CodeGen_Specification {
         // need this case since some versions of mysql do not have boolean as a valid column type and hence it
         // is changed to tinyint. hopefully after 2 yrs this case can be removed.
         $field['sqlType'] = 'tinyint';
-        $field['phpType'] = $type;
+        $field['phpType'] = 'bool';
         $field['crmType'] = 'CRM_Utils_Type::T_' . strtoupper($type);
         break;
 
@@ -340,6 +340,7 @@ class CRM_Core_CodeGen_Specification {
         $field['phpType'] = $this->value('phpType', $fieldXML, $type);
         $field['sqlType'] = $type;
         if ($type == 'int unsigned') {
+          $field['phpType'] = 'int';
           $field['crmType'] = 'CRM_Utils_Type::T_INT';
         }
         else {