WordPress.org

1. Anonymous
   Unregistered
   Posted 8 years ago #

   The Links bookmarklet works well with Safari 1.1 (v100). There is a small problem, though. If the title of a page contains one or more escaped characters like ', " or \ the title of the new link will have backslashes inserted before each one of these chars.
   The fix is very simple. In wp-admin/linkmanager.php, line 423 or so :
   Find:
   
$link_url =$HTTP_GET_VARS["linkurl"];
$link_name = $HTTP_GET_VARS["name"];

   Replace with:
   
$link_url = stripslashes($HTTP_GET_VARS["linkurl"]);
$link_name = stripslashes($HTTP_GET_VARS["name"]);


2. Anonymous
   Unregistered
   Posted 8 years ago #

   Ahem, the \ was actually a backslash.

3. JackSim
   Member
   Posted 8 years ago #

   Ok, and a space is missing between the first = sign and $HTTP_GET_VARS.

4. Alex King
   Member
   Posted 8 years ago #

   Hi JackSim, nice to see you here. :)

5. Mike Little
   Member
   Posted 8 years ago #

   Hi JackSim,
   I've committed this fix to CVS now.
   Thanks a lot.
   Mike

Topic Closed

This topic has been closed to new replies.