Ticket #1317 (closed defect: fixed)

Opened 10 months ago

Last modified 9 months ago

stumbleupon bookmarklet not passing proper URL

Reported by: npeterson Owned by: npeterson
Priority: high Milestone: 0.9.3_rc2
Component: ambra Version: 0.9.3
Keywords: Cc:

Description

Select any article in any journal, and check the stumbleupon bookmarklet URL, for example:

 http://www.stumbleupon.com/submit?url=/article/info%3Adoi%2F10.1371%2Fjournal.pmed.1000081

Then go to StumbleUpon?, and the URL is malformed:

 http:///article/info:doi/10.1371/journal.pmed.1000081

and the site will not let you add a review. Looks like we need to also pass the dx.plos.org prefix. The correct URL should be:

 http://www.stumbleupon.com/submit?url=dx.plos.org/article/info%3Adoi%2F10.1371%2Fjournal.pmed.1000081

see attached screenshot

Attachments

stumbleupon - URL error.png (199.3 KB) - added by npeterson 10 months ago.

Change History

Changed 10 months ago by npeterson

Changed 10 months ago by josowski

(In [7719]) The journal context was being read out by a freemarker template before the value was actually set. This was causing the Stumbled apon social bookmarking link to not be complete.

This resolves this issue

References #1317

Changed 10 months ago by josowski

  • status changed from new to verify
  • owner changed from josowski to npeterson
  • resolution set to fixed

Changed 9 months ago by rcave

  • status changed from verify to reopened
  • owner changed from npeterson to josowski
  • resolution fixed deleted

Changed 9 months ago by rcave

The StumbleUpon? bookmarklet URL is still incorrect:

http://www.stumbleupon.com/submit?url=/article/info%3Adoi%2F10.1371%2Fjournal.pbio.1000133
http://www.stumbleupon.com/submit?url=/article/info%3Adoi%2F10.1371%2Fjournal.pbio.0020222

Changed 9 months ago by josowski

  • status changed from reopened to verify
  • owner changed from josowski to npeterson
  • resolution set to fixed

Committed changes to CVS.

Changed 9 months ago by rcave

  • status changed from verify to closed

Verified fixed

Note: See TracTickets for help on using tickets.