Opened 5 years ago

Last modified 5 years ago

#11690 confirmed Bug

Placeholder with forbidden characters

Reported by: Piotr Jasiun Owned by:
Priority: Normal Milestone:
Component: General Version: 4.3
Keywords: Cc:

Description

  1. Open placeholder sample (plugins/placeholder/samples/placeholder.html).
  2. In the source mode enter:
    <p>This is a [[sample ]placeholder]].</p>
    
  3. Switch back to the WYSIWYG mode.

Result: there is no placeholder.

  1. In the source mode enter:
    <p>This is a [[sample >placeholder]].</p>
    
  2. Switch back to the WYSIWYG mode.

Result: there is a placeholder.

  1. Open placeholder dialog.
  2. Press "Ok".

Result: Error message:

The placeholder can not be empty and can not contain any of following characters: [, ], <, >


On the one hand it is not possible to upcast placeholder with ']' character, because it is forbidden, but on the other we can do it with '>'.

In my opinion it should not be possible to upcast placeholder with forbidden character but also it would be good if user would be informed that the placeholder contains forbidden character.

Change History (2)

comment:1 Changed 5 years ago by Jakub Ś

Status: newconfirmed

comment:2 Changed 5 years ago by Piotrek Koszuliński

There are two problems actually. Placeholder does not encode <> characters when downcasting. I'm not sure if that shouldn't be done by htmlParser.text (constructor or writer).

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