undefined - Contrib.social

🇵🇭Philippines paraderojether

I reviewed MR!21 and confirmed it fixes the Create new account button spacing issue.

I added screenshots for reference.
Thank you.

🐛 | Rover | In User register page, Email address needs space from top.

🇵🇭Philippines paraderojether

Hi alok_singh,

I reviewed MR!23 and it is not applied successfully.

➜  rover git:(main) ✗ curl https://git.drupalcode.org/project/rover/-/merge_requests/23.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   427    0   427    0     0    320      0 --:--:--  0:00:01 --:--:--   321
patching file 'css/style.css'
No such line 1099 in input file, ignoring
1 out of 1 hunks failed--saving rejects to 'css/style.css.rej'
➜  rover git:(main) ✗

Kindly check and advise, please.
Thank you.

🐛 | Industrial Zymphonies Theme | In Advance search field needs some space.

🇵🇭Philippines paraderojether

I reviewed MR!19, and confirmed it fixes the spacing issue of the field.

I added a screenshot for reference.
Thank you.

🐛 | File De-duplicator | Php coding standads

🇵🇭Philippines paraderojether

Hi szato,

I reviewed patch #2, applied it against File De-duplicator 1.2.2, and confirmed it fixes the issues reported by phpcs.

➜  file_de_duplicator git:(main) ✗ curl https://www.drupal.org/files/issues/2025-01-23/phpcs-fixes.3501752-2.patch | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 24380  100 24380    0     0  46707      0 --:--:-- --:--:-- --:--:-- 47339
patching file drush.services.yml
patching file file_de_duplicator.module
patching file file_de_duplicator.permissions.yml
patching file file_de_duplicator.routing.yml
patching file file_de_duplicator.services.yml
patching file 'src/Commands/DeDuplicator.php'
patching file 'src/Controller/DuplicatesController.php'
patching file 'src/DuplicateFinder.php'
patching file 'src/Form/FindDuplicatesForm.php'
➜  file_de_duplicator git:(main) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig file_de_duplicator
➜  contrib git:(main) ✗

Moving this RTBC.
Thank you.

🐛 | Small Business theme | Got some errors on home page.

🇵🇭Philippines paraderojether

I reviewed MR!2, applied it against Small Business theme 2.0.1, and confirmed it fixes the errors on the homepage.

I added screenshots for reference.
Thank you.

📌 | Term Manager | Fix the issues reported by phpcs

🇵🇭Philippines paraderojether

Hi I reviewed MR!2 and there are still PHP_CodeSniffer remaining.

➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig term_manager

FILE: /Users/jetherparadero/Q1/q1contributions/web/modules/contrib/term_manager/src/Form/TermManagerMoveForm.php
----------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 3 LINES
----------------------------------------------------------------------------------------------------------------
 102 | WARNING | Vocabulary::loadMultiple calls should be avoided in classes, use dependency injection instead
 169 | WARNING | Term::load calls should be avoided in classes, use dependency injection instead
 173 | WARNING | Term::load calls should be avoided in classes, use dependency injection instead
----------------------------------------------------------------------------------------------------------------

Time: 245ms; Memory: 10MB

I reviewed MR!6 and it failed on eslint, phpcs, phpstan using Gitlab CI.

Kindly check and advise, please.
Thank you.

📌 | Views Load More | Coding Standard issues

🇵🇭Philippines paraderojether

Hi I reviewed MR!11 and it is not applied cleanly.

➜  views_load_more git:(2.x) curl https://git.drupalcode.org/project/views_load_more/-/merge_requests/11.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 22775    0 22775    0     0  69763      0 --:--:-- --:--:-- --:--:-- 72301
patching file .gitlab-ci.yml
patching file README.md
3 out of 3 hunks failed--saving rejects to README.md.rej
patching file 'src/Ajax/VLMAppendCommand.php'
5 out of 5 hunks failed--saving rejects to 'src/Ajax/VLMAppendCommand.php.rej'
patching file 'src/EventSubscriber/VLMEventSubscriber.php'
3 out of 4 hunks failed--saving rejects to 'src/EventSubscriber/VLMEventSubscriber.php.rej'
patching file 'src/Plugin/views/pager/LoadMore.php'
3 out of 3 hunks failed--saving rejects to 'src/Plugin/views/pager/LoadMore.php.rej'
patching file views_load_more.module
4 out of 4 hunks failed--saving rejects to views_load_more.module.rej
➜  views_load_more git:(2.x) ✗

Kindly check and advise, please.
Thank you.

📌 | Configuration Provider | GitLabCI: Fix existing PHPCS issues

🇵🇭Philippines paraderojether

I reviewed MR!9, and confirmed PHPCS & PHPStan issues are fixed on the Gitlab CI

However, I noticed a phpcs issue using PHP_CodeSniffer:

➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig config_provider

FILE: /Users/jetherparadero/Q1/q1contributions/web/modules/contrib/config_provider/src/ConfigProviderConfigInstaller.php
-------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------------------------------------------
 12 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\config_provider\Plugin\ConfigCollectorInterface.
-------------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/jetherparadero/Q1/q1contributions/web/modules/contrib/config_provider/src/Plugin/ConfigCollector.php
-----------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------------------------------------
 8 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Config\StorageInterface.
-----------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------------

Time: 391ms; Memory: 10MB

Command used: phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig config_provider

Kindly check and advise, please.
Thank you.

📌 | FlexiStyle | Fix issues reported by PHPCS

🇵🇭Philippines paraderojether

Hi nidhish,

I reviewed MR!11, again and it did not applied cleanly on my end.

➜  flexi_style git:(1.4.1) curl https://git.drupalcode.org/project/flexi_style/-/merge_requests/11.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  5945    0  5945    0     0  16466      0 --:--:-- --:--:-- --:--:-- 16841
patching file 'css/base/elements.css'
patching file 'css/base/formating.css'
patching file 'css/component/buttons.css'
patching file flexi_style.theme
1 out of 1 hunks failed--saving rejects to flexi_style.theme.rej
patching file 'fonts/fontawesome/LICENSE.txt'
patching file 'templates/form/input--submit.html.twig'
patching file theme-settings.php
➜  flexi_style git:(1.4.1) ✗

