Project

General

Profile

Bug #2560

Chamilo 1.8.8 alpha- : Imported HotPotatoes tests are not visible/accessible in the "Tests" tool

Added by Ivan Tcholakov about 8 years ago. Updated almost 8 years ago.

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

80%

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

Description

In the "Tests" tool import the attached testing file (a HotPotatoes test). You will discover that the test does not show up in the list of tests. You can't access the imported test.

See also the note http://support.chamilo.org/issues/1339#note-5


Files

chamilo_1862_hotpotatoes_1.png (56.4 KB) chamilo_1862_hotpotatoes_1.png Ivan Tcholakov, 18/01/2011 00:10
chamilo_1862_hotpotatoes_2.png (48.5 KB) chamilo_1862_hotpotatoes_2.png Ivan Tcholakov, 18/01/2011 00:10
a.zip (11.7 KB) a.zip Ivan Tcholakov, 18/01/2011 00:13
hotpotatoes_imported_1.png (67.4 KB) hotpotatoes_imported_1.png Ivan Tcholakov, 03/02/2011 18:29
hotpotatoes_deleted_1.png (69.3 KB) hotpotatoes_deleted_1.png Ivan Tcholakov, 03/02/2011 18:29
hotpotatoes_imported_2.png (67.8 KB) hotpotatoes_imported_2.png Ivan Tcholakov, 03/02/2011 21:19

History

#2

Updated by Ivan Tcholakov about 8 years ago

Ivan Tcholakov wrote:

Two pictures have been added.

... and a.html again, compressed.

#3

Updated by Ivan Tcholakov about 8 years ago

  • File deleted (a.html)
#4

Updated by Julio Montoya about 8 years ago

  • Status changed from New to Needs more info
  • % Done changed from 0 to 50

What about now? I test it with your a.zip and is working fine, I removed the hotpotatoes support due a lot of changes in exercises ..

#5

Updated by Ivan Tcholakov about 8 years ago

I see two issues at the moment:

1. The old implementation read the title of the test by retrieving it form the title tag inside the file - this I think should be restored too. Now the name of the test is equal by default to the file name which is not user friendly. At the first image I expected the title of the imported test to be "Test de prueba" not "a.html".

There is a tricky moment related with proper encoding detection and conversion of the title. If you agree, I can implement and test this title reading feature.

2. When I deleted the imported test, a strange SQL error appeared, see the second picture.

#6

Updated by Ivan Tcholakov about 8 years ago

1. About retrieving the HotPotatoes title (from html):

13542:a2b93a1305f7 Task #2560 - Fixing some indentations that I broke in the previous transaction.
http://code.google.com/p/chamilo/source/detail?r=a2b93a1305f7c97127e1149df7b5c3c4601513dd&repo=classic

13541:1547235a25b2 Task #2560 - On HotPotatoes import the test's title is retrieved from the (html) title tag.
http://code.google.com/p/chamilo/source/detail?r=1547235a25b21f7a2f2fdc67403d2e878e49de81&repo=classic

2. The SQL error disappeared, I don't know why.

#7

Updated by Julio Montoya about 8 years ago

For 1, I thought it was something like that because in campus.chamilo.org (1.8.7.1) we have the filename and not the title

#8

Updated by Julio Montoya about 8 years ago

I fix the metadata error with this commit
http://code.google.com/p/chamilo/source/detail?r=68d10c0ab34bf368db0dbd60ea2ade5cabd6d79d&repo=classic

I think we should delete that metadata support ... i don't know if somebody use it ..

#9

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

Julio Montoya wrote:

I fix the metadata error with this commit
http://code.google.com/p/chamilo/source/detail?r=68d10c0ab34bf368db0dbd60ea2ade5cabd6d79d&repo=classic

I think we should delete that metadata support ... i don't know if somebody use it ..

Sorry I talk in Spanish:
No lo usan porque no está desarrollado, pero los metadatos son importantes en los paquetes para su identificación y en su caso para realizar búsquedas.
En el caso de borrar la actual funcionalidad de metadatos, al menos debería haber un campo tag en el que poder incluir palabras clave. Este campo tag con palabras clave estarí muy bien que pudiera acompañar también a cada test, cada, cada pregunta de test, cada documento, etc., esto permitiría unas búsquedas muy eficientes con un buscador global.

#10

Updated by Julio Montoya about 8 years ago

  • % Done changed from 50 to 80

Juan Carlos Raña wrote:

Julio Montoya wrote:

I fix the metadata error with this commit
http://code.google.com/p/chamilo/source/detail?r=68d10c0ab34bf368db0dbd60ea2ade5cabd6d79d&repo=classic

I think we should delete that metadata support ... i don't know if somebody use it ..

Sorry I talk in Spanish:
No lo usan porque no está desarrollado, pero los metadatos son importantes en los paquetes para su identificación y en su caso para realizar búsquedas.
En el caso de borrar la actual funcionalidad de metadatos, al menos debería haber un campo tag en el que poder incluir palabras clave. Este campo tag con palabras clave estarí muy bien que pudiera acompañar también a cada test, cada, cada pregunta de test, cada documento, etc., esto permitiría unas búsquedas muy eficientes con un buscador global.

Ok, buen argumento habría que abrir otra tarea para retomar o documentar este tema de metadatos ...

#11

Updated by Yannick Warnier about 8 years ago

  • Target version changed from 1.8.8 alpha to 1.8.8 beta
#12

Updated by Julio Montoya about 8 years ago

creo que podemos cerrar esta tarea ...

#13

Updated by Julio Montoya almost 8 years ago

  • Status changed from Needs more info to Feature implemented

Also available in: Atom PDF