Project

General

Profile

Feature #7961

Implement wiris math editor for ckeditor

Added by José Loguercio almost 4 years ago. Updated over 3 years ago.

Status:
Feature implemented
Priority:
Normal
Category:
Plugins
Target version:
Start date:
13/11/2015
Due date:
% Done:

100%

Estimated time:
Complexity:
Normal
SCRUM pts - complexity:
?

Associated revisions

Revision beb11dc9 (diff)
Added by José Loguercio almost 4 years ago

Added Wiris Editor for CKEditor - Refs #7961

Revision 350d6f4f
Added by Yannick Warnier almost 4 years ago

Merge pull request #934 from jloguercio/10450

Added Wiris Editor for CKEditor - Refs #7961

History

#1

Updated by José Loguercio almost 4 years ago

  • Status changed from New to Needs testing
  • Assignee changed from José Loguercio to Yannick Warnier
  • % Done changed from 0 to 90

PR : https://github.com/chamilo/chamilo-lms/pull/934

Needs to update the Settings Description for plugin activation, change fckeditor to ckeditor and change the current download link for this : http://www.wiris.com/es/plugins3/ckeditor/download

#3

Updated by Yannick Warnier almost 4 years ago

  • Status changed from Feature implemented to Assigned
  • % Done changed from 100 to 90

There's still an issue with loading configurationjs.php, which is loaded through web/assets/ when it should be loaded through main/inc/lib/javascript/.
José is working on that.

#4

Updated by José Loguercio almost 4 years ago

Ok, Al parecer existe un bug en el archivo plugin.js del ckeditor_wiris, en las lineas 6, 36 y 37.

El problema está en que si se instala CKEDITOR por composer y los plugins fuera de la carpeta base del ckeditor, al usar "CKEDITOR.basePath" obtenemos la url absoluta de la instalación del ckeditor, más no la ruta relativa.
En este caso los plugins estan instalados en http://local/main/inc/lib/javascript/ckeditor/plugins y la ruta del ckeditor instalada por composer es : http://local/web/assets/ckeditor/
Otros Third Party Plugins que tenemos instalados usan "CKEDITOR.getUrl(this.path)" para obtener la ruta correcta de los plugins,
El problema es que esta solución solo funciona si es declarado dentro del init del plugin, a menos que exista alguna otra funcion para llamar a la ruta relativa antes de la init
Para este caso una solución parcial es cambiar las rutas de las 3 lineas a rutas relativas en plugin.js dentro de ckeditor_wiris, pero esto implicaría que el plugin esté específico a Chamilo:
Linea 6 : var _wrs_conf_path = CKEDITOR.basePath + '/plugins/ckeditor_wiris';
Linea 36 : var _wrs_int_editorIcon = CKEDITOR.basePath + '/plugins/ckeditor_wiris/core/icons/formula.gif';
Linea 37 : var _wrs_int_CASIcon = CKEDITOR.basePath + '/plugins/ckeditor_wiris/core/icons/cas.gif';
Por
Linea 6 : var _wrs_conf_path = '/main/inc/lib/javascript/ckeditor/plugins/ckeditor_wiris';
Linea 36 : var _wrs_int_editorIcon = '/main/inc/lib/javascript/ckeditor/plugins/ckeditor_wiris/core/icons/formula.gif';
Linea 37 : var _wrs_int_CASIcon = '/main/inc/lib/javascript/ckeditor/plugins/ckeditor_wiris/core/icons/cas.gif';

Otra (y mejor) solución sería incluir estas declaraciones dentro del init, y usar CKEDITOR.getUrl(this.path)

#5

Updated by Yannick Warnier almost 4 years ago

  • % Done changed from 90 to 50

Correo enviado al equipo de Wiris para que nos comenten.

#6

Updated by Yannick Warnier almost 4 years ago

  • Target version changed from 1.10.2 to 2.0

Wiris ha aceptado nuestros comentarios e indica que un parche será publicado como parte de su versión de enero 2016.

Muevo esta tarea a la 2.0, pero en realidad es simplemente para tenerla presente sin que bloquee la publicación de la 1.10.2

#7

Updated by José Loguercio over 3 years ago

  • Status changed from Assigned to Feature implemented
  • Assignee changed from José Loguercio to Yannick Warnier
  • % Done changed from 50 to 100

Wiris corrigió el inconveniente que, actualmente, en su última versión funciona de maravilla, he actualizado el Readme.txt en la carpeta donde va el plugin con información de como configurarlo correctamente

PR : https://github.com/chamilo/chamilo-lms/pull/1197

Saludos

Also available in: Atom PDF