[#23104] - Update IFW to work with CE, remove JSONP requirement
- Fixed in Code Base
- 22 Nov 2018
- Medium
- Build: 4.0-dev
- # 23104
- Diff
- mbabker:unbreak-ifw-js
User tests: Successful: Unsuccessful:
Pull Request for Issue #23039
Summary of Changes
Fixes the broken install from web tab on the (what appears to be presently untested/unreviewed) extension manager's install view.
Notable changes:
- Removes custom loading element in favor of the
Joomla.loadingLayerscript (as specified in #19429) - Removes the requirement for JSONP in the calls to the remote server as CORS has now been implemented
- Refactors the script initialization to remove the jQuery and Bootstrap dependencies and work with the page's CEs
Testing Instructions
On an environment where you can run npm, apply this patch and recompile the UI elements, then go to the extension manager and ensure the install from web UI goes further than the "Loading..." initial display.
Expected result
IFW actually works
Actual result
IFW has been broken for approximately 3 months (since the conversion of the install view of the extension manager to use CEs)
Documentation Changes Required
Doubtful any documentation change can fix a workflow issue
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | JavaScript Repository |
| Labels |
Added:
?
|
||
I have tested this item
Loading screen only before patch. IFW loads after applying patch.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/23104.
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2018-11-22 11:12:52 |
| Closed_By | ⇒ | wilsonge |
Thanks!
I have tested this item✅ successfully on 328e570
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/23104.