Kindly check and advise.
Thank you.

📌 | FlexiStyle | Fix issues reported by PHPCS

🇵🇭Philippines paraderojether

Hi @nidhish,

I reviewed MR!11, applied against FlexiStyle 1.4.1, and there are still remaining phpcs issues show bellow:

➜  flexi_style git:(main) ✗ curl https://git.drupalcode.org/project/flexi_style/-/merge_requests/11.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  5209    0  5209    0     0  13844      0 --:--:-- --:--:-- --:--:-- 14040
patching file 'css/base/elements.css'
patching file 'css/base/formating.css'
patching file 'css/component/buttons.css'
patching file flexi_style.theme
patching file 'fonts/fontawesome/LICENSE.txt'
patching file theme-settings.php
➜  flexi_style git:(main) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig flexi_style

FILE: .../Q1/q1contributions/web/themes/contrib/flexi_style/css/component/field.css
--------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------
 26 | ERROR | [x] Expected 1 newline at end of file; 0 found
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------


FILE: ...ions/web/themes/contrib/flexi_style/templates/form/input--submit.html.twig
--------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------
 15 | ERROR | [x] Expected 1 newline at end of file; 0 found
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------

Time: 2.03 secs; Memory: 60MB

Command used: phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig flexi_style

Please check and advise.
Thank you.

📌 | Matomo reporting API | Fix the issues reported by PHP_CodeSniffer

🇵🇭Philippines paraderojether

📌 | Matomo reporting API | Fix the issues reported by PHP_CodeSniffer

🇵🇭Philippines paraderojether

Hi ismaelromero,

I reviewed MR!7, applied against Matomo reporting API 3.0.x-dev and there is still on phpcs remaining:

➜  matomo_reporting_api git:(3.0.x) curl https://git.drupalcode.org/project/matomo_reporting_api/-/merge_requests/7.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 20598    0 20598    0     0  55511      0 --:--:-- --:--:-- --:--:-- 56587
patching file .gitlab-ci.yml
patching file matomo_reporting_api.install
patching file matomo_reporting_api.module
patching file matomo_reporting_api.post_update.php
patching file 'matomo_reporting_api_example/readme.md'
patching file 'matomo_reporting_api_example/src/Plugin/Block/MatomoStatisticsBlock.php'
patching file readme.md
patching file 'src/ConfigHandler.php'
patching file 'src/ConfigHandlerInterface.php'
patching file 'src/EventSubscriber/MatomoReportingApiSubscriber.php'
patching file 'src/Exception/MissingMatomoServerUrlException.php'
patching file 'src/Form/SettingsForm.php'
patching file 'src/MatomoQueryFactory.php'
patching file 'src/MatomoQueryFactoryInterface.php'
patching file 'tests/src/Kernel/AccessDeniedTest.php'
patching file 'tests/src/Unit/ConfigHandlerTest.php'
➜  matomo_reporting_api git:(3.0.x) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig matomo_reporting_api

FILE: /Users/jetherparadero/Q1/q1contributions/web/modules/contrib/matomo_reporting_api/tests/src/Unit/ConfigHandlerTest.php
------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------------------------------------
 10 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\matomo_reporting_api\ConfigHandler.
------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------------------

Time: 306ms; Memory: 10MB

➜  contrib git:(main) ✗

Please check and advise.
Thank you.

🐛 | Page Notifications | Bunch of Undefined Array Warnings After Install

🇵🇭Philippines paraderojether

I reviewed MR!3, applied it against the Page Notifications 3.0.4, and confirmed it fixes the Warnings and Issues after a fresh install of the module.

I added screenshots for reference.
Thank you.

🐛 | One Page Zymphonies Theme | On mobile view subject input box going out from container.

🇵🇭Philippines paraderojether

Hi @alok_singh,

I reviewed MR!13, and it did not applied successfully.

 onepage_zymphonies_theme git:(main) ✗ curl https://git.drupalcode.org/project/onepage_zymphonies_theme/-/merge_requests/13.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  3735    0  3735    0     0   9581      0 --:--:-- --:--:-- --:--:--  9751
patching file .gitlab-ci.yml
patching file 'css/global.css'
patching file 'sass/_global-styles.scss'
patching file 'templates/form/select.html.twig'
2 out of 2 hunks failed--saving rejects to 'templates/form/select.html.twig.rej'
➜  onepage_zymphonies_theme git:(main)

Kindly check and advise, please.
Thank you.

📌 | Recombee | Coding standards issue

🇵🇭Philippines paraderojether

I reviewed MR!7 again and confirmed it passed the phpcs using Gitlab CI and PHP_CodeSniffer.

➜  recombee git:(2.0.x) curl https://git.drupalcode.org/project/recombee/-/merge_requests/7.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  4790    0  4790    0     0  10822      0 --:--:-- --:--:-- --:--:-- 10961
patching file 'js/recombee.properties.js'
patching file 'js/recombee.public_scenario.js'
patching file 'js/recombee.tracker.js'
patching file 'src/Form/SettingsForm.php'
patching file 'src/Plugin/Block/RecombeePublicScenario.php'
➜  recombee git:(2.0.x) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig recombee
➜  contrib git:(main) ✗

Moving this to RTBC.
Thank you.

📌 | Recombee | Coding standards issue

🇵🇭Philippines paraderojether

Hi kinyein,

I reviewed MR!7, and confirmed it passed the phpcs using Gitlab CI. However there are still phpcs issues reported using the PHP_CodeSniffer.

➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig recombee

FILE: ...rs/jetherparadero/Q1/q1contributions/web/modules/contrib/recombee/src/Form/SettingsForm.php
-------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 7 WARNINGS AFFECTING 7 LINES
-------------------------------------------------------------------------------------------------
  44 | WARNING | t() calls should be avoided in classes, use
     |         | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
  45 | WARNING | t() calls should be avoided in classes, use
     |         | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
  50 | WARNING | t() calls should be avoided in classes, use
     |         | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
  51 | WARNING | t() calls should be avoided in classes, use
     |         | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
  96 | WARNING | t() calls should be avoided in classes, use
     |         | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 102 | WARNING | t() calls should be avoided in classes, use
     |         | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
 118 | WARNING | t() calls should be avoided in classes, use
     |         | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
