stylesCombo_stylesSet doesn't work for full url
|Reported by:||alfonsoml||Owned by:||fredck|
|Component:||Core : Styles||Version:||3.0|
The code tries to split the configuration entry based on ':' and get a maximum of 2 splitted blocks, but the way that split() works is just ignoring the rest, not putting everything in the second value.
config.stylesCombo_stylesSet = 'default:http://ckeditor.com/apps/ckeditor/3.0/plugins/stylescombo/styles/default.js?t=97KD'
means that it will try to load a file from the "http" url
This could have been avoided easily choosing any other separator as long as it isn't a valid part of a url.
The patch provides a solution without changing the separator, because although no one is able to use the full url setting, some people might have used the name:relative url syntax.
Change History (6)
Changed 7 years ago by alfonsoml
comment:1 Changed 7 years ago by fredck
- Keywords Review- added; Review? removed
- Owner changed from alfonsoml to fredck
- Status changed from new to assigned