Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[4.0] Image manipulation broken when inserting image in article #19461

Closed
infograf768 opened this issue Jan 26, 2018 · 4 comments
Closed

[4.0] Image manipulation broken when inserting image in article #19461

infograf768 opened this issue Jan 26, 2018 · 4 comments
Labels
No Code Attached Yet

Comments

@infograf768
Copy link
Member

infograf768 commented Jan 26, 2018 •
edited by joomla-cms-bot

Insert Image via the image xtd in TinyMCE

image_manip

As the feature is designed to be used from com_media (and does not really work there), it breaks in the xtd popup
@ghost
Copy link

ghost commented Jan 26, 2018

Issue confirmed.

This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19461.

@amone4
Copy link
Contributor

amone4 commented Mar 24, 2018

The following change in administrator>components>com_media>tmpl>media>default.php solved the bug

before
'editViewUrl' => JUri::root() . 'administrator/index.php?option=com_media&view=file' . $tmpl,
after:
'editViewUrl' => JUri::root() . 'administrator/index.php?option=com_media&view=file&tmpl=' . $tmpl,

But for some reason, the images won't get inserted into the article.
I have been experiencing this problem in 4.0-dev before I made this change though.
This particular issue has been addressed in #19922

This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19461.

@amone4 amone4 mentioned this issue Mar 24, 2018
Closed
@joomla-cms-bot
Copy link

Set to "closed" on behalf of @franz-wohlkoenig by The JTracker Application at issues.joomla.org/joomla-cms/19461

@ghost
Copy link

ghost commented Mar 25, 2018

closed as having Pull Request #19976

This was referenced Apr 2, 2018
Closed
Merged
laoneo pushed a commit that referenced this issue Apr 9, 2018
@amone4 @laoneo
[Re] [#19461] [4.0] Image manipulation broken when inserting image in…
01abf43 
… article (#20087)

* issue-19461

* Added check to see if tmpl is empty
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
No Code Attached Yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@infograf768 @joomla-cms-bot @amone4