-------------------------------------------------------------------------------------------------


FILE: ...Q1/q1contributions/web/modules/contrib/recombee/src/Plugin/Block/RecombeePublicScenario.php
-------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------
 131 | WARNING | t() calls should be avoided in classes, use
     |         | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
-------------------------------------------------------------------------------------------------

Time: 165ms; Memory: 10MB

Command used: phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig recombee

Kindly check and advise, please.
Thank you.

📌 | Button Field | Automated Drupal 11 compatibility fixes for button_field

🇵🇭Philippines paraderojether

mradcliffe → credited paraderojether → .

🐛 | Fepper | On Mobile view, Add new comment - subject input box needs to be styled.

🇵🇭Philippines paraderojether

Hi alok_singh,

I reviewed MR!15, applied it against Fepper 10.0.2, and confirmed the comment subject field is now full width on mobile.

I added a screenshot for reference.
Thank you.

🐛 | Market Wave | On Mobile view, Advance search input box, needs space from right side.

🇵🇭Philippines paraderojether

Hi alok_singh,

I reviewed MR!8, applied it against the Market Wave 1.0.6, and confirmed the Advance search input box overlapping issue is fixed.

I added a screenshot for reference.
Thank you.

🐛 | FlexiStyle DXP | Search button overlap Search input box.

🇵🇭Philippines paraderojether

I reviewed MR!8, applied it against the FlexiStyle DXP 1.0.0, and confirmed the Search button overlap Search input box issue if fixed.

I added a screenshot for reference.
Thank you.

🐛 | FlexiStyle DXP | On Mobile view, Login page input box going out from right side.

🇵🇭Philippines paraderojether

Hi alok_singh,

I reviewed MR!8, applied it against the FlexiStyle DXP 1.0.0, and confirmed the Login page input box overlapping issue if fixed.

I added a screenshot for reference.
Thank you.

🐛 | FlexiStyle Bootstrap SCSS | Read more and Add new comment, Needs to be be styled.

🇵🇭Philippines paraderojether

Hi alok_singh,

I reviewed MR!2 against FlexiStyle Bootstrap SCSS 1.0.8 and it did not apply successfully.

➜  flexistyle_bootstrap_scss git:(main) ✗ curl https://git.drupalcode.org/project/flexistyle_bootstrap_scss/-/merge_requests/2.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  5630    0  5630    0     0  14253      0 --:--:-- --:--:-- --:--:-- 14776
patching file 'css/style.css'
1 out of 1 hunks failed--saving rejects to 'css/style.css.rej'
patching file 'scss/theme/style.scss'
1 out of 1 hunks failed--saving rejects to 'scss/theme/style.scss.rej'
➜  flexistyle_bootstrap_scss git:(main) ✗ curl https://git.drupalcode.org/project/flexistyle_bootstrap_scss/-/merge_requests/2.diff | git apply -v
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  5630    0  5630    0     0  12922      0 --:--:-- --:--:-- --:--:-- 13404
Skipped patch 'css/style.css'.
Skipped patch 'scss/theme/style.scss'.
➜  flexistyle_bootstrap_scss git:(main) ✗

Please check and advise.
Thank you.

🐛 | FlexiStyle Bootstrap | On mobile view header is overlapping on toolbaar.

🇵🇭Philippines paraderojether

I reviewed MR!4, applied it against the FlexiStyle Bootstrap 1.2.9, and confirmed that the header is overlapping on the toolbar issue is fixed.

I added a screenshot for reference.
Thank you.

🐛 | Users Bulk Delete | Drupal Coding Standard is not matched

🇵🇭Philippines paraderojether

I reviewed MR!2, applied it against Users Bulk Delete, and confirmed phpcs are now fixed.

Code Sniffer

➜  user_bulk_delete git:(main) ✗ curl https://git.drupalcode.org/project/user_bulk_delete/-/merge_requests/2.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  2693    0  2693    0     0   9171      0 --:--:-- --:--:-- --:--:--  9318
patching file .gitlab-ci.yml
patching file user_bulk_delete.install
patching file user_bulk_delete.module
➜  user_bulk_delete git:(main) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig user_bulk_delete
➜  contrib git:(main) ✗

It also passed on the Gitlab CI.

Moving this to RTBC!
Thank you.

🐛 | JSON-LD REST Services | runtime error after installing module

🇵🇭Philippines paraderojether

I reviewed MR!2, applied it against JSON-LD REST Services 3.0.2 and confirmed it fixes the runtime error displayed on the configuration page.

➜  jsonld git:(main) ✗ curl https://git.drupalcode.org/project/jsonld/-/merge_requests/2.diff |
 patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  1104    0  1104    0     0   3604      0 --:--:-- --:--:-- --:--:--  3680
patching file 'src/Form/JsonLdSettingsForm.php'
➜  jsonld git:(main)

I also added screenshots for reference.
Thank you.

✨ | Matomo reporting API | Add configure link to info.yml

🇵🇭Philippines paraderojether

Hi tostinni,

I reviewed MR!9, applied it against the Matomo reporting API 3.0.x-dev, and confirmed the configure link on the module list was added successfully.

➜  matomo_reporting_api git:(3477552-fix-the-issues) curl https://git.drupalcode.org/project/matomo_reporting_api/-/merge_requests/9.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   465    0   465    0     0   1550      0 --:--:-- --:--:-- --:--:--  1597
patching file matomo_reporting_api.info.yml
➜  matomo_reporting_api git:(3477552-fix-the-issues)

I added screenshots for reference.
Thank you.

📌 | Year Only | Coding standard issues

🇵🇭Philippines paraderojether

I reviewed MR!13 again, applied it against against Year Only 9.x-dev and confirmed the remaining phpcs are now fixed.

