🇺🇦Ukraine @andrii momotov

Hi Boris,

Thank you very much! I'll be glad to assist in supporting and improving this module!

Have a good time!

andrii momotov → created an issue.

Yes, you are right, I think it will be safer not to change anything at this stage, so I will just close this issue. If necessary, it will be possible to open it again.

I did not delete those unused variables that have TODO comments next to them, which means that these variables can be used in the future.

Please review MR. Thanks!

Added composer.json file and described it. Please review MR. Thanks!

I realize that I am not sure if I understood correctly what needs to be done in the TODO comment. If the module maintainer provides additional comments, I will do so. If someone already understands, feel free to fix this issue.

Replace t() with $this->t(). Also I removed t() function from this code:

throw new FieldException(t('A field with a different type already exists on destination entity type @entity-type.', [
  '@entity-type' => $destination_entity_type_id,
]));

in the FieldCloner.php because exceptions should not be translated.

Replaced all array() with []. Please review MR. Thanks!

Created a new module permission. Please review MR. Thanks!

Set up GitLab CI. Please review MR. Thanks!

I assigned the task myself because I plan to fix this issue and make a Merge Request.
Please do not work on this issue.

Andrii Momotov → created an issue.

I assigned the task myself because I plan to fix this issue and make a Merge Request.
Please do not work on this issue.

Andrii Momotov → created an issue.

I assigned the task myself because I plan to fix this issue and make a Merge Request.
Please do not work on this issue.

Andrii Momotov → created an issue.

I assigned the task myself because I plan to fix this issue and make a Merge Request.
Please do not work on this issue.

Andrii Momotov → created an issue.

I assigned the task myself because I plan to fix this issue and make a Merge Request.
Please do not work on this issue.

Andrii Momotov → created an issue.

I assigned the task myself because I plan to fix this issue and make a Merge Request.
Please do not work on this issue.

Andrii Momotov → created an issue.

I assigned the task myself because I plan to fix this issue and make a Merge Request.
Please do not work on this issue.

Andrii Momotov → created an issue.

I assigned the task myself because I plan to fix this issue and make a Merge Request.
Please do not work on this issue.

Andrii Momotov → created an issue.

The merge request has been merged. Thank you!

The merge request has been merged. Thank you!

Fixed this issue. Please review MR. Thanks!

Added more usage details in the hook_help() function. Please review MR. Thanks!

Before Merging please have someone test if everything works the same as before! Because I haven't tested it yet. Thank you!

Fixed this issue. Please review MR. Thanks!

Fixed this issue. Please review MR. Thanks!

Created a new module permission. Please review MR. Thanks!

I assigned the task myself because I plan to fix this issue and make a Merge Request.
Please do not work on this issue.

Andrii Momotov → created an issue.

I assigned the task myself because I plan to fix this issue and make a Merge Request.
Please do not work on this issue.

Andrii Momotov → created an issue.

I assigned the task myself because I plan to fix this issue and make a Merge Request.
Please do not work on this issue.

Andrii Momotov → created an issue.

I assigned the task myself because I plan to fix this issue and make a Merge Request.
Please do not work on this issue.

Andrii Momotov → created an issue. See original summary → .

I assigned the task to myself because I plan to fix this issue and make a Merge Request.
Please do not work on this issue.

Andrii Momotov → created an issue. See original summary → .

I assigned the task to myself because I plan to fix this issue and make a Merge Request.
Please do not work on this issue.

Andrii Momotov → created an issue.

Fixed Typo: In the variable '$output'. Removed needless check and namespace from hook_help()

MR is merged. Thanks!

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

Merged. Thanks!

Set up GitLab CI. Please review MR. Thanks!

Fixed the warning reported by PHP_CodeSniffer and created MR. Please review. Thanks!

Added a composer.json file and included all the required information. Please review MR. Thanks!

I assigned the task to myself because I plan Set up GitLab CI and make a Merge Request.

Please do not work on this issue.

Andrii Momotov → created an issue.

I assigned the task myself because I plan to create a composer.json file, describe it, and make a Merge Request.

Please do not work on this issue.

Andrii Momotov → created an issue.

Fixed the warning reported by PHP_CodeSniffer and created MR. Please review. Thanks!

Andrii Momotov → created an issue.

MR was merged. Thanks.

Merged. Thanks!

thank you all!

This has already been fixed in another issue:
https://www.drupal.org/project/popin/issues/3425549 📌 Drupal 10 Compatibility Fixed

Thank you very much! I'll be glad to assist you in supporting and improving this module!

Andrii Momotov → created an issue.

Thank you all!

Replaced direct database query in bulk_update_fields_entity_operation_alter() with \Drupal::configFactory()->listAll().

Please review MR. Thanks!

I assigned the task to myself because I plan to work on this issue.
Please do not work on this issue.

Andrii Momotov → created an issue. See original summary → .

I reviewed and tested the patch https://www.drupal.org/files/issues/2023-04-16/bulk_update_fields-add-sc... → created by @hkirsman, and it is correctly done and works well. Therefore, I created a Merge Request based on this patch. Please review. Thanks!

Added a composer.json file and included all the required information. Please review MR. Thanks!

Set up GitLab CI. Please review MR. Thanks!

Fixed Typo: In word 'exlude'. Please review my MR:
https://git.drupalcode.org/project/bulk_update_fields/-/merge_requests/13
Thanks!

@deepak5423 You acted very unfairly! This is not acceptable in our Drupal community!
You saw:
1. That I assigned this issue to myself.
2. I wrote a comment stating that I planned to work on this issue.
Nevertheless, you still went ahead and started working on it.

Fixed the errors/warnings reported by PHP_CodeSniffer and created MR. Please review. Thanks!

I assigned the task to myself because I plan to create a composer.json file, describe it, and make a Merge Request.

Andrii Momotov → created an issue.

I assigned the task to myself because I plan Set up GitLab CI and make a Merge Request.

Andrii Momotov → created an issue.

I assigned the task to myself because I plan to fix this typo and make a Merge Request.