FIx the issues reported by phpcs

FILE: C:\xamppnew\htdocs\s\web\modules\nextcloud_dam\nextcloud_dam.module ------------------------------------------------------------------------- FOUND 1 ERROR AFFECTING 1 LINE ------------------------------------------------------------------------- 124 | ERROR | [x] Expected 1 newline at end of file; 2 found ------------------------------------------------------------------------- PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY -------------------------------------------------------------------------

FILE: ...mppnew\htdocs\s\web\modules\nextcloud_dam\src\NextcloudDamApiInterface.php -------------------------------------------------------------------------------- FOUND 2 ERRORS AFFECTING 2 LINES -------------------------------------------------------------------------------- 94 | ERROR | Type hint "array" missing for $share_options 143 | ERROR | Type hint "array" missing for $path --------------------------------------------------------------------------------

FILE: ...w\htdocs\s\web\modules\nextcloud_dam\src\Plugin\media\Source\Nextcloud.php -------------------------------------------------------------------------------- FOUND 3 ERRORS AFFECTING 2 LINES -------------------------------------------------------------------------------- 127 | ERROR | Expected type hint "nextcloudDamInterface"; found | | "NextcloudDamApiInterface" for $nextcloud_dam_api_service 329 | ERROR | Type hint "Drupal\media\MediaInterface" missing for $media 329 | ERROR | Type hint "array" missing for $metadata --------------------------------------------------------------------------------

FILE: ...docs\s\web\modules\nextcloud_dam\src\Plugin\QueueWorker\UpdateMetadata.php -------------------------------------------------------------------------------- FOUND 2 ERRORS AFFECTING 1 LINE -------------------------------------------------------------------------------- 99 | ERROR | Expected type hint "LoggerChannel"; found | | "LoggerChannelInterface" for $logger_channel 99 | ERROR | Expected type hint "EventDispatcher"; found | | "EventDispatcherInterface" for $event_dispatcher --------------------------------------------------------------------------------

Comments & Activities

Issue created by @sonam_sharma
Comment almost 2 years ago →
🇮🇳India sonam_sharma
@sonam_sharma opened merge request.
Open in Jenkins → Open on Drupal.org →
Core: 10.0.7 + Environment: PHP 8.1 & MariaDB 10.3.22
last update almost 2 years ago
6 fail
Status changed to Needs work almost 2 years ago9:57pm 1 July 2023
Comment almost 2 years ago →
🇪🇸Spain aleix
Thank's for contributing.

I haven't the same results with phpcs running
phpcs --standard=Drupal,DrupalPractice src/NextcloudDamApiInterface.php
Could you explain how the command was launched
Status changed to Postponed: needs info almost 2 years ago10:56pm 1 July 2023
Comment almost 2 years ago →
🇪🇸Spain aleix
Status changed to Closed: outdated 5 months ago12:12am 23 November 2024
Comment 5 months ago →
🇪🇸Spain aleix

FIx the issues reported by phpcs

Problem/Motivation

Comments & Activities