🇲🇽Mexico @GafgarionMorua

Hello again!

I modified the code to utilize array_walk function as suggested for a clearer error message.

Please check the changes when you have a minute, and let me know if additional work is required.

Kind regards!

Hello!

I changed the base of the MR from 11.0.x to 11.x.

Please review the changes and let me know if additional work is required.

Kind regards!

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

The error message indicates that your current user does not have the requisite privileges to create symbolic links when inserting Drush with Composer. The lack of privileges could be due to your user account type, or you might be running the command prompt without administrative rights.

You could try running the operation as an administrator to overcome this error. Below are the steps:

1. Press the "Windows" key and type "cmd" or "Command Prompt".
2. From the results, right-click on "Command Prompt" and opt to "Run as administrator" (you might need to confirm or key in a password).
3. Once inside the command prompt, navigate to your project directory using the cd command (e.g., cd D:\xampp\htdocs\test\).
4. After that, try running the Composer command once more (composer require drush/drush).

If you're facing challenges with the manual installation of Drush and other elements of a Drupal development environment, you might consider using DDEV. DDEV is an open-source tool that uses Docker to build a pre-packaged environment containing everything you need for Drupal development.

With DDEV, you don't have to worry about setting up Apache, Nginx, PHP, MySQL, Solr, or other services. Plus, it comes with Drush and other useful tools preinstalled.

To install DDEV on your machine, follow the instructions in the official DDEV documentation (https://ddev.readthedocs.io/en/stable/).

driverok → credited GafgarionMorua → .

I updated the summary to represent the changes better. Whenever you have a minute, please review the MR. I appreciate your time.

Updated comments for clarity and grammar conformance. Replaced ambiguous language and terms with more explicit phrases.

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

Hello!

I added Documentation for Error Handling in getFieldStorageDefinition() Function. However, I see the spell-checking pipeline is failing, but I can't identify why. I would greatly appreciate it if you could review the changes and let me know how to address the error to ensure all the tests can run.

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

My apologies; I have fixed the issues flagged by the pipelines. The MR is ready for review.

Refactor multiple catch blocks into union type catch block.

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

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

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

The following issues were flagged by the upgrade_status module
================================================================================
Diff, --
Scanned on Tue, 02/13/2024 - 18:28

FILE: web/modules/contrib/diff/src/Controller/PluginRevisionController.php

STATUS LINE MESSAGE
--------------------------------------------------------------------------------
Fix later 135 Call to deprecated method loadRevision() of class
Drupal\Core\Entity\EntityStorageInterface. Deprecated in
drupal:10.1.0 and is removed from drupal:11.0.0. Use
Drupal\Core\Entity\RevisionableStorageInterface::loadRevisio
n instead.
--------------------------------------------------------------------------------

FILE: web/modules/contrib/diff/src/Plugin/views/field/DiffPluginBase.php

STATUS LINE MESSAGE
--------------------------------------------------------------------------------
Fix later 153 Call to deprecated method loadRevision() of class
Drupal\Core\Entity\EntityStorageInterface. Deprecated in
drupal:10.1.0 and is removed from drupal:11.0.0. Use
Drupal\Core\Entity\RevisionableStorageInterface::loadRevisio
n instead.
--------------------------------------------------------------------------------

FILE: web/modules/contrib/diff/diff.info.yml

STATUS LINE MESSAGE
--------------------------------------------------------------------------------
Check manually 0 Value of core_version_requirement: ^9.3 || ^10 is not
compatible with the next major version of Drupal core. See
https://drupal.org/node/3070687.
--------------------------------------------------------------------------------

FILE: web/modules/contrib/diff/composer.json

STATUS LINE MESSAGE
--------------------------------------------------------------------------------
Check manually 0 The drupal/core requirement is not compatible with the next
major version of Drupal. Either remove it or update it to be
compatible. See
https://drupal.org/node/2514612#s-drupal-9-compatibility.
--------------------------------------------------------------------------------

Crated a merge request to fix compatibility issues by updating deprecated 'loadRevision()' method usage, adjusted 'drupal/core' version in composer.json file

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

Replaced deprecated PHPUnit returnValue() method with the willReturn() method in the module's tests.

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

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

driverok → credited GafgarionMorua → .

GafgarionMorua → created an issue.

driverok → credited GafgarionMorua → .

The patch in #60 works for me on Drupal 9.5.5

driverok → credited GafgarionMorua → .

GafgarionMorua → created an issue.

I tested the module, and it seems to be working as expected after applying the changes in the latest merge request:

Drupal version
10.0.3

Web Server
nginx/1.22.1

PHP
Version
8.1.13
Memory limit
1024M

Database
Version
10.4.27-MariaDB-1:10.4.27+maria~ubu2004-log
System
MariaDB

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

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

GafgarionMorua → created an issue.