➜  yearonly git:(9.x) curl https://git.drupalcode.org/project/yearonly/-/merge_requests/13.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  3583    0  3583    0     0   9313      0 --:--:-- --:--:-- --:--:--  9453
patching file 'src/Feeds/Target/YearOnly.php'
patching file 'src/Plugin/Field/FieldType/YearOnlyItem.php'
patching file 'src/Plugin/Field/FieldWidget/YearOnlyDefaultWidget.php'
➜  yearonly git:(9.x) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig yearonly
➜  contrib git:(main) ✗

Moving this to RTBC!
Thank you.

📌 | Database logging ban operation | Coding standard issues

🇵🇭Philippines paraderojether

Hi divyansh.gupta,

I double checked MR!8 and this is now passed on the Gitlab CI

Moving this to RTBC.
Thank you.

🐛 | Simple WYSIWYG | Fix errors reported by scpell

🇵🇭Philippines paraderojether

I reviewed MR!8 and confirmed all coding standard issues are fixed and it passed on the Gitlab CI

I also checked PHP_CodeSniffer and no phpcs issues found.

➜  simple_wysiwyg git:(1.0.x) curl https://git.drupalcode.org/project/simple_wysiwyg/-/merge_requests/8.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 12342    0 12342    0     0  41947      0 --:--:-- --:--:-- --:--:-- 43153
patching file .cspell-project-words.txt
patching file .gitlab-ci.yml
patching file 'css/simple-wysiwyg.css'
patching file 'js/simple-wysiwyg.js'
➜  simple_wysiwyg git:(1.0.x) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig simple_wysiwyg
➜  contrib git:(main) ✗

Moving this to RTBC!
Thank you.

📌 | Upload size per role | Coding standards

🇵🇭Philippines paraderojether

I reviewed MR!6, applied it against Upload size per role 1.0.x-dev and confirmed it fixes the issues reported by PHP_CodeSniffer.

 upload_size_per_role git:(1.0.x) curl https://git.drupalcode.org/project/upload_size_per_role/-/merge_requests/6.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  8787    0  8787    0     0  29328      0 --:--:-- --:--:-- --:--:-- 29989
patching file .cspell-project-words.txt
patching file .gitlab-ci.yml
patching file 'src/Form/UploadSizePerRoleSettingsForm.php'
patching file upload_size_per_role.module
➜  upload_size_per_role git:(1.0.x) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig upload_size_per_role
➜  contrib git:(main) ✗

phpcs, phpstan, and cspell also passed on the Gitlab CI

Moving this to RTBC!
Thank you.

📌 | Database logging ban operation | Coding standard issues

🇵🇭Philippines paraderojether

I reviewed MR!8, applied it against the Database logging ban operation 2.0.x and confirmed it fixes the issues reported by PHP_CodeSniffer.

dblog_ban git:(2.0.x) curl https://git.drupalcode.org/project/dblog_ban/-/merge_requests/8.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  5137    0  5137    0     0  12003      0 --:--:-- --:--:-- --:--:-- 12201
patching file CHANGELOG.txt
patching file README.md
patching file 'src/Controller/IpBanUnbanController.php'
patching file 'src/Form/BanUnbanConfirmFormBase.php'
patching file 'tests/modules/dblog_ban_test/dblog_ban_test.info.yml'
patching file 'tests/src/Unit/RouteProcessorCsrfAjaxTest.php'
➜  dblog_ban git:(2.0.x) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig dblog_ban
➜  contrib git:(main) ✗

However, it display phpcs issues on the Gitlab CI

Kindly check and advise, please.
Thank you.

🐛 | Bootstrap PointNet | Breadcrumb needs to be aligned.

🇵🇭Philippines paraderojether

Hi alok_singh,

I reviewed MR!7, applied it against Bootstrap PointNet and it did not apply successfully, I got the error below:

error: patch failed: css/bootstrap_pointnet_style.css:3
error: css/bootstrap_pointnet_style.css: patch does not apply

I did not change the issue status so other contributors can also review it.
Thank you.

🐛 | Bootstrap PointNet | Breadcrumb needs to be aligned.

🇵🇭Philippines paraderojether

I will review this.

📌 | helpfulness | Coding Standard Issues

🐛 | Holy - atomic theme | In Advance search page, Advance button is not in full width, Needs to be styled.

🇵🇭Philippines paraderojether

I reviewed MR!4, applied it against the helpfulness 2.x-dev, and confirmed it fixes the issues reported by phpcs.

helpfulness git:(2.x) curl https://git.drupalcode.org/project/helpfulness/-/merge_requests/4.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  2611    0  2611    0     0    775      0 --:--:--  0:00:03 --:--:--   778
patching file 'src/Form/HelpfulnessBlockForm.php'
patching file 'src/Form/HelpfulnessConfirmDeleteFeedbackForm.php'
patching file 'src/Form/HelpfulnessReportForm.php'
patching file 'src/Plugin/Block/HelpfulnessBlock.php'
➜  helpfulness git:(2.x) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig helpfulness
➜  contrib git:(main) ✗

Thank you.

🇵🇭Philippines paraderojether

I reviewed MR!9, applied it against Holy - atomic theme 3.0.0-alpha1, and confirmed the "Advanced search" button is now fixed.

I added screenshots for reference.
Thank you.

🐛 | Holy - atomic theme | Remove extra space from, Add new comment section.

🇵🇭Philippines paraderojether

I reviewed MR!10, applied it against Holy - atomic theme 3.0.x-dev, and confirmed the extra spacing on the Add new comment section is removed.

I added screenshots for reference.
Thank you.

📌 | AI Translation Management (TMGMT) | Add gitlab CI and fix issues

🇵🇭Philippines paraderojether

scott_euser → credited paraderojether → .

🐛 | Dark Awesome | Need to be implement the close icon Button on search page.

🇵🇭Philippines paraderojether

🐛 | Disci | Table Css Distorted.

🇵🇭Philippines paraderojether

I reviewed patch #2 and confirmed it is applied successfully and it fixes the rendering issue of the table.

