Merge branch 'madmatt-pulls/apidocs-package-list' into 3.1

control/RequestFilter.php

@@ -7,6 +7,8 @@
  *
  * @author marcus@silverstripe.com.au
  * @license BSD License http://silverstripe.org/bsd-license/
+ * @package framework
+ * @subpackage control
  */
 interface RequestFilter {
 	/**

core/DAG.php

@@ -3,6 +3,9 @@
 /**
  * A Directed Acyclic Graph - used for doing topological sorts on dependencies, such as the before/after conditions
  * in config yaml fragments
+ *
+ * @package framework
+ * @subpackage manifest
  */
 class SS_DAG implements IteratorAggregate {
 	/**
@@ -88,10 +91,20 @@ class SS_DAG implements IteratorAggregate {
 	}
 }
 
+/**
+ * Exception thrown when the {@link SS_DAG} class is unable to resolve sorting the DAG due to cyclic dependencies.
+ *
+ * @package framework
+ * @subpackage manifest
+ */
 class SS_DAG_CyclicException extends Exception {
 
 	public $dag;
 
+	/**
+	 * @param string $message The Exception message
+	 * @param SS_DAG $dag The remainder of the Directed Acyclic Graph (DAG) after the last successful sort
+	 */
 	public function __construct($message, $dag) {
 		$this->dag = $dag;
 		parent::__construct($message);
@@ -99,6 +112,10 @@ class SS_DAG_CyclicException extends Exception {
 
 }
 
+/**
+ * @package framework
+ * @subpackage manifest
+ */
 class SS_DAG_Iterator implements Iterator {
 
 	protected $data;

core/TempPath.php

@@ -2,6 +2,9 @@
 /**
  * Returns the temporary folder path that silverstripe should use for its cache files.
  *
+ * @package framework
+ * @subpackage core
+ *
  * @param $base The base path to use for determining the temporary path
  * @return string Path to temp
  */
@@ -20,6 +23,9 @@ function getTempFolder($base = null) {
 
 /**
  * Returns as best a representation of the current username as we can glean.
+ *
+ * @package framework
+ * @subpackage core
  */
 function getTempFolderUsername() {
 	$user = getenv('APACHE_RUN_USER');
@@ -38,6 +44,9 @@ function getTempFolderUsername() {
  * Return the parent folder of the temp folder.
  * The temp folder will be a subfolder of this, named by username.
  * This structure prevents permission problems.
+ *
+ * @package framework
+ * @subpackage core
  */
 function getTempParentFolder($base = null) {
 	if(!$base && defined('BASE_PATH')) $base = BASE_PATH;

core/manifest/ConfigStaticManifest.php

@@ -9,6 +9,7 @@
  * optionally catch attempts to modify the config statics (otherwise the modification will appear
  * to work, but won't actually have any effect - the equvilent of failing silently)
  *
+ * @package framework
  * @subpackage manifest
  */
 class SS_ConfigStaticManifest {
@@ -150,6 +151,9 @@ class SS_ConfigStaticManifest {
  * We can't do this using TokenisedRegularExpression because we need to keep track of state
  * as we process the token list (when we enter and leave a namespace or class, when we see
  * an access level keyword, etc)
+ *
+ * @package framework
+ * @subpackage manifest
  */
 class SS_ConfigStaticManifest_Parser {
 

core/manifest/ManifestCache.php

@@ -2,6 +2,9 @@
 
 /**
  * A basic caching interface that manifests use to store data.
+ *
+ * @package framework
+ * @subpackage manifest
  */
 interface ManifestCache {
 	public function __construct($name);
@@ -12,6 +15,9 @@ interface ManifestCache {
 
 /**
  * Stores manifest data in files in TEMP_DIR dir on filesystem
+ *
+ * @package framework
+ * @subpackage manifest
  */
 class ManifestCache_File implements ManifestCache {
 	function __construct($name) {
@@ -37,6 +43,9 @@ class ManifestCache_File implements ManifestCache {
 /**
  * Same as ManifestCache_File, but stores the data as valid PHP which gets included to load
  * This is a bit faster if you have an opcode cache installed, but slower otherwise
+ *
+ * @package framework
+ * @subpackage manifest
  */
 class ManifestCache_File_PHP extends ManifestCache_File {
 	function load($key) {
@@ -58,6 +67,9 @@ class ManifestCache_File_PHP extends ManifestCache_File {
 /**
  * Stores manifest data in APC.
  * Note: benchmarks seem to indicate this is not particularly faster than _File
+ *
+ * @package framework
+ * @subpackage manifest
  */
 class ManifestCache_APC implements ManifestCache {
 	protected $pre;

dev/SilverStripeListener.php

@@ -1,8 +1,12 @@
 <?php
-
+/**
-// Inject SilverStripe 'setUpOnce' and 'tearDownOnce' unittest extension methods into phpunit
+ * Inject SilverStripe 'setUpOnce' and 'tearDownOnce' unittest extension methods into PHPUnit.
-// This is already in later SilverStripe 2.4 versions, but having it here extends compatibility to older versions
+ *
-
+ * This is already in later SilverStripe 2.4 versions, but having it here extends compatibility to older versions.
+ *
+ * @package framework
+ * @subpackage testing
+ */
 class SilverStripeListener implements PHPUnit_Framework_TestListener {
 	
 	protected function isValidClass($name) {

dev/TeamCityListener.php

@@ -1,7 +1,10 @@
 <?php
-
+/**
-// Bind TeamCity test listener. Echos messages to stdout that TeamCity interprets into the test results
+ * Bind TeamCity test listener. Echos messages to stdout that TeamCity interprets into the test results
-
+ *
+ * @package framework
+ * @subpackage testing
+ */
 class TeamCityListener implements PHPUnit_Framework_TestListener {
 	
 	private function escape($str) {

email/Mailer.php

@@ -437,6 +437,8 @@ class Mailer extends Object {
 }
 
 /**
+ * @package framework
+ * @subpackage email
  * @deprecated 3.1
  */
 function htmlEmail($to, $from, $subject, $htmlContent, $attachedFiles = false, $customheaders = false,
@@ -449,6 +451,8 @@ function htmlEmail($to, $from, $subject, $htmlContent, $attachedFiles = false, $
 }
 
 /**
+ * @package framework
+ * @subpackage email
  * @deprecated 3.1
  */
 function plaintextEmail($to, $from, $subject, $plainContent, $attachedFiles, $customheaders = false) {
@@ -459,6 +463,8 @@ function plaintextEmail($to, $from, $subject, $plainContent, $attachedFiles, $cu
 }
 
 /**
+ * @package framework
+ * @subpackage email
  * @deprecated 3.1
  */
 function encodeMultipart($parts, $contentType, $headers = false) {
@@ -469,6 +475,8 @@ function encodeMultipart($parts, $contentType, $headers = false) {
 }
 
 /**
+ * @package framework
+ * @subpackage email
  * @deprecated 3.1
  */
 function wrapImagesInline($htmlContent) {
@@ -479,6 +487,8 @@ function wrapImagesInline($htmlContent) {
 }
 
 /**
+ * @package framework
+ * @subpackage email
  * @deprecated 3.1
  */
 function wrapImagesInline_rewriter($url) {
@@ -490,6 +500,8 @@ function wrapImagesInline_rewriter($url) {
 }
 
 /**
+ * @package framework
+ * @subpackage email
  * @deprecated 3.1
  */
 function processHeaders($headers, $body = false) {
@@ -500,6 +512,8 @@ function processHeaders($headers, $body = false) {
 }
 
 /**
+ * @package framework
+ * @subpackage email
  * @deprecated 3.1
  */
 function encodeFileForEmail($file, $destFileName = false, $disposition = NULL, $extraHeaders = "") {
@@ -510,6 +524,8 @@ function encodeFileForEmail($file, $destFileName = false, $disposition = NULL, $
 }
 
 /**
+ * @package framework
+ * @subpackage email
  * @deprecated 3.1
  */
 function QuotedPrintable_encode($quotprint) {
@@ -520,6 +536,8 @@ function QuotedPrintable_encode($quotprint) {
 }
 
 /**
+ * @package framework
+ * @subpackage email
  * @deprecated 3.1
  */
 function validEmailAddr($emailAddress) {

filesystem/GD.php

@@ -486,7 +486,10 @@ class GDBackend extends Object implements Image_Backend {
 }
 
 /**
- * Backwards compatibility
+ * This class is maintained for backwards-compatibility only. Please use the {@link GDBackend} class instead.
+ *
+ * @package framework
+ * @subpackage filesystem
  */
 class GD extends GDBackend {
 

forms/HtmlEditorField.php

@@ -759,6 +759,8 @@ class HtmlEditorField_Toolbar extends RequestHandler {
  * such as file name or the URL.
  *
  * @todo Remove once core has support for remote files
+ * @package forms
+ * @subpackage fields-formattedinput
  */
 class HtmlEditorField_File extends ViewableData {
 
@@ -827,6 +829,13 @@ class HtmlEditorField_File extends ViewableData {
 
 }
 
+/**
+ * Encapsulation of an oembed tag, linking to an external media source.
+ *
+ * @see Oembed
+ * @package forms
+ * @subpackage fields-formattedinput
+ */
 class HtmlEditorField_Embed extends HtmlEditorField_File {
 	protected $oembed;
 
@@ -910,6 +919,12 @@ class HtmlEditorField_Embed extends HtmlEditorField_File {
 	}
 }
 
+/**
+ * Encapsulation of an image tag, linking to an image either internal or external to the site.
+ *
+ * @package forms
+ * @subpackage fields-formattedinput
+ */
 class HtmlEditorField_Image extends HtmlEditorField_File {
 
 	protected $width;

forms/HtmlEditorSanitiser.php

@@ -7,7 +7,8 @@
  * See www.tinymce.com/wiki.php/configuration:valid_elements for details on the spec of TinyMCE's
  * whitelist configuration
  *
- * Class HtmlEditorSanitiser
+ * @package forms
+ * @subpackage fields-formattedinput
  */
 class HtmlEditorSanitiser {
 

forms/MoneyField.php

@@ -6,7 +6,7 @@
  * 
  * @author Ingo Schommer, SilverStripe Ltd. (<firstname>@silverstripe.com)
  * 
- * @package framework
+ * @package forms
  * @subpackage fields-formattedinput
  */
 class MoneyField extends FormField {

forms/UploadField.php

@@ -22,8 +22,8 @@
  * </code>
  * 
  * @author Zauberfisch
- * @package framework
+ * @package forms
- * @subpackage forms
+ * @subpackages fields-files
  */
 class UploadField extends FileField {
 
@@ -1325,8 +1325,8 @@ class UploadField extends FileField {
  * RequestHandler for actions (edit, remove, delete) on a single item (File) of the UploadField
  * 
  * @author Zauberfisch
- * @package framework
+ * @package forms
- * @subpackage forms
+ * @subpackages fields-files
  */
 class UploadField_ItemHandler extends RequestHandler {
 
@@ -1486,6 +1486,9 @@ class UploadField_ItemHandler extends RequestHandler {
 
 /**
  * File selection popup for attaching existing files.
+ *
+ * @package forms
+ * @subpackages fields-files
  */
 class UploadField_SelectHandler extends RequestHandler {
 

forms/gridfield/GridField.php

@@ -15,7 +15,7 @@
  * 
  * @see SS_List
  * 
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 class GridField extends FormField {
@@ -764,7 +764,7 @@ class GridField extends FormField {
  * This class is the base class when you want to have an action that alters 
  * the state of the {@link GridField}, rendered as a button element. 
  * 
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 class GridField_FormAction extends FormAction {

forms/gridfield/GridFieldAddExistingAutocompleter.php

@@ -15,7 +15,7 @@
  * For easier setup, have a look at a sample configuration in 
  * {@link GridFieldConfig_RelationEditor}.
  *
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 class GridFieldAddExistingAutocompleter 

forms/gridfield/GridFieldAddNewButton.php

@@ -6,7 +6,7 @@
  * Only returns a button if {@link DataObject->canCreate()} for this record 
  * returns true.
  *
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 class GridFieldAddNewButton implements GridField_HTMLProvider {

forms/gridfield/GridFieldButtonRow.php

@@ -8,7 +8,7 @@
  * This row provides two new HTML fragment spaces: 'toolbar-header-left' and 
  * 'toolbar-header-right'.
  *
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 class GridFieldButtonRow implements GridField_HTMLProvider {

forms/gridfield/GridFieldComponent.php

@@ -3,7 +3,7 @@
 /**
  * Base interface for all components that can be added to GridField.
  *
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 interface GridFieldComponent {	
@@ -13,7 +13,7 @@ interface GridFieldComponent {
  * A GridField manipulator that provides HTML for the header/footer rows, or f
  * or before/after the template.
  *
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 interface GridField_HTMLProvider extends GridFieldComponent {
@@ -43,7 +43,7 @@ interface GridField_HTMLProvider extends GridFieldComponent {
  *
  * Used once per record/row.
  *
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 interface GridField_ColumnProvider extends GridFieldComponent {
@@ -110,7 +110,7 @@ interface GridField_ColumnProvider extends GridFieldComponent {
  *
  * @see {@link GridField_FormAction}.
  *
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 interface GridField_ActionProvider extends GridFieldComponent {
@@ -151,7 +151,7 @@ interface GridField_ActionProvider extends GridFieldComponent {
  * Generally, the data manipulator will make use of to {@link GridState} 
  * variables to decide how to modify the {@link DataList}.
  *
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 interface GridField_DataManipulator extends GridFieldComponent {
@@ -177,7 +177,7 @@ interface GridField_DataManipulator extends GridFieldComponent {
  * For example a URL that will return JSON-formatted data for a javascript 
  * grid control.
  *
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 interface GridField_URLHandler extends GridFieldComponent {
@@ -196,7 +196,7 @@ interface GridField_URLHandler extends GridFieldComponent {
  * A component which is used to handle when a {@link GridField} is saved into 
  * a record.
  *
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 interface GridField_SaveHandler extends GridFieldComponent {

forms/gridfield/GridFieldConfig.php

@@ -16,7 +16,7 @@
  * - {@link GridFieldConfig_RecordEditor}
  * - {@link GridFieldConfig_RelationEditor}
  *
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 class GridFieldConfig {
@@ -138,7 +138,7 @@ class GridFieldConfig {
  * A simple readonly, paginated view of records, with sortable and searchable 
  * headers.
  *
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 class GridFieldConfig_Base extends GridFieldConfig {
@@ -163,7 +163,7 @@ class GridFieldConfig_Base extends GridFieldConfig {
 /**
  * Allows viewing readonly details of individual records.
  *
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 class GridFieldConfig_RecordViewer extends GridFieldConfig_Base {
@@ -178,7 +178,10 @@ class GridFieldConfig_RecordViewer extends GridFieldConfig_Base {
 }
 
 /**
- * @package framework
+ * Allows editing of records contained within the GridField, instead of only allowing the ability to view records in
+ * the GridField.
+ *
+ * @package forms
  * @subpackage fields-gridfield
  */
 class GridFieldConfig_RecordEditor extends GridFieldConfig {
@@ -224,7 +227,7 @@ class GridFieldConfig_RecordEditor extends GridFieldConfig {
  * 	->setSearchFields('MyField');
  * </code>
  *
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 class GridFieldConfig_RelationEditor extends GridFieldConfig {

forms/gridfield/GridFieldDataColumns.php

@@ -2,7 +2,7 @@
 /**
  * @see GridField
  * 
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 class GridFieldDataColumns implements GridField_ColumnProvider {

forms/gridfield/GridFieldDeleteAction.php

@@ -15,7 +15,7 @@
  * $action = new GridFieldDeleteAction(true); 
  * </code>
  *
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 class GridFieldDeleteAction implements GridField_ColumnProvider, GridField_ActionProvider {

forms/gridfield/GridFieldDetailForm.php

@@ -13,7 +13,7 @@
  *  - <FormURL>/field/<GridFieldName>/item/<RecordID>
  *  - <FormURL>/field/<GridFieldName>/item/<RecordID>/edit
  *
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 class GridFieldDetailForm implements GridField_URLHandler {
@@ -196,7 +196,7 @@ class GridFieldDetailForm implements GridField_URLHandler {
 }
 
 /**
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 class GridFieldDetailForm_ItemRequest extends RequestHandler {

forms/gridfield/GridFieldEditButton.php

@@ -10,7 +10,7 @@
  * The default routing applies to the {@link GridFieldDetailForm} component, 
  * which has to be added separately to the {@link GridField} configuration.
  *
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 class GridFieldEditButton implements GridField_ColumnProvider {

forms/gridfield/GridFieldExportButton.php

@@ -3,7 +3,7 @@
 /**
  * Adds an "Export list" button to the bottom of a {@link GridField}.
  *
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 

forms/gridfield/GridFieldFilterHeader.php

@@ -5,7 +5,7 @@
  * 
  * @see GridField
  * 
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 class GridFieldFilterHeader implements GridField_HTMLProvider, GridField_DataManipulator, GridField_ActionProvider {

forms/gridfield/GridFieldFooter.php

@@ -12,7 +12,7 @@
  * The purpose of this class is to have a footer that can round off 
  * {@link GridField} without having to use pagination.
  *
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 class GridFieldFooter implements GridField_HTMLProvider {

forms/gridfield/GridFieldLevelup.php

@@ -4,7 +4,7 @@
  * hierarchical data. Requires the managed record to have a "getParent()" 
  * method or has_one relationship called "Parent".
  *
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 class GridFieldLevelup extends Object implements GridField_HTMLProvider {

forms/gridfield/GridFieldPageCount.php

@@ -6,7 +6,7 @@
  * 
  * Depends on {@link GridFieldPaginator} being added to the {@link GridField}.
  * 
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 class GridFieldPageCount implements GridField_HTMLProvider {

forms/gridfield/GridFieldPaginator.php

@@ -3,7 +3,7 @@
  * GridFieldPaginator paginates the {@link GridField} list and adds controls 
  * to the bottom of the {@link GridField}.
  * 
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 class GridFieldPaginator implements GridField_HTMLProvider, GridField_DataManipulator, GridField_ActionProvider {

forms/gridfield/GridFieldPrintButton.php

@@ -3,7 +3,7 @@
 /**
  * Adds an "Print" button to the bottom or top of a GridField.
  *
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 class GridFieldPrintButton implements GridField_HTMLProvider, GridField_ActionProvider, GridField_URLHandler {

forms/gridfield/GridFieldSortableHeader.php

@@ -6,7 +6,7 @@
  * 
  * @see GridField
  * 
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 class GridFieldSortableHeader implements GridField_HTMLProvider, GridField_DataManipulator, GridField_ActionProvider {

forms/gridfield/GridFieldToolbarHeader.php

@@ -6,7 +6,7 @@
  *
  * The header serves to display the name of the data the GridField is showing.
  *
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 class GridFieldToolbarHeader implements GridField_HTMLProvider {

forms/gridfield/GridFieldViewButton.php

@@ -4,7 +4,7 @@
  * disabled by default and intended for use in readonly {@link GridField}
  * instances.
  *
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 class GridFieldViewButton implements GridField_ColumnProvider {

forms/gridfield/GridState.php

@@ -7,7 +7,7 @@
  * 
  * @see GridField
  * 
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 class GridState extends HiddenField {
@@ -121,7 +121,7 @@ class GridState extends HiddenField {
  *
  * @see GridState
  * 
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 class GridState_Data {
@@ -175,7 +175,7 @@ class GridState_Data {
 /**
  * @see GridState
  * 
- * @package framework
+ * @package forms
  * @subpackage fields-gridfield
  */
 class GridState_Component implements GridField_HTMLProvider {

i18n/i18nSSLegacyAdapter.php

@@ -6,7 +6,6 @@ require_once 'Zend/Translate/Adapter.php';
  * @package framework
  * @subpackage i18n
  */
-
 class i18nSSLegacyAdapter extends Zend_Translate_Adapter implements i18nTranslateAdapterInterface {
 	
 	/**
@@ -65,6 +64,10 @@ class i18nSSLegacyAdapter extends Zend_Translate_Adapter implements i18nTranslat
 	
 }
 
+/**
+ * @package framework
+ * @subpackage i18n
+ */
 class i18nSSLegacyAdapter_Iterator extends RecursiveIteratorIterator {
 
 	protected $keyStack = array();

i18n/i18nTextCollector.php

@@ -479,6 +479,9 @@ class i18nTextCollector extends Object {
 /**
  * Allows serialization of entity definitions collected through {@link i18nTextCollector}
  * into a persistent format, usually on the filesystem.
+ *
+ * @package framework
+ * @subpackage i18n
  */
 interface i18nTextCollector_Writer {
 	/**
@@ -495,6 +498,9 @@ interface i18nTextCollector_Writer {
 
 /**
  * Legacy writer for 2.x style persistence.
+ *
+ * @package framework
+ * @subpackage i18n
  */
 class i18nTextCollector_Writer_Php implements i18nTextCollector_Writer {
 
@@ -573,6 +579,9 @@ class i18nTextCollector_Writer_Php implements i18nTextCollector_Writer {
 
 /**
  * Writes files compatible with {@link i18nRailsYamlAdapter}.
+ *
+ * @package framework
+ * @subpackage i18n
  */
 class i18nTextCollector_Writer_RailsYaml implements i18nTextCollector_Writer {
 
@@ -627,6 +636,9 @@ class i18nTextCollector_Writer_RailsYaml implements i18nTextCollector_Writer {
 
 /**
  * Parser that scans through a template and extracts the parameters to the _t and <%t calls
+ *
+ * @package framework
+ * @subpackage i18n
  */
 class i18nTextCollector_Parser extends SSTemplateParser {
 

model/Filterable.php

@@ -7,6 +7,8 @@
  * applied, rather than applying the filter in place
  *
  * @see SS_List, SS_Sortable, SS_Limitable
+ * @package framework
+ * @subpackage model
  */
 interface SS_Filterable {
 

model/Limitable.php

@@ -7,6 +7,8 @@
  * applied, rather than applying the limit in place
  *
  * @see SS_List, SS_Sortable, SS_Filterable
+ * @package framework
+ * @subpackage model
  */
 interface SS_Limitable {
 

model/Sortable.php

@@ -7,6 +7,8 @@
  * applied, rather than applying the sort in place
  *
  * @see SS_List, SS_Filterable, SS_Limitable
+ * @package framework
+ * @subpackage model
  */
 interface SS_Sortable {
 

security/PermissionFailureException.php

@@ -1,9 +1,11 @@
 <?php
-
 /**
  * Throw this exception to register that a user doesn't have permission to do the given action
  * and potentially redirect them to the log-in page.  The exception message may be presented to the
  * user, so it shouldn't be in nerd-speak.
+ *
+ * @package framework
+ * @subpackage security
  */
 class PermissionFailureException extends Exception {
 

view/SSTemplateParser.php

@@ -20,6 +20,9 @@ else {
  * This is the exception raised when failing to parse a template. Note that we don't currently do any static analysis,
  * so we can't know if the template will run, just if it's malformed. It also won't catch mistakes that still look
  * valid.
+ *
+ * @package framework
+ * @subpackage view
  */
 class SSTemplateParseException extends Exception {
 	
@@ -64,6 +67,9 @@ class SSTemplateParseException extends Exception {
   * 
   * Angle Bracket: angle brackets "<" and ">" are used to eat whitespace between template elements
   * N: eats white space including newlines (using in legacy _t support)
+  * 
+  * @package framework
+  * @subpackage view
   */
 class SSTemplateParser extends Parser implements TemplateParser {
 

view/SSTemplateParser.php.inc

@@ -41,6 +41,9 @@ else {
  * This is the exception raised when failing to parse a template. Note that we don't currently do any static analysis,
  * so we can't know if the template will run, just if it's malformed. It also won't catch mistakes that still look
  * valid.
+ *
+ * @package framework
+ * @subpackage view
  */
 class SSTemplateParseException extends Exception {
 	
@@ -85,6 +88,9 @@ class SSTemplateParseException extends Exception {
   * 
   * Angle Bracket: angle brackets "<" and ">" are used to eat whitespace between template elements
   * N: eats white space including newlines (using in legacy _t support)
+  *
+  * @package framework
+  * @subpackage view
   */
 class SSTemplateParser extends Parser implements TemplateParser {
 

view/SSViewer.php

@@ -18,7 +18,9 @@
  * We also keep the index of the current starting point for lookups. A lookup is a sequence of obj calls -
  * when in a loop or with tag the end result becomes the new scope, but for injections, we throw away the lookup
  * and revert back to the original scope once we've got the value we're after
- * 
+ *
+ * @package framework
+ * @subpackage view
  */
 class SSViewer_Scope {
 	
@@ -182,6 +184,13 @@ class SSViewer_Scope {
 	}
 }
 
+/**
+ * Defines an extra set of basic methods that can be used in templates
+ * that are not defined on sub-classes of {@link ViewableData}.
+ *
+ * @package framework
+ * @subpackage view
+ */
 class SSViewer_BasicIteratorSupport implements TemplateIteratorProvider {
 
 	protected $iteratorPos;
@@ -345,6 +354,9 @@ class SSViewer_BasicIteratorSupport implements TemplateIteratorProvider {
  * (like $FirstLast etc).
  * 
  * It's separate from SSViewer_Scope to keep that fairly complex code as clean as possible.
+ *
+ * @package framework
+ * @subpackage view
  */
 class SSViewer_DataPresenter extends SSViewer_Scope {