undefined - Contrib.social

🇮🇳India rushiraval

I am changing the issue priority as per issue priorities → .

📌 | stub project from issue ref | [1.0.x] Social Media Platforms

🇮🇳India rushiraval

I am changing the issue priority as per issue priorities → .

📌 | stub project from issue ref | [1.0.x] Register With OTP

🇮🇳India rushiraval

I am changing the issue priority as per issue priorities → .

📌 | stub project from issue ref | [1.0.x] Commerce Order Data Formatter

🇮🇳India rushiraval

I am changing the issue priority as per issue priorities → .

📌 | stub project from issue ref | [2.x] ImageLightbox

🇮🇳India rushiraval

I am changing the issue priority as per issue priorities → .

📌 | stub project from issue ref | [1.0.x] Image Scale and Fill Background Effect

🇮🇳India rushiraval

Fix phpcs issue

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

FILE: image_scale_fill/README.md
------------------------------------------------------------------------
FOUND 1 ERROR AND 13 WARNINGS AFFECTING 13 LINES
------------------------------------------------------------------------
3 | WARNING | [ ] Line exceeds 80 characters; contains 420 characters
7 | WARNING | [ ] Line exceeds 80 characters; contains 89 characters
12 | WARNING | [ ] Line exceeds 80 characters; contains 106 characters
18 | WARNING | [ ] Line exceeds 80 characters; contains 95 characters
19 | WARNING | [ ] Line exceeds 80 characters; contains 93 characters
20 | WARNING | [ ] Line exceeds 80 characters; contains 137 characters
21 | WARNING | [ ] Line exceeds 80 characters; contains 161 characters
22 | WARNING | [ ] Line exceeds 80 characters; contains 141 characters
26 | WARNING | [ ] Line exceeds 80 characters; contains 88 characters
28 | WARNING | [ ] Line exceeds 80 characters; contains 81 characters
37 | WARNING | [ ] Line exceeds 80 characters; contains 138 characters
41 | WARNING | [ ] Line exceeds 80 characters; contains 106 characters
51 | WARNING | [ ] Line exceeds 80 characters; contains 94 characters
51 | ERROR | [x] Expected 1 newline at end of file; 2 found
------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------

FILE: image_scale_fill/src/Plugin/ImageEffect/ScaleFillBackgroundEffect.php
-------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 3 LINES
-------------------------------------------------------------------------------------------
59 | WARNING | Line exceeds 80 characters; contains 81 characters
142 | WARNING | Line exceeds 80 characters; contains 95 characters
310 | WARNING | Line exceeds 80 characters; contains 88 characters
-------------------------------------------------------------------------------------------

FILE: image_scale_fill/image_scale_fill.info.yml
----------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------
7 | ERROR | [x] Expected 1 newline at end of file; 2 found
----------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------

Time: 90ms; Memory: 14MB

📌 | stub project from issue ref | [1.0.x] Calculation Fields

🇮🇳India rushiraval

Fix phpcs issues

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

FILE: calculation_fields/README.md
----------------------------------------------------------------------
FOUND 0 ERRORS AND 10 WARNINGS AFFECTING 10 LINES
----------------------------------------------------------------------
4 | WARNING | Line exceeds 80 characters; contains 403 characters
9 | WARNING | Line exceeds 80 characters; contains 141 characters
16 | WARNING | Line exceeds 80 characters; contains 86 characters
21 | WARNING | Line exceeds 80 characters; contains 118 characters
24 | WARNING | Line exceeds 80 characters; contains 512 characters
31 | WARNING | Line exceeds 80 characters; contains 178 characters
50 | WARNING | Line exceeds 80 characters; contains 109 characters
53 | WARNING | Line exceeds 80 characters; contains 199 characters
72 | WARNING | Line exceeds 80 characters; contains 118 characters
74 | WARNING | Line exceeds 80 characters; contains 164 characters
----------------------------------------------------------------------

FILE: calculation_fields/calculation_fields.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
--------------------------------------------------------------------------------

FILE: calculation_fields/modules/calculation_fields_example/calculation_fields_example.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
--------------------------------------------------------------------------------

FILE: calculation_fields/modules/webform_calculation_fields/README.md
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 14 WARNINGS AFFECTING 14 LINES
--------------------------------------------------------------------------------
4 | WARNING | Line exceeds 80 characters; contains 395 characters
9 | WARNING | Line exceeds 80 characters; contains 178 characters
18 | WARNING | Line exceeds 80 characters; contains 111 characters
21 | WARNING | Line exceeds 80 characters; contains 118 characters
24 | WARNING | Line exceeds 80 characters; contains 325 characters
25 | WARNING | Line exceeds 80 characters; contains 81 characters
28 | WARNING | Line exceeds 80 characters; contains 213 characters
31 | WARNING | Line exceeds 80 characters; contains 216 characters
34 | WARNING | Line exceeds 80 characters; contains 177 characters
40 | WARNING | Line exceeds 80 characters; contains 223 characters
43 | WARNING | Line exceeds 80 characters; contains 223 characters
46 | WARNING | Line exceeds 80 characters; contains 134 characters
49 | WARNING | Line exceeds 80 characters; contains 271 characters
51 | WARNING | Line exceeds 80 characters; contains 161 characters
--------------------------------------------------------------------------------

