- Issue created by @urvashi_vora
- š®š³India Ashutosh Ahirwal India
I reviewed provided patch and it get applied cleanly.
There are one remaining issue.
Providing updated patch too.
Please review. - Status changed to RTBC
about 1 year ago 7:26am 5 October 2023 - šµšPhilippines clarkssquared
Hi
I applied your patch #2 and I confirmed that it fixes all the PHPCS issues except for the project, datestamp, and version in info.yml file which I believe will appear when I install the module via composer. I will retain the status to needs review to give way for other people to give their feedback regarding your patch, RTBC +1 from me.
ā simple_media_bulk_upload git:(master) ā curl https://www.drupal.org/files/issues/2023-10-05/phpcs-issue-fixes.patch | patch -p1 % Total % Received % Xferd Average Speed Time Time Time Current Dload Upload Total Spent Left Speed 100 4560 100 4560 0 0 17386 0 --:--:-- --:--:-- --:--:-- 17952 patching file 'src/Form/BulkUploadForm.php' ā simple_media_bulk_upload git:(master) ā .. ā contrib git:(master) ā phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml simple_media_bulk_upload FILE: /Users/clarksubing-subing/Projects/d9/d9-local/web/modules/contrib/simple_media_bulk_upload/simple_media_bulk_upload.info.yml ----------------------------------------------------------------------------------------------------------------------------------- FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 1 LINE ----------------------------------------------------------------------------------------------------------------------------------- 1 | WARNING | Remove "project" from the info file, it will be added by drupal.org packaging automatically 1 | WARNING | Remove "datestamp" from the info file, it will be added by drupal.org packaging automatically 1 | WARNING | Remove "version" from the info file, it will be added by drupal.org packaging automatically ----------------------------------------------------------------------------------------------------------------------------------- Time: 348ms; Memory: 12MB
- Status changed to Needs review
3 months ago 8:49pm 16 September 2024 - ššŗHungary nagy.balint
The patch is a bit outdated.
Here is a reroll for 2.0.x, but likely applies to 1.0.x too.
- ššŗHungary nagy.balint
Actually the FormBase already have
use LoggerChannelTrait;
use MessengerTrait;
So we don't even need some of those injections. -
nagy.balint ā
committed a0061cfa on 2.0.x
Issue #3372423 by nagy.balint, urvashi_vora, Ashutosh Ahirwal,...
-
nagy.balint ā
committed a0061cfa on 2.0.x
- Status changed to Fixed
3 months ago 8:46am 17 September 2024 Automatically closed - issue fixed for 2 weeks with no activity.