Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#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;
}

Change History (1)

comment:1 Changed 7 years ago by Jakub Ś

Resolution: invalid
Status: newclosed

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

Last edited 7 years ago by Jakub Ś (previous) (diff)
Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy