🇬🇧United Kingdom @marcelovani

PHPCS is in vendor/bin folder.

I checked out the repo of coder manually to run the tests.

Here is the info about php_codesniffer

composer show squizlabs/php_codesniffer
name     : squizlabs/php_codesniffer
descrip. : PHP_CodeSniffer tokenizes PHP, JavaScript and CSS files and detects violations of a defined set of coding standards.
keywords : phpcs, standards, static analysis
versions : * 3.13.0
released : 2025-05-11, last week
type     : library
license  : BSD 3-Clause "New" or "Revised" License (BSD-3-Clause) (OSI approved) https://spdx.org/licenses/BSD-3-Clause.html#licenseText
homepage : https://github.com/PHPCSStandards/PHP_CodeSniffer
source   : [git] https://github.com/PHPCSStandards/PHP_CodeSniffer.git 65ff2489553b83b4597e89c3b8b721487011d186
dist     : [zip] https://api.github.com/repos/PHPCSStandards/PHP_CodeSniffer/zipball/65ff2489553b83b4597e89c3b8b721487011d186 65ff2489553b83b4597e89c3b8b721487011d186
path     : /Users/marcelovani/Development/LegalConnection/Drupal/legal_connection_headless_drupal/vendor/squizlabs/php_codesniffer
names    : squizlabs/php_codesniffer

support
issues : https://github.com/PHPCSStandards/PHP_CodeSniffer/issues
security : https://github.com/PHPCSStandards/PHP_CodeSniffer/security/policy
source : https://github.com/PHPCSStandards/PHP_CodeSniffer
wiki : https://github.com/PHPCSStandards/PHP_CodeSniffer/wiki

requires
ext-simplexml *
ext-tokenizer *
ext-xmlwriter *
php >=5.4.0

requires (dev)
phpunit/phpunit ^4.0 || ^5.0 || ^6.0 || ^7.0 || ^8.0 || ^9.3.4

Any updates on this?

Agreed. This issue was created few years ago and should no longer be an issue.

BTW schemata_json_schema is indeed required, for example if you have the User module enabled.

The code in the pull request fixes the issue.
I added a test to make sure the output is accessible https://git.drupalcode.org/issue/openapi_rest-3343816/-/commit/b17300b04...

I am getting these fatal errors when I visit the endpoint without enabling REST for blocks and menus

Done in Gitlab

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

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

We have a gitlab template now, can we close this?

marcelovani → created an issue.

I have tested this and can confirm that it is good to be merged

@Klauysi I don't have phpcs.xml.

I am running it with this command
phpcs --standard=Drupal --extensions=php,module,inc,install,test,profile,theme,info,txt,md --ignore=node_modules,bower_components,vendor -s --colors --report-width=120 --report-junit=junit.xml --report-full --report-summary --report-source web/modules/custom

Thanks for the initiative. I tested the PR and still see some deprecations:

Drupal.CSS.ClassDefinitionNameSpacing
This sniff has been deprecated since Coder 8.3.30 and will be removed in Coder 9.0.0. Checking CSS coding standards
is not supported anymore, use Stylelint instead with the Drupal core .stylelintrc.json configuration file.
https://git.drupalcode.org/project/drupal/-/blob/11.x/core/.stylelintrc....
- Drupal.CSS.ColourDefinition
This sniff has been deprecated since Coder 8.3.30 and will be removed in Coder 9.0.0. Checking CSS coding standards
is not supported anymore, use Stylelint instead with the Drupal core .stylelintrc.json configuration file.
https://git.drupalcode.org/project/drupal/-/blob/11.x/core/.stylelintrc....
- Drupal.Strings.UnnecessaryStringConcat
This sniff has been deprecated since Coder 8.3.30 and will be removed in Coder 9.0.0. The custom
UnnecessaryStringConcatSniff is deprecated and will be removed in Coder 9.0.0. Use
Generic.Strings.UnnecessaryStringConcat instead.

Thansk for the fix. PR merged.

What would be the use case for this? Please explain what you are trying to achieve and cannot do today.

marcelovani → created an issue.

The permissions that I normally use are Api, Read/Write Repo.

Thanks for your help Stew.

Humm, I am not sure what is happening on your local setup.

Things that I would look:
1 - Make sure you pulled the latest dev code https://www.drupal.org/project/config_pr/releases/8.x-1.x-dev →
2 - Make sure you require the dependencies https://git.drupalcode.org/project/config_pr/-/blob/8.x-1.x/modules/conf...
3 - Clear caches