FILE: calculation_fields/modules/webform_calculation_fields/webform_calculation_fields.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
--------------------------------------------------------------------------------

FILE: calculation_fields/modules/calculation_fields_example/webform_calculation_fields_examples.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: 165ms; Memory: 10MB

🌱 | AT Tool 2.0 | Drupal 11

Comment 12 days ago →

🇮🇳India rushiraval

@mattbloomfield We need to update summary and Description of this project page. Because its supports Drupal 10.

So user can view this in search view while searching module.

🐛 | AT Tool 2.0 | Layout Builder sections not rendering

Comment 29 days ago →

🇮🇳India rushiraval

📌 | stub project from issue ref | [1.0.x] Personality Assessment Test/Quiz

🇮🇳India rushiraval

📌 | stub project from issue ref | [1.0.x] Personality Assessment Test/Quiz

🇮🇳India rushiraval

Please refer comment #4

📌 | stub project from issue ref | [1.0.x] Node field values

🇮🇳India rushiraval

Remember to change status, when the project is ready to be reviewed. In this queue, projects are only reviewed when the status is Needs review.

📌 | stub project from issue ref | [1.0.x] Node field values

🇮🇳India rushiraval

Thank you for applying!

The important notes are the following.

If you have not done it yet, you should run phpcs --standard=Drupal,>DrupalPractice on the project, which alone fixes most of what reviewers would report.
For the time this application is open, only your commits are allowed.
The purpose of this application is giving you a new drupal.org role that allows you to opt projects into security advisory coverage, either projects you already created, or projects you will create. The project status will not be changed by this application; no other user will be able to opt projects into security advisory policy.
We only accept an application per user. If you change your mind about the project to use for this application, or it is necessary to use a different project for the application, please update the issue summary with the link to the correct project and the issue title with the branch to review and the project name.

To the reviewers

Please read How to review security advisory coverage applications → , Application workflow → , What to cover in an application review → , and Tools to use for reviews → .

The important notes are the following.

It is preferable to wait for a Code Review Administrator before commenting on newly created applications, even to leave a comment similar to the following one. Code Review Administrators will do some preliminary checks that are necessary before any change on the project files is suggested.
It is also preferable to wait before using a CLI tool → to report what needs to be changed, especially because the comment left from Code Review Administrators suggests to use PHP_CodeSniffer. Before that, manual reviews should be done.
Reviewers should not copy-paste the output of a CLI tool. They should use a CLI tool only once per application. When they do that, they should later verify the code has been correctly changed; this means, for example, that adding a documentation comment that is not correct just to avoid to get a warning/error is not a correct change that should be reported in a further comment.
It may be best to have the applicant fix things before further review.

For new reviewers, I would also suggest to first read In which way the issue queue for coverage applications is different from other project queues → .

📌 | stub project from issue ref | [1.0.x] Ckeditor(5) Emoji

🇮🇳India rushiraval

I think you can continue with this application. You need to change description after you upload new module.
Then follow the comment #2.
When project is ready for for review then change status to "Needs Review"

📌 | stub project from issue ref | [1.0.x] Ckeditor(5) Emoji

🇮🇳India rushiraval

Thank you for applying!

The important notes are the following.

If you have not done it yet, you should run phpcs --standard=Drupal,>DrupalPractice on the project, which alone fixes most of what reviewers would report.
For the time this application is open, only your commits are allowed.
The purpose of this application is giving you a new drupal.org role that allows you to opt projects into security advisory coverage, either projects you already created, or projects you will create. The project status will not be changed by this application; no other user will be able to opt projects into security advisory policy.
We only accept an application per user. If you change your mind about the project to use for this application, or it is necessary to use a different project for the application, please update the issue summary with the link to the correct project and the issue title with the branch to review and the project name.

To the reviewers

Please read How to review security advisory coverage applications → , Application workflow → , What to cover in an application review → , and Tools to use for reviews → .

The important notes are the following.

