Removal of (resolved) TODOS

Author: tersmitten

Model/Datasource/RedisSource.php

@@ -220,7 +220,6 @@ protected function _setPrefix() {
  * @return bool Always true
  */
 	public function close() {
-		// TODO: Remove useless condition
 		if ($this->isConnected()) {
 			$this->_connection->close();
 		}
@@ -245,7 +244,6 @@ public function isConnected() {
  *
  * @param mixed $data List of tables
  * @return array Array of sources available in this datasource
- * @todo: Remove useless method?
  */
 	public function listSources($data = null) {
 		return parent::listSources($data);
@@ -256,25 +254,11 @@ public function listSources($data = null) {
  *
  * @param Model|string $model Name of database table to inspect or model instance
  * @return array Array of Metadata for the $model
- * @todo: Remove useless method?
  */
 	public function describe($model) {
 		return parent::describe($model);
 	}
 
-/**
- * Returns an SQL calculation, i.e. COUNT() or MAX()
- *
- * @param Model $Model The model to get a calculated field for
- * @param string $func Lowercase name of SQL function, i.e. 'count' or 'max'
- * @param array $params Function parameters (any values must be quoted manually)
- * @return string An SQL calculation function
- * @todo Remove useless method?
- */
-	public function calculate(Model $Model, $func, $params = array()) {
-		return array('count' => true);
-	}
-
 }
 
 /**

Test/Case/Model/Datasource/RedisSourceTest.php

@@ -688,21 +688,4 @@ public function testDescribe() {
 		$this->assertNull($result);
 	}
 
-/**
- * testCalculate method
- *
- * @return void
- */
-	public function testCalculate() {
-		$Source = new TestRedisSource();
-		$Model = $this->getMockForModel('Model');
-		$func = 'foo';
-		$params = array('b', 'a', 'r');
-
-		$result = $Source->calculate($Model, $func, $params);
-		$expected = array('count' => true);
-
-		$this->assertIdentical($expected, $result);
-	}
-
 }