I changed the following line (57) in eu_cookie_compliance.module

from
'popup_link' => $popup_settings['popup_link'],
to
'popup_link' => url($popup_settings['popup_link']),

So that node/79 (or whatever) gets nicely translated to yourwebsite.co/privacy-policy (or whatever)

Comments

Miszel’s picture

Miszel commented

What if someone inputs an alias rather than a node/xxx url into the field?

code-brighton’s picture

code-brighton commented

Should still work, the drupal url() function just checks for an alias otherwise returns original string

dakku’s picture

dakku commented
Version: 7.x-1.0 » 7.x-1.x-dev
Assigned: Unassigned » dakku
Status: Active » Fixed

patched in 7 dev

Status: Fixed » Closed (fixed)

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

  • Commit 7028d51 on 7.x-1.x, 8.x-1.x, 7.x-2.x by dakku: 
    #1600686 patch added