It is preferable to wait for a Code Review Administrator before commenting on newly created applications, even to leave a comment similar to the following one. Code Review Administrators will do some preliminary checks that are necessary before any change on the project files is suggested.
It is also preferable to wait before using a CLI tool → to report what needs to be changed, especially because the comment left from Code Review Administrators suggests to use PHP_CodeSniffer. Before that, manual reviews should be done.
Reviewers should not copy-paste the output of a CLI tool. They should use a CLI tool only once per application. When they do that, they should later verify the code has been correctly changed; this means, for example, that adding a documentation comment that is not correct just to avoid to get a warning/error is not a correct change that should be reported in a further comment.
It may be best to have the applicant fix things before further review.

For new reviewers, I would also suggest to first read In which way the issue queue for coverage applications is different from other project queues → .

🐛 | AT Tool 2.0 | Layout Builder sections not rendering

🇮🇳India rushiraval

@jandewit6 can you provide step to reproduce issue or share screenshot for issue.

🐛 | AT Tool 2.0 | Fatal error when activate AT Theme Generator

🇮🇳India rushiraval

Since there is no activity on this issue since longtime. So closing it as outdated.

📌 | stub project from issue ref | [1.0.x] Advanced Sanitize

🇮🇳India rushiraval

I am changing the issue priority as per issue priorities → .

📌 | stub project from issue ref | [1.0.x] Error Reporting

🇮🇳India rushiraval

I am changing the issue priority as per issue priorities → .

📌 | stub project from issue ref | [1.0.x] Darkmode JS

🇮🇳India rushiraval

I have reviewed this project findings in comment #3 and #6 are fixed. Rest looks fine to me.

Let’s wait for a Code Review Administrator to take a look and if everything goes fine, you will get the role.

📌 | stub project from issue ref | Request for 'confirmed' user role

🇮🇳India rushiraval

@tmotions
I have already reviewed both post. But there is not any meaningful contribution to community it is only link to contribution. This post you mentioning are committed by some other user. You can contribute community without confirmed role also.

📌 | stub project from issue ref | Request for 'confirmed' user role

🇮🇳India rushiraval

📌 | stub project from issue ref | Request for 'confirmed' user role

🇮🇳India rushiraval

📌 | stub project from issue ref | Request for 'confirmed' user role

🇮🇳India rushiraval

This user has already made one request which is clsoed. All post by user is not contributing to community. User has just copied URL of other user.

🐛 | Human Behavior | Missing readme file

🇮🇳India rushiraval

Please follow Readme template →

📌 | stub project from issue ref | Request for 'confirmed' user role

🇮🇳India rushiraval

Hello, and a warm welcome to the Drupal community!

You can contribute to drupal.org without having the role.

The 'confirmed' role is for users that contribute to this website. In this case, you've not contributed any content except this post, so there is no content to review. Postponing for now, after you have posted some content on Drupal.org.

You can add a comment to this issue to request a new review in order to get 'confirmed' or you will get that automatically.

Please visit the Become a confirmed user → page for information.

Here is a list of resources that will assist you in making helpful contributions:

📌 | Drupal core | Change the enabled alignments checkbox options to titlecase

🇮🇳India rushiraval

I have applied patch #17 is working fine for me. I have attached screenshot for before and after patch.

📌 | stub project from issue ref | [1.0.X] SSO Connector

🇮🇳India rushiraval

Rest looks fine to me.

Let’s wait for a Code Review Administrator to take a look and if everything goes fine, you will get the role.

📌 | stub project from issue ref | [1.0.X] SSO Connector

🇮🇳India rushiraval

Thanks @vishal.kadam. command added in comment.

📌 | stub project from issue ref | [1.0.X] SSO Connector

🇮🇳India rushiraval

Fix the phpcs Error :

FILE: /sso_connector/README.md
------------------------------------------------------------------------
FOUND 1 ERROR AND 13 WARNINGS AFFECTING 13 LINES
------------------------------------------------------------------------
3 | WARNING | [ ] Line exceeds 80 characters; contains 300 characters
22 | WARNING | [ ] Line exceeds 80 characters; contains 103 characters
24 | WARNING | [ ] Line exceeds 80 characters; contains 150 characters
30 | WARNING | [ ] Line exceeds 80 characters; contains 93 characters
34 | WARNING | [ ] Line exceeds 80 characters; contains 86 characters
36 | WARNING | [ ] Line exceeds 80 characters; contains 123 characters
37 | WARNING | [ ] Line exceeds 80 characters; contains 86 characters
38 | WARNING | [ ] Line exceeds 80 characters; contains 201 characters
39 | WARNING | [ ] Line exceeds 80 characters; contains 138 characters
45 | WARNING | [ ] Line exceeds 80 characters; contains 97 characters
57 | WARNING | [ ] Line exceeds 80 characters; contains 81 characters
58 | WARNING | [ ] Line exceeds 80 characters; contains 186 characters
64 | WARNING | [ ] Line exceeds 80 characters; contains 100 characters
64 | ERROR | [x] Expected 1 newline at end of file; 2 found
------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------

