Opened 10 years ago

Closed 10 years ago

#4340 closed New Feature (fixed)

Implement Email protection

Reported by: Alfonso Martínez de Lizarrondo Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.1
Component: Core : Output Data Version: SVN (CKEditor) - OLD
Keywords: Confirmed Review+ Cc:

Description

Port #2220 to CKEditor (or something better if it's even possible :) )

Attachments (4)

4340.patch (7.5 KB) - added by Garry Yao 10 years ago.
4340_2.patch (8.6 KB) - added by Garry Yao 10 years ago.
4340_3.patch (9.9 KB) - added by Garry Yao 10 years ago.
4340_4.patch (9.9 KB) - added by Garry Yao 10 years ago.

Download all attachments as: .zip

Change History (14)

Changed 10 years ago by Garry Yao

Attachment: 4340.patch added

comment:1 Changed 10 years ago by Garry Yao

Component: GeneralCore : Output Data
Keywords: Confirmed Review? added
Owner: set to Garry Yao
Status: newassigned
Version: 3.0SVN (CKEditor)

comment:3 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Review- added; Review? removed

Having it implemented as filters definitely have its benefits, like automatic protection even when the user is not aware about. But, the drawback is that all the code goes into the core.

We must keep an eye in the file size also, so I would propose having this as a dialog feature, much like V2. If we understand that having this enabled as a filter is critical we'll then re-implement it later. We must consider the contents products with the editor for now.

Changed 10 years ago by Garry Yao

Attachment: 4340_2.patch added

comment:4 Changed 10 years ago by Garry Yao

Keywords: Review? added; Review- removed

Now the protection only affect dialog operation.
Ticket Test updated.

comment:5 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Review- added; Review? removed
  • This quite back TC is broken after the patch:
  1. With default config, load this HTML:
<p><a href="http://www.example.com">Test</a></p>
  1. Right click into "Test" and open the link properties dialog.

The href value will not be loaded.


  • The config should be named emailProtection, not mailProtection.
  • The config documentation is wrong. The @name tag cannot be the first thing there. The very first thing must be the description. I think jsdoc will not work otherwise.
  • Again in the documentation, the type is "String", but the default value is "false" (???). I think you wanted to say empty string there.
  • And finally, there is no ASC2... but ASCII :).
  • I have used the following setting:
config.mailProtection = 'abcd(NAME,SUBJECT,BODY,DOMAIN)';

... having these results by filling the e-mail address only with "test@…":

<p><a href="javascript:abcd('test','example.com')">Test</a></p>

... expected:

<p><a href="javascript:abcd('test','','','example.com')">Test</a></p>

comment:6 Changed 10 years ago by Garry Yao

Keywords: Review? added; Review- removed

Changed 10 years ago by Garry Yao

Attachment: 4340_3.patch added

comment:7 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Review- added; Review? removed

We're almost there. I've just found one (quite hidden) issue:

  1. Load the following HTML:
<p><a name="Target"></a>This is a <a href="#Target">sample</a>.</p>
  1. Right-click the "sample" link and open the link dialog to edit it.

The dialog doesn't load the Anchor properly. The "Link Type" field should be "Link to anchor in the text".

Changed 10 years ago by Garry Yao

Attachment: 4340_4.patch added

comment:8 Changed 10 years ago by Garry Yao

Keywords: Review? added; Review- removed

comment:9 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:10 Changed 10 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Fixed with [4608] at 3.1.x branch.

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy