Feeling Lucky
NPM Resource Changed
?
?
[#32825] - [4.0] Fix Media Manager cant insert in content unless logged in as admin
- Fixed in Code Base
- 25 Mar 2021
- Medium
- Build: 4.0-dev
- # 32825
- Diff
- rdeutz:fix_32734
User tests: Successful: Unsuccessful:
Pull Request for Issue #32734 .
Summary of Changes
Fix the base path in js
Testing Instructions
- Ensure you are not logged into site administrator
- Login to front-end of site as super user
- Edit any article and click Image in the CMS Content dropdown to bring up Media Manager
- Select and attempt to insert any image
- Appy patch
- npm run build:js
Do the same steps, bonus points when you try the same in the backend.
Actual result BEFORE applying this Pull Request
Image is not inserted
Expected result AFTER applying this Pull Request
Image is inserted
Documentation Changes Required
None
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | JavaScript Repository NPM Change |
alikon
- comment
- 24 Mar 2021
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/32825.
| Status | Pending | ⇒ | Ready to Commit |
alikon
- comment
- 24 Mar 2021
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/32825.
SharkyKZ
- comment
- 24 Mar 2021
I have tested this item
Broken in backend.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/32825.
| Status | Ready to Commit | ⇒ | Pending |
alikon
- comment
- 24 Mar 2021
back to pending
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/32825.
infograf768
- comment
- 24 Mar 2021
Please modify
const apiBaseUrl = `${Joomla.getOptions('system.paths').rootFull}index.php?option=com_media&format=json`;
to
`const apiBaseUrl = `${Joomla.getOptions('system.paths').baseFull}index.php?option=com_media&format=json`;`
rdeutz
- comment
- 24 Mar 2021
Broken in backend.
Tested this also, how do you make it fail?
| Labels |
Added:
NPM Resource Changed
?
|
||
rdeutz
- comment
- 24 Mar 2021
fixed, please re-test. Sorry for not looking into it properly
infograf768
- comment
- 24 Mar 2021
infograf768
- comment
- 24 Mar 2021
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/32825.
dgrammatiko
- comment
- 24 Mar 2021
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/32825.
| Status | Pending | ⇒ | Ready to Commit |
infograf768
- comment
- 25 Mar 2021
rtc
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/32825.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-03-25 08:42:50 |
| Closed_By | ⇒ | drmenzelit | |
| Labels |
Added:
?
|
||
drmenzelit
- comment
- 25 Mar 2021
Thanks!
I have tested this item✅ successfully on 2237de9
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/32825.