FILE: /sso_connector/sso_connector.info.yml
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------
1 | WARNING | Remove "version" from the info file, it will be added by
| | drupal.org packaging automatically
--------------------------------------------------------------------------------

FILE: /sso_connector/src/tests/src/Kernel/SsoTokenTest.php
--------------------------------------------------------------------------------
FOUND 10 ERRORS AND 2 WARNINGS AFFECTING 12 LINES
--------------------------------------------------------------------------------
7 | WARNING | [x] Unused use statement
11 | ERROR | [x] Use statements should be sorted alphabetically. The first
| | wrong one is Drupal\KernelTests\KernelTestBase.
15 | WARNING | [x] Unused use statement
74 | ERROR | [x] Whitespace found at end of line
76 | ERROR | [x] Whitespace found at end of line
93 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
94 | ERROR | [x] Expected 5 space(s) before asterisk; 3 found
95 | ERROR | [x] Expected 5 space(s) before asterisk; 3 found
96 | ERROR | [x] Expected 1 blank line before function; 0 found
98 | ERROR | [x] Namespaced classes/interfaces/traits should be referenced
| | with use statements
139 | ERROR | [x] Expected 1 blank line after function; 2 found
142 | ERROR | [x] The closing brace for the class must have an empty line
| | before it
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 12 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------

Time: 1.17 secs; Memory: 10MB

✨ | Stable | Release for Drupal 11

🇮🇳India rushiraval

✨ | Stable | Release for Drupal 11

🇮🇳India rushiraval

Rushikesh Raval → created an issue.

📌 | stub project from issue ref | [1.0.x] Reading Rating

🇮🇳India rushiraval

I think this project is too small for us and it doesn't contain enough PHP code. Senior Reviewer may review this.

📌 | stub project from issue ref | [10.x-1.x] languagewire_translation_provider

🇮🇳India rushiraval

I am changing the issue priority as per issue priorities → .

📌 | stub project from issue ref | [1.0.x] AI Field

🇮🇳India rushiraval

I am changing the issue priority as per issue priorities → .

📌 | stub project from issue ref | [1.1.x] Address Static Map

🇮🇳India rushiraval

I am changing the issue priority as per issue priorities → .

📌 | stub project from issue ref | [2.0.x] Basic Slider

🇮🇳India rushiraval

I am changing the issue priority as per issue priorities → .

🐛 | Marvelous | Add a space between the email form and the copyright on mobile.

🇮🇳India rushiraval

🐛 | Marvelous | Add a space between the email form and the copyright on mobile.

🇮🇳India rushiraval

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

🐛 | AT Tool 2.0 | AT Theme generator - error: GeneratorForm.php,

🇮🇳India rushiraval

@voloda86

Please following step. It working fine for me

Install

composer require drupal/stable:^2.0
composer require 'drupal/at_tool:^2.0'

This will install

drupal/stable (2.0.0)
drupal/adaptivetheme (5.4.0)
drupal/at_tool (2.0.5)

then Enable AT Tool and AT Theme Generator from UI.

It works fine for me for on Drupal : 10.2.5 ( PHP : 8.2.7)

🐛 | AT Tool 2.0 | Does not uninstall properly

🇮🇳India rushiraval

📌 | stub project from issue ref | Request for 'confirmed' user role

🇮🇳India rushiraval

📌 | stub project from issue ref | Request for 'confirmed' user role

🇮🇳India rushiraval

I have confirmed your account based on your contribution for issue The display background of the name in the Media library module is white 🐛 The display background of the name in the Media library module is white Fixed

📌 | stub project from issue ref | [1.0.x] AesirX-Analytics

🇮🇳India rushiraval

📌 | stub project from issue ref | [1.0.x] AesirX-Analytics

🇮🇳India rushiraval

mainis wrong names for branch. Release branch names always end with the literal .x as described in Release branches → .

After updating as per comment #2 change status to Needs Review

📌 | stub project from issue ref | [1.0.x] AesirX-Analytics

🇮🇳India rushiraval

Thank you for applying!

The important notes are the following.

If you have not done it yet, you should run phpcs --standard=Drupal,>DrupalPractice on the project, which alone fixes most of what reviewers would report.
For the time this application is open, only your commits are allowed.
The purpose of this application is giving you a new drupal.org role that allows you to opt projects into security advisory coverage, either projects you already created, or projects you will create. The project status will not be changed by this application; no other user will be able to opt projects into security advisory policy.
We only accept an application per user. If you change your mind about the project to use for this application, or it is necessary to use a different project for the application, please update the issue summary with the link to the correct project and the issue title with the branch to review and the project name.

