CommentFileSizeAuthor
#2 pop_links.js_.patch180 bytesbwinett

Comments

bwinett’s picture

Status: Active » Closed (fixed)

Re-opening.

Figured out that you have to set permissions for the visitor (in this case, anonymous) to "record external clicks". But the url still seems to be incorrect. It does not work with the url formed by the JavaScript code (get a 404):

http://localhost/drupal/pop_links/handle?operation=track_click&url=http%...

but it DOES work when I change the url to:

http://localhost/drupal/?q=pop_links/handle&operation=track_click&url=ht...

I'll attach the patch that works for me.

bwinett’s picture

Status: Closed (fixed) » Active
StatusFileSize
new180 bytes

Attached is the patch.

sikjoy’s picture

Hi bwinett:

Thanks for catching this and for doing the work. Quick question: are you running without clean URLs? Shame on me for not testing that scenario.

I will get this tested.

Best regards,

sicjoy

bwinett’s picture

Hello,

Sorry for taking so long to answer your question - I must have missed the email informing me that there was an update to this issue.

Anyway, answer to your question: yes, I was running without clean URL's.

Thanks!

sikjoy’s picture

Good stuff - will be pushing out an update shortly.

Thanks so much,

sicjoy