silverstripe / silverstripe-mssql

code/MSSQLDatabase.php

Doc Comments +3 added lines, -3 removed lines

@@ -106,7 +106,7 @@ 
 	 * Checks whether the current SQL Server version has full-text
 	 * support installed and full-text is enabled for this database.
 	 *
-	 * @return boolean
+	 * @return string|boolean
 	 */
 	public function fullTextEnabled() {
 		if($this->fullTextEnabled === null) {
@@ -119,7 +119,7 @@ 
 	 * Checks whether the current SQL Server version has full-text
 	 * support installed and full-text is enabled for this database.
 	 *
-	 * @return boolean
+	 * @return false|string
 	 */
 	protected function updateFullTextEnabled() {
 		// Check if installed
@@ -308,7 +308,7 @@ 
 	 * Returns a SQL fragment for querying a fulltext search index
 	 *
 	 * @param $tableName specific - table name
-	 * @param $keywords string The search query
+	 * @param string $keywords string The search query
 	 * @param $fields array The list of field names to search on, or null to include all
 	 *
 	 * @returns null if keyword set is empty or the string with JOIN clause to be added to SQL query

code/MSSQLSchemaManager.php

Doc Comments +6 added lines, -5 removed lines

@@ -86,7 +86,7 @@ 
 	 * Check if a fulltext index exists on a particular table name.
 	 * 
 	 * @param string $tableName
-	 * @return boolean TRUE index exists | FALSE index does not exist | NULL no support
+	 * @return null|boolean TRUE index exists | FALSE index does not exist | NULL no support
 	 */
 	function fulltextIndexExists($tableName) {
 		// Special case for no full text index support
@@ -124,7 +124,7 @@ 
 	 * Gets the identity column of a table
 	 * 
 	 * @param string $tableName
-	 * @return string|null
+	 * @return string
 	 */
 	function getIdentityColumn($tableName) {
 		return $this->preparedQuery("
@@ -168,7 +168,7 @@ 
 	 * @param $options An map of additional options.  The available keys are as follows:
 	 *   - 'MSSQLDatabase'/'MySQLDatabase'/'PostgreSQLDatabase' - database-specific options such as "engine" for MySQL.
 	 *   - 'temporary' - If true, then a temporary table will be created
-	 * @return The table name generated.  This may be different from the table name, for example with temporary tables.
+	 * @return string table name generated.  This may be different from the table name, for example with temporary tables.
 	 */
 	public function createTable($tableName, $fields = null, $indexes = null, $options = null, $advancedOptions = null) {
 		$fieldSchemas = $indexSchemas = "";
@@ -233,7 +233,7 @@ 
 	 * 
 	 * @param string $tableName Table name column resides in
 	 * @param string $columnName Column name the constraint is for
-	 * @return string|null
+	 * @return string
 	 */
 	public function getConstraintName($tableName, $columnName) {
 		return $this->preparedQuery("
@@ -288,7 +288,7 @@ 
 	 * 
 	 * @param string $tableName Name of the table
 	 * @param string $colName Name of the column
-	 * @return string|null
+	 * @return string
 	 */
 	protected function defaultConstraintName($tableName, $colName) {
 		return $this->preparedQuery("
@@ -625,6 +625,7 @@ 
 	 * For a given table name, get all the internal index names,
 	 * except for those that are primary keys and fulltext indexes.
 	 *
+	 * @param string $tableName
 	 * @return array
 	 */
 	public function indexNames($tableName) {

code/SQLServerConnector.php

Doc Comments -1 removed lines

@@ -177,7 +177,6 @@
 	 * Quotes a string, including the "N" prefix so unicode
 	 * strings are saved to the database correctly.
 	 *
-	 * @param string $string String to be encoded
 	 * @return string Processed string ready for DB
 	 */
 	public function quoteString($value) {