To the reviewers

Please read How to review security advisory coverage applications → , Application workflow → , What to cover in an application review → , and Tools to use for reviews → .

The important notes are the following.

It is preferable to wait for a Code Review Administrator before commenting on newly created applications, even to leave a comment similar to the following one. Code Review Administrators will do some preliminary checks that are necessary before any change on the project files is suggested.
It is also preferable to wait before using a CLI tool → to report what needs to be changed, especially because the comment left from Code Review Administrators suggests to use PHP_CodeSniffer. Before that, manual reviews should be done.
Reviewers should not copy-paste the output of a CLI tool. They should use a CLI tool only once per application. When they do that, they should later verify the code has been correctly changed; this means, for example, that adding a documentation comment that is not correct just to avoid to get a warning/error is not a correct change that should be reported in a further comment.
It may be best to have the applicant fix things before further review.

For new reviewers, I would also suggest to first read In which way the issue queue for coverage applications is different from other project queues → .

📌 | stub project from issue ref | [1.x] TFA Migration

🇮🇳India rushiraval

Thank you for applying!

The important notes are the following.

If you have not done it yet, you should run phpcs --standard=Drupal,>DrupalPractice on the project, which alone fixes most of what reviewers would report.
For the time this application is open, only your commits are allowed.
The purpose of this application is giving you a new drupal.org role that allows you to opt projects into security advisory coverage, either projects you already created, or projects you will create. The project status will not be changed by this application; no other user will be able to opt projects into security advisory policy.
We only accept an application per user. If you change your mind about the project to use for this application, or it is necessary to use a different project for the application, please update the issue summary with the link to the correct project and the issue title with the branch to review and the project name.

To the reviewers

Please read How to review security advisory coverage applications → , Application workflow → , What to cover in an application review → , and Tools to use for reviews → .

The important notes are the following.

It is preferable to wait for a Code Review Administrator before commenting on newly created applications, even to leave a comment similar to the following one. Code Review Administrators will do some preliminary checks that are necessary before any change on the project files is suggested.
It is also preferable to wait before using a CLI tool → to report what needs to be changed, especially because the comment left from Code Review Administrators suggests to use PHP_CodeSniffer. Before that, manual reviews should be done.
Reviewers should not copy-paste the output of a CLI tool. They should use a CLI tool only once per application. When they do that, they should later verify the code has been correctly changed; this means, for example, that adding a documentation comment that is not correct just to avoid to get a warning/error is not a correct change that should be reported in a further comment.
It may be best to have the applicant fix things before further review.

For new reviewers, I would also suggest to first read In which way the issue queue for coverage applications is different from other project queues → .

📌 | stub project from issue ref | [2.x] Title Length

🇮🇳India rushiraval

I have reviewed code. But I think there is not enough php code to review.

Senior reviewer may review code with this point.

📌 | stub project from issue ref | [1.0.x] mosparo Integration

🇮🇳India rushiraval

I am changing the issue priority as per issue priorities → .

📌 | stub project from issue ref | [2.0.x] Summary Only

🇮🇳India rushiraval

I am changing the issue priority as per issue priorities → .

📌 | stub project from issue ref | [1.0.x] CSP Google Supported Domains

🇮🇳India rushiraval

I am changing the issue priority as per issue priorities → .

📌 | stub project from issue ref | Request for 'confirmed' user role

🇮🇳India rushiraval

I am confirming your role based on your contribution to project (Personalized Content Dashboard)

📌 | stub project from issue ref | [1.0.x] Copy Text to Clipboard

🇮🇳India rushiraval

I have reviewed code. But I think there is not enough php code to review.

Senior reviewer may review code with this point.

📌 | Copy Text to Clipboard | How to Install

🇮🇳India rushiraval

There was not any release when I have posted. Then contributor have created initial release. Thanks

now this issue may be marked fixed by contributor.

📌 | Copy Text to Clipboard | How to Install

🇮🇳India rushiraval

Rushikesh Raval → created an issue.

📌 | stub project from issue ref | [1.x] Easy Adsense

🇮🇳India rushiraval

I have reviewed code. Its looks clean. But I think there is not enough code to review.

Senior reviewer may review this further.

📌 | stub project from issue ref | [1.0.x] Momento

🇮🇳India rushiraval

Please solve the following phpcs Error

phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml web/modules/contrib/drupal-cache/

