Support » Plugin: ThickBox Content » [Plugin: ThickBox Content] Suggestion for update

  • Resolved istarnet


    The problem is that you cannot add querystring arguments to a url when creating a link to thickbox content. This is because the plugin *always* postpends ?keepThis=true&more=args.

    I solved this by adding this code after line 61 of lib/shortcodes.php

    // inline href strings might contain querystring arguments.
    // If the $url contains a ? then postpend with a &
    if (strpos($url,'?')) {$conjunctive = '&'; } else { $conjunctive = '?'; }

    Then I changed (former line 61, now line 65 to:

    $out = $opentag . '<a' . $hid . $class . ' href="' . $url . $conjunctive . 'keepThis=true'.$iframe.'&height=' . $height . '&width=' . $width .'" title="' . $title . '" class="thickbox">' . $anchortext .'</a>' . $closetag;

    I suggest you add this or a more elegant solution and re-release.

    And of course, THANK YOU for this plugin. I always include it and train my clients to add their own thickbox content!

Viewing 2 replies - 1 through 2 (of 2 total)
  • Plugin Author Max Chirkov


    Yes, looks like I totally “ignored” people that don’t use permalinks 🙂 Thanks for pointing that out. I’ll include your patch into the next update. Thanks again!

    Plugin Author Max Chirkov


    Committed and released. Thanks!

Viewing 2 replies - 1 through 2 (of 2 total)
  • The topic ‘[Plugin: ThickBox Content] Suggestion for update’ is closed to new replies.