Project

General

Profile

Bug #7244

Doublons dans le dossier Documents

Added by Alain Deschênes almost 5 years ago. Updated over 4 years ago.

Status:
Bug resolved
Priority:
Normal
Assignee:
Category:
Documents
Target version:
Start date:
27/08/2014
Due date:
% Done:

100%

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

Description

J'ai fait l'installation de 1.9.8.2 et je constate que lors d'un Upload d'un document sur mon serveur à plus d'une reprise, l'affichage de ce fichier dans le dossier se dédouble autant de fois qu'il est Uploadé.

Cependant, il n'existe qu'une seule copie sur le serveur.

Testé sur stable.chamilo.org et sur mon serveur.

Un problème similaire a été soulevé par Dauzeine, j'ai appliqué le correctif proposé, ça enlève le mot Overwrite, mais l'affichage du nom de fichier persiste.

[[http://www.chamilo.org/phpBB3/viewtopic.php?f=14&t=5346]]

Alain


Files

Doublons.png (67 KB) Doublons.png Alain Deschênes, 27/08/2014 02:53
Doublons_2.jpg (116 KB) Doublons_2.jpg Alain Deschênes, 04/09/2014 14:22

Associated revisions

Revision 56f891ee (diff)
Added by Julio Montoya almost 5 years ago

Upload ajax by default rename see #7244

Revision d7719972 (diff)
Added by Julio Montoya almost 5 years ago

Changing default upload document ajax to overwrite. see #7244

Revision 25f5896c (diff)
Added by Julio Montoya almost 5 years ago

Fixing duplicating document see #7244

History

#1

Updated by Alain Deschênes almost 5 years ago

Voir aussi le post https://support.chamilo.org/issues/7199 émis par Éric Marguin

#2

Updated by Yannick Warnier almost 5 years ago

Pas encore revu les références, mais un bug dans une version récente de Chrome affecte d'autres applications web (dont Redmine ici) avec un problème similaire.

#3

Updated by Alain Deschênes almost 5 years ago

Salut Yannick,

Je viens tout juste de faire un test un ordinateu en Linux avec Firefox 31 et le problème est le même.

Il est à noter que cette machine ne contient ni IE, ni Chrome.

Alain

#4

Updated by Andre Boivin almost 5 years ago

Bonjour Yannick,
Plusieurs enseignants m'ont signalé le problème. J'ai utilisé les fichiers github du 25 aout en local sur wamp et j'ai le problème avec firefox et chrome. C'est assez urgent,si un prof. supprime un de ces fichiers en croyant en conserver un, il le perd car l'entrée restante dans la bd ne pointe vers aucun fichier.

#5

Updated by Andre Boivin almost 5 years ago

J'ai une piste intéresante: le problème n'apparait que si le document est dans un sous répertoire. Un fichier à la racine "document" ne pose pas problème, il es écrasé sans doublon.

#6

Updated by Alain Deschênes almost 5 years ago

Bonjour Messieurs,

J'ai compris qu'il s'agissait uniquement d'un doublon d'affichage dans la base de données pointant tous vers un fichier unique sur le serveur.

Ce qui veut dire que l'effacement d'un doublon implique l'effacement du fichier unique. Ainsi, le reste des doublons vont continuer à pointer vers le fichier effacé...

#7

Updated by Yannick Warnier almost 5 years ago

Julio, si tu peux y jeter un oeil... ça a l'air grave.

#8

Updated by Julio Montoya almost 5 years ago

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

Je ne peux pas reproduire l'erreur dans https://stable.chamilo.org ni dans mon serveur local.

Je voudrais savoir la valeur d'id des 2 document install.notes est la même ou pas?

#9

Updated by Alain Deschênes almost 5 years ago

Bonjour Julio,

2 choses:

  1. Le fichier install.notes est un fichier .txt identique en toute part, c'est juste un fichier léger pour faire des tests.
  1. J'ai essayé à nouveau ce matin sur le site stable.chamilo.org et effectivement, on ne peut reproduire cette erreur, cependant, le comportement du transfert de fichier a changé...

On ne peut plus écraser un fichier existant comme le montre l'image ci-dessous:

Il y a donc quelque chose qui a changé dans la programmation dernièrement.

Toujours dispo pour faire de plus amples tests,

Alain

#10

Updated by Julio Montoya almost 5 years ago

Je pense qu'avant par défaut on faisait un "rename" au lieu de bloquer comme ca.
Je vais regarder ...

#11

Updated by Julio Montoya almost 5 years ago

je viens de corriger l'erreur, maintenant le fichier est renommé à chaque fois, il me semble qu'avant c'était comme ca.

J'ai réglé un autre bug aussi, car le titre c'était le même à chaque fois au lieux de file1, file2, file3, etc.

https://github.com/chamilo/chamilo-lms/commit/56f891ee1bcbc8e82e21253be52112aedd17f600

#12

Updated by Alain Deschênes almost 5 years ago

Bonjour Julio,

Cette approche ne peut pas fonctionner.

Il faut absolument que la dernière version du fichier téléversé ait le nom d'origine puisque dans les PARCOURS, on fait appel au nom d'origine du ficher.

L'idée de téléverser à nouveau un fichier, c'est de le mettre à jour afin que l'apprenant ait accès à la dernière version dans son cours.

Alain

#13

Updated by Julio Montoya almost 5 years ago

Donc, il reste 2 options "overwrite" ou "warning".

#14

Updated by Alain Deschênes almost 5 years ago

Pour ma part, je préfère Overwrite comme on faisait dans les versions précédentes.

#15

Updated by Julio Montoya almost 5 years ago

Je viens d'ajouter la correction.

#16

Updated by Andre Boivin almost 5 years ago

quels fichiers svp?

#17

Updated by Alain Deschênes almost 5 years ago

Afin de faire le suivi, voici le résultat de mes récents tests sur stable.chamilo.org

Dans le dossier principal (maître), aucun problème, tout fonctionne correctement et le fichier est remplacé (overwrite).

Toutefois, comme le mentionnait André un peu plus tôt, le problème de doublon persiste dans les sous-répertoires.

Alain

#18

Updated by Alain Deschênes almost 5 years ago

En passant, j'ai fait un test dans le cours https://stable.chamilo.org/courses/ADPARCOURS/index.php dont j'ai inscrit John Doe comme enseignant

#19

Updated by Julio Montoya almost 5 years ago

je viens d'ajouter une autre correction pour les sous-répertoires.

#20

Updated by Alain Deschênes almost 5 years ago

Salut Julio,

Je viens de tester le tout sur stable.chamilo.org et tout fonctionne parfaitement ;-)

Je vais attendre la mise à jour des fichiers sur Github et je finalise les tests sur mon serveur.

Merci,

Alain

#21

Updated by Julio Montoya almost 5 years ago

  • Status changed from Needs more info to Needs testing
  • % Done changed from 10 to 90

Bien!

#22

Updated by Alain Deschênes almost 5 years ago

Bonjour Julio,

Je viens de mettre à jour mon serveur avec la dernière version dispo sur Github, cependant rien n'a changé, le problème persiste toujours contrairement à stable.chamilo.org .

Si j'observe les dates de modification des fichiers dont tu as fait les corrections, ça me laisse à penser qu'ils n'ont pas été mis à jour sur Github avec les dernières modifications.

Merci,

Alain

#23

Updated by Julio Montoya almost 5 years ago

Alain Deschênes wrote:

Bonjour Julio,

Je viens de mettre à jour mon serveur avec la dernière version dispo sur Github, cependant rien n'a changé, le problème persiste toujours contrairement à stable.chamilo.org .

Si j'observe les dates de modification des fichiers dont tu as fait les corrections, ça me laisse à penser qu'ils n'ont pas été mis à jour sur Github avec les dernières modifications.

Merci,

Alain

Pour avoir la dernière version zip de chamilo 1.9.x il faut aller ici:
https://github.com/chamilo/chamilo-lms/tree/1.9.x

Ensuite cliquer sur "Download zip"

Si vous utilisez git il faudra recuperer les donnes de la branch "1.9.x".

#24

Updated by Alain Deschênes almost 5 years ago

Salut Julio,

Je viens tout juste de tester avec la dernière version dispo sur Github, tel que mentionné plus haut et tout fonctionne parfaitement - Enfin .... Ça commençait à être le délire dans mes dossiers !

Merci,

Alain

#25

Updated by Yannick Warnier almost 5 years ago

  • Target version set to 1.9.10
#26

Updated by Yannick Warnier almost 5 years ago

  • Category set to Documents
  • Status changed from Needs testing to Bug resolved
  • Assignee set to Julio Montoya
  • % Done changed from 90 to 100

Approved by issue reporter.

#27

Updated by Yannick Warnier over 4 years ago

Pour référence, je note que ce changement affecte un changement antérieur fait par Éric Marguin après la release de la 1.9.8.2. Je le note ici au cas où on reviendrait là-dessus plus tard https://github.com/chamilo/chamilo-lms/commit/9dcb404889e25682f28278765803f13e7c80cb03

Also available in: Atom PDF