FILE: web/modules/contrib/drupal-cache/README.md
--------------------------------------------------------------------------------
FOUND 1 ERROR AND 6 WARNINGS AFFECTING 6 LINES
--------------------------------------------------------------------------------
13 | WARNING | [ ] Line exceeds 80 characters; contains 632 characters
19 | WARNING | [ ] Line exceeds 80 characters; contains 107 characters
30 | WARNING | [ ] Line exceeds 80 characters; contains 253 characters
32 | WARNING | [ ] Line exceeds 80 characters; contains 420 characters
34 | WARNING | [ ] Line exceeds 80 characters; contains 422 characters
36 | WARNING | [ ] Line exceeds 80 characters; contains 220 characters
36 | ERROR | [x] Expected 1 newline at end of file; 2 found
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------

📌 | stub project from issue ref | [1.0.x] Momento

https://www.drupal.org/docs/getting-started/system-requirements/overview

🇮🇳India rushiraval

This module is for Drupal 9 and 10 but you requires php 7.4 in info.

Drupal 10 Requires php 8.1 and above. Please refer

📌 | stub project from issue ref | [1.0.x] Search Tracking

🇮🇳India rushiraval

@vishal.kadam thank for your guidance. It is noted for future review.

📌 | stub project from issue ref | [1.0.x] Search Tracking

🇮🇳India rushiraval

solve the phpcs Error
phpcs --standard=Drupal,DrupalPractice web/modules/contrib/search_tracking/

FILE: web/modules/contrib/search_tracking/src/Controller/searchTrackingController.php
------------------------------------------------------------------------------------------------------------------------------------------
FOUND 12 ERRORS AND 8 WARNINGS AFFECTING 16 LINES
------------------------------------------------------------------------------------------------------------------------------------------
6 | WARNING | [x] Unused use statement
7 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Symfony\Component\HttpFoundation\JsonResponse.
8 | WARNING | [x] Unused use statement
8 | ERROR | [x] When importing a class with "use", do not include a leading \
9 | WARNING | [x] Unused use statement
14 | ERROR | [ ] Class name must begin with a capital letter
16 | ERROR | [x] Missing function doc comment
18 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected "TRUE" but found "true"
19 | ERROR | [x] Expected 1 space after IF keyword; 0 found
19 | ERROR | [x] Expected 1 space after closing parenthesis; found 0
20 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
24 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
24 | ERROR | [x] Whitespace found at end of line
25 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
33 | ERROR | [ ] Public method name "searchTrackingController::search_tracking" is not in lowerCamel format
33 | ERROR | [x] Missing function doc comment
34 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
37 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
42 | ERROR | [x] Expected 1 blank line after function; 0 found
43 | ERROR | [x] The closing brace for the class must have an empty line before it
------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 13 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------------------------

📌 | stub project from issue ref | [1.0.x] FIFO Lock (Ticket lock)

🇮🇳India rushiraval

Solve following phpcs Error

phpcs --standard=Drupal,DrupalPractice web/modules/contrib/fifo_lock/

📌 | stub project from issue ref | [2.x] User Field Login

🇮🇳India rushiraval

Solve phpcs Error

phpcs --standard=Drupal,DrupalPractice web/modules/contrib/field_login/

FILE: web/modules/contrib/field_login/src/Form/FieldLoginSettingsForm.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------
74 | WARNING | \Drupal calls should be avoided in classes, use dependency
| | injection instead
--------------------------------------------------------------------------------

FILE: .../drupal104/web/modules/contrib/field_login/src/FieldLoginPluginManager.php
--------------------------------------------------------------------------------
FOUND 1 ERROR AND 2 WARNINGS AFFECTING 3 LINES
--------------------------------------------------------------------------------
47 | WARNING | \Drupal calls should be avoided in classes, use dependency
| | injection instead
48 | WARNING | Unused variable $plugin_id.
64 | ERROR | Missing parameter comment
--------------------------------------------------------------------------------

FILE: web/modules/contrib/field_login/src/FieldLoginPluginManagerInterface.php
--------------------------------------------------------------------------------
FOUND 4 ERRORS AFFECTING 4 LINES
--------------------------------------------------------------------------------
1 | ERROR | [x] End of line character is invalid; expected "\n" but found
| | "\r\n"
7 | ERROR | [x] Missing interface doc comment
9 | ERROR | [x] Missing function doc comment
11 | ERROR | [x] Missing function doc comment
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------

FILE: web/modules/contrib/field_login/src/UserAuthDecorator.php
--------------------------------------------------------------------------------
FOUND 4 ERRORS AND 1 WARNING AFFECTING 4 LINES
--------------------------------------------------------------------------------
8 | ERROR | [x] Use statements should be sorted alphabetically. The first
| | wrong one is
| | Drupal\Core\DependencyInjection\DependencySerializationTrait.
55 | ERROR | [ ] Parameter $flood is not described in comment
55 | ERROR | [ ] Parameter $config_factory is not described in comment
63 | ERROR | [x] Parameter comment indentation must be 3 spaces, found 4
| | spaces
107 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency
| | injection instead
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------