Also, you can try to uninstall and reinstall the module

The tokens are created on your repository, you need to check with them if there is any limit.

The Config Pull Request only needs one token per user and you can use the same token for many users if you want.

Hi Stew

I did some refactoring and also worked on automated tests, which is something that I wanted to do for a long time, the tests are all passing now https://git.drupalcode.org/project/config_pr/-/jobs/4416444

I believe that this ticket will pass manual testing now.

Kind Regards,
Marcelo

Had some issues merging the PR, I opened a new PR and it worked.
Thanks.

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

Did some refactoring, moved some of the functionality to sub-module.
Needs testing

marcelovani → created an issue.

Hi Steve, thanks for confirming.

I have done something that you may help me to test https://www.drupal.org/project/config_pr/issues/3507408 ✨ Support Self Managed Gitlab Active

For now, I merged to dev branch only.

Thanks
Marcelo

marcelovani → created an issue.

Thanks for the explanation.

But on your use case, you are saying that people that log in as admin should not see the field. This is not a common pattern and people that log in as admin need to know what they are doing and will see a lot of configurations that may not make sense to non technical users.

I agree that the field should not be visible to users that do not have the permission to create pull requests. This makes sense and I committed the code for that.

If you want to hide the field from admin users, I recommend that you implement the same hook on a custom module.

function my_module_form_user_form_alter(&$form, FormStateInterface $form_state, $form_id) {
  $field_name = 'field_config_pr_auth_token';
  if (isset($form[$field_name])) {
    $form[$field_name]['#access'] = FALSE;
  }
}

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

@eiriksm, now that I am reading this again, I have a different comment.

The access token is configured on the user's profile.
This tokens are not supposed to be shared between users, each user should generate their own access token on their repositories and save on their own user accounts.

The token can be scoped to specific repos and you can limit the permissions to allow only reading the repo and creating pull requests. There is no big danger.

What are your concerns?

I am pushing a fix to the dev branch, I tested it but it would be nice to have someone else to try

marcelovani → created an issue.

marcelovani → created an issue.

I found the reason for the original issue reported, I am working on a fix and will update soon

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

Hi @nelo_drup

Thanks for reporting.

In order for me to debug this I need some information.

1 - Can you please provide detailed steps to reproduce?
2 - Are you using other modules that could be conflicting and causing this error?
3 - Can you try it on a fresh Drupal installation?

Done

marcelovani → created an issue.

Thanks for everyone that was involved, sorry for the delay. I merged the PR and will work on the Drupal 11 support https://www.drupal.org/project/config_token/issues/3505745 💬 Drupal 11 version Active

Hi Lee

I am not clear on what is happening.

Captcha Keypad is working for me as standalone and also with Captcha module.

I will create a release and see if there is any issues that will be affecting anyone else.

Thanks for helping to test this.

Marcelo

Thanks Lee

Lets wait until you find which module is causing the issue. I can install the same module on my side and see if there is anything we can do at Captcha Keypad.

Thanks
Marcelo

@Iee56 I tested it locally, with the steps you mentioned, but I did not get any issues.
Perhaps it is another module that is enabled and causing this, can you please check the list of enabled modules using drush pml command?

I have pushed more changes to the dev branch, please pull the latest changes first.

The automated tests are passing https://git.drupalcode.org/project/captcha_keypad/-/pipelines/414110

I pushed the fixes for Drupal 11.
If you could test the Dev branch and let me know if works, I will make a new release.
Thanks

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

This is a very good idea, I will put in my list of todos.
If you want to contribute, feel free to send a PR and I will review.

Pushed latest changes and merged PR

Hi Maxilein, I believe this issue is not related to this project.

Can you please review and add more details to explain the problem?

Reopening to work on bugs

Fixed, thanks.

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

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

Merged

I commented in the PR. Feel free to do phpcs fixes

There are lots of changes in the MR, please review.

I fixed this on another ticket and gave you credit https://www.drupal.org/project/sharerich/issues/3289584#comment-15898445 📌 Automated Drupal 10 compatibility fixes Needs review
Can you please help to check latest tag 2.0.3 https://www.drupal.org/project/sharerich →

I pushed some fixes, need some help with testing

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

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

It has been merged, but needs to be tested and confirm that everything works before making a release

Thanks for reporting, can you please add screenshots with steps to reproduce?

vincenzo gambino → credited marcelovani → .

Thanks Ankit

Thanks, PR merged.