From 28153443a55290877f4507a06b404f39c1cd6ec1 Mon Sep 17 00:00:00 2001 From: Côme Chilliet Date: Thu, 26 Sep 2024 11:11:20 +0200 Subject: chore(rector): Add rule to import Nextcloud classes in all files MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit We skip all files outside of OC/OCA/OCP from "use" auto-import because it’s not always desirable to import classes when they have generic names, so it should be the developer choice. Signed-off-by: Côme Chilliet --- build/rector.php | 25 +++++++++++++++++++++++++ 1 file changed, 25 insertions(+) (limited to 'build') diff --git a/build/rector.php b/build/rector.php index ed8b748663e..9df995a0044 100644 --- a/build/rector.php +++ b/build/rector.php @@ -7,10 +7,32 @@ declare(strict_types=1); * SPDX-License-Identifier: AGPL-3.0-only */ +use PhpParser\Node; +use Rector\CodingStyle\Contract\ClassNameImport\ClassNameImportSkipVoterInterface; use Rector\Config\RectorConfig; +use Rector\StaticTypeMapper\ValueObject\Type\FullyQualifiedObjectType; +use Rector\ValueObject\Application\File; $nextcloudDir = dirname(__DIR__); +class NextcloudNamespaceSkipVoter implements ClassNameImportSkipVoterInterface { + private array $namespacePrefixes = [ + 'OC', + 'OCA', + 'OCP', + ]; + public function shouldSkip(File $file, FullyQualifiedObjectType $fullyQualifiedObjectType, Node $node) : bool { + foreach ($this->namespacePrefixes as $prefix) { + if (str_starts_with($fullyQualifiedObjectType->getClassName(), $prefix . '\\')) { + // Import Nextcloud namespaces + return false; + } + } + // Skip everything else + return true; + } +} + $config = RectorConfig::configure() ->withPaths([ $nextcloudDir . '/apps', @@ -30,9 +52,12 @@ $config = RectorConfig::configure() ]) // uncomment to reach your current PHP version // ->withPhpSets() + ->withImportNames(importShortClasses:false) ->withTypeCoverageLevel(0); +$config->registerService(NextcloudNamespaceSkipVoter::class, tag:ClassNameImportSkipVoterInterface::class); +/* Ignore all files ignored by git */ $ignoredEntries = shell_exec('git status --porcelain --ignored ' . escapeshellarg($nextcloudDir)); $ignoredEntries = explode("\n", $ignoredEntries); $ignoredEntries = array_filter($ignoredEntries, static fn (string $line) => str_starts_with($line, '!! ')); -- cgit v1.2.3