Project

General

Profile

Bug #3933

Path problem when uploading media with fckeditor (both advanced and basic)

Added by Hubert Borderiou almost 8 years ago. Updated about 7 years ago.

Status:
Bug resolved
Priority:
High
Assignee:
Category:
-
Target version:
Start date:
27/09/2011
Due date:
% Done:

70%

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

Description

Hi,
I open a new ticket for this bug. It follows tickets n°563 and 3197.
It is still the same issue, see [[http://support.chamilo.org/attachments/1506/2011-04-04_1352_editeur_wysiwyg.swf]]

It works on chamilodev.beeznest.com but I haven't got the same server configuration (and may be I'm not alone in this case).
In my configuration $_configuration['url_append'] isn't empty. My URLs are something like http://chamilo.grenet.fr/_[name_of_the_university]_/index.php

This problem isn't simple because Chamilo enable 2 differents WYSIWYG interface : advanced and basic. These interface doesn't write the same src path for images.
If you put an image in a document with BASIC WYSIWYG interface, the path of the image in the HTML will be : images/gallery/essai_17202_sylphe.png
If you put an image in a document with ADVANCED interface, the path of the document will be : /courses/MYCOURSEDIR/document/images/gallery/essai_17202_sylphe.png

Other medias (flv, mp3, swf) added to a document from both interfaces have a long path that include the course dir /courses/MYCOURSEDIR/document/...

Which path do you want for medias in the html code of a document ? The long one, with course dir, or the short one ? It should be the same for every medias, and every WYSIWYG editor (advanced or basic).
If you choose the long one, it doesn't work with a server conf that has an url_append (but I know how to fix it, no pb).
It is a important choice. For the moment, if you copy document of your course in another course (using Chamilo tools) only medias with the long path (including the course dir) are copied in the destination course.

What's your point of view ?

(I've got to patch my Chamilo to make both editors work and I'll submit it.)


Files

fckeditor.tgz (29.2 KB) fckeditor.tgz Hubert Borderiou, 29/09/2011 11:54

Associated revisions

Revision 595b3f07 (diff)
Added by Hubert Borderiou over 7 years ago

Ref #3933 - long URL Chamilo site - path upload problem

Revision 9ee380ce (diff)
Added by Julio Montoya over 7 years ago

Adding possible fix to the long url (chamilo in a folder) only to the images. Need feedback, don't be mad for this change :) just testing! see #3933

Revision ff75a7d3 (diff)
Added by Julio Montoya over 7 years ago

Cosmetic changes + fixing error when loading course images see #3933

Revision e5967fbd (diff)
Added by Julio Montoya over 7 years ago

Should fix bug when loading images using the simple html editor see #3933

Revision 28de7333 (diff)
Added by Julio Montoya over 7 years ago

Fixing course validation see #3933

Revision d7eeeab7 (diff)
Added by Julio Montoya over 7 years ago

My fck changes waiting Hubert changes in order to merge see #3933

Revision 484a501e (diff)
Added by Hubert Borderiou over 7 years ago

Upload media in long URLchamilo site for advanced editor - ref #3933

Revision 905a6e62 (diff)
Added by Hubert Borderiou over 7 years ago

Upload media in long URLchamilo site for basic editor - ref #3933

History

#1

Updated by Hubert Borderiou almost 8 years ago

Hi,

I did the patch.
I've tested it on 2 Chamilo install. One with url_append empty (URL = http://chamilo-test.grenet.fr/index.php, the most common config I guess) and the other with url_append = "test" (URL = http://chamilo.grenet.fr/test/index.php, my config).

I've tested all medias with both editors : basic and advanced.
The URL written in the HTML source of the document is, for every medias, the long URL with the course_dir include.

I've modified files :
// for the html source with advanced wysiwyg editor that not contain the url_append
./fckeditor/editor/dialog/fck_flash/fck_flash.js
./fckeditor/editor/plugins/MP3/fck_mp3.js
./fckeditor/editor/plugins/audio/fck_audio.js
./fckeditor/editor/plugins/fckEmbedMovies/fck_embedmovies.js
./fckeditor/editor/plugins/flvPlayer/flvPlayer.js

// + files for the unreachable ressources directory with basic wysiwyg editor
./fckeditor/editor/filemanager/connectors/php/commands.php config.php io.php

// for the html source with course_dir for adding an image on a doc with the basic wysiwyg editor
./fckeditor/editor/plugins/ImageManager/images.php

These are not .patch files, nor .diff, but each lines I've modified contain word "hub" in comments.
Don't hesitate if you have any question.

Regards,

Hubert

#2

Updated by Hubert Borderiou almost 8 years ago

Hi,
I've tested the behavior of chamilo 1.9 concerning HTML editors :
I'll write the path generated in the HTML code

<with advanced HTML editor>
Image : images/avatar2.png
FLV : /courses/COURS00001/document/video/flv/cuepoints.flv
Flash : flash/2010-01-13_1339-1024x767.swf
MP3 : /courses/COURS00001/document/audio/JNAX.mp3

<with BASIC HTML editor>
Image : images/avatar2.png
FLV : /courses/COURS00001/document/video/flv/cuepoints.flv
Flash : flash/2010-01-13_1339-1024x767.swf
MP3 : /courses/COURS00001/document/audio/JNAX.mp3

I think it's going to be a good idea to choose the behaviour you want for the FCKEditor, because the copy course tool parse HTML of documents. I'll be able to do the patche.

Do you want to have the course directory in the path of media ?

#3

Updated by Yannick Warnier almost 8 years ago

Adding Julio as an observer (he was fiddling around the document paths code a few days ago).

#4

Updated by Hubert Borderiou almost 8 years ago

If we put the path of the media from the document folder (without the course folder name in it) we'll not have problem with the long URL any longer (for URL likes http://server.com/chamilo/university/index.php).
The URL of the media will be src="my_image.png"
Otherwise, the URL of the media will be src="/chamilo/university/courses/COURS00001/document/my_image.png"

As a result, I suggest not to have the course directory in the path of the media.

#5

Updated by Julio Montoya almost 8 years ago

  • Status changed from New to Needs more info
  • Assignee changed from Hubert Borderiou to Yannick Warnier
  • % Done changed from 0 to 10

I agree with those changes

Yannick?

Hubert Borderiou wrote:

If we put the path of the media from the document folder (without the course folder name in it) we'll not have problem with the long URL any longer (for URL likes http://server.com/chamilo/university/index.php).
The URL of the media will be src="my_image.png"
Otherwise, the URL of the media will be src="/chamilo/university/courses/COURS00001/document/my_image.png"

As a result, I suggest not to have the course directory in the path of the media.

#6

Updated by Yannick Warnier almost 8 years ago

  • Status changed from Needs more info to Assigned
  • Assignee changed from Yannick Warnier to Hubert Borderiou

Let's see... I agree but this fix should then include a fix for all courses restored from a previous version of Chamilo. That is, in the coursecopy/classes/CourseRestorer.class.php, there should be a check on the version of the course being <1.9 and then, if it is the case, the restoration should "trim" the name of the course.
Also, when upgrading from version<1.9, all contents should be updated, which I see as a little bit of a problem for large portals... (because it could take several days to upgrade).

Are you aware that you're jumping into that kind of problems? :-)
Of course, we're as crazy as you are and we'll jump hand in hand with you, but we need a minimum of engagement from you to at least try to cover the minimum requirements...

#7

Updated by Hubert Borderiou almost 8 years ago

I think it's worth it.

It'll solve the issue of uploading media in portals with longs url (like mine), and it'll put every url the same shape.

I already had such problem of upgrading large portals when I did my tools to "clean" Dokeos portals.
I'll have to do such a tool to migrates thousand courses from Dok 1.8.6.1 to Chamilo 1.9

Also, when upgrading from version<1.9, all contents should be updated, which I see as a little bit of a problem for large portals... (because it could take several days to upgrade).

It's true but it is not as dangerous as it seems, because for the moment url are like this
-------------
Image : images/avatar2.png
FLV : /courses/COURS00001/document/video/flv/cuepoints.flv
Flash : flash/2010-01-13_1339-1024x767.swf
MP3 : /courses/COURS00001/document/audio/JNAX.mp3
-------------
and both works (if your portal doesn't have a long url)
We'll just choose to shape every url like "images/avatar2.png", without the course name.
url like "/courses/COURS00001/document/audio/JNAX.mp3", from older portal, will still works fine.
But it should be nicer, cleaner and better to upgrade contents replacing every url ^^

#8

Updated by Julio Montoya over 7 years ago

  • Status changed from Assigned to Needs more info
  • Assignee changed from Hubert Borderiou to Yannick Warnier

Yannick, Hubert says that old courses will continue to work so it will not be necessary to migrate all courses
So when you make an "import" or "export" of courses the URLs will continue to work, otherwise the migration will be a pain in the *** :)

#9

Updated by Julio Montoya over 7 years ago

I added Juan Carlos as a watcher for this task

#10

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

Julio Montoya wrote:

I added Juan Carlos as a watche for this task

Julio, te importaría escribir en español (pues has seguido más este hilo y quizá Hubert no lo hable)concretamente ¿qué problema hay?, no acabo de ver concretamente el bug para intentar reproducirlo.

#11

Updated by Julio Montoya over 7 years ago

Hola Juan Carlos,

el problema que presenta Hubert es el siguiente:

Cuando se sube un archivo mp3, una imagen, etc la URL es diferente

Con el editor avanzado "advanced HTML editor"

Image : images/avatar2.png
FLV : /courses/COURS00001/document/video/flv/cuepoints.flv
Flash : flash/2010-01-13_1339-1024x767.swf
MP3 : /courses/COURS00001/document/audio/JNAX.mp3

con el editor básico
Image : images/avatar2.png
FLV : /courses/COURS00001/document/video/flv/cuepoints.flv
Flash : flash/2010-01-13_1339-1024x767.swf
MP3 : /courses/COURS00001/document/audio/JNAX.mp3

lo que propone Hubert es "estandarizar"/"normalizar" estas URLs y solo usar esto:

src="my_image.png"

en lugar de

src="/chamilo/university/courses/COURS00001/document/my_image.png" o
src="/courses/COURS00001/document/my_image.png" o
src="/document/my_image.png"

El estado actual es que de todas maneras funciona en una instalación "clásica" (en un dominio), pero el bug sucede cuando tienes instalado un chamilo dentro de directorios como está en el ejemplo /dir/chamilo/university/courses/COURS00001/document/my_image.png

#12

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

Bueno entonces la solución (lo tengo que probar) es que se incorpore también ese directorio a la ruta?

#13

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

El caso que yo tengo instalado el chamilo de pruebas en un directorio y no me presenta mayores problemas aquí. Pero volveré a mirar...

related
http://support.chamilo.org/issues/563
http://support.chamilo.org/issues/3197

#14

Updated by Hubert Borderiou over 7 years ago

Hi,

if the path of your media is long, eg link=/courses/COURS00001/document/video/flv/cuepoints.flv, when you backup your course and import it in a new course, in the new course you'll have a document with a link to another course.

Example :
I've got a HTML file in my course COURSE0001 with link=/courses/COURS00001/document/video/flv/cuepoints.flv

I backup this course and import it in course COURSE00002, in this course I'll have a document with a HTML link link=/courses/COURS00001/document/video/flv/cuepoints.flv. I is not the behaviour of Chamilo to share documents between courses. Or we have to edit the HTML file...

#15

Updated by Julio Montoya over 7 years ago

Hola Juan Carlos,

alguna novedad? La idea de Hubert me parece buena, de esta manera los nuevos cursos estarán "limpios" y los antiguos funcionarán como siempre, creo que es una buena oportunidad en esta version alpha de agregarlo para poder probarlo antes de la beta.


quelque chose de nouveau? L'idée de Hubert semble bonne, de sorte que les nouveaux cours seront "propres" et les autres vont vont marcher comme d'habitude, je pense que la version chamilo 1.9 alpha est une bonne ocassion pour l'ajouter avant la beta

#16

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

No me parece mal el objetivo final; ahora bien seamos claros, nunca se ha tratado realmente de un bug sino más bien de una petición de otra forma de funcionar del editor.
La cuestión es lo que se puede ganar y el precio a pagar. Es decir, no tengo claro que estos cambios no puedan suponer problemas de seguridad y relajamiento en el control de acceso a los documentos. Por otro lado, habría que realizar concienzudas pruebas en todos los botones de fckeditor que implicados, en los modos profesor/alumno y en todas las herramientas de Chamilo, incluidas las que se acceden desde fuera del curso, claro está: tanto en un modo simplificado como en el editor ampliado. Yo no tengo tiempo para realizar un seguimiento de esto, así que si consideráis que los beneficios del cambio merecen este trabajo y el riesgo, adelante, pero por favor no realizar los cambios en el código sin comprobar TODO esto.

#17

Updated by Hubert Borderiou over 7 years ago

  • % Done changed from 10 to 80

Hi,

N.B. : this ticket is for chamilo site with url_append and images/flv/swf/mp3 files uploaded. Today, if your site is http://dom.org/UNIV/index.php you cannot use uploaded media files, because there path will be, for example, <img src="/course/COURS0001/document/images/totoro.gif />. It should be <img src="/UNIV/course/COURS0001/document/images/totoro.gif />

The patch will do :
- if the URL of your site doesn't have url_append (e.g. http://campus.chamilo.org/index.php) my patch will actually do nothing. No URLs will be changed.
- if the URL of your site has url_append (e.g. http://dom.org/UNIV/index.php) my patch will concatenate the url_append text (e.g. /UNIV/) before the media path (e.g. <img src="/UNIV/course/COURSE00001/document/images/totoro.png" /> ou <img src="/UNIV/home/default_platform_document/essai_17202_sylphe.png"/>

(I did it for dok€os many years now, because my Dok's URL have url_append. I'd redo it for chamilo.)

MORE INFORMATION
I.
I cannot use relative URL for all media. We already saw in this post that they are used for images and swf file (<img src="totoro.png"/>), and not for flv and mp3 files (<...file=/course/COURS00001/document/video/flv/cuepoint.flv ...>)
The reason is the player used : player.swf which is in folder /main/inc/lib/mediaplayer
This player reads mp3 and flv files. To use this player you have to write in the <embed> tag the attribute file with SEMI-ABSOLUTE URL or RELATIVE URL from the player folder (here /main/inc/lib/mediaplayer)
As a result your URL will be, in the first case (SEMI-ABSOLUTE URL), /course/COURS00001/document/video/flv/cuepoint.flv
and in the second case, for RELATIVE URL, ../../../course/COURS00001/document/video/flv/cuepoint.flv
In order to have URL like "video/flv/cuepoint.flv, we should put the player.swf file in the course directory, which is really a bad idea.
That's why I cannot put short relative url for all type of files.

II.
FCKeditor has the option to use relative URL.
For example in file, main/inc/lib/fckeditor/editor/plugin/flvPlayer/flvplayer.js, fucntion setURL line 367, replace
url = FCK.GetUrl( url, FCK.SEMI_ABSOLUTE_URL ) ;
with
url = FCK.GetUrl( url, FCK.DOCUMENT_RELATIVE_URL ) ;
But it doesn't work for us because of /main/inc/lib/mediaplayer/player.swf (see above)

Regards,

#18

Updated by Yannick Warnier over 7 years ago

  • Target version set to 1.9 Beta
#19

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

Como advertía hace unos días en el post n 16 se han introducido los cambios, conocidos y discutidos desde hace tiempo y no se ha verificado sus repercusiones. Vuelvo a subrayar que esto funcionaba, aunque de otra forma, pero funcionaba. Ahora con los nueos cambios sí que hay un bug, pues no funciona la inserción al menos entre otros desde el editor avanzado. Así que sería bueno que quien ha introducido los cambios los revise para que TODO funcione igual o mejor que lo hacía antes, pero por favor, nunca peor. Un poco de prudencia.

#20

Updated by Julio Montoya over 7 years ago

Los cambios hechos por Hubert fueron anulados con este commit:

http://code.google.com/p/chamilo/source/detail?spec=svn.classic.c6b368065552a4bd6c0b4e623c9cf54ebdfb331b&repo=classic&r=e76a3a4de6b278fce3927f2ea6ec59c474d33bcb

personalmente no he encontrado el bug que indica Juan Carlos. Esta correción es importante para los portales que tienen chamilo instalado en una carpeta.

Se podría detallar el bug para poder resolverlo y discutirlo?


Les changements fait par Hubert on ete annulé avec le commit

http://code.google.com/p/chamilo/source/detail?spec=svn.classic.c6b368065552a4bd6c0b4e623c9cf54ebdfb331b&repo=classic&r=e76a3a4de6b278fce3927f2ea6ec59c474d33bcb

Personnellement je n'ai pas vu le bug que signale Juan Carlos. Le changement de Hubert, est important pour les portails installes dans un dossier.

On pourrai détailler le bug pour pouvoir le résoudre et en parler?

#21

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

Hace tres semanas el bug estaba presente y no permitía insertar absolutamente nada. De todas formas si tú lo has comprobado ahora y con esos cambios todo funciona igual o mejor que en 1.8.4 no hay ningún problema en una mejora

#22

Updated by Julio Montoya over 7 years ago

Ok, Hubert va enviar sus cambios vamos a ver si encontramos errores

#23

Updated by Hubert Borderiou over 7 years ago

Hola, utiliso este patch desde algunos anos. Lo hice la primera vez para Dokeos1.8.5 en 2009. Lo utilisamos en la universidad de Grenoble desde 4 anos sin problema, en 13 instalaciones de Dokeos. Conozco bien el problema porque lo verifico en cada version de Dokeos et lo volvi a hacer para Chamilo. Puedo explicar con mas detalles mis modificaciones si lo desean y buscar problemas.

#24

Updated by Yannick Warnier over 7 years ago

  • Status changed from Needs more info to Assigned
  • Priority changed from Normal to High
  • Target version changed from 1.9 Beta to 1.9 RC1

Los talentos secretos de Hubert se revelan :-)

Hola a todos y gracias por la colaboración en esta tarea. Como lo indican todos, esta tarea es super importante (pues el hecho de poder subir contenidos multimedia es lo que hace un 20% por lo menos de la sencillez de Chamilo).

Por lo tanto, les pido no cerrarla con demasiado prisa (la voy a postergar hasta RC1) para que se puedan hacer las pruebas correspondientes en varios navegadores.

De lo que tengo entendido, faltaría pruebas con- y sin- uso de "subcarpeta". En todo caso, confio en la experiencia de Hubert que gestiona efectivamente esto desde hace un buen tiempo, pero como medida general es bueno apoyarnos el uno el otro revisándolo. Este fin de semana trataré instalarme uno con sub-url para ver como nos sale.

Fue una agradable sorpresa ver que Hubert maneja tan bien el español (aunque se haya introducido un "et" discretamente - en lugar de "y" - en su respuesta :-p)

#25

Updated by Hubert Borderiou over 7 years ago

Fue gracias a Karine que pude escribir mi texto en espanol, una estudiante que esta haciendo sus practicas en la empresa y que me ayuda para las migraciones de Chamilo. Estoy a su disposicion para hacer màs pruebas sobre este problema. (en ingles porque desafortunadamente Karine se va la semana proxima).

http://code.google.com/p/chamilo/source/detail?r=ea46ed0e2fcdc173dc3ff8650bfd24a01fa9b915&repo=classic

#26

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

Aunque a veces me da la impresión de estar perdiendo el tiempo al escribir, aprecio a Julio y probablemente también acabe haciendo lo mismo con Hubert; así que que sacaré fuerzas para escribir de nuevo. Los comentarios los haré en español porque me resulta más fácil y empleo menos tiempo, así que como he de decidir si explico en español o no gasto mi tiempo en intentar evitar una y otra vez un desastre, escribiré en español y seguro que alguien echa una mano para la traducción o hay alguna ayudita de google.

1. En primer lugar, ni tan siquiera hay que mirar el código del commit que introduce las modificaciones para ver que algo va a dejar de funcionar, simplemente viendo los ficheros que modifica está claro que será así porque no se ha tocado ningún fichero del editor avanzado. El cual ha dejado de funcionar.

2. La modificación se ha hecho sólo teniendo en cuenta el editor simple que no es el editor por defecto.Hay una regla general que dice:

ninguna implementación debe romper una implementación por defecto para abrirse paso. Así que debería haber sido lo primero que se tendría

que haber comprobado.

3. Sin entrar en los dos puntos anteriores, que ya de por sí dicen bastante, entraré en concreto en lo que afecta al simple, cuya implementación tampoco es correcta y planteará varios problemas (de entre ellos me limitaré a las imágenes, habría que revisar también videos, flash, sonidos...) ya que entre otras cosas, no se ha tenido en cuenta a los estudiantes.

3.a. Las modificaciones en el editor simple solo funcionan cuando la inserción de una imagen existente y la realiza un profesor.

3.b. Cuando un profesor sube una imagen, por defecto se sube siempre a la carpeta de galería de imágenes del curso. Esto no siempre es deseable en todas las herramientas por los siguientes motivos:
3.b.1 Un curso puede tener varios profesores y bastará que uno de ellos borre una imagen que subió a un foro, al wiki, al blog, etc., para que borre la imagen que otros pueden usar en un hilo de otro foro por ejemplo. Además estas imágenes no tienen por qué ser lo suficientemente significativas para mostrarse a los alumnos en la carpeta galería de imágenes, en el caso de que aquí se deseara desplegar otras para su distribución, o por ejemplo cuando se muestran como imágenes disponibles para la galería de svg que se abre desde dentro de la herramienta svg-editor
3.b.2. Es mejor que las imágenes se suban por defecto a la carpeta de cada usuario en el curso, independientemente de que éste sea alumno o profesor. En lo posible hay que ligar usuario y documento en vez de ligar curso y documento. Si se quieren poner expresamente estas imágenes en la carpeta galería de imágenes también sería posible moviéndose por el árbol de directorios.
3.b.3. En el editor simple no está desarrollada una identificación clara de las carpetas de los usuarios mediante su nombre como ocurre en el avanzado, quizá sea esta una de las razones por la que se mantiene funcionando de la forma anterior.

3.c. Como Chamilo es un sistema integrado, es lógico que si se modifica en un sitio algo habrá que hacerlo en otros, por ejemplo, parece que nadie ha comprobado que las exportadiciones a pdf de las imágenes en los documentos ha dejado de funcionar, y probablemente algunas cosas más.

3.d. Cuando el que sube las imágenes tiene el estatus de administrador de la plataforma, el funcionamiento debe ser similar y subirse a su carpeta general de usuario en la plataforma (no de curso), pues un administrador puede dejar de serlo. Sin embargo cuando se trate de editar la página principal de la plataforma estas imágenes deben residir en otra carpeta que estará accesible al editor para todo aquel que tenga el estatus de administrador. Sin embargo aquí también hay errores: cuando se intenta insertar alguna imagen que se haya subido tampoco será posible, entre otras cosas por algo tan obvio si se mira el código insertado como que falta algo en el path
<p><img width="1024" height="768" align="bottom" src="/chamilo190/courses//document/Hydrangeas.jpg" alt="Hydrangeas.jpg" /></p>

3.e. Finalmente nuestros alumnos, son los más perjudicados con los cambios. En este caso las modificaciones les han privado por completo de su capacidad para insertar imágenes o cualquir otro objeto en los foros, wiki, documentos html de sus portafolios de curso, etc. La imagen sigue subiéndose al sitio correcto, pero la herramienta documentos ya no sincroniza los cambios que se han realizado, probablemente porque ahora a mysql se le envía información errónea. Además tampoco se puede visualizar en el documento la imagen que intenten subir e insertar, debido a una cuestión tan obvia como que los cambios introducen un path erróneo del tipo ../../shared_folder/sf_user_5//chamilo190/courses/GESTION/document/3.jpg Es probable que en algunos entornos pedagógicos no sea importante los materiales que realicen los alumnos y que la plataforma sólo se emplee para distribuir información, aunque creo que lo normal y cada vez más frecuente es que el alumnado no sólo sea lector de materiales, sino también procesador, reelaborador y autor.

No he hecho más pruebas por ej., en el modo de sesiones, que sin embargo adelanto que tampoco funcionará correctamente.

En suma, que quedan bastantes cosas para acabar esta tarea... y lo dicho un poco de prudencia...

#27

Updated by Hubert Borderiou over 7 years ago

[Españoles por debajo]
Bonjour,
Je crois qu'il y a un malentendu. Mon patche ne touche en aucun cas l'outil de téléchargement d'un fichier média (image, son, flv etc...)
Il se contente de modifier, si $_configuration['url_append'] n'est pas vide, le chemin web pour pouvoir afficher le fichier média dans les 2 type d'éditeurs, basique et avancé. Il ne modifie aucun droit, aucun dossier de destination, aucune variable.

Dans votre exemple, <img width="1024" height="768" align="bottom" src="/chamilo190/courses//document/Hydrangeas.jpg" alt="Hydrangeas.jpg" /> il manque le code du cours entre /courses/ et /document/.
Cela devrait être <img width="1024" height="768" align="bottom" src="/chamilo190/courses/ CODE_DU_COURS /document/Hydrangeas.jpg" alt="Hydrangeas.jpg" />

C'est une erreur à corriger.

----------------------------------------------------------------
Hola, creo que hay un malentendido. Mi patch no toca en ningun caso la parte que sube un documento media (imagen, sonido, fly etc...) No modifica ningun derecho, ninguna carpeta de destinacion, ninguna variable.
Solo modifica, si $_configuration['url_append'] no esta vacio, el camino web para poder mostrar el documento media; esto para los dos tipos de editores, simple y avanzado.
En su ejemplo, <img width="1024" height="768" align="bottom" src="/chamilo190/courses//document/Hydrangeas.jpg" alt="Hydrangeas.jpg" />, falta el codigo del curso entre /courses/ y /document/.

Deberia ser <img width="1024" height="768" align="bottom" src="/chamilo190/courses/ CODE_DU_COURS /document/Hydrangeas.jpg" alt="Hydrangeas.jpg" />
Es un error que hay que corregir.
(gracias Karine por la traducción)

Hubert

#28

Updated by Julio Montoya over 7 years ago

Hola Juan Carlos aqui mis comentarios primero voy a responder los primeros puntos para luego ir avanzando con los otros puntos que mencionas ...

Juan Carlos Raña wrote:

Aunque a veces me da la impresión de estar perdiendo el tiempo al escribir, aprecio a Julio y probablemente también acabe haciendo lo mismo con Hubert; así que que sacaré fuerzas para escribir de nuevo. Los comentarios los haré en español porque me resulta más fácil y empleo menos tiempo, así que como he de decidir si explico en español o no gasto mi tiempo en intentar evitar una y otra vez un desastre, escribiré en español y seguro que alguien echa una mano para la traducción o hay alguna ayudita de google.

1. En primer lugar, ni tan siquiera hay que mirar el código del commit que introduce las modificaciones para ver que algo va a dejar de funcionar, simplemente viendo los ficheros que modifica está claro que será así porque no se ha tocado ningún fichero del editor avanzado. El cual ha dejado de funcionar.

2. La modificación se ha hecho sólo teniendo en cuenta el editor simple que no es el editor por defecto.Hay una regla general que dice:

ninguna implementación debe romper una implementación por defecto para abrirse paso. Así que debería haber sido lo primero que se tendría

que haber comprobado.

Sobre el punto 1 y 2: tengo activado el editor avanzado con chamilo instalado en una URL (no dentro de un directorio) y he podido crear/subir una imagen y un mp3. He hecho la misma prueba en chamilodev.beeznest.com y tambien funciona bien.

Este es el curso:

http://chamilodev.beeznest.com/courses/PRUEBACURSOFCKEDITORMODIF/index.php

En los documentos:

http://chamilodev.beeznest.com/main/document/document.php?cidReq=PRUEBACURSOFCKEDITORMODIF&id_session=0&gidReq=0

Tal vez el único detalle es que cuando subo una imagen el thumbnail no se crea automáticamente, hay que darle un "actualizar" sin embargo en mi instalación local no tuve ese problema.

Un detalle para Hubert cuando agrego la imagen la ruta es:

<img width="170" height="213" alt="" src="1316359911825.jpg" />

pero cuando creo un MP3 la URL: file=/courses/PRUEBACURSOFCKEDITORMODIF/document/You_Say_You_Dont_Love_Me_-Live_at_Le_Poisson_Rouge-.mp3


<embed width="300" height="20" 
flashvars="file=/courses/PRUEBACURSOFCKEDITORMODIF/document/You_Say_You_Dont_Love_Me_-Live_at_Le_Poisson_Rouge-.mp3&amp;autostart=false" allowscriptaccess="always" allowfullscreen="false" src="/main/inc/lib/mediaplayer/player.swf" bgcolor="#FFFFFF" pluginspage="http://www.macromedia.com/go/getflashplayer" type="application/x-shockwave-flash"></embed>

#29

Updated by Julio Montoya over 7 years ago

Luego de hablar con Hubert (por el IRC) me dice que su patch solo afecta cuando se tiene una URL larga es decir algo asi: chamilodev.beeznest.com/chamilo19/ en lugar de chamilodev.beeznest.com

Sobre el MP3 me dice que no es posible realizarlo por el comentario en 17 por el problema del swf

Ahora voy a instalar chamilo en una "URL larga" para verificar si hay errores

#30

Updated by Julio Montoya over 7 years ago

  • Assignee changed from Yannick Warnier to Hubert Borderiou
  • % Done changed from 80 to 70

Acabo de instalar chamilo en un directorio URL:

http://my.sub.net/subfolder/chamilo-read-only/

y he agregado una imagen en un documento HTML (editor avanzado) y tengo esto en el src:

<img src="/subfolder/chamilo-read-only53808_132020570182273_132020536848943_176378_5199987_o.jpg" alt="" />

falta una "/"

asignando a Hubert ...

(he comunicado esto a Hubert por IRC)

#31

Updated by Julio Montoya over 7 years ago

En una URL larga "http://my.sub.net/subfolder/chamilo-read-only/"

Cuando voy a editar la homepage main/admin/configure_homepage.php agrego una imagen y tengo esto:

editor simple:

<img src="/subfolder/chamilo-read-only/courses//document/01_1_.jpg" alt="01_1_.jpg" align="baseline" width="1000" height="1093" />

editor avanzado:

<img src="/subfolder/chamilo-read-only/subfolder/chamilo-read-only/home/default_platform_document/01_1_.jpg" alt="" />

no puedo ver las imagenes claro

------------------------------
fr

quand je veux modifier la homepage dans: main/admin/configure_homepage.php et j'ajoute un imagej'ai ca:

editor simple:

<img src="/subfolder/chamilo-read-only/courses//document/01_1_.jpg" alt="01_1_.jpg" align="baseline" width="1000" height="1093" />

editor avancé

<img src="/subfolder/chamilo-read-only/subfolder/chamilo-read-only/home/default_platform_document/01_1_.jpg" alt="" />
#32

Updated by Hubert Borderiou over 7 years ago

This issue, of wrong source path for uploaded media is due to the long url of the site, and to a different name between the chamilo folder and is symbolic link in /var/www ( for exemple)

First issue cause : long url (http://www.site.com/SUBFOLDER)
-------------------
Second issue cause : chamilo folder name != symbolic link
--------------------
e.g. :
Chamilo installed in folder /home/mychamilo/www/chamilo.classic
URL link is /var/www/chamilo19 -> /home/mychamilo/www/chamilo.classic
chamilo.classic != chamilo19

If you've got first cause, Julio modification of my patch works fine
See http://code.google.com/p/chamilo/source/detail?spec=svn.classic.6362a8732cc0567e66474a6c83ae0381e51c9686&repo=classic&r=e4ee7f67c8d1c049cd8b0e9b03cfb0c481e63aba

If you've got first and second cause, you need my patch.

#33

Updated by Hubert Borderiou over 7 years ago

I'm going to modify the patche with Julio modif.
As a result, the configuration server has to have this configuration :
The site folder in disk must have the same name has the url_append.
If not, in your install (like in mine) it is quite easy to change in UNIX server with symbolic links.

The configuration should be like that :
URL : http://www.cham.org/chamilo19
In folder /home/myaccount/www/chamilo19
(as a result we have /var/www/chamilo19 -> /home/myaccount/www/chamilo19)

And we have in configuration.php file :
$_configuration['root_web'] = 'http://www.cham.org/chamilo19/';
$_configuration['root_sys'] = '/home/myaccount/www/chamilo19/';
$_configuration['url_append'] = '/chamilo19';

If your folder's name is (for example) /home/myaccount/www/chamiloLastVersion
Rename it to /home/myaccount/www/chamilo19
and create a symbolic link chamilo19 -> chamiloLastVersion

It's ok now with advanced editor.
http://code.google.com/p/chamilo/source/detail?r=a95913480e574008740b370ec62275dc726ef97b&repo=classic

#35

Updated by Yannick Warnier about 7 years ago

  • Assignee changed from Hubert Borderiou to Eric Petitdemange

Coursenligne, the goal here is to try and upload files as indicated at the very beginning of the bug, and see if you can reproduce the problem. If you can't, then we all win.
Please make sure you try with the basic and the advanced filemanagers (they can be switched from the WYSIWYG section in the platform settings).

#36

Updated by Hubert Borderiou about 7 years ago

The current version of the patch doesn't work for this specific configuration
URL got a non empty URI : http://domain.org/my_university/index.php
Web site installed in a folder /home/user/www/chamilo
And symbolic link with a different name in /var/www/my_university -> /home/user/www/chamilo

regards,

#37

Updated by Eric Petitdemange about 7 years ago

Hi,

I do have a point regarding this issue, long path doesn't exist anymore if you replace absolute path by relative path.

I've done it manually to a course (140 images to manualy repoint out) and it's working.

I have 4 platforms running and only 1 is under production.
I have 3 1.8.8.4 and 1 1.9 (updated after every "nightly" to the latest version).

I have the production ith a public IP address and a specific name, the "local" 1.8.8.4 named cham18 and the lates nightly named cham19xx, where xx is the day (so the actual version is cham1917).

I create a course on cham18 (from my home computer), if I upload it to cham1917, no image, or video can display, because instead of ../../images/<image_name>, I have a path like /cham18/courses/<name of the course>/documents/images/<name of the image>.
This path is obviously wrong on cham1917.

From 1.8.8.4 platform, I edited all pathes and replaced them using relative path...
I upload the course after saving and IT'S WORKING ;)

The problem is:
- Templates don't have the same path for resources (images)
- Exercices don't use the same path when you add images (even inside the test if you create clickable image).

I'm sure there is something to do about it :)

GOOD LUCK Guys!

#38

Updated by Yannick Warnier about 7 years ago

  • Status changed from Needs more info to Assigned
  • Assignee changed from Eric Petitdemange to Julio Montoya
  • Target version changed from 1.9 RC1 to 1.9 Stable

Moving to stable. It's rather minor but let's ask Julio to do his magic.

#39

Updated by Yannick Warnier about 7 years ago

  • Status changed from Assigned to Bug resolved

Closing. As Coursenligne said, "IT'S WORKING", but I want to try and improve the rest a little, but this should go to another task.

Also available in: Atom PDF