@dgt41 I think this is one for you

The button is the one on the modals footer or the one inside the modal?

The button inside the modal from media/images that calls the image insertion routine into the field.

I cannot replicate this! @anibalsanchez is your browser's cache cleared? Which extension failed?

If Isis view override is removed, then the image selection works OK.

administrator/templates/isis/html/com_media/images/default.php

Yes i confirm this issue,

isis backend template has an template override file ...
administrator/templates/isis/html/com_media/images/default.php

• which uses modal that is bootstrap based (if i remember correctly)

but com_media component has:
administrator/components/com_media/views/images/tmpl/default.php

• which uses modal that is mootools based (if i remember correctly)

I have already made a workaround in my extension to load a copy of the component template file ...

JPATH_BASE.'/templates/'.$app->getTemplate().'/html/com_media/images/my_extensionname.php

but there must be several extensions with custom code,

• that may be effected by this, they will not be able to select media files

[EDIT]
by using creating file: my_extensionname.php i am not breaking anything in isis template or in any other extension

Hmmm @ggppdk that doesn't make much sense:
The field SHOULD work out of the box. The current ISIS override DOESN'T need the onclick call (when a fieldId is passed) as that is done in the mediafield.js.
can you share the xml definition for such a failing field?

@dgt41
you are right but ...
it is not an XML file not JForm !

I am calling inside media manager inside custom PHP code ...
similar as @anibalsanchez example
and i have seen 1 or 2 other extensions doing this

[EDIT]
which extensions are propably not updated yet

OK, now it makes sense. I will change the code to incorporate such cases

@anibalsanchez @ggppdk @wilsonge This is one of those chicken-egg situation: providing the old JinsertImage() breaks the repeatable field

OK How exactly are you guys using this field. I know you say PHP - but I need more information to make a decision about b/c. Like honestly my gut is that JFormFields are designed to work through JForm - if you are using things outside of this then we don't offer b/c on that. But willing to study code if you guys feel you are "using" the API correctly :)

Ok, this is not a complete solution, but maybe duplicate

administrator/components/com_media/views/images/tmpl/default.php
as:
administrator/components/com_media/views/images/tmpl/default_legacy.php

And require that all extensions using custom code to call media manager will need to add to their URL:
&layout=default_legacy

But i think that there is 1 more problem here:

• which is bad too: adding an override that uses bootstrap modal in isis template and thus behaves differently than the template file of component, makes little sense and creates another problem

It means that if any site uses a different backend template that does not have an override like isis
then these will break ?

• media field
• and repeatable field

[EDIT]
i mean if template was to be changed like this then the change should have been inside the component and not via a template override

@wilsonge
Because i felt i was not using the API correctly

• that is why i did not submit an issue here

I am using JForm,

• but the field is not extending JFormField

About my code:

// We will use the mootools based media manager
JHtml::_('behavior.framework', true);

// Load the modal behavior script.
JHtml::_('behavior.modal'/*, '.fc_image_field_mm_modal'*/);

// Include media field JS, detecting different version of Joomla
if( file_exists($path = JPATH_ROOT.'/media/media/js/mediafield-mootools.min.js') ) $media_js = 'media/mediafield-mootools.min.js';
else if( file_exists($path = JPATH_ROOT.'/media/media/js/mediafield.min.js') ) $media_js = 'media/mediafield.min.js';
else $media_js = 'media/mediafield.js';

JHtml::_('script', $media_js, $mootools_framework = true, $media_folder_relative_path = true, false, false, true);

// Tooltips for image path and image popup preview
JHtml::_('behavior.tooltip', '.hasTipImgpath', array('onShow' => 'jMediaRefreshImgpathTip'));
JHtml::_('behavior.tooltip', '.hasTipPreview', array('onShow' => 'jMediaRefreshPreviewTip'));
...
...
...
$mm_link = 'index.php?option=com_media&view=images&layout=default_fc&tmpl=component&asset=com_flexicontent&author=&fieldid=\'+mm_id+\'&folder=';
$select_existing = '
<div class="'.$input_grp_class.'">
    <div class="media-preview '.$add_on_class.' ">
        '.JHtml::tooltip($tooltip, $tooltip_options).'
    </div>
    <input type="text" name="'.$fieldname_n.'[existingname]" id="'.$mm_id.'" value="'.htmlspecialchars($img_path, ENT_COMPAT, 'UTF-8').'" readonly="readonly"
        class="existingname input-xxlarge field-media-input hasTipImgpath"  title="'.htmlspecialchars('<span id="TipImgpath"></span>', ENT_COMPAT, 'UTF-8').'" data-basepath="'.JUri::root().'"
    />
    <a class="fc_image_field_mm_modal btn '.$tooltip_class.'" title="'.JText::_('FLEXI_SELECT_IMAGE').'" onclick="var mm_id=jQuery(this).parent().find(\'.existingname\').attr(\'id\'); currElement'.$field->id.'=mm_id; SqueezeBox.open(\''.$mm_link.'\', {size:{x: ((screen.width-120) > 1360 ? 1360 : (screen.width-120)), y: ((screen.height-220) > 800 ? 800 : (screen.height-220))}, handler: \'iframe\', onClose: function() { incrementValCnt'.$field->id.'(); } });  return false;">
        '.JText::_('FLEXI_SELECT').'
    </a>
    <a class="btn '.$tooltip_class.'" href="javascript:;" title="'.JText::_('FLEXI_CLEAR').'" onclick="var mm_id=jQuery(this).parent().find(\'.existingname\').attr(\'id\');  clearField'.$field->id.'(this); jInsertFieldValue(\'\', mm_id); return false;" >
        <i class="icon-remove"></i>
    </a>
</div>
';

In my case, the original button from media/images fixes the issue.

@anibalsanchez

With your changes in the insert button

• it works in my case too without using a template override

And now i remember that indeed that was the problem, the changes in the insert button, not the use of bootstrap modal (sorry i recalled wrong)

• in my case i use Squeezebox modal and it works with the changes that you made in the insert button

So this seems to be backwards compatible.

Does your change break the repeatable field ?

Closed as we have a PR for testing #9466

Hi,

I am also facing the same problem with JA Platon theme. It was working fine with Joomla 3.4. The issue cause after Joomla Version update. Kindly help me.

@nareshvida you should check with the maker if the template for an update