Project

General

Profile

Bug #3927

can't edit and save a svg image

Added by Juan Carlos Raña Trabado almost 8 years ago. Updated over 7 years ago.

Status:
Feature implemented
Priority:
Normal
Assignee:
Category:
-
Target version:
Start date:
25/09/2011
Due date:
% Done:

100%

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

Files

svg.png (12.6 KB) svg.png Michela Mosquera, 20/05/2012 05:05
dibujo-svg.png (88 KB) dibujo-svg.png Michela Mosquera, 20/05/2012 05:05

Associated revisions

Revision f5b9c8e4 (diff)
Added by Julio Montoya almost 8 years ago

Fixing Image Library when using the "Draw" ( document/create_draw.php) option in documents see #3927

History

#1

Updated by Juan Carlos Raña Trabado almost 8 years ago

Es descorazonador que tantas cosas que funcionaban perfectamente antes y tanto trabajo costó que así fuera hayan dejado de funcionar. Es ir atrás una y otra vez. Cuando se realiza algún cambio se debe ser más exigente en las repercusiones que puede tener en el resto. Es bonito introducir código pero mucho más que todo funcione.

#2

Updated by Yannick Warnier almost 8 years ago

No te descorazones demasiado Juan Carlos, lo vamos a corregir. Es normal (aunque dolorozo para sus desarrolladores) que ocurran cosas así por mientras no tengamos un sistema de gestión de pruebas automatizadas más completo y tengamos que hacer merges grandes y cambios de permisos, pero trataremos de hacerlos con un poco más cuidado. Debido a la escsez de recursos a veces es difícil de tomar el tiempo necesario en cada momento, pero como viste hicimos mucho para estabilizarlo todo antes de la 1.8.8.4. Personalmente me siento mucho más tranquilo que en la época de Patrick que rompía algo cada dos commits.

De paso, recibí el siguiente comentario de Adrian Jones, desarrollador de SVGEdit, que no he tenido tiempo de analizar todavía:

This is in response to a message posted on the SVGedit wiki page.

Just took a look at your site - seems like a great integration of SVG-edit, but thought you should know that two of the image libraries you have added result in PHP foreach errors and the IAN library won't actually load any of the images - seems to be an iframe issue, because when I load up your editor directly it works fine, ie http://chamilodev.beeznest.com/main/inc/lib/svg-edit/svg-editor.php?url=../../../../courses/AAA/document/image.svg&lang=nl

#3

Updated by Juan Carlos Raña Trabado almost 8 years ago

Es cierto, aunque me parece que el error es anterior y creo no está generado por los últimos cambios sino por la introducción de un id para identificar a los directorios y documentos en lugar del path (tendría que comprobar si es así). En el curso actual, en una nueva instalación he necesitado usar estas herramientas y no he podido, lo peor es que ahora no tengo tiempo para arreglarlas. En el editor de imágenes he hecho algún arreglo, que tendría que pulir aún más, aunque por lo menos ahora funciona.
Respecto al mensaje de Adrian Jones sería bueno saber si hizo la prueba con el sistema con los bugs introducidos o no. Supongo que los errores de foreach se refieren a los directorios de Chamilo. Originariamente todo marchaba bien y el editor podía no sólo crear y editar imágenes svg y png desde los grupos (sólo los miembros de ese grupo), desde la galería principal del curso y desde una galería personal que se encuentra en el user shared folder, en cuarto lugar en el caso de la IAN library no lo puedo asegurarlo con tanta rotundidad pues no hice pruebas exhaustivas al ser una galería externa.

Nadie está libre de errores, por ejemplo el error que te atribuyes en http://code.google.com/p/chamilo/source/detail?r=5104459cf5d36c1e91d1289f443063e82ad0aa38&repo=classic realmente no es tuyo sino mío, pero no tiene que ver con las herramientas que estamos hablando.

En fin, que a cada uno lo suyo

#4

Updated by Yannick Warnier almost 8 years ago

