[Resolved] Issue between Auto Thickbox Plus and Connections
The reason is because that plugin re-writes the links but does so incorrectly. I could work around that but I think a better solution would be for the dev to better target the re-writing of the links.
The plugin has the possibility to deactivate the re-writing of the links on text links. Unfortunately that does not work. So I suppose an update is needed.
Thanks for using Auto Thickbox Plus, and sorry for late reply.
I don’t have cMap-Template of Connections (it’s not free), so I can’t debug the plugin in reproduce environment.
But I guess this problem caused by iframe function in auto-thickbox.php. Please try to modify auto-thickbox.php at line 55.
* iframe function is defined by Auto Thickbox plugin (not Plus). I think this function is buggy.
The plugin has the possibility to deactivate the re-writing of the links on text links. Unfortunately that does not work.
“ThickBox on Text Links” option targets links to image (i.e. link URL ends with image extensions).
But links like “Show Map”, “Show Bio”, “Show Notes” and so on are not linked to image (link URL may be only “#”).
This is the reason why this option does not work.
To Steven Zahm
The reason is because that plugin re-writes the links but does so incorrectly.
Please give me the correct URL and incorrect URL rewritten by Auto Thickbox so that I can fix the problem.
But I guess this problem caused by iframe function in auto-thickbox.php.
My comment may be wrong. Auto Thickbox plugin rewrite links only in the following condition.
- a@href value ends with image extensions (png, jpg, jpeg, gif, …)
- a@class value contains “thickbox”
But “Show Map”, “Show Bio” and “Show Notes” links are not matched with above condition.
<a class="cn-map-anchor toggle-map" id="map-anchor-64f2df9c9a4f76" href="#" ...>Show Map</a>
<a class="cn-note-anchor toggle-div" id="note-anchor-64f2df9c9a4f76" href="#" ...>Show Notes</a>
<a class="cn-bio-anchor toggle-div" id="bio-anchor-64f2df9c9a4f76" href="#" ...>Show Bio</a>
After all, I don’t know what causes this problem. Since I can’t debug any more without c-Map template.
Thanks for looking into this. When Auto Thickbox is activated the links are rewritten like so:
<a class="cn-note-anchor toggle-div" id="note-anchor-32384f2f3786995e9" href="#" 0="data-uuid='32384f2f3786995e9'" 1="data-div-id='note-block-32384f2f3786995e9'" 2="data-str-show='Show" 3="Notes'" 4="data-str-hide='Close" 5="Notes'">Show Notes</a> <a class="cn-bio-anchor toggle-div" id="bio-anchor-33054f2f37869bcaf" href="#" 0="data-uuid='33054f2f37869bcaf'" 1="data-div-id='bio-block-33054f2f37869bcaf'" 2="data-str-show='Show" 3="Bio'" 4="data-str-hide='Close" 5="Bio'">Show Bio</a> <a class="cn-map-anchor toggle-map" id="map-anchor-32384f2f3786995e9" href="#" 0="data-uuid=" 32384f2f3786995e9""="" 1="data-str-show=" show"="" 2="Map" "="" 3="data-str-hide=" close"="" 4="Map">Show Map</a>
I noticed that I’m using single quotes for a couple of the data attributes instead of double quotes. I wonder if that is throwing off the re-writing. I’ll have to make them double quotes to see if that resolves the issue.
I made sure all quotes are now double quotes and the links still are rewritten incorrectly.
To Steven Zahm
Thanks a lot for more helpful information.
I was able to reproduce the bug by writing the links directly such as “Show Notes”, “Show Bio” and “Show Map”. And I found that shortcode_parse_atts() function called from anchor-utils.php is the cause.
shortcode_parse_atts() in wp-includes/shortcodes.php does not accept custom data attributes (data-*). shortcode_parse_atts() splits data-* attribute name with ‘-‘, then links would corrupt. This is a bug in Auto Thickbox Plugin (original), and for that matter, a bug in WordPress Shortcode API.
I’m not the original author, so I don’t understand yet how anchor-utils.php works. But I added the workaround for this issue. I would appreciate it if you check latest revision of Auto Thickbox Plus plugin.
Thanks for your help to solve the issue.
Sorry for the delay, the .7 version seems to work just fine now. Thanks for looking into this! I’ve upped your rating.
Thanks to you both for looking into this issue and finally solving it! Great work! 🙂
- The topic ‘[Resolved] Issue between Auto Thickbox Plus and Connections’ is closed to new replies.