Composer requiring drupal/ckeditor_lts instead of drupal/ckeditor

Created on 26 September 2023, about 1 year ago
Updated 9 October 2023, about 1 year ago

Problem/Motivation

Doing a composer update where we require drupal/ckeditor and now composer is downloading drupal/ckeditor_lts instead:

  - Upgrading drupal/ckeditor (1.0.2 => dev-1.0.x)
  - Locking drupal/ckeditor_lts (dev-1.0.x a6f6c97)

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

πŸ› Bug report
Status

Fixed

Version

1.0

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States dave reid Nebraska USA

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @dave reid
  • πŸ‡ΊπŸ‡ΈUnited States dave reid Nebraska USA
  • πŸ‡«πŸ‡·France prudloff Lille

    The existence of this module breaks composer update --lock on all our projects:

      Problem 1
        - Root composer.json requires drupal/ckeditor == 1.0.1.0, found drupal/ckeditor[1.0.1] in the lock file but not in remote repositories, make sure you avoid updating this package to keep the one from the lock file.
      Problem 2
        - Root composer.json requires drupal/ckeditor_config == 3.2.0.0 -> satisfiable by drupal/ckeditor_config[3.2.0].
        - drupal/ckeditor_config 3.2.0 requires drupal/ckeditor ^1.0 -> found drupal/ckeditor[1.0.1] in the lock file but not in remote repositories, make sure you avoid updating this package to keep the one from the lock file.
      Problem 3
        - Root composer.json requires drupal/editor_file == 1.6.0.0 -> satisfiable by drupal/editor_file[1.6.0].
        - drupal/editor_file 1.6.0 requires drupal/ckeditor * -> found drupal/ckeditor[1.0.1] in the lock file but not in remote repositories, make sure you avoid updating this package to keep the one from the lock file.
    

    It seems to happen because drupal/ckeditor now points to the ckeditor_lts module:

    name     : drupal/ckeditor
    descrip. : WYSIWYG editing for rich text fields using CKEditor.
    keywords : 
    versions : 10.0.0-alpha6, 10.0.0-alpha5, 10.0.0-alpha4, 10.0.0-alpha3, 10.0.0-alpha2, 10.0.0-alpha1, 9.4.3, 9.4.2, 9.4.1, 9.4.0, 9.4.0-rc2, 9.4.0-rc1, 9.4.0-beta1, 9.4.0-alpha1, 1.0.x-dev, dev-1.0.x
    type     : metapackage
    license  : GNU General Public License v2.0 or later (GPL-2.0-or-later) (OSI approved) https://spdx.org/licenses/GPL-2.0-or-later.html#licenseText
    homepage : https://www.drupal.org/project/ckeditor_lts
    source   : []  
    dist     : []  
    names    : drupal/ckeditor
    
    support
    source : https://git.drupalcode.org/project/ckeditor_lts
    

    We require ckeditor_config and it requires ckeditor. But for some reason ckeditor_lts claims to be drupal/ckeditor so Composer tries to use it instead of https://www.drupal.org/project/ckeditor β†’ .
    Is there a reason the package for this module is named drupal/ckeditor and not drupal/ckeditor_lts? (I am not even sure that is possible.

  • πŸ‡«πŸ‡·France prudloff Lille
  • πŸ‡ΊπŸ‡ΈUnited States dave reid Nebraska USA

    I reported this in the #drupal-infrastructure Slack thread and there is already a fix in place.

  • πŸ‡΅πŸ‡±Poland salmonek

    Thank you for reporting the issue and I apologize for the problem we raised.
    I will contact infrastructure team to discuss how we can avoid this in case of future releases.
    Our intention was to have this module available as drupal/ckeditor_lts, not overriding drupal/ckeditor for users that doesn't use LTS version.
    We do base our module on ckeditor having same namespaces to simplify update process and as far I'm aware CKSource no longer maintains original ckeditor module thus the decision to deploy new module.

    Once more apologies for the mess.

  • πŸ‡ΊπŸ‡ΈUnited States dave reid Nebraska USA

    The Drupal Infrastructure team was able to adjust things yesterday with https://www.drupal.org/project/project_composer/issues/3309838#comment-1... πŸ› Incorrect version of CKEditor being parsed from info.yml files Active so we're good!

  • Status changed to Fixed about 1 year ago
  • πŸ‡΅πŸ‡±Poland salmonek

    Just small info - we have released version 1.0.0 and there were no further problems with package override.

  • Automatically closed - issue fixed for 2 weeks with no activity.

Production build 0.71.5 2024