- Issue created by @dineshkumarbollu
- Issue was unassigned.
- Status changed to Needs review
about 1 year ago 4:43am 3 May 2023 - Status changed to RTBC
about 1 year ago 5:04am 3 May 2023 - ๐ต๐ญPhilippines paraderojether
Hi dineshkumarbollu
I reviewed patch #2, and verified it resolved the errors reported by phpcs. Drupal core 9.5.8, Nice Login 2.10.
I added screenshots for reference.
Thank You. - Status changed to Needs work
about 1 year ago 2:44pm 20 May 2023 - ๐ฎ๐นItaly apaderno Brescia, ๐ฎ๐น
- $link->getUrl()->setOption('attributes', ['class' => ['nice-login', 'nice-login-reset-password']]); + $link->getUrl()->setOption( + 'attributes', + [ + 'class' => ['nice-login', 'nice-login-reset-password'], + ] + );
- $link->getUrl()->setOption('attributes', ['class' => ['nice-login', 'nice-login-create-account']]); + $link->getUrl()->setOption( + 'attributes', + [ + 'class' => ['nice-login', 'nice-login-create-account'], + ] + );
- $link->getUrl()->setOption('attributes', ['class' => ['nice-login', 'nice-login-login']]); + $link->getUrl()->setOption( + 'attributes', + [ + 'class' => ['nice-login', 'nice-login-login'], + ] + );
- $link->getUrl()->setOption('attributes', ['class' => ['nice-login', 'nice-login-login']]); + $link->getUrl()->setOption( + 'attributes', + [ + 'class' => ['nice-login', 'nice-login-login'], + ] + );
Code lines are not required to be shorter than 81 characters. Line length and wrapping โ , part of the Drupal coding standards, says:
- Lines containing longer function names, function/class definitions, variable declarations, etc are allowed to exceed 80 characters.
- Control structure conditions may exceed 80 characters, if they are simple to read and understand.
(Emphasis is mine.)
- ๐ฎ๐ณIndia kbk1992 Hyderabad
bharath-kondeti โ made their first commit to this issueโs fork.
- @bharath-kondeti opened merge request.
- Status changed to Needs review
about 1 year ago 3:05pm 20 May 2023 - ๐ฎ๐ณIndia kbk1992 Hyderabad
Made one change on top the patch and raised a MR. Please review
- Status changed to Needs work
about 1 year ago 4:20pm 20 May 2023 - ๐ฎ๐นItaly apaderno Brescia, ๐ฎ๐น
The MR just changes the nice_login.libraries.yml file, while the patch changes more files.
- Status changed to Needs review
about 1 year ago 4:29pm 20 May 2023 - ๐ฎ๐ณIndia kbk1992 Hyderabad
@aparderno, As per the comments #5 "Lines containing longer function names, function/class definitions, variable declarations, etc are allowed to exceed 80 characters.
Control structure conditions may exceed 80 characters, if they are simple to read and understand."The code in nice_login.module is reverted to its existing state. Hence, we have only one file in the MR raised.