diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index c655bf951700b..1337e7d05b3c1 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -60,7 +60,7 @@ jobs: needs: [composer] strategy: matrix: - command: ['php-cs-fixer fix -vvv --dry-run --diff', 'phpcs --extensions=php -p --standard=ruleset.xml .'] + command: ['php-cs-fixer fix -vvv', 'phpcs --extensions=php -p --standard=ruleset.xml .'] steps: - uses: actions/checkout@v4 - uses: actions/cache/restore@v4 @@ -68,6 +68,8 @@ jobs: path: libraries/vendor key: ${{ runner.os }}-composer-${{ hashFiles('composer.lock') }} - name: Check PHP code style + id: check-codestyle + continue-on-error: true env: PHP_CS_FIXER_IGNORE_ENV: true run: | @@ -76,6 +78,7 @@ jobs: - uses: parkerbxyz/suggest-changes@v2 with: comment: 'Your code does not comply with our codestyle. Please commit the suggested changes.' + event: 'REQUEST_CHANGES' code-style-js-css: name: Check Javascript & CSS code style diff --git a/libraries/src/MVC/Factory/MVCFactory.php b/libraries/src/MVC/Factory/MVCFactory.php index b21b3a1021f54..be1fe28deb2e2 100644 --- a/libraries/src/MVC/Factory/MVCFactory.php +++ b/libraries/src/MVC/Factory/MVCFactory.php @@ -79,7 +79,7 @@ class MVCFactory implements MVCFactoryInterface, FormFactoryAwareInterface, Site public function __construct($namespace, ?LoggerInterface $logger = null) { $this->namespace = $namespace; - $this->logger = $logger; + $this->logger = $logger; } /** diff --git a/libraries/src/MVC/View/HtmlView.php b/libraries/src/MVC/View/HtmlView.php index aa82b04dae6aa..1aa5cda393415 100644 --- a/libraries/src/MVC/View/HtmlView.php +++ b/libraries/src/MVC/View/HtmlView.php @@ -14,6 +14,7 @@ use Joomla\CMS\Event\View\DisplayEvent; use Joomla\CMS\Factory; use Joomla\CMS\Language\Text; +use Joomla\CMS\Router\SiteRouterAwareTrait; use Joomla\CMS\Uri\Uri; use Joomla\CMS\User\CurrentUserInterface; use Joomla\CMS\User\CurrentUserTrait; diff --git a/libraries/src/Service/Provider/CaptchaRegistry.php b/libraries/src/Service/Provider/CaptchaRegistry.php index 59585a04f1abe..061060eae750e 100644 --- a/libraries/src/Service/Provider/CaptchaRegistry.php +++ b/libraries/src/Service/Provider/CaptchaRegistry.php @@ -38,19 +38,19 @@ class CaptchaRegistry implements ServiceProviderInterface public function register(Container $container) { $container->alias('captcharegistry', Registry::class) - ->share( - Registry::class, - function (Container $container) { - $dispatcher = $container->get(DispatcherInterface::class); - $registry = new Registry(); - $registry->setDispatcher($dispatcher); + ->share( + Registry::class, + function (Container $container) { + $dispatcher = $container->get(DispatcherInterface::class); + $registry = new Registry(); + $registry->setDispatcher($dispatcher); - PluginHelper::importPlugin('captcha', null, true, $dispatcher); - $registry->initRegistry(); + PluginHelper::importPlugin('captcha', null, true, $dispatcher); + $registry->initRegistry(); - return $registry; - }, - true - ); + return $registry; + }, + true + ); } }