pluxopolis / ckeditor Goto Github PK

ckeditor 4.4.6.1
Dans la dernière version du plugin oEmbed il y a une erreur.
Sur une installation toute fraiche PluXml + CKEditor, le code généré par oEmbed sur les vidéos Youtube est erroné.

Cette erreur vient du fichier js jquery.oembed.min.js utilisé par ce plugin qui n'est pas à jour.

Le problème peut être résolu en remplaçant ce fichier par celui sur cette page
https://github.com/nfl/jquery-oembed-all.

Bug 1 : Problème avec la fonction protectEmails()

Lorsqu'un email ou une adresse SSH ou tout ce qui peut être confondu par un email est traité par la fonction protectEmails() à l'intérieur d'une balise pre celà affiche en clair le cryptage javascript du genre :

<script type="text/javascript">eval(unescape('%64%6f%63%75%6d%65%6e%74%2e%77%72%69%74%65%28%27%72%6f%6f%74%40%69%70%27%29'))

Au lieu de

[email protected]

Bug 2 : Désactiver les retours à la ligne

Dans le bloc "Ajouter du code" > Avancé de CKEditor, l'option "Désactiver les retours à la ligne." ne semblent pas être prise en compte.

Bug 3 : Les lignes sont mal comptées

Apparemment dés qu'une ligne est trop grande et fait un saut de ligne, ça compte une ligne de plus alors que ça ne devrait pas.
Du coup ça décale et à la fin il manque des lignes.
Par exemple :
Voici une phrase assez longue de test.
Une autre ligne.
Je devrais obtenir quelque chose du genre :

1. Voici une phrase assez
   longue de test
2. Une autre ligne

Hors, j'obtiens :

1. Voici une phrase assez
2. longue de test
   Une autre ligne

J'ai mis en ligne les bugs sur un site provisoire que je supprimerais par la suite pour bien comprendre :
https://tetsumaki.net/test123/

Bonjour,
Est-il possible d'utiliser le fichier de configuration native de l'éditeur - config.js ?
J'ai besoin d'ajouter des commandes au code:

> // interdire la suppression des tags s'il n'y a rien entre eux
> 	CKEDITOR.dtd.$removeEmpty.span = 0;
> 	CKEDITOR.dtd.$removeEmpty.i = 0;
> 	CKEDITOR.dtd.$removeEmpty.div = 0;
> 	CKEDITOR.dtd.$removeEmpty.em = 0;
> 	CKEDITOR.dtd.$removeEmpty.b = 0;
> // Autoriser les tags et les codes individuels
>         config.protectedSource.push(/<(style)[^>]*>.*<\/style>/ig); // Style tag
> 	config.protectedSource.push(/<(script)[^>]*>.*<\/script>/ig); // Script code
> 	config.protectedSource.push(/<\?[\s\S]*?\?>/g); // PHP code
> 	config.protectedSource.push(/<%[\s\S]*?%>/g); // ASP code
> 	config.protectedSource.push(/(]+>[\s|\S]*?<\/asp:[^\>]+>)|(]+\/>)/gi); // ASP.Net code

La réécriture des mails ne fonctionne pas lorsqu'il y a un lien mailto="..." avec un autre attribut, comme class ou title, dans la balise a.

Solution: remplacer dans ckeditor/ckeditor.php la ligne 166 (fonction protectEmails)

if(preg_match_all('/<a.+href=[\'"]mailto:([\._a-zA-Z0-9-@]+)((\?.*)?)[\'"]>([\._a-zA-Z0-9-@]+)<\/a>/i', $txt, $matches)) {

par

if(preg_match_all('/<a.+href=[\'"]mailto:([\._a-zA-Z0-9-@]+)((\?.*)?)[\'"][^>]*>([\._a-zA-Z0-9-@]+)<\/a>/i', $txt, $matches)) {

Dans l'expression régulière, ajouter [^>]* avant la fermeture de la balise a, pour laisser la place à d'éventuels autres attributs.