I added screenshots for reference.
Thank you.

🐛 | Disci | Table Css Distorted.

🇵🇭Philippines paraderojether

🐛 | Dark Awesome | Need to be implement the close icon Button on search page.

🇵🇭Philippines paraderojether

I reviewed patch #10 and confirmed it is applied successfully the close button is added when the search is open. I tested the close button on desktop and touch devices and no issues was found while testing.

I added screenshots for reference.
Thank you.

🐛 | Dark Awesome | Need to be implement the close icon Button on search page.

🇵🇭Philippines paraderojether

I will review this.

📌 | Facets Pretty Paths | Add configuration link in .info.yml

🇵🇭Philippines paraderojether

I reviewed MR!18, applied it against Facets Pretty Paths 2.0.x-dev, and confirmed the configure link is added on the module listing page.

I added screenshots for reference.
Thank you.

📌 | Pretty Checkbox Radio | Coding Standard issues

🇵🇭Philippines paraderojether

Hi,

I reviewed MR!5, applied it against PCR 2.0.x-dev and confirmed it fixes the issues reported by phpcs.

 contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig pcr

FILE: /Users/jetherparadero/Q4/q4contributions/web/modules/contrib/pcr/pcr.module
------------------------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AND 1 WARNING AFFECTING 3 LINES
------------------------------------------------------------------------------------------------------------------------------------
  1 | ERROR   | [x] Missing file doc comment
 11 | ERROR   | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Routing\RouteMatchInterface.
 44 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 3 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/jetherparadero/Q4/q4contributions/web/modules/contrib/pcr/src/Plugin/better_exposed_filters/filter/PrettyCheckboxesRadios.php
-------------------------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 6 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\better_exposed_filters\Plugin\better_exposed_filters\filter\RadioButtons.
-------------------------------------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------------------------------------------------------------------------------------

Time: 338ms; Memory: 10MB

➜  contrib git:(main) ✗ cd pcr
➜  pcr git:(2.0.x) curl https://git.drupalcode.org/project/pcr/-/merge_requests/5.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  1436    0  1436    0     0   3388      0 --:--:-- --:--:-- --:--:--  3443
patching file pcr.module
patching file 'src/Plugin/better_exposed_filters/filter/PrettyCheckboxesRadios.php'
➜  pcr git:(2.0.x) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig pcr
➜  contrib git:(main) ✗

It is also passed on the on the Gitlab CI https://git.drupalcode.org/issue/pcr-3474014/-/jobs/2734627.

Thank you.

📌 | Year Only | Coding standard issues

🇵🇭Philippines paraderojether

Hi alyaj2a,

I reviewed MR!3, applied against Year Only 9.x-dev and there are still error show below:

➜  contrib ✗ cd yearonly
➜  yearonly git:(9.x) curl https://git.drupalcode.org/project/yearonly/-/merge_requests/13.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  2441    0  2441    0     0   5693      0 --:--:-- --:--:-- --:--:--  5784
patching file 'src/Feeds/Target/YearOnly.php'
patching file 'src/Plugin/Field/FieldWidget/YearOnlyDefaultWidget.php'
➜  yearonly git:(9.x) ✗ cd ..
➜  contrib ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig yearonly

FILE: /Users/jetherparadero/Contributions/recommended-project/web/modules/contrib/yearonly/src/Plugin/Field/FieldWidget/YearOnlyDefaultWidget.php
-------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------------------------------------------
 7 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Field\WidgetBase.
-------------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/jetherparadero/Contributions/recommended-project/web/modules/contrib/yearonly/src/Plugin/Field/FieldType/YearOnlyItem.php
-------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------------------------------------
 6 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Field\FieldStorageDefinitionInterface.
-------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------------------------------------------------------

Please check and advise, the issue exists on 9.x-dev.
Thank you.

📌 | Project Browser | Cursor pointer for results tabs

🐛 | Display Suite | Errors and warnings when creating new taxonomy and managing display in Display Suite module when config is empty

🇵🇭Philippines paraderojether

I reviewed MR!570, applied it against Project Browser 2.0.x-dev, and confirmed the cursor is changed to a pointer when hovering over the results tab.

I added a recorded video for reference.
Thank you.

🇵🇭Philippines paraderojether

Hi mohammad-fayoumi,

I was not able to reproduce the issue, my Drupal version is 10.3.2 with PHP version 8.3.6, can you please specify in what Drupal core version the issue occurred?

Thank you.

📌 | View Password | Coding Standard Issues

🇵🇭Philippines paraderojether

Hi dharmeshbarot89,

I reviewed MR!22, and there is still 1 warning remaining shown below:

➜  view_password git:(6.0.x) curl https://git.drupalcode.org/project/view_password/-/merge_requests/22.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  7774    0  7774    0     0   4335      0 --:--:--  0:00:01 --:--:--  4350
patching file CHANGELOG.txt
patching file README.md
patching file 'config/install/view_password.settings.yml'
patching file 'config/schema/view_password.schema.yml'
patching file 'src/Form/PasswordSettingsForm.php'
patching file view_password.libraries.yml
patching file view_password.module
➜  view_password git:(6.0.x) ✗ cd ..
➜  contrib git:(1.0.0) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig view_password

FILE: /Users/jetherparadero/Contributions/recommended-project/web/modules/contrib/view_password/src/Form/PasswordSettingsForm.php
---------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------------------------------------
 45 | WARNING | Unused variable $trailing_slash_message.
---------------------------------------------------------------------------------------------------------------------------------

Time: 116ms; Memory: 10MB

Thank you.

📌 | Single User Role | Fix the issues reported by PHP_CodeSniffer

🇵🇭Philippines paraderojether

Hi,

I reviewed MR!7, applied against Single User Role 8.x-1.x-dev and confirmed phpcs issues are now resolved.

PHP_CodeSniffer

