Tooltip for image field in article shows wrong path

[Harmageddon]

Steps to reproduce the issue

1. Create or edit an article in backend.
2. In the tab "Images and links", set two different images for intro and fulltext image.
3. Move your mouse over the fulltext image field.

Expected result

The path for the fulltext image should be displayed when hovering over the respective field.

Actual result

Instead, the tooltip always shows the path for the intro image.

Additional comments

Reported by user Cartho on joomla-bugs.de (original post), confirmed by @zero-24 and me. The JavaScript for this behavior comes from /libraries/cms/form/field/media.php (JFormFieldMedia class), but I wasn't able to fix it yet.

[dgrammatiko]

There is #5871 which corrects this problem but will not be available before 3.5, unless @phproberto decides otherwise.

PS: the problem is fixed #5871 only for the bootstrap layout not for the classic mootools js.

[Harmageddon]

@dgt41 Thank you, I didn't know this PR. I just applied it and the bug here is not really fixed, but the tooltip for the path is removed completely in Isis instead. I'm not sure if this is desireable. Also, as you wrote, the bug still exists in Hathor.

[dgrammatiko]

@Harmageddon I just tested it again here and it looks ok:

Also in article (multiple media fields):

But yes the mootols is still wrong

[Harmageddon]

@dgt41 I re-tested and commented on the PR.

[dgrammatiko]

@Harmageddon I had a quick look at the code and I have to say that tips don’t respect the current field because the code behind them doesn’t care about the field id, so the tip in a multiple fields page will always show the first field...

[brianteeman]

Note this is also true on ANY page that has two media fields as reported #7285

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/6799.}

[gunjanpatel]

So, do we have code fix for this issue? I guess Yes as far as I read the above conversation. But we have that for Joomla 3.5 series.
In summery, We have fix for this issue and it should be in "Confirmed" status. We should change it "pending" and close this issue in reference with the patch link?

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/6799.}

[Bakual]

The issue is basically coming from this line: https://github.com/joomla/joomla-cms/blob/staging/libraries/cms/form/field/media.php#L260
The ID is given there hardcoded into the JavaScript code. Since that code is only loaded once per pageload and is supposed to work for all instances of the field, this obviously fails and takes the path of the first media field.
The fix is to either determine the field ID in the JavaScript code (or pass it when called) or have that code added multiple times.
Either way should work and will fix it in staging.

[dgrammatiko]

@gunjanpatel It’s half solved, meaning the code for media layouts that used jquery+bootstrap is working fine but the old one with mootools still buggy. @Bakual is right with the comment above. Now someone needs to code that as well...

[luisorozoli]

@gunjanpatel @Bakual @dgt41 Do you know if this issue will be fix for the version 3.5?? Thanks.

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/6799.}

[dgrammatiko]

@luisorozoli @Harmageddon @gunjanpatel can you test #8100 ?

[gunjanpatel]

Thanks @dgt41 for fix. Tested and works fine.

[zero-24]

Closing as we have a now RTC PR here: #8100 Thanks.