So far, it is easy (and slick) to add a link to any other node on the same site.

However, if you later edit an existing link to a node, it does not show in the plugin (the Drupal field of the link window is simply empty). Could this be achieved?

Comments

anrikun’s picture

anrikun commented

Actually, it should behave the way you want! (Please try the demonstration site to see how it is supposed to work.)
So there is obviously a problem with your site.
Can you please provide more information on your install or maybe send me a login to your site from my contact form?

anrikun’s picture

anrikun commented

sorry, double post!

vacilando’s picture

vacilando commented

Thanks for your reply. I've looked at it again and I see that it works it should for links in format "/node/N". It only does not work for aliases. If the link is an alias, e.g. "/services", then the "Drupal" field is empty even though that is an alias to a node.

anrikun’s picture

anrikun commented

Ok I get it!
Well, I think it is not a good practice to directly enter an alias as the URL, because if you change the alias afterwards, this might result in broken links.
So I suggest you replace your aliases by /node/N URLs instead (using CKEditor Link).
CKEditor Link Filter (you're using it too, don't you?) will then convert them to aliases when displayed to end users.
And if an alias is changed afterwards, CKEditor Link will update the alias on next cron run :-)

vacilando’s picture

vacilando commented

@anrikun - you are absolutely correct about the better and recommended way of linking. However, people are not so reliable as to consistently use node numbers instead of aliases and also correction of existing links, esp. on huge existing sites, does not seem practical.

Perhaps you could find a way how to detect an alias and - if it is a node - find it's node number link and show that one. That would progressively correct all links as well.

anrikun’s picture

anrikun commented
Title: Also allow editing existing Drupal links » When editing a link that is a node alias, convert it back to the BASE/node/N form

Ok, let's add this to the roadmap :-)
I don't have much time to work on this at the moment so any patch is welcome meanwhile!

anrikun’s picture

anrikun commented
Version: 6.x-1.x-dev » 6.x-2.x-dev
Status: Active » Fixed

This is now in 6.x-2.x-dev.
Note that conversion is committed only if user clicks the dialog's OK button.

Status: Fixed » Closed (fixed)

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

ludo.r’s picture

ludo.r
Location Brussels
commented
Version: 6.x-2.x-dev » 7.x-2.3
Category: Feature request » Support request
Issue summary: View changes
Status: Closed (fixed) » Active

Does this feature exist in version 7.x-2.3?