➜  single_user_role git:(8.x-1.0-beta2) curl https://git.drupalcode.org/project/single_user_role/-/merge_requests/7.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  4773    0  4773    0     0   2846      0 --:--:--  0:00:01 --:--:--  2859
patching file .gitlab-ci.yml
patching file single_user_role.module
patching file 'src/Form/SingleUserRoleConfigForm.php'
➜  single_user_role git:(8.x-1.0-beta2) ✗ cd ..
➜  contrib git:(1.0.0) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig single_user_role
➜  contrib git:(1.0.0) ✗

It also passed on the Gitlab CI: https://git.drupalcode.org/issue/single_user_role-3452991/-/jobs/2681621

Thank you.

🐛 | Micro Site | Error opening the media library when adding a new micro site

🇵🇭Philippines paraderojether

I reviewed MR!8, applied against Micro Site 8.x-1.x-dev, and confirmed the error when opening the media library when adding a new micro site is fixed.

I added screenshots and a video for reference.
Thank you.

📌 | Views Load More | Coding Standard issues

🇵🇭Philippines paraderojether

Hi alyaj2a,

I reviewed MR!11, applied it against it Views Load More 2.x-dev, and there is still 1 error remaining after applying the MR.

views_load_more git:(2.x) curl https://git.drupalcode.org/project/views_load_more/-/merge_requests/11.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 20900    0 20900    0     0  21990      0 --:--:-- --:--:-- --:--:-- 22116
patching file README.md
patching file 'src/Ajax/VLMAppendCommand.php'
patching file 'src/EventSubscriber/VLMEventSubscriber.php'
patching file 'src/Plugin/views/pager/LoadMore.php'
patching file views_load_more.module
➜  views_load_more git:(2.x) ✗ cd ..
➜  contrib phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig views_load_more

FILE: /Users/jetherparadero/Contributions/recommended-project/web/modules/contrib/views_load_more/src/EventSubscriber/VLMEventSubscriber.php
--------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------------------------------------------------------------------
 6 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\views\Ajax\ViewAjaxResponse.
--------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------------------------------------------------------------------

Time: 143ms; Memory: 10MB

Please check.
Thank you.

🐛 | Lunr search | Add configure route to .info.yml

🇵🇭Philippines paraderojether

Hi lhridley,

I reviewed MR!3, applied it against Lunr Search 8.x-1.x-dev, and confirmed the configure link is added on the module listing page successfully.

I added screenshots for reference.
Thank you.

🐛 | Layout disable | Error on disabling the "navigation_layout"

🇵🇭Philippines paraderojether

Hi f0ns,

I reviewed patch #5, applied it against Layout disable 2.0.3, and confirmed the "navigation_layout" can't be disabled anymore.

I added screenshots for reference.
Thank you.

📌 | Page Templates | Access denied warning on "Delete" operation.

🇵🇭Philippines paraderojether

Hi v.dovhaliuk,

I've tried to reproduce the issue but am unable to reproduce it.
Please provide the detailed Steps to Reproduce the issue.

Thank you.

🐛 | Bootstrap DateTime Picker | Cannot select Sunday as a date

🇵🇭Philippines paraderojether

I reviewed MR!6, applied it against Bootstrap DateTime Picker 4.0.0-beta3, and confirmed I can select Sunday as a date now.

I added a before and after screen recording for reference.
Thank you.

🐛 | revolt | On mobile view, Advance search input box cut from right side.

🇵🇭Philippines paraderojether

Hi @alok_singh,

I tested MR!18 and confirmed it fixes the advanced search input fields overlapping issue on mobile devices.

I added screenshots for reference.
Thank you.

📌 | Media Entity: Unsplash | Fix the errors/warnings reported by PHP_CodeSniffer

🇵🇭Philippines paraderojether

I created an MR to fix the issues reported by phpcs, please review.

Thank you.

📌 | Media Entity: Unsplash | Fix the errors/warnings reported by PHP_CodeSniffer

🇵🇭Philippines paraderojether

paraderojether → created an issue.

📌 | Environment Indicator | Fix the errors/warnings reported by PHP_CodeSniffer

🇵🇭Philippines paraderojether

Fix the issues reported by phpcs, please review.

Thank you.

📌 | Environment Indicator | Fix the errors/warnings reported by PHP_CodeSniffer

🇵🇭Philippines paraderojether

paraderojether → created an issue.

📌 | Layout Builder Paragraphs | Fix the issues reported by phpcs

🇵🇭Philippines paraderojether

📌 | Layout Builder Paragraphs | Fix the issues reported by phpcs

🇵🇭Philippines paraderojether

I fixed the issues reported by phpcs, please review.

Thank you.

📌 | Layout Builder Paragraphs | Fix the issues reported by phpcs

🇵🇭Philippines paraderojether

paraderojether → created an issue.

🐛 | Media Thumbnails | Fix the warning reported by PHP_CodeSniffer

🇵🇭Philippines paraderojether

I fixed the remaining issue, please review.

Thank you.

🐛 | Media Thumbnails | Fix the warning reported by PHP_CodeSniffer

🇵🇭Philippines paraderojether

I reviewed MR!4, applied against Media Thumbnails 8.x-1.x-dev and there is one phpcs error shown below:

✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig media_thumbnails

FILE: /Users/jetherparadero/Contributions/recommended-project/web/modules/contrib/media_thumbnails/src/Plugin/MediaThumbnailManager.php
---------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------------------------------------------
 8 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Cache\CacheBackendInterface.
---------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------------------------------------------------

Time: 350ms; Memory: 10MB

Thank you.

🐛 | Media Thumbnails | Fix the warning reported by PHP_CodeSniffer

🇵🇭Philippines paraderojether

📌 | Encrypted sql:dump | Fix the issues reported by phpcs

🇵🇭Philippines paraderojether

I created MR to fix the remaining issues, please review.

Thank you.

📌 | Encrypted sql:dump | Fix the issues reported by phpcs

🇵🇭Philippines paraderojether

✨ | Purge | Include link to purge config page in admin menu

🇵🇭Philippines paraderojether

I reviewed MR!33, applied against the Purge 8.x-3.x-dev and confirmed the configure link is added on the Purge UI module.

I added screenshots for reference.
Thank you.

