From 4a011303b99d1f8cd7981943f80472508793d105 Mon Sep 17 00:00:00 2001
From: Damian Mooyman <damian@silverstripe.com>
Date: Mon, 24 Aug 2015 16:15:38 +1200
Subject: [PATCH] Add missing packages

---
 admin/code/LeftAndMain.php                      | 9 ++++++---
 control/CookieJar.php                           | 3 +++
 control/Cookie_Backend.php                      | 3 +++
 core/startup/ErrorControlChain.php              | 3 +++
 core/startup/ParameterConfirmationToken.php     | 3 +++
 dev/phpunit/PhpUnitWrapper_Generic.php          | 3 +++
 model/connect/MySQLStatement.php                | 3 +++
 model/queries/SQLQuery.php                      | 7 +++++--
 security/CMSSecurity.php                        | 2 ++
 src/SilverStripe/Framework/Injector/Factory.php | 3 +++
 10 files changed, 34 insertions(+), 5 deletions(-)

diff --git a/admin/code/LeftAndMain.php b/admin/code/LeftAndMain.php
index 0a7fa7917..ed84f83a2 100644
--- a/admin/code/LeftAndMain.php
+++ b/admin/code/LeftAndMain.php
@@ -1,13 +1,16 @@
 <?php
+
+/**
+ * @package framework
+ * @subpackage admin
+ */
+
 /**
  * LeftAndMain is the parent class of all the two-pane views in the CMS.
  * If you are wanting to add more areas to the CMS, you can do it by subclassing LeftAndMain.
  *
  * This is essentially an abstract class which should be subclassed.
  * See {@link CMSMain} for a good example.
- *
- * @package framework
- * @subpackage admin
  */
 class LeftAndMain extends Controller implements PermissionProvider {
 
diff --git a/control/CookieJar.php b/control/CookieJar.php
index ab4912558..2b2a80c58 100644
--- a/control/CookieJar.php
+++ b/control/CookieJar.php
@@ -10,6 +10,9 @@
  * @todo A getter for cookies that haven't been sent to the browser yet
  * @todo Tests / a way to set the state without hacking with $_COOKIE
  * @todo Store the meta information around cookie setting (path, domain, secure, etc)
+ *
+ * @package framework
+ * @subpackage misc
  */
 class CookieJar implements Cookie_Backend {
 
diff --git a/control/Cookie_Backend.php b/control/Cookie_Backend.php
index 857120254..bd1c40ec9 100644
--- a/control/Cookie_Backend.php
+++ b/control/Cookie_Backend.php
@@ -4,6 +4,9 @@
  * The Cookie_Backend interface for use with `Cookie::$inst`.
  *
  * See Cookie_DefaultBackend and Cookie
+ *
+ * @package framework
+ * @subpackage misc
  */
 interface Cookie_Backend {
 
diff --git a/core/startup/ErrorControlChain.php b/core/startup/ErrorControlChain.php
index dffd8290f..496de8615 100644
--- a/core/startup/ErrorControlChain.php
+++ b/core/startup/ErrorControlChain.php
@@ -13,6 +13,9 @@
  *
  * WARNING: This class is experimental and designed specifically for use pre-startup in main.php
  * It will likely be heavily refactored before the release of 3.2
+ *
+ * @package framework
+ * @subpackage misc
  */
 class ErrorControlChain {
 	public static $fatal_errors = null; // Initialised after class definition
diff --git a/core/startup/ParameterConfirmationToken.php b/core/startup/ParameterConfirmationToken.php
index 206e8f302..6c98300a0 100644
--- a/core/startup/ParameterConfirmationToken.php
+++ b/core/startup/ParameterConfirmationToken.php
@@ -9,6 +9,9 @@
  *
  * WARNING: This class is experimental and designed specifically for use pre-startup in main.php
  * It will likely be heavily refactored before the release of 3.2
+ *
+ * @package framework
+ * @subpackage misc
  */
 class ParameterConfirmationToken {
 
diff --git a/dev/phpunit/PhpUnitWrapper_Generic.php b/dev/phpunit/PhpUnitWrapper_Generic.php
index 651488c04..2d2e1ac02 100644
--- a/dev/phpunit/PhpUnitWrapper_Generic.php
+++ b/dev/phpunit/PhpUnitWrapper_Generic.php
@@ -4,6 +4,9 @@
  * Generic PhpUnitWrapper.
  * Originally intended for use with Composer based installations, but will work
  * with any fully functional autoloader.
+ *
+ * @package framework
+ * @subpackage dev
  */
 class PhpUnitWrapper_Generic extends PhpUnitWrapper {
 
diff --git a/model/connect/MySQLStatement.php b/model/connect/MySQLStatement.php
index f70445043..b02938e76 100644
--- a/model/connect/MySQLStatement.php
+++ b/model/connect/MySQLStatement.php
@@ -5,6 +5,9 @@
  *
  * By default streams unbuffered data, but seek(), rewind(), or numRecords() will force the statement to
  * buffer itself and sacrifice any potential performance benefit.
+ *
+ * @package framework
+ * @subpackage model
  */
 class MySQLStatement extends SS_Query {
 
diff --git a/model/queries/SQLQuery.php b/model/queries/SQLQuery.php
index 58404e902..fcbd25bc1 100644
--- a/model/queries/SQLQuery.php
+++ b/model/queries/SQLQuery.php
@@ -1,11 +1,14 @@
 <?php
 
+/**
+ * @package framework
+ * @subpackage model
+ */
+
 /**
  * Object representing a SQL SELECT query.
  * The various parts of the SQL query can be manipulated individually.
  *
- * @package framework
- * @subpackage model
  * @deprecated since version 4.0
  */
 class SQLQuery extends SQLSelect {
diff --git a/security/CMSSecurity.php b/security/CMSSecurity.php
index 3fb66ef8f..8ca5f183f 100644
--- a/security/CMSSecurity.php
+++ b/security/CMSSecurity.php
@@ -2,6 +2,8 @@
 
 /**
  * Provides a security interface functionality within the cms
+ * @package framework
+ * @subpackage security
  */
 class CMSSecurity extends Security {
 
diff --git a/src/SilverStripe/Framework/Injector/Factory.php b/src/SilverStripe/Framework/Injector/Factory.php
index ffc61ddf3..22d51fdef 100644
--- a/src/SilverStripe/Framework/Injector/Factory.php
+++ b/src/SilverStripe/Framework/Injector/Factory.php
@@ -4,6 +4,9 @@ namespace SilverStripe\Framework\Injector;
 
 /**
  * A factory which is used for creating service instances.
+ *
+ * @package framework
+ * @subpackage injector
  */
 interface Factory {