FILE: web/modules/contrib/field_login/src/UserVerificationInterface.php
--------------------------------------------------------------------------------
FOUND 3 ERRORS AND 2 WARNINGS AFFECTING 5 LINES
--------------------------------------------------------------------------------
1 | ERROR | [x] End of line character is invalid; expected "\n" but found
| | "\r\n"
5 | WARNING | [x] Unused use statement
9 | WARNING | [ ] The class short comment should describe what the class does
| | and not simply repeat the class name
15 | ERROR | [ ] Missing parameter comment
17 | ERROR | [x] Additional blank lines found at end of doc comment
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 3 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------

FILE: web/modules/contrib/field_login/src/UserVerification.php
--------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
--------------------------------------------------------------------------------
50 | ERROR | [x] Parameter comment must end with a full stop
52 | ERROR | [x] Parameter comment indentation must be 3 spaces, found 4
| | spaces
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------

FILE: web/modules/contrib/field_login/field_login.api.php
--------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------
13 | ERROR | Missing parameter type
--------------------------------------------------------------------------------

Time: 420ms; Memory: 10MB

📌 | stub project from issue ref | [1.0.x] Entity limit extend

🇮🇳India rushiraval

I am changing the issue priority as per issue priorities → .

📌 | stub project from issue ref | [1.0.x] Search Tracking

🇮🇳India rushiraval

I am changing the issue priority as per issue priorities → .

📌 | stub project from issue ref | [1.0.x] Facets Taxonomy Multilevel

🇮🇳India rushiraval

I am changing the issue priority as per issue priorities → .

🐛 | AT Tool 2.0 | SEO: tap targets are too small in book list

🇮🇳India rushiraval

🐛 | AT Tool 2.0 | when updating to latest 1.4 version i got error

🇮🇳India rushiraval

🐛 | AT Tool 2.0 | Theme "at_tools" missing

🇮🇳India rushiraval

🐛 | Marvelous | slider issue

🇮🇳India rushiraval

🐛 | Marvelous | slider issue

🇮🇳India rushiraval

Fixed

🐛 | Marvelous | slider issue

🇮🇳India rushiraval

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

📌 | stub project from issue ref | [1.0.x] Darkmode JS

🇮🇳India rushiraval

I am changing the issue priority as per issue priorities → .

📌 | stub project from issue ref | [1.0.x] AI Field

🇮🇳India rushiraval

I am changing the issue priority as per issue priorities → .

🐛 | Great Zymphonies Theme | Twitter logo needs to be changed to the X

🇮🇳India rushiraval

Updated Library.

🐛 | Drupal core | $form_state storage disappears after form submit + submit on F5

🇮🇳India rushiraval

@cilefen I think it is not bug. It normal to reinitialize of variable on refresh. So no need for any code. But if @xandeadx needs to retain value then he may store value in session.

🐛 | Virgo Zymphonies Theme | Twitter logo needs to be changed to the X

🇮🇳India rushiraval

I have updated logo for X

🐛 | Virgo Zymphonies Theme | Twitter logo needs to be changed to the X

🇮🇳India rushiraval

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

🐛 | Great Zymphonies Theme | Twitter logo needs to be changed to the X

🇮🇳India rushiraval

I have updated logo for X.

🐛 | Great Zymphonies Theme | Twitter logo needs to be changed to the X

🇮🇳India rushiraval

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

💬 | Easy Install | Offering to co-maintain Easy Install

🇮🇳India rushiraval

Hello Team,

Can you please grant me access of co-maintainer so we will upgrade this module for Drupal 10.

Thanks

🐛 | Star Zymphonies Theme | Twitter logo needs to be changed to the X

🇮🇳India rushiraval

I have reviewed the Patch #2. It changed the logo of twitter to X. so moving it RTBC+

🐛 | Drupal core | $form_state storage disappears after form submit + submit on F5

🇮🇳India rushiraval

It may not be bug. When you refresh or reload page it will build form again. So form state will reinitialize.

📌 | stub project from issue ref | [1.0.x] Generative Summary

🇮🇳India rushiraval

📌 | stub project from issue ref | [2.x] REST Log

📌 | stub project from issue ref | [8.x-1.x] Powerling translation provider for Translation Management Tool (TMGMT)

🇮🇳India rushiraval

🇮🇳India rushiraval

📌 | stub project from issue ref | [1.0.x] JQuery form validation

🇮🇳India rushiraval

Please solve phpcs Error

FILE: jquery_form_validation/README.md
------------------------------------------------------------------------------------------
FOUND 1 ERROR AND 1 WARNING AFFECTING 2 LINES
------------------------------------------------------------------------------------------
5 | WARNING | [ ] Line exceeds 80 characters; contains 81 characters
26 | ERROR | [x] Expected 1 newline at end of file; 0 found
------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------

