Allow node-picked links to wrap other elements

You mean you want to link a image for example? Haven't though about that ... not sure how that would work with the input filter. I'll have a look.

+1 on this feature for me. I'd love to be able to select an image and link it to an existing node.

I'm happy to help test out code for this feature, I just don't have the module-writing chops to start it out on my own :)

Why does Node Picker insert special code -- e.g. [nodepicker==node/58==Title%20of%20node==word] -- instead of a normal link? It seems like if it did the latter, it would be easier to wrap it around other objects -- but I might be totally missing a critical reason it doesn't create normal links, so feel free to school me :)

It creates a special tag so even if you change the URL-alias for a node the link would still work. But this feature causes some other problems, wrapping images etc for instance.

Hello.
I suggested change line 274 nodepicker.module

from

  $link = l($attr['link_text'], drupal_get_path_alias($attr['href']), array('attributes' => array( 'title' => $attr['title'], 'class' => 'nodepicker-link')));

to

  $link = l($attr['link_text'], drupal_get_path_alias($attr['href']), array('attributes' => array( 'title' => $attr['title'], 'class' => 'nodepicker-link'), 'html' => TRUE));

and i don't understand

why do you use utf8_encode and urldecode on title?

I have to convert title text to urlencode code, but i want to set title on my language without url-encoding and to see my title on my language in editor

Is the action in linking ajax?

may be use t() for title?

For self I removed the actions with title because l() function checked it

Are my changes wrong?

For example, now i use

[nodepicker==node/315==My title 315==[#340:field_image_cache:1,formatter="image_plain",nolink]]

without url-encoding

and nodepicker work with linodef

Best regards.

I'm using urlencode because the tag decoding doesn't work with special characters otherwise. Tags get decoded on an AJAX request like nodepicker/ajax?tag=[nodepicker==node/1==My title==My link text] and if this URL contains special characters you get all sorts of strange behaviors.

+1 on this feature

Agree with #4 rootwork.
Why not to use Drupal internal paths 'node/123'? see no reason for special tags.

This feature is dependant on the outcome of #819512: Use <a>-tags (instead of custom tags/tokens) and do NOT use URL aliases. If I decide to skip the coding of links wrapping other elements will be implemented.

Sub

+1

+1

the feature works if you use #6, and you aren't use wisiwyg mode!

[nodepicker==node/315==My title 315==[#340:field_image_cache:1,formatter="image_plain",nolink]]