Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sarvjeetsingh created an issue. See original summary.

sarvjeetsingh’s picture

Assigned: sarvjeetsingh » Unassigned
Status: Active » Needs review
FileSize
336 bytes

added core_version_requirement: ^8 || ^9 to info.yml file.

sarvjeetsingh’s picture

FileSize
336 bytes

added core_version_requirement: ^8 || ^9 to info.yml file.

meghasharma’s picture

Status: Needs review » Needs work

please provide the screenshots of rector and drupal_check command..

meghasharma’s picture

its giving error while run the drupal_check command
please fix these errors..
drupal-8.8.1 % vendor/bin/drupal-check -ad modules/contrib/colordialog
Warning: Class 'PHPUnit\Framework\MockObject\Matcher\InvokedRecorder' not found in /Users/Documents/standar_drupal/drupal-8.8.1/core/tests/bootstrap.php on line 197
2/2 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

------ ----------------------------------------
Line colordialog.install
------ ----------------------------------------
16 Function libraries_get_path not found.
------ ----------------------------------------

------ -------------------------------------------
Line src/Plugin/CKEditorPlugin/ColorDialog.php
------ -------------------------------------------
25 Function libraries_get_path not found.
------ -------------------------------------------

[ERROR] Found 2 errors

ankitsingh0188’s picture

Assigned: Unassigned » ankitsingh0188
ankitsingh0188’s picture

Assigned: ankitsingh0188 » Unassigned
Status: Needs work » Needs review
FileSize
16.19 KB
1.96 KB
1.59 KB

Created the patch and make it compatible with D9.
after applying patch

priyanka.sahni’s picture

Assigned: Unassigned » priyanka.sahni
priyanka.sahni’s picture

Assigned: priyanka.sahni » Unassigned
kkalashnikov’s picture

Assigned: Unassigned » kkalashnikov
kkalashnikov’s picture

Assigned: kkalashnikov » Unassigned
FileSize
36.75 KB

This module is compatible with Drupal version 9.

Test with the drupal-check utility and found 0 errors. For me it's RTBC.

Ravi_3596’s picture

The patch #7 works fine for me. It would be better if we merge patch in dev branch to make module compatible with D9.

Ravi_3596’s picture

Status: Needs review » Reviewed & tested by the community
GGH’s picture

When can we expect a stable release for this module ported for Drupal 9,Can we have some timelines

adinac’s picture

Webbeh’s picture

Per #15, can you please provide context for the patch, and how this differs from the work in #7?

szeidler’s picture

Wouldn't it be better to drop the libraries dependencies and use the new core service for it? See https://www.drupal.org/node/3099614

szeidler’s picture

Status: Reviewed & tested by the community » Needs review
FileSize
1.95 KB

Here would be an example, without using the libraries module at all.

  • szeidler authored 9982594 on 8.x-1.x
    Issue #3135491 by szeidler, kkalashnikov: Drupal 9 readiness
    
g_miric’s picture

Status: Needs review » Fixed
g_miric’s picture

Status: Fixed » Closed (fixed)