From b05b5a193330b7f924b014cc483170e06081a189 Mon Sep 17 00:00:00 2001
From: Eileen McNaughton <eileen@fuzion.co.nz>
Date: Thu, 15 Jan 2015 10:01:06 +1300
Subject: [PATCH] a few tidy ups & comment fixes

---
 CRM/Core/Controller.php       | 2 --
 CRM/Core/DAO.php              | 2 +-
 CRM/SMS/Controller/Send.php   | 6 +++++-
 CRM/SMS/StateMachine/Send.php | 1 -
 4 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/CRM/Core/Controller.php b/CRM/Core/Controller.php
index 1cb6fceffa..cd62a31fe7 100644
--- a/CRM/Core/Controller.php
+++ b/CRM/Core/Controller.php
@@ -172,8 +172,6 @@ class CRM_Core_Controller extends HTML_QuickForm_Controller {
    *   Should we add a unique sequence number to the end of the key.
    * @param bool $ignoreKey
    *   Should we not set a qfKey for this controller (for standalone forms).
-   *
-   * @internal param bool $whether controller is modal
    */
   function __construct(
     $title = NULL,
diff --git a/CRM/Core/DAO.php b/CRM/Core/DAO.php
index ed999153f0..24643ee900 100644
--- a/CRM/Core/DAO.php
+++ b/CRM/Core/DAO.php
@@ -1158,7 +1158,7 @@ FROM   civicrm_domain
     if ($freeDAO ||
       preg_match('/^(insert|update|delete|create|drop|replace)/i', $queryStr)
     ) {
-      // we typically do this for insert/update/delete stataments OR if explicitly asked to
+      // we typically do this for insert/update/delete statements OR if explicitly asked to
       // free the dao
       $dao->free();
     }
diff --git a/CRM/SMS/Controller/Send.php b/CRM/SMS/Controller/Send.php
index 68a2ed858a..2febbdfc99 100644
--- a/CRM/SMS/Controller/Send.php
+++ b/CRM/SMS/Controller/Send.php
@@ -1,5 +1,5 @@
 <?php
-/*
+/**
  +--------------------------------------------------------------------+
  | CiviCRM version 4.6                                                |
  +--------------------------------------------------------------------+
@@ -36,6 +36,10 @@ class CRM_SMS_Controller_Send extends CRM_Core_Controller {
 
   /**
    * Class constructor
+   *
+   * @param null $title
+   * @param bool|int $action
+   * @param bool $modal
    */
   public function __construct($title = NULL, $action = CRM_Core_Action::NONE, $modal = TRUE) {
     parent::__construct($title, $modal, NULL, FALSE, TRUE);
diff --git a/CRM/SMS/StateMachine/Send.php b/CRM/SMS/StateMachine/Send.php
index 57f724a371..c51f4dee7b 100644
--- a/CRM/SMS/StateMachine/Send.php
+++ b/CRM/SMS/StateMachine/Send.php
@@ -45,7 +45,6 @@ class CRM_SMS_StateMachine_Send extends CRM_Core_StateMachine {
    * @param object $controller
    * @param \const|int $action
    *
-   * @internal param \CRM_SMS_Controller $object
    * @return \CRM_SMS_StateMachine_Send CRM_SMS_StateMachine
    */
   public function __construct($controller, $action = CRM_Core_Action::NONE) {
-- 
2.25.1