silverstripe / silverstripe-framework

core/Config.php

Doc Comments +27 added lines, -11 removed lines

@@ -470,11 +470,17 @@ 
 
 	protected $extraConfigSources = array();
 
+	/**
+	 * @param string $class
+	 */
 	public function extraConfigSourcesChanged($class) {
 		unset($this->extraConfigSources[$class]);
 		$this->cache->clean("__{$class}");
 	}
 
+	/**
+	 * @param integer $sourceOptions
+	 */
 	protected function getUncached($class, $name, $sourceOptions, &$result, $suppress, &$tags) {
 		$tags[] = "__{$class}";
 		$tags[] = "__{$class}__{$name}";
@@ -603,7 +609,7 @@ 
 	 * replace the current array value, you'll need to call remove first.
 	 *
 	 * @param $class string - The class to update a configuration value for
-	 * @param $name string - The configuration property name to update
+	 * @param string $name string - The configuration property name to update
 	 * @param $value any - The value to update with
 	 *
 	 * Arrays are recursively merged into current configuration as "latest" - for associative arrays the passed value
@@ -648,15 +654,6 @@ 
 	 *
 	 * @param string $class The class to remove a configuration value from
 	 * @param string $name The configuration name
-	 * @param mixed $key An optional key to filter against.
-	 *   If referenced config value is an array, only members of that array that match this key will be removed
-	 *   Must also match value if provided to be removed
-	 * @param mixed $value And optional value to filter against.
-	 *   If referenced config value is an array, only members of that array that match this value will be removed
-	 *   If referenced config value is not an array, value will be removed only if it matches this argument
-	 *   Must also match key if provided and referenced config value is an array to be removed
-	 *
-	 * Matching is always by "==", not by "==="
 	 */
 	public function remove($class, $name /*,$key = null*/ /*,$value = null*/) {
 		$argc = func_num_args();
@@ -735,6 +732,10 @@ 
 		$this->cache = $cloned;
 	}
 
+	/**
+	 * @param integer $key
+	 * @param integer $val
+	 */
 	public function set($key, $val, $tags = array()) {
 		// Find an index to set at
 		$replacing = null;
@@ -775,7 +776,7 @@ 
 	 * For a more robust cache checking, use {@link checkAndGet()}
 	 *
 	 * @param  string $key The cache key
-	 * @return variant     Cached value, if hit. False otherwise
+	 * @return boolean     Cached value, if hit. False otherwise
 	 */
 	public function get($key) {
 		list($hit, $result) = $this->checkAndGet($key);
@@ -803,6 +804,9 @@ 
 		}
 	}
 
+	/**
+	 * @param string $tag
+	 */
 	public function clean($tag = null) {
 		if ($tag) {
 			foreach ($this->cache as $i => $v) {
@@ -841,6 +845,10 @@ 
 		$this->cache = array();
 	}
 
+	/**
+	 * @param integer $key
+	 * @param integer $val
+	 */
 	public function set($key, $val, $tags = array()) {
 		foreach($tags as $t) {
 			if(!isset($this->tags[$t])) {
@@ -859,6 +867,11 @@ 
 		return $this->miss ? ($this->hit / $this->miss) : 0;
 	}
 
+	/**
+	 * @param integer $key
+	 *
+	 * @return boolean
+	 */
 	public function get($key) {
 		list($hit, $result) = $this->checkAndGet($key);
 		return $hit ? $result : false;
@@ -878,6 +891,9 @@ 
 		}
 	}
 
+	/**
+	 * @param string $tag
+	 */
 	public function clean($tag = null) {
 		if($tag) {
 			if(isset($this->tags[$tag])) {

email/Email.php

Doc Comments +6 added lines, -6 removed lines

@@ -224,42 +224,42 @@
 	}
 
 	/**
-	 * @return string|null
+	 * @return string
 	 */
 	public function Subject() {
 		return $this->subject;
 	}
 
 	/**
-	 * @return string|null
+	 * @return string
 	 */
 	public function Body() {
 		return $this->body;
 	}
 
 	/**
-	 * @return string|null
+	 * @return string
 	 */
 	public function To() {
 		return $this->to;
 	}
 
 	/**
-	 * @return string|null
+	 * @return string
 	 */
 	public function From() {
 		return $this->from;
 	}
 
 	/**
-	 * @return string|null
+	 * @return string
 	 */
 	public function Cc() {
 		return $this->cc;
 	}
 
 	/**
-	 * @return string|null
+	 * @return string
 	 */
 	public function Bcc() {
 		return $this->bcc;

core/manifest/ManifestCache.php

Doc Comments +18 added lines

@@ -7,9 +7,27 @@
  * @subpackage manifest
  */
 interface ManifestCache {
+
+	/**
+	 * @return void
+	 */
 	public function __construct($name);
+
+	/**
+	 * @param string $key
+	 */
 	public function load($key);
+
+	/**
+	 * @param string $key
+	 *
+	 * @return void
+	 */
 	public function save($data, $key);
+
+	/**
+	 * @return void
+	 */
 	public function clear();
 }
 

view/ThemeResourceLoader.php

Doc Comments +3 added lines

@@ -30,6 +30,9 @@
 		self::$instance = $instance;
 	}
 
+	/**
+	 * @param string $base
+	 */
 	public function __construct($base = null) {
 		$this->base = $base ? $base : BASE_PATH;
 	}

view/SSViewer.php

Doc Comments +15 added lines, -6 removed lines

@@ -370,7 +370,6 @@ 
 	/**
 	 * Returns the modulus of the numerical position of the item in the data set.
 	 * The count starts from $startIndex, which defaults to 1.
-	 * @param int $Mod The number to perform Mod operation to.
 	 * @param int $startIndex Number to start count from.
 	 * @return int
 	 */
@@ -444,6 +443,10 @@ 
 		$this->underlay = $underlay ? $underlay : array();
 	}
 
+	/**
+	 * @param string $interfaceToQuery
+	 * @param string $variableMethod
+	 */
 	protected function createCallableArray(&$extraArray, $interfaceToQuery, $variableMethod, $createObject = false) {
 		$implementers = ClassInfo::implementorsOf($interfaceToQuery);
 		if($implementers) foreach($implementers as $implementer) {
@@ -818,7 +821,7 @@ 
 	 * Assign the list of active themes to apply.
 	 * If default themes should be included add $default as the last entry.
 	 *
-	 * @param array $themes
+	 * @param string[] $themes
 	 */
 	public static function set_themes($themes = []) {
 		Config::inst()->remove('SSViewer', 'themes');
@@ -863,7 +866,7 @@ 
 	 *
 	 * @param $className string - valid class name
 	 * @param $suffix string
-	 * @param $baseClass string
+	 * @param string $baseClass string
 	 *
 	 * @return array
 	 */
@@ -893,7 +896,7 @@ 
 	}
 
 	/**
-	 * @param string|array $templateList If passed as a string with .ss extension, used as the "main" template.
+	 * @param string|array $templates If passed as a string with .ss extension, used as the "main" template.
 	 *  If passed as an array, it can be used for template inheritance (first found template "wins").
 	 *  Usually the array values are PHP class names, which directly correlate to template names.
 	 *  <code>
@@ -1246,7 +1249,7 @@ 
 	 * @param string $template Template name
 	 * @param mixed $data Data context
 	 * @param array $arguments Additional arguments
-	 * @return string Evaluated result
+	 * @return DBHTMLText Evaluated result
 	 */
 	public static function execute_template($template, $data, $arguments = null, $scope = null) {
 		$v = new SSViewer($template);
@@ -1272,6 +1275,9 @@ 
 		return $v->process($data, $arguments);
 	}
 
+	/**
+	 * @param string $content
+	 */
 	public function parseTemplateContent($content, $template="") {
 		return $this->getParser()->compileString(
 			$content,
@@ -1301,7 +1307,7 @@ 
 	 * Return an appropriate base tag for the given template.
 	 * It will be closed on an XHTML document, and unclosed on an HTML document.
 	 *
-	 * @param $contentGeneratedSoFar The content of the template generated so far; it should contain
+	 * @param string $contentGeneratedSoFar The content of the template generated so far; it should contain
 	 * the DOCTYPE declaration.
 	 */
 	public static function get_base_tag($contentGeneratedSoFar) {
@@ -1342,6 +1348,9 @@ 
 	 */
 	protected $cacheTemplate;
 
+	/**
+	 * @param string $content
+	 */
 	public function __construct($content, TemplateParser $parser = null) {
 		if ($parser) {
 			$this->setParser($parser);