[11.x][PHPDOC] Fix bad phpdoc for \Illuminate\Database\Connection::$schemaGrammar

[dimitriBouteille]

This PR fix bad phpdoc in \Illuminate\Database\Connection :

• $schemaGrammar : This property can be null or \Illuminate\Database\Schema\Grammars\Grammar

Here is an example where the property $schemaGrammar is null :

framework/src/Illuminate/Database/Connection.php

Lines 301 to 307 in 8f6a9b0

	public function getSchemaBuilder()
	{
	if (is_null($this->schemaGrammar)) {
	$this->useDefaultSchemaGrammar();
	}
	return new SchemaBuilder($this);

framework/src/Illuminate/Database/Connection.php

Lines 266 to 274 in 8f6a9b0

	/**
	* Get the default schema grammar instance.
	*
	* @return \Illuminate\Database\Schema\Grammars\Grammar|null
	*/
	protected function getDefaultSchemaGrammar()
	{
	//
	}

[shaedrich]

Doesn't

protected function getDefaultGrammar()
{
    //
}

result in void rather than null? Sure, it's type-cast to null, but having the PHPDoc say void would be more correct, wouldn't it?

[dimitriBouteille]

@shaedrich Is good question. For me, getDefaultGrammar must be return null by default. The function useDefaultSchemaGrammar call getDefaultGrammar, void is invalid type ...

framework/src/Illuminate/Database/Connection.php

Lines 256 to 274 in 8f6a9b0

	/**
	* Set the schema grammar to the default implementation.
	*
	* @return void
	*/
	public function useDefaultSchemaGrammar()
	{
	$this->schemaGrammar = $this->getDefaultSchemaGrammar();
	}
	/**
	* Get the default schema grammar instance.
	*
	* @return \Illuminate\Database\Schema\Grammars\Grammar|null
	*/
	protected function getDefaultSchemaGrammar()
	{
	//
	}

[shaedrich]

Since when is void invalid? Sure, it's only a return type, but that doesn't make it less valid