Feeling Lucky
?
[#7184] - Better external resource detection in JHtml::includeRelativeFiles()
- Closed
- 11 Jul 2015
- Medium
- Build: staging
- Easy Test
- # 7184
- Diff
- Fedik:jhtml-better-external-resourse-detection
User tests: Successful: Unsuccessful:
Better external resource detection in JHtml::includeRelativeFiles()
Test
Add in template index.php:
JHtml::_('script', '//maps.googleapis.com/maps/api/js?v=3&sensor=false');
Open site in browser and check the page source.
Actual result
JHtml does not add //maps.googleapis.com/maps/api/js to the page <head>
Expected
Somewhere between <head> tags you should be able to see:
<script src="//maps.googleapis.com/maps/api/js?v=3&sensor=false" type="text/javascript"></script>
zero-24
- | Category | ⇒ | Libraries |
| Status | New | ⇒ | Pending |
| Easy | No | ⇒ | Yes |
joomla-cms-bot
- | Labels |
Added:
?
|
||
| Status | Pending | ⇒ | Ready to Commit |
brianteeman
- comment
- 18 Jun 2015
Setting RTC thanks
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7184.
| Labels |
Added:
?
|
||
| Milestone |
Added: |
||
| Milestone |
Added: |
||
mbabker
- | Status | Ready to Commit | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-07-11 15:00:19 |
| Closed_By | ⇒ | mbabker |
| Labels |
Removed:
?
|
||
Confirmed issue and confirmed the PR fixes it
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7184.