jQuery find url within string and wrap with href if not already wrapped

Question

I need to wrap href around matched urls within an imported string - however in some cases the urls already have the href attribute set therefore they do not need wrapping. My code at this early stage is as follows

if(jQuery('.single-ai1ec_event')) {
        var httpSelector = '.single-ai1ec_event .entry-content, .single-ai1ec_event .p-location';
        var searchText = jQuery(httpSelector).text();
        var expression = /https?:\/\/(www\.)?[-a-zA-Z0-9@:%_\+.~#?&//=]{2,256}\.[a-z]{2,4}\b(\/[-a-zA-Z0-9@:%_\+.~#?&//=]*)?/gi;
        var urls = searchText.match(expression);
        var wrappedUrl;
        if (urls) {
            for (var i = 0, il = urls.length; i < il; i++) {
                jQuery(httpSelector).html(function () {
                    wrappedUrl = wrap(urls[i]);
                    jQuery(this).html(jQuery(this).html().replace(urls[i], wrappedUrl));
                });
            }
        }
    }

    function wrap(str) {
        return '<a href="' + str + '" target="_blank">' + str + '<\/a>';
    }

I am currently looking for a way to exclude wrapping urls that aleady have the href attribute set but without much success.

What if we add a check that we are not inside <a> tag: var expression = /https?:\/\/(www\.)?[-a-zA-Z0-9@:%_\+.~#?&//=]{2,256}\.[a-z]{2,4}\b(\/[-a-zA-Z0-9@:%_\+.~#?&//=]*)?(?!(?:[^>]*?>)?[^<]*?<\/a>)/gi;? See regex101.com/r/oZ7jE4/3 — Wiktor Stribiżew
– Wiktor Stribiżew, Commented Apr 24, 2015 at 12:08
Thanks - I have implemented the regex and this so far seems to work great - will continue testing — Sideshow
– Sideshow, Commented Apr 24, 2015 at 12:31

Drakes · Accepted Answer · 2015-04-24 12:18:17Z

1

You can change your selector to exclude anchors with no href tag like this:

:not([href])

so then your selector becomes:

var httpSelector = 'a:not([href]).single-ai1ec_event ...';

edited Apr 24, 2015 at 12:18

answered Apr 24, 2015 at 12:12

Drakes

23.8k3 gold badges58 silver badges97 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

jQuery find url within string and wrap with href if not already wrapped

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related