Comments

mizage@gmail.com’s picture

I can confirm this issue. Using the following:

Firefox or Chrome
Drupal 7.36
Linkit 3.3
IMCE 1.9
Wysiwyg 2.2 configured with TinyMCE 3.5.11

See screen shots.

tylergbass’s picture

Hello,

I can also confirm that this is an issue with Linkit 7.x-3.3.

joverstreet001’s picture

Has there been any fix to this issue? Customer is experiencing the same behavior.

Thanks!

tylergbass’s picture

Version: 7.x-3.3 » 7.x-3.x-dev
Status: Active » Needs review
StatusFileSize
new1.3 KB

Worked with @cmanalansan to create this patch.

mizage@gmail.com’s picture

Thanks @tylergbass and @cmanalansan! The patched worked for me.

jaydarnell’s picture

I can also confirm that patch #4 works for version 7.x-3.4

granticusiv’s picture

I can also confirm patch #4 work width 7.x-3.4

anon’s picture

Whats the reason for this?

linkit_profile_load_by_format reutrns the first profile that has a given text format.

To just move the tests isset($profile->data['imce']) && $profile->data['imce'] == 1 may fix the problem, but not the root problem.

Also, I can't see that ckeditor has to do with this, please explain.

leonblueeyes’s picture

Thanks tylergbass and cmanalansan! The patch #4 worked for me.

danbarron’s picture

I am having the same problem even when CKEditor is present, and the problem seems to be in the same bit of code.

I can fix it by selecting all text formats to use the LinkIt profile, but this is clearly an accidental work around that will break if we change our set up.

I'm not 100% sure what this code is meant to be doing, but it does seem that something is amiss in the logic. The $profile variable is used to determine whether or not to add the button, but that variable will be FALSE if the last text format checked in the foreach is not using the profile.

anon’s picture

I can confirm this issue now. I will have a look at it. First look at the provided patch seems to solves this.

zipymonkey’s picture

Path #4 works for me as well.

  • anon committed 2843e5b on 7.x-3.x authored by tylergbass
    Issue #2471555 by tylergbass: No "Open File Browser" button on content...
anon’s picture

Status: Needs review » Fixed

Thanks for the patch.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.