Project

General

Profile

Bug #2198

Students can not see the images, flash and other resources embedded in the documents, although they have the eye open

Added by Juan Carlos Raña Trabado about 9 years ago. Updated almost 9 years ago.

Status:
Bug resolved
Priority:
High
Category:
-
Target version:
Start date:
22/10/2010
Due date:
% Done:

80%

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

Associated revisions

Revision 6b90406b (diff)
Added by Julio Montoya almost 9 years ago

Fixing document visibility see #2198

History

#1

Updated by Juan Carlos Raña Trabado about 9 years ago

  • Target version set to 1.8.8 alpha

The problem is in commint 12709 Tuning coach and student in a session permissions see BT#1652

#2

Updated by Juan Carlos Raña Trabado about 9 years ago

Julio, las modificaciones que has hecho parecen correctas, pero si bloqueas también los directorios que tengan el ojo cerrado, cualquier imagen dentro de un documento no será visible por los alumnos. Este problema ya se producía cuando se implementó seguridad para archivos y directorios, así que la solución que le dí es que fuera sobre los archivos, de manera que si el archivo tiene el ojo abierto, aunque el directorio lo tenga cerrado siempre será accesible. Ya sé que siempre alguien dirá que un archivo puede ser accesible si se conoce la url, pero la cuestión es que efectivamente debe serlo si se ha puesto el ojo abierto junto al mismo. Si tienes una forma mejor de solucionarlo genial, si no sería bueno que revertieras esta parte del código a la situación original pues de otra forma dejas la plataforma como un mero almacén seguro pero los alumnos no podrán acceder a nada, secuencias del learning path con imágenes, bibliotecas locales del svg, archivos html visibles que contengan elementos de la plataforma en directorios ocultos, etc.

#3

Updated by Juan Carlos Raña Trabado about 9 years ago

el archivo implicado es inc/lib/document.lib.php

#4

Updated by Julio Montoya about 9 years ago

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

Updated by Julio Montoya almost 9 years ago

  • Status changed from Assigned to Needs more info
  • Assignee changed from Julio Montoya to Juan Carlos Raña Trabado
  • % Done changed from 0 to 70

Hola Juan Carlos,
he hecho algunos cambios para que funcione como me indicas. Espero tu feedback para cerrar la tarea.

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

#6

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

Lo miro a finales de semana o la próxima semana, please. Esta semana estoy muyyy ocupado.

#7

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

Parece que ahora funciona de nuevo bien pero no acabo de ver la razón para que api_get_cidreq() desaparezca de la línea 173 de showinframes.php que se introdujo Yannick en el commit 3023 (bab8531eedf3) [svn r14478] Added cidReq to header frame , aunque la imposibilidad de acceder a esta tarea ahora hace que se me escape el problema específico que resolvió.

#8

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

  • Assignee changed from Juan Carlos Raña Trabado to Julio Montoya
#10

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

  • Assignee changed from Julio Montoya to Juan Carlos Raña Trabado
#11

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

  • Status changed from Needs more info to Bug resolved
  • % Done changed from 70 to 80

Also available in: Atom PDF