Bueno, es posible que sea el ID, sí, porque el modo con la ruta del documento nos traía problemas también de otro lado con clientes con documentos migrados desde Dokeos 1.8.4 (por lo menos con los IDs evitas una serie de problemas al filtrar las rutas contra inyecciones de ficheros).
No sabía que usabas versiones de desarrollo en producción para cursos. OK, vamos a tratar de hacer algo este fin de semana al respecto, porque es cierto que las demos no se ven muy bien con una falla ahí.

#5

Updated by Juan Carlos Raña Trabado almost 8 years ago

El error se mantiene desde hace bastante tiempo y está presente en la versión supuestamente estable de Chamilo 1.8.8.4

#6

Updated by Julio Montoya almost 8 years ago

  • Status changed from New to Assigned
  • Assignee set to Julio Montoya
#7

Updated by Julio Montoya almost 8 years ago

  • % Done changed from 0 to 10

Acabo de hacer unas prueba en chamilodev en Documentos

1. Editar y guardar un archivo png (ya existente en documentos). Funciona
2. Crear y guardar un nuevo archivo png (Link "Photo retouching" document/create_paint.php ) Funciona
3. Editar y guardar un archivo svg (ya subido en los documentos). Funciona
4. Crear y guardar un archivo svg (create_draw.php) Funciona
5. Dentro de create_draw.php Hice click en "Main Menu" > Image Library -> Local gallery Funciona
6. Dentro de create_draw.php Hice click en "Main Menu" > Image Library -> Group gallery - Aparece un mensaje:
"Only accessible from your group"
7. Dentro de create_draw.php Hice click en "Main Menu" > Image Library -> My files gallery No funciona
error

Select a picture
    Warning: Invalid argument supplied for foreach() in /var/www/chamilodev.beeznest.com/www/main/inc/lib/svg-edit/extensions/imagelib/users.php on line 44 Call Stack: 0.0012 681488 1. {main}() /var/www/chamilodev.beeznest.com/www/main/inc/lib/svg-edit/extensions/imagelib/users.php:0 

No tenía idea de que existía esta opción de "Main Menu" > Image Library tal vez por eso no lo probé cuando agregué el tema de los ids.

Voy a ver el problema para 6 y 7

#8

Updated by Juan Carlos Raña Trabado almost 8 years ago

La implementación de svg-edit la amplié para poder usar galerías png y svg dentro del curso (creo que no hay otra implementación conocida que lo haga), además de las externas. Es realmente útil pues permite utilizar con más comodidad el editor y reaprovechar el trabajo antes realizado que puede ser a su vez parte de un nuevo dibujo.
Ahora no puedo echar una mano en esto pero te indico un par de ideas:

-Julio el punto 6 no tiene por qué estar mal. Si accedemos desde cualquier sitio de un curso que no sea desde la herramienta grupos no se debe acceder a este directorio porque el grupo puede ser privado. Se podría intentar comprobar la pertenencia al grupo desde cualquier sitio fuera de la herramienta grupos, pero esta funcionalidad no la llegué a implementar. Por ahora, bastaría con que cada miembro de un grupo pudiera acceder a las herramienta desde SU grupo.

-Por el contrario el error del punto 7 si es un bug. Desde cualquier sitio del curso debería mostrar los documentos del usuario que estén en su carpeta de shared files

#9

Updated by Julio Montoya almost 8 years ago

  • Status changed from Assigned to Needs more info
  • % Done changed from 10 to 50

Holas,

para el punto 6 solo decía que aparecía un mensaje, no dije que no funciona. Por lo que funciona bien ahora que lo he probado (Agregando svgs y entrando al grupo).

para el punto 7 he agregado un fix, ahora ya funciona

#10

Updated by Julio Montoya almost 8 years ago

  • Assignee deleted (Julio Montoya)
#11

Updated by Yannick Warnier over 7 years ago

  • Target version set to 1.9 Beta
#12

Updated by Michela Mosquera over 7 years ago

Revisado importar (si) , editar (si ) y guardar (si ) en IE8 y FF12

Also available in: Atom PDF