Feeling Lucky
?
[#16762] - remove extra slash
- Fixed in Code Base
- 20 Jun 2017
- Medium
- Build: staging
- # 16762
- Diff
- brianteeman:jdrag
User tests: Successful: Unsuccessful:
JUri::root() returns the domain with a slash at the end so there is no need for the slash in the second part
To test
View the source of any page with th tinymce editor and your will see
{"jdragdrop":"http:\/\/127.0.0.1\/cms\/\/media\/editors\/tinymce\/js\/plugins\/dragdrop\/plugin.min.js"}
Note the double escaped slash \/\/
After the pr view the source
{"jdragdrop":"http:\/\/127.0.0.1\/cms\/media\/editors\/tinymce\/js\/plugins\/dragdrop\/plugin.min.js"}
Note the single escaped slash \/
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End Plugins |
franz-wohlkoenig
- comment
- 19 Jun 2017
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16762.
| Status | Pending | ⇒ | Ready to Commit |
franz-wohlkoenig
- comment
- 19 Jun 2017
RTC after two successful tests.
rdeutz
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-06-20 11:25:54 |
| Closed_By | ⇒ | rdeutz | |
| Labels |
Added:
?
|
||
I have tested this item✅ successfully on b34e889
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16762.