Closed (fixed)
Project:
CKEditor Anchor Link
Version:
8.x-2.4
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
26 Jan 2018 at 20:38 UTC
Updated:
21 Mar 2021 at 15:01 UTC
Jump to comment: Most recent
Could this module depend on the fakeobjects module rather than include it?
I've already got the 'fakeobjects' module as a dependency, because I'm also using the ckeditor_drupalbreaks module which has it as a dependency. So, it'd be nice to not have this in twice.
The one wrinkle is that the 'fakeobjects' module doesn't include the CKEditor plugin in the module, which adds an extra step to put it in /libraries. But I think not including the plugin is the right thing to do because you need the plugin version to match the CKEditor version, which is different depending on which version of Drupal 8 core you have.
Comments
Comment #2
rajab natshahTotally agree with you David!
CKEditor FakeObjects
I wish that FakeObjects and the Anchor Link comes with Drupal 8 Core CKEditor package.
Drupal.org Library Packaging Whitelist
#2940014: CKEditor Fake Objects
I had created the following issue 7 months ago
#2888309: CKEditor Anchor link
Comment #3
rajab natshahStarting a new branch 8.x-2.x to follow with the best practice
Comment #4
rajab natshahComment #5
rajab natshahComment #6
rajab natshahComment #8
rajab natshahComment #9
rajab natshahComment #10
rajab natshahComment #12
anybodyHi @RajabNatshah,
we had the problem on many sites where we upgraded anchor_link that drupal/fakeobjects was missing but composer didn't tell. Later on we had errors in Drupal:
So I looked into the composer.json of anchor_link and saw that drupal/anchor link isn't listed there? Do you know why? Perhaps we should reopen the issue and ensure it's required?
Current 8.x-2.4:
While it is in 8.x-2.x-dev.
I'm not sure how this could happen. Perhaps a build mistake from drupal.org? I guess we need a new 2.5 release?
Comment #13
karolus commentedI'm running into this same issue, and like Anybody have been hit with errors when attempting to edit/create nodes. So, for the moment, it may be good to at least have a patch ready, for users who aren't comfortable with running development versions of a module on the production environments.
Comment #14
anybody@RajabNatshah could you perhaps have a look at #12 and reopen this eventually?
Comment #15
rajab natshah#3129944: Add Fakeojects requirement to composer.json
Comment #16
anybodyThank you :)