Add psalm types for the migration schema closure

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
This commit is contained in:
Christoph Wurst 2020-10-15 10:58:51 +02:00 committed by backportbot[bot]
parent c9d196af78
commit 5290cc77bc
2 changed files with 15 additions and 12 deletions

View File

@ -1,4 +1,7 @@
<?php <?php
declare(strict_types=1);
/** /**
* @copyright Copyright (c) 2017 Joas Schilling <coding@schilljs.com> * @copyright Copyright (c) 2017 Joas Schilling <coding@schilljs.com>
* *
@ -49,6 +52,7 @@ interface IMigrationStep {
/** /**
* @param IOutput $output * @param IOutput $output
* @param \Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper` * @param \Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
* @psalm-param Closure():ISchemaWrapper $schemaClosure
* @param array $options * @param array $options
* @since 13.0.0 * @since 13.0.0
*/ */
@ -57,6 +61,7 @@ interface IMigrationStep {
/** /**
* @param IOutput $output * @param IOutput $output
* @param \Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper` * @param \Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
* @psalm-param Closure():ISchemaWrapper $schemaClosure
* @param array $options * @param array $options
* @return null|ISchemaWrapper * @return null|ISchemaWrapper
* @since 13.0.0 * @since 13.0.0
@ -66,6 +71,7 @@ interface IMigrationStep {
/** /**
* @param IOutput $output * @param IOutput $output
* @param \Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper` * @param \Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
* @psalm-param Closure():ISchemaWrapper $schemaClosure
* @param array $options * @param array $options
* @since 13.0.0 * @since 13.0.0
*/ */

View File

@ -1,4 +1,7 @@
<?php <?php
declare(strict_types=1);
/** /**
* @copyright Copyright (c) 2017 Joas Schilling <coding@schilljs.com> * @copyright Copyright (c) 2017 Joas Schilling <coding@schilljs.com>
* *
@ -25,8 +28,6 @@
namespace OCP\Migration; namespace OCP\Migration;
use OCP\DB\ISchemaWrapper;
/** /**
* @since 13.0.0 * @since 13.0.0
*/ */
@ -52,19 +53,16 @@ abstract class SimpleMigrationStep implements IMigrationStep {
} }
/** /**
* @param IOutput $output * {@inheritDoc}
* @param \Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper` *
* @param array $options
* @since 13.0.0 * @since 13.0.0
*/ */
public function preSchemaChange(IOutput $output, \Closure $schemaClosure, array $options) { public function preSchemaChange(IOutput $output, \Closure $schemaClosure, array $options) {
} }
/** /**
* @param IOutput $output * {@inheritDoc}
* @param \Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper` *
* @param array $options
* @return null|ISchemaWrapper
* @since 13.0.0 * @since 13.0.0
*/ */
public function changeSchema(IOutput $output, \Closure $schemaClosure, array $options) { public function changeSchema(IOutput $output, \Closure $schemaClosure, array $options) {
@ -72,9 +70,8 @@ abstract class SimpleMigrationStep implements IMigrationStep {
} }
/** /**
* @param IOutput $output * {@inheritDoc}
* @param \Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper` *
* @param array $options
* @since 13.0.0 * @since 13.0.0
*/ */
public function postSchemaChange(IOutput $output, \Closure $schemaClosure, array $options) { public function postSchemaChange(IOutput $output, \Closure $schemaClosure, array $options) {