#9908 closed Bug (invalid)
Link dialog code is outdated after JS links has been forbidden
| Reported by: | Piotrek Koszuliński | Owned by: | |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | General | Version: | 4.0 |
| Keywords: | Cc: |
Description
Since #9289 JS links aren't allowed, but I found some evidences that that patch wasn't complete. E.g.:
urlOnChangeTestOther = /^((javascript:)|[#\/\.\?])/i;
Few lines later part of this pattern is invalidated:
if ( (/javascript\:/).test( this.getValue() ) ) {
alert( commonLang.invalidValue );
return false;
}

Taking into consideration comment 10 from #9661 and comment 8 from #9637, I'm closing this issue as invalid.