- Issue created by @paraderojether
- First commit to issue fork.
- last update
over 1 year ago 18 pass - @_pratik_ opened merge request.
- last update
over 1 year ago 18 pass - Status changed to Needs review
over 1 year ago 6:26am 10 May 2023 - Issue was unassigned.
- last update
over 1 year ago 18 pass - 🇵ðŸ‡Philippines paraderojether
Created a patch to fix the issue reported by phpcs.
Please review.
Thank You. - Status changed to RTBC
over 1 year ago 2:04pm 3 July 2023 - 🇵ðŸ‡Philippines roberttabigue
Hi,
I reviewed the changes and confirmed the Patch was applied cleanly.
Checking patch pelcro_auth.module... Checking patch src/Controller/UserController.php... Checking patch src/EventSubscriber/AccessDeniedSubscriber.php... Checking patch src/Util.php... Checking patch tests/src/Functional/PelcroRedirectTest.php... Checking patch tests/src/Unit/UtilTest.php... Applied patch pelcro_auth.module cleanly. Applied patch src/Controller/UserController.php cleanly. Applied patch src/EventSubscriber/AccessDeniedSubscriber.php cleanly. Applied patch src/Util.php cleanly. Applied patch tests/src/Functional/PelcroRedirectTest.php cleanly. Applied patch tests/src/Unit/UtilTest.php cleanly.
And all PHPCS errors have been fixed.
I ran this command:
phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml
Applied to Pelcro Authorization module with 1.x-dev version and with the Drupal core version of 9.5.x.
Attaching screenshots and moving this to RTBC.
Thank you!