Project

General

Profile

Bug #3261

chamilo 1.8.8 alfa edit svg with svg-edit and edit jpg png with pixlr don't work

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

Status:
Bug resolved
Priority:
High
Category:
-
Target version:
Start date:
13/04/2011
Due date:
% Done:

80%

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

Description

Julio, si no me equivoco creo que el error está en el commit

14116 (59f9de54ebd9) Fixing documents tool move functionality, replacing curdirpath in document list,

más concretamente creo que el problema está en que se cambió $curdirpath = urlencode($document_data['path']); por $curdirpath = dirname($document_data['path']);

esto produce que donde se devolvía / ahora se devuelva \

El resultado es que varias funcionalidades de la herramienta documentos han dejado de funcionar pues reciben una url incorrecta: por ejemplo: edición de imágenes jpg, png mediante el editor pixlr, también ha dejado de funcionar la edición de imágenes svg mediante svg edit...

Como hiciste los cambios no me atrevo a cambiar este valor pues estropearía tu trabajo, así que si estoy en lo cierto creo que eres el más indicado para arreglar esto.


Files

bug_documents.png (58.8 KB) bug_documents.png Juan Carlos Raña Trabado, 14/04/2011 21:31
Screenshot.png (40.1 KB) Screenshot.png Julio Montoya, 15/04/2011 11:12

Associated revisions

Revision e4de11d3 (diff)
Added by Julio Montoya over 8 years ago

More fixes in the document tool see #3261

History

#1

Updated by Julio Montoya over 8 years ago

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

a ver ...

#2

Updated by Julio Montoya over 8 years ago

Hola Juan Carlos,

Bueno en principio he estado tratado de sacar los $curdirpath para reemplazarlo por el document id, luego, gracias a este document id puedo tener el path

seguro que se me olvidó algo, ahora lo corrigo

#4

Updated by Julio Montoya over 8 years ago

  • Status changed from Assigned to Needs more info
  • Assignee deleted (Julio Montoya)
  • % Done changed from 50 to 60
#5

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

Sigue sin funcionar. Insisto, el problema está en la barra invertida \ que se envía en la url cuando se pulsa editar pixlr o svg desde el icono de la tabla de la herramienta documentos. Puedes comprobarlo con tan sólo ver la url que desde documentos se le envía a editar svg o a editar pixlr. De todas formas incluyo un clip.

Es esto lo que causa el error y esto es así porque se cambió $curdirpath = urlencode($document_data['path']); por $curdirpath = dirname($document_data['path']); que hace que se cambie / por \ Se puede ver en línea 359 de document.inc.php y en la línea 73 de document.php. Si esto no puede volver a estar como estaba habría que hacer que la url que nos lleve desde documents hasta edit svg o edit pixlr sea la correcta.

#6

Updated by Julio Montoya over 8 years ago

MMmmm esto es raro pues ahora cuando paso el mouse solo tengo esto:

Seguro que usamos la misma versión?

#7

Updated by Julio Montoya over 8 years ago

Una pregunta Juan Carlos,
estás usando un XAMPP en windows?

#8

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

Julio Montoya wrote:

Una pregunta Juan Carlos,
estás usando un XAMPP en windows?

Buen olfato, efectivamente así es

#9

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

ahora entiendo la razón por la que a tí sí te funcionaba. Pero de todas formas no debemos mezclar sin más direcciones reales en disco y direcciones web

#10

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

En el caso de convertir un documento en una plantilla o viceversa también aparece esta barra invertida (no se envía una url correcta) aunque la herramienta parece no haber dejado de funcionar.

#11

Updated by Yannick Warnier over 8 years ago

  • Target version changed from 1.8.8 beta to 1.8.8 stable

I think it is acceptable to move this for stable, considering this is an optional feature.

#13

Updated by Yannick Warnier about 8 years ago

Juan Carlos Raña wrote:

Esto soluciona el tema por el momento
http://code.google.com/p/chamilo/source/detail?r=16fc4afc4803f43ddee74ab70306339259d4f244&repo=classic

Apparently putting "/" is valid even on Windows.
http://alanhogan.com/tips/php/directory-separator-not-necessary

#14

Updated by Yannick Warnier about 8 years ago

  • Target version changed from 1.8.8 stable to 1.8.8.4

Moving to 1.8.8.1 for a nicer solution

#15

Updated by Yannick Warnier about 8 years ago

  • Target version changed from 1.8.8.4 to 1.8.8.2
#16

Updated by Yannick Warnier about 8 years ago

  • Target version changed from 1.8.8.2 to 1.8.8.4

1.8.8.1 renamed 1.8.8.3

#17

Updated by Yannick Warnier almost 8 years ago

  • Target version changed from 1.8.8.4 to 1.9 Stable
#18

Updated by Yannick Warnier about 7 years ago

  • Target version changed from 1.9 Stable to 1.9 Beta
#19

Updated by Michela Mosquera about 7 years ago

  • Target version changed from 1.9 Beta to 1.9.2
#20

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

  • Status changed from Needs more info to Bug resolved
  • Assignee set to Juan Carlos Raña Trabado

esto ya lo arreglé hace tiempo pero no se cerró la tarea

Also available in: Atom PDF