Skip to content

fix(ExceptionT): Only check for translation within the exception#106

Merged
klausi merged 1 commit into
pfrenssen:8.x-3.xfrom
arkener:3123061
May 16, 2020
Merged

fix(ExceptionT): Only check for translation within the exception#106
klausi merged 1 commit into
pfrenssen:8.x-3.xfrom
arkener:3123061

Conversation

@arkener
Copy link
Copy Markdown
Collaborator

@arkener arkener commented May 15, 2020

Copy link
Copy Markdown
Owner

@pfrenssen pfrenssen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the quick fix!

@pfrenssen
Copy link
Copy Markdown
Owner

pfrenssen commented May 15, 2020

Here are results for a "before" and "after" check in my project which was affected by 3 false positives. The result is perfect now!

Before

$ ./vendor/bin/phpcs
........................................................S...  60 / 706 (8%)
............................................................ 120 / 706 (17%)
............................................................ 180 / 706 (25%)
............S.........................................W..... 240 / 706 (34%)
........................W......W............................ 300 / 706 (42%)
............................................................ 360 / 706 (51%)
............................................................ 420 / 706 (59%)
............................................................ 480 / 706 (68%)
............................................................ 540 / 706 (76%)
............................................................ 600 / 706 (85%)
..S........................................S................ 660 / 706 (93%)
..............................................               706 / 706 (100%)



FILE: /home/pieter/v/joinup-dev/web/modules/custom/joinup_federation/src/JoinupFederationStepPluginBase.php
-----------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------------------
 93 | WARNING | Exceptions should not be translated
-----------------------------------------------------------------------------------------------------------


FILE: /home/pieter/v/joinup-dev/web/modules/custom/joinup_federation/src/Plugin/pipeline/Step/UserSelectionFilter.php
---------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------------------------
 127 | WARNING | Exceptions should not be translated
---------------------------------------------------------------------------------------------------------------------


FILE: /home/pieter/v/joinup-dev/web/modules/custom/joinup_federation/src/Plugin/pipeline/Step/ManualUpload.php
--------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------------------------------------
 76 | WARNING | Exceptions should not be translated
--------------------------------------------------------------------------------------------------------------

Time: 8.17 secs; Memory: 54MB

After

$ ./vendor/bin/phpcs
........................................................S...  60 / 706 (8%)
............................................................ 120 / 706 (17%)
............................................................ 180 / 706 (25%)
............S............................................... 240 / 706 (34%)
............................................................ 300 / 706 (42%)
............................................................ 360 / 706 (51%)
............................................................ 420 / 706 (59%)
............................................................ 480 / 706 (68%)
............................................................ 540 / 706 (76%)
............................................................ 600 / 706 (85%)
..S........................................S................ 660 / 706 (93%)
..............................................               706 / 706 (100%)


Time: 8.15 secs; Memory: 54MB

👍

@klausi klausi merged commit d187981 into pfrenssen:8.x-3.x May 16, 2020
@arkener arkener deleted the 3123061 branch November 16, 2021 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants