The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.OK, I very much want to get this module into a working state, so I spent quite some time hacking it to resolve a number of issues and integrating different patches contributed in the issues listed here.
Thank you for everyone's contributions (in no particular order):
@fgm
@osopolar
@mukundhraj
@bbu23
@mikogotyk
This patch is based on the current 2.x dev version, it is compatible with Drupal 10.2 and I believe will at least partially solve the following issues :
https://www.drupal.org/project/alinks/issues/3171893
https://www.drupal.org/project/alinks/issues/3407251
https://www.drupal.org/project/alinks/issues/3081917
https://www.drupal.org/project/alinks/issues/3042282
https://www.drupal.org/project/alinks/issues/3283793
https://www.drupal.org/project/alinks/issues/2943906
https://www.drupal.org/project/alinks/issues/3301335
In the current state of the module, *some* keywords get replaced.
At this moment, I don't know why the module doesn't replace all keywords, I guess it has something to do with paragraph instances.
So it's not working perfectly, but at least a handful of links are placed on content pages if content and paragraphs are configured, which should already help with SEO.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | 3412674-cumulative-patch.patch | 8.32 KB | drupaldope |
Issue fork alinks-3412674
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
- 2.x
changes, plain diff MR !6 - 3412674-new-2.x-dev changes, plain diff MR !5
Comments
Comment #3
drupaldope commentedComment #4
drupaldope commentedComment #5
greg boggsIt makes it really difficult to review each patch when you create a new issue and combine 7 patches into a single patch. Good for your local site, I'm sure, but for reviewing and merging, it's much more time consuming.
Comment #6
greg boggsHrm, there's something different about the fork on this issue, there's no merge request for me to merge.
Comment #11
greg boggsYour work on Alinks has been merged.
Comment #12
greg boggsThanks DrupalDope! :)
Comment #13
drupaldope commented@thank you Greg
Sorry about the mess :)
The trouble about making several patches is the order in which they have to be applied.
I hope this patch works, I just had trouble applying it to a new site I'm working on.
(aaaah - you updated 2.x-dev in the meanwhile that's why my patch didn't apply anymore!)