FILE: query_form_validation.settings.yml
-----------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------
9 | ERROR | [x] Expected 1 newline at end of file; 0 found
-----------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------

FILE: jquery_form_validation/src/Form/TestValidationExampleForm.php
-----------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------
98 | WARNING | #options values usually have to run through t() for translation
-----------------------------------------------------------------------------------------------

FILE: jquery_form_validation/src/Form/JQueryValidationSettingsForm.php
-----------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 6 WARNINGS AFFECTING 5 LINES
-----------------------------------------------------------------------------------------------
114 | WARNING | Unused variable $name_field.
118 | WARNING | Unused variable $name_field.
130 | WARNING | t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
138 | WARNING | t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
249 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
249 | WARNING | t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
-----------------------------------------------------------------------------------------------

📌 | stub project from issue ref | [1.0.x] mosparo Integration

🇮🇳India rushiraval

Sorry, Its my mistake I have check on beta version instead of dev. I found there is not phpcs error in dev Version.

📌 | stub project from issue ref | [1.0.x] mosparo Integration

🇮🇳India rushiraval

Fix phpcs issues.

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

FILE: mosparo_integration/mosparo_integration.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
-------------------------------------------------------------------------------------------------------------

FILE: mosparo_integration/modules/mosparo_webform/mosparo_webform.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
-------------------------------------------------------------------------------------------------------------------------

FILE: mosparo_integration/modules/mosparo_contact/mosparo_contact.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
-------------------------------------------------------------------------------------------------------------------------

FILE: mosparo_integration/modules/mosparo_contact/src/Plugin/Field/FieldType/MosparoItem.php
-------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------------------
6 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Field\FieldItemBase.
-------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------------------------------------

FILE: mosparo_integration/modules/mosparo_contact/src/Plugin/Field/FieldWidget/MosparoWidget.php
-------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
-------------------------------------------------------------------------------------------------------------------------
33 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
34 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
-------------------------------------------------------------------------------------------------------------------------

FILE: mosparo_integration/modules/mosparo_captcha/mosparo_captcha.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
-------------------------------------------------------------------------------------------------------------------------

FILE: mosparo_integration/src/Form/MosparoConnectionForm.php
-----------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------------------------------
6 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Entity\EntityForm.
-----------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------

FILE: mosparo_integration/src/Service/MosparoService.php
------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------------
116 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
------------------------------------------------------------------------------------------------------------

FILE: .mosparo_integration/tests/src/Unit/Controller/MosparoConnectionListBuilderTest.php
-------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------------------
8 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is
| | Drupal\Core\Entity\EntityStorageInterface.
-------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------------------------------------

FILE: .mosparo_integration/tests/src/Unit/Service/MosparoServiceTest.php
-------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------------------
23 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Tests\UnitTestCase.
-------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------------------------------------

📌 | stub project from issue ref | [1.0.x] mosparo Integration

🇮🇳India rushiraval

Thank you for applying!

The important notes are the following.

If you have not done it yet, you should run phpcs --standard=Drupal,>DrupalPractice on the project, which alone fixes most of what reviewers would report.
For the time this application is open, only your commits are allowed.
The purpose of this application is giving you a new drupal.org role that allows you to opt projects into security advisory coverage, either projects you already created, or projects you will create. The project status will not be changed by this application; no other user will be able to opt projects into security advisory policy.
We only accept an application per user. If you change your mind about the project to use for this application, or it is necessary to use a different project for the application, please update the issue summary with the link to the correct project and the issue title with the branch to review and the project name.

To the reviewers

Please read How to review security advisory coverage applications → , Application workflow → , What to cover in an application review → , and Tools to use for reviews → .

The important notes are the following.

It is preferable to wait for a Code Review Administrator before commenting on newly created applications, even to leave a comment similar to the following one. Code Review Administrators will do some preliminary checks that are necessary before any change on the project files is suggested.
It is also preferable to wait before using a CLI tool → to report what needs to be changed, especially because the comment left from Code Review Administrators suggests to use PHP_CodeSniffer. Before that, manual reviews should be done.
Reviewers should not copy-paste the output of a CLI tool. They should use a CLI tool only once per application. When they do that, they should later verify the code has been correctly changed; this means, for example, that adding a documentation comment that is not correct just to avoid to get a warning/error is not a correct change that should be reported in a further comment.
It may be best to have the applicant fix things before further review.

For new reviewers, I would also suggest to first read In which way the issue queue for coverage applications is different from other project queues → .

📌 | stub project from issue ref | Request for 'confirmed' user role

🇮🇳India rushiraval

📌 | stub project from issue ref | Request for 'confirmed' user role