Skip to content

Comments

[4.0] Fix media URLs when SEF suffix is enabled#30688

Merged
HLeithner merged 1 commit intojoomla:4.0-devfrom
SharkyKZ:j4/fix/media-urls
Sep 19, 2020
Merged

[4.0] Fix media URLs when SEF suffix is enabled#30688
HLeithner merged 1 commit intojoomla:4.0-devfrom
SharkyKZ:j4/fix/media-urls

Conversation

@SharkyKZ
Copy link
Contributor

Fixes #30665.

Summary of Changes

Use non-SEF URLs for media paths.

Testing Instructions

Enable Add Suffix to URL option in Global Configuration.
Edit an article in frontend.
Click CMS Content -> Image button.

Actual result BEFORE applying this Pull Request

Images shown in modal.

Expected result AFTER applying this Pull Request

Images not shown in modal.

Documentation Changes Required

No.

@ghost
Copy link

ghost commented Sep 19, 2020

I have tested this item ✅ successfully on 7e859f4


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

@softforge
Copy link
Contributor

I have tested this item ✅ successfully on 7e859f4

Does what it said on the tin


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

@richard67
Copy link
Member

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Sep 19, 2020
@HLeithner HLeithner merged commit 23292c6 into joomla:4.0-dev Sep 19, 2020
@HLeithner
Copy link
Member

Thanks

@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Sep 19, 2020
@HLeithner HLeithner added this to the Joomla 4.0 milestone Sep 19, 2020
sakiss pushed a commit to sakiss/joomla-cms that referenced this pull request Oct 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants