Project

General

Profile

Bug #5656

Imposible eliminar documentos recibidos en dropbox

Added by Laura Guirao about 9 years ago. Updated almost 9 years ago.

Status:
Bug resolved
Priority:
Normal
Assignee:
-
Category:
-
Target version:
Start date:
16/10/2012
Due date:
% Done:

80%

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

Description

Error reportado por Emiliano en el foro ([[http://www.chamilo.org/forum/viewtopic.php?f=15&t=2632]])
Yo he probado en dos instalaciones de Chamilo, una nuestra de la versión de 1.9.2 y la otra es la de stable.chamilo.org
No me permite eliminar el documento de la bandeja de recibidos, ni como administrador/profesor ni como alumno
Aparece en pantalla un mensaje indicando que se ha eliminado, pero el documento permanece.
Adjunto captura:


Files

eliminar.png (17.3 KB) eliminar.png Laura Guirao, 16/10/2012 17:47

Associated revisions

Revision d7b750b3 (diff)
Added by Julio Montoya about 9 years ago

Should fix bug when deleting dropbox items queries where not corrected see #5656

Revision 54d7bb9a (diff)
Added by Julio Montoya about 9 years ago

Should fix bug when deleting items in dropbox works as in 1.8.8.4 see #5656

History

#1

Updated by Julio Montoya about 9 years ago

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

testing

#2

Updated by Julio Montoya about 9 years ago

  • Status changed from Assigned to Needs testing
  • Assignee deleted (Julio Montoya)
  • Target version set to 1.9.4
  • % Done changed from 0 to 80

Should be fixed now

#3

Updated by Laura Guirao about 9 years ago

Hola Julio

Hemos probado en stable.chamilo.org y funciona correctamente
Queríamos bajarnos la versión 1.9.4 actualizada de ayer con mercurial para probarla en nuestra demo pero no sabemos cómo, si nos puedes indicar...

Ahora usamos esto: hg clone https://code.google.com/p/chamilo.classic --rev 1.9.x chamilo19x
Pero en esa versión no parecen estar los cambios porque cuando lo hemos probado seguia fallando

#4

Updated by Julio Montoya about 9 years ago

Laura Guirao wrote:

Hola Julio

Hemos probado en stable.chamilo.org y funciona correctamente
Queríamos bajarnos la versión 1.9.4 actualizada de ayer con mercurial para probarla en nuestra demo pero no sabemos cómo, si nos puedes indicar...

Ahora usamos esto: hg clone https://code.google.com/p/chamilo.classic --rev 1.9.x chamilo19x
Pero en esa versión no parecen estar los cambios porque cuando lo hemos probado seguia fallando

si exacto esta bien con ese comando, para confirmar que tienen la última version pueden cotejar la última version algo asi:

/var/www/chamilonet$ hg heads
changeset: 20092:7da6eb08805a
branch: 1.9.x
tag: tip
user: Julio Montoya

última version es 20092:7da6eb08805a

Revisar en google code
http://code.google.com/p/chamilo/source/list?repo=classic&name=1.9.x

el último cambio es 7da6eb08805a

Normalmente todo esto no es necesario de hacerlo porque se asume que se tiene todos los cambios cuando se hace un:

hg pull
hg update -C (sobre escribe todos los archivos modificados por la última versión del repositorio)

saludos!

#5

Updated by Yoselyn Castillo about 9 years ago

  • Status changed from Needs testing to Assigned
  • Assignee set to Yoselyn Castillo
#6

Updated by Laura Guirao about 9 years ago

Hemos bajado la última versión, lo hemos comprobado como dices, y no nos funciona
También hemos probado a hacerlo con una instalación limpia con esta versión y tampoco repara el error.
A Emiliano le ocurre lo mismo en su instalación local.

#7

Updated by Yoselyn Castillo about 9 years ago

I will check and report my feedback

#8

Updated by Jose Manuel Abuin Mosquera about 9 years ago

Hi all,

this was happening to me in version 1.9.0, now we updated to 1.9.2 and I see it is happening again.
I "solved" the problem by adding this lines in the end of file /main/dropbox/dropbox_class.inc.php at the end of function deleteReceivedWork($id)

The code is this:

// Delete entries in person table concerning received works
Database::query("DELETE FROM ".$dropbox_cnf['tbl_person']." WHERE c_id = $course_id AND user_id='".$this->userId."' AND file_id='".$id."'");
//NEW CODE
$sql = "DELETE FROM ".$dropbox_cnf['tbl_post']." WHERE c_id = $course_id AND dest_user_id = ".$this->userId." AND file_id = ".$id;
Database::query($sql);
//END NEW CODE
removeUnusedFiles(); // Check for unused files

I hope this helps :)

PS: It needs testing, I didn't have time to test it myself :(

#9

Updated by Julio Montoya about 9 years ago

El usuario que envió el archivo debería de poder eliminarlo y no el que lo recibió.

#10

Updated by Laura Guirao about 9 years ago

La modificación de Jose Manuel Abuin si que nos funciona.

Julio, lo que dices de de que no deberia poder eliminar el archivo el que lo recibe, podría ser lógico, aunque a mi me parece más interesante que sí lo pueda eliminar. Pero en cualquier caso, si no pudiera hacerlo no deberían entonces aparecer las opciones de eliminar, ni tampoco el mensaje de confirmación, no?

#11

Updated by Julio Montoya about 9 years ago

Laura Guirao wrote:

La modificación de Jose Manuel Abuin si que nos funciona.

Julio, lo que dices de de que no deberia poder eliminar el archivo el que lo recibe, podría ser lógico, aunque a mi me parece más interesante que sí lo pueda eliminar. Pero en cualquier caso, si no pudiera hacerlo no deberían entonces aparecer las opciones de eliminar, ni tampoco el mensaje de confirmación, no?

Hola Laura! Exacto, voy a revisar como funcionaba en 1.8.8.4

#12

Updated by Julio Montoya about 9 years ago

  • Assignee changed from Yoselyn Castillo to Julio Montoya

te robo la tarea Yoselyn :)

#13

Updated by Julio Montoya about 9 years ago

en 1.8.8.4 el usuario que recibe la tarea puede eliminarlo de su bandeja, sin embargo el archivo sigue presente para el usuario que envió la tarea.
Voy a revisarlo para que funcione como antes.

#14

Updated by Julio Montoya about 9 years ago

  • Status changed from Assigned to Needs testing
  • Assignee deleted (Julio Montoya)

hehe encontré el bug, al parece Laurent quizo arreglar el doble registro en la tabla c_dropbox_person


    /**
             * Poster is already added when work is created - not so good to split logic 
             */
             if ($user_id != $user_id) {
                // Insert entries into person table
                $sql = "INSERT INTO $table_person (c_id, file_id, user_id)
                        VALUES ($course_id, $file_id, $user_id)";

                // Do not add recipient in person table if mailing zip or just upload.
                if (!$justSubmit) {
                    $result = Database::query($sql);    // If work already exists no error is generated
                }
            }

código original

      $sql = "INSERT INTO $table_person (c_id, file_id, user_id)
                        VALUES ($course_id, $file_id, $user_id)";

                // Do not add recipient in person table if mailing zip or just upload.
                if (!$justSubmit) {
                    $result = Database::query($sql);    // If work already exists no error is generated
                }

Por ello la 1.9 funciona diferente a la 1.8.8.4.

Ahora la 1.9.4 funciona como antes.

#15

Updated by Julio Montoya about 9 years ago

bueno no puedo enviar mis cambios porque el servidor google code está en modo solo lectura

searching for changes
remote: abort: Mercurial hosting is temporarily read-only

#16

Updated by Julio Montoya about 9 years ago

Julio A. Montoya wrote:

bueno no puedo enviar mis cambios porque el servidor google code está en modo solo lectura

searching for changes
remote: abort: Mercurial hosting is temporarily read-only

los cambios ya fueron enviados

#17

Updated by Hubert Borderiou almost 9 years ago

I can delete the file if I am the receiver but there is still a problem, it deletes the file in the Sender tool.

If the sender send the same file again to the same receiver, he gets the error :

DATABASE ERROR #1062:
Duplicate entry '8-1-33' for key 'PRIMARY'
QUERY :
INSERT INTO `chamilo192`.`c_dropbox_person` (c_id, file_id, user_id)
VALUES (8,
'1'
, '33'
)
FILE :
****************/main/dropbox/dropbox_class.inc.php
LINE :
157
CLASS :
Dropbox_Work
METHOD :
_createNewWork

see screencast here : [[http://www.screencast.com/users/hborderi/folders/Jing/media/3cd5fe41-43e2-4f75-80eb-dc57308d46f9]]

I think, if I delete a file in my Dropbox tool (even Send or Receive), it musn't delete files in other users Dropbox tool (even Send or Receive) otherwise, it is not understandable.

#18

Updated by Julio Montoya almost 9 years ago

Hubert Borderiou wrote:

I can delete the file if I am the receiver but there is still a problem, it deletes the file in the Sender tool.

If the sender send the same file again to the same receiver, he gets the error :

DATABASE ERROR #1062:
Duplicate entry '8-1-33' for key 'PRIMARY'
QUERY :
INSERT INTO `chamilo192`.`c_dropbox_person` (c_id, file_id, user_id)
VALUES (8,
'1'
, '33'
)
FILE :
****************/main/dropbox/dropbox_class.inc.php
LINE :
157
CLASS :
Dropbox_Work
METHOD :
_createNewWork

see screencast here : [[http://www.screencast.com/users/hborderi/folders/Jing/media/3cd5fe41-43e2-4f75-80eb-dc57308d46f9]]

I think, if I delete a file in my Dropbox tool (even Send or Receive), it musn't delete files in other users Dropbox tool (even Send or Receive) otherwise, it is not understandable.

Hello hb,
the screencast contains my latest fix?

#19

Updated by Julio Montoya almost 9 years ago

I sent some fixes now it should work in stable.chamilo.org

#20

Updated by Hubert Borderiou almost 9 years ago

Fine, it works for me

#21

Updated by Yoselyn Castillo almost 9 years ago

  • Status changed from Needs testing to Assigned
  • Assignee set to Yoselyn Castillo
#22

Updated by Yoselyn Castillo almost 9 years ago

  • Status changed from Assigned to Bug resolved
  • Assignee deleted (Yoselyn Castillo)

I have done the test. I have sent one file as teacher to 2 users and it was successfully received by them. Later, i have logged in as receivers and i could remove the files. Later i have logged in as teacher again and the file was still there as sent file.
So it works fine.

Also available in: Atom PDF