📌 | Menu Admin per Menu | Fix issues reported by phpcs

🇵🇭Philippines paraderojether

I reviewed MR!6, applied it against Menu Admin per Menu 8.x-1.x-dev and there are still remaining phpcs issue shown below:

➜  menu_admin_per_menu git:(98d34cb) curl https://git.drupalcode.org/project/menu_admin_per_menu/-/merge_requests/16.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  4928    0  4928    0     0   9338      0 --:--:-- --:--:-- --:--:--  9476
patching file README.md
patching file 'src/Routing/RouteSubscriber.php'
patching file 'tests/src/Kernel/MenuAdminPerMenuAccessTest.php'
patching file 'tests/src/Kernel/MenuAdminPerMenuPermissionsTest.php'
➜  menu_admin_per_menu git:(98d34cb) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig menu_admin_per_menu

FILE: /Users/jetherparadero/q3do/q3issues/web/modules/contrib/menu_admin_per_menu/src/Access/MenuAdminPerMenuAccess.php
-----------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
-----------------------------------------------------------------------------------------------------------------------
 26 | WARNING | Menu::loadMultiple calls should be avoided in classes, use dependency injection instead
 35 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
-----------------------------------------------------------------------------------------------------------------------


FILE: /Users/jetherparadero/q3do/q3issues/web/modules/contrib/menu_admin_per_menu/src/MenuAdminPerMenuPermissions.php
----------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------------------------------------------------
 6 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\StringTranslation\StringTranslationTrait.
----------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/jetherparadero/q3do/q3issues/web/modules/contrib/menu_admin_per_menu/src/MenuAdminPerMenuAccessInterface.php
----------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------------------------------
 6 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Menu\MenuLinkInterface.
----------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------------------------

Time: 548ms; Memory: 16MB

Kindly check and advise, please.
Thank you.

📌 | Pager | Fix the errors/warnings reported by Php_Codesniffer

🇵🇭Philippines paraderojether

I reviewed MR!2, applied it against Pager 3.0.0, and confirmed it fixes the issue reported by phcs.

➜  pager git:(main) ✗ curl https://git.drupalcode.org/project/pager/-/merge_requests/2.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  3554    0  3554    0     0  11521      0 --:--:-- --:--:-- --:--:-- 11729
patching file 'src/PagerStorage.php'
patching file 'src/Plugin/Block/PagerBlock.php'
➜  pager git:(main) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig pager
➜  contrib git:(main) ✗

Thank you.

📌 | Recently Read | Fix the issues reported by phpcs

🇵🇭Philippines paraderojether

I reviewed MR!6, applied against Recently Read 8.x-1.x-dev, and confirmed it fixes the issues reported by phpcs.

➜  recently_read git:(2eb445a) curl https://git.drupalcode.org/project/recently_read/-/merge_requests/6.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 37599    0 37599    0     0  87599      0 --:--:-- --:--:-- --:--:-- 89521
patching file README.txt
patching file recently_read.info.yml
patching file recently_read.install
patching file recently_read.module
patching file recently_read.views.inc
patching file 'src/Entity/RecentlyRead.php'
patching file 'src/Entity/RecentlyReadInterface.php'
patching file 'src/Entity/RecentlyReadType.php'
patching file 'src/Entity/RecentlyReadTypeInterface.php'
patching file 'src/Form/RecentlyReadConfigForm.php'
patching file 'src/Form/RecentlyReadTypeForm.php'
patching file 'src/Plugin/views/relationship/RecentlyReadRelationship.php'
patching file 'src/RecentlyReadHtmlRouteProvider.php'
patching file 'src/RecentlyReadService.php'
patching file 'src/RecentlyReadServiceInterface.php'
patching file 'src/RecentlyReadStorageSchema.php'
patching file 'src/RecentlyReadTypeListBuilder.php'
patching file 'tests/src/Functional/RecentlyReadBlockTest.php'
➜  recently_read git:(2eb445a) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig recently_read
➜  contrib git:(main) ✗

Thank you.

📌 | System Tags | Coding standards

🇵🇭Philippines paraderojether

I reviewed MR!5, and applied it against Systems Tags 2.0.1 and it is not applied cleanly.

➜  system_tags git:(2.0.1) curl https://git.drupalcode.org/project/system_tags/-/merge_requests/5.diff | git apply -v
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 19046    0 19046    0     0  59017      0 --:--:-- --:--:-- --:--:-- 61637
Checking patch README.md...
Checking patch modules/theme/system_tags_theme.module...
error: while searching for:
 * Implements hook_theme_suggestions_HOOK_alter().
 */
function system_tags_theme_theme_suggestions_node_alter(array &$suggestions, array $variables) {
  $suggestions = array_merge($suggestions, _system_tag_node_suggestions('node__system_tag__', $variables['elements']['#node']));
}

/**

error: patch failed: modules/theme/system_tags_theme.module:27
error: modules/theme/system_tags_theme.module: patch does not apply
Checking patch src/Annotation/SystemTagFinder.php...
Checking patch src/Config/SystemPageConfigOverrider.php...
Checking patch src/Config/SystemTagDefinitions.php...
Checking patch src/Entity/SystemTag.php...
Checking patch src/Entity/SystemTagInterface.php...
Checking patch src/Form/SystemTagDeleteForm.php...
Checking patch src/Form/SystemTagForm.php...
Checking patch src/Plugin/Condition/SystemTags.php...
Checking patch src/Plugin/SystemTagFinder/SystemTagBlockFinder.php...
Checking patch src/Plugin/SystemTagFinder/SystemTagNodeFinder.php...
Checking patch src/SystemTagAccessControlHandler.php...
Checking patch src/SystemTagFinder/SystemTagFinderInterface.php...
Checking patch src/SystemTagFinder/SystemTagFinderManager.php...
Checking patch src/SystemTagFinder/SystemTagFinderManagerInterface.php...
Checking patch src/SystemTagFinder/SystemTagFinderPluginBase.php...
error: while searching for:
      $storage = $this->entityTypeManager->getStorage($entityTypeId);

      $query = $storage->getQuery()
        ->sort('changed', 'DESC', $langcode);
      $orCondition = $query->orConditionGroup();
      foreach ($fields as $field) {

error: patch failed: src/SystemTagFinder/SystemTagFinderPluginBase.php:95
error: src/SystemTagFinder/SystemTagFinderPluginBase.php: patch does not apply
Checking patch src/SystemTagHelper.php...
Checking patch src/SystemTagHelperInterface.php...
Checking patch src/SystemTagHtmlRouteProvider.php...
Checking patch src/SystemTagListBuilder.php...
Checking patch src/TwigExtension/SystemTagsTwigExtension.php...
error: while searching for:
namespace Drupal\system_tags\TwigExtension;

use Drupal\system_tags\SystemTagFinder\SystemTagFinderManagerInterface;

/**
 * Class SystemTagsTwigExtension.
 */
class SystemTagsTwigExtension extends \Twig_Extension {

  /**
   * The system tag finder manager.

error: patch failed: src/TwigExtension/SystemTagsTwigExtension.php:3
error: src/TwigExtension/SystemTagsTwigExtension.php: patch does not apply
Checking patch system_tags.module...
Checking patch system_tags.tokens.inc...
Checking patch tests/src/Functional/DefaultSystemTagsTest.php...

Kindly check and advise, please.
Thank you.

📌 | Youtube Integration | Fix the issues reported by phpcs

🇵🇭Philippines paraderojether

I reviewed MR!1, applied it against Youtube Integration 1.0.1 and confirmed it fixes the issues reported by phpcs.

➜  youtubeintegrate git:(main) ✗ curl https://git.drupalcode.org/project/youtubeintegrate/-/merge_requests/1.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 47286    0 47286    0     0  85015      0 --:--:-- --:--:-- --:--:-- 86131
patching file 'css/youtube.css'
patching file 'src/Controller/YouTubeIntegrationController.php'
patching file 'src/Entity/YouTubePlaylistEntity.php'
patching file 'src/YouTubeAPI.php'
patching file 'templates/youtube-integration-playlist-item.html.twig'
patching file 'templates/youtube-integration-playlists.html.twig'
patching file youtube_integration.links.menu.yml
patching file youtube_integration.module
patching file youtube_integration.routing.yml
➜  youtubeintegrate git:(main) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig youtubeintegrate
➜  contrib git:(main) ✗

Moving this to RTBC.
Thank you.

📌 | CAS | Make cas configuration translatable

🇵🇭Philippines paraderojether

Hi LOBsTerr,

I reviewed MR!25, applied it against CAS 2.1.0, and confirmed Cas setting is now translatable.

I added screenshots for reference.
Thank you.

📌 | Unique entity title | Fix the issues reported by phpcs

🇵🇭Philippines paraderojether

I fix the remaining issues, please review.

Thank you.

📌 | Unique entity title | Fix the issues reported by phpcs

🇵🇭Philippines paraderojether

paraderojether → made their first commit to this issue’s fork.

📌 | Content Dependency Tracker | Fix the issues reported by phpcs

🇵🇭Philippines paraderojether

I fix the remaining issue, please review.

Thank you.

📌 | Content Dependency Tracker | Fix the issues reported by phpcs

🇵🇭Philippines paraderojether

paraderojether → made their first commit to this issue’s fork.

🐛 | Views Default User Taxonomy Argument | Drupal Coding Standards Issues | phpcs

🇵🇭Philippines paraderojether

I fix the remaining issues, please review.

Thank you

🐛 | Views Default User Taxonomy Argument | Drupal Coding Standards Issues | phpcs

🇵🇭Philippines paraderojether

paraderojether → made their first commit to this issue’s fork.

📌 | Fancy Login | Coding standard issues

🇵🇭Philippines paraderojether

paraderojether → made their first commit to this issue’s fork.

📌 | United Nations Maps & Geoservices | Fix the issue reported by phpcs

🇵🇭Philippines paraderojether

I reviewed MR!7, applied it against United Nations Maps & Geoservices 1.2.0, and confirmed it fixes the issues reported by phpcs.

➜  unep_maps git:(main) ✗ curl https://git.drupalcode.org/project/unep_maps/-/merge_requests/7.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 11473    0 11473    0     0  27730      0 --:--:-- --:--:-- --:--:-- 28120
patching file README.md
patching file 'css/edw_mapbox.css'
patching file edw_maps.api.php
patching file edw_maps.module
patching file 'src/Controller/EdwMapsController.php'
patching file 'src/EdwMapsElementsTrait.php'
patching file 'src/Plugin/views/style/MapboxMapStyle.php'
patching file 'src/Services/EdwMapsDataService.php'
➜  unep_maps git:(main) ✗ cd ..
➜  contrib git:(main) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,info,txt,md,yml,twig unep_maps
➜  contrib git:(main) ✗

Thank you.

✨ | Event Scheduler | Add Configuration settings in info file

🇵🇭Philippines paraderojether

I reviewed MR!8, applied it against Event Scheduler 2.3.0-rc3 and it confirmed the configuration was added successfully.

I added screenshots for reference.
Thank you.

✨ | LB Plus | Add configure link to info.yml

🇵🇭Philippines paraderojether

I reviewed MR!50, applied it against Layout Builder + and it confirmed the configuration was added successfully.

I added screenshots for reference.
Thank you.

📌 | OPENAI Search | config missing in info.yml file

🇵🇭Philippines paraderojether

I reviewed patch #2, applied it against OPENAI Search 2.0.1 and it confirmed the configuration was added successfully.

I added screenshots for reference.
Thank you.

📌 | Watchdog Mailer | Add configure link to the .info.yml

🇵🇭Philippines paraderojether

I reviewed MR!19, applied against Watchdog Mailer and it confirmed the configuration is added successfully.

I added screenshots for reference.
Thank you.

📌 | Shy One-Time | Fix coding standards issues

🇵🇭Philippines paraderojether

I updated the MR to fix the remaining issue, please review.

Thank you.

📌 | Shy One-Time | Fix coding standards issues