Project

General

Profile

Actions

Bug #5089

closed

Impossible d'ajouter/modifier un fichier sur un message existant

Added by Yohann Chastagnier almost 9 years ago. Updated 19 days ago.

Status:
Closed
Priority:
Normal
Start date:
11/08/2013
Due date:
% Done:

100%

Estimated time:
Navigateur:
Tous
Votre version de Silverpeas:
5.13
Système d'exploitation:
Votre base de données:
Toutes
Livraison en TEST:
Livraison en PROD:

Description

Se connecter avec un utilisateur qui est "participant" à forum (cela se définit dans les paramètre de l'application).
Créer un sujet avec un fichier joint (ou répondre à un sujet ou message avec un fichier joint).
Se diriger sur le sujet pour visualiser tous les messages qui lui sont associés et essayer d'ajouter un autre fichier ... ou essayer de modifier un fichier existant ...

Actions #1

Updated by Nicolas Eysseric over 7 years ago

  • Status changed from New to Assigned
  • Assignee set to Kevin FONTAINE

Dans un premier temps, il faut reproduire le problème décrit ci-dessus...

Actions #2

Updated by Kevin FONTAINE over 7 years ago

  • Status changed from Assigned to In progress...

Bug confirmé, il ne se produit que pour l'utilisateur postant les messages en réponse au premier.
Il est lié à une erreur Javascript, à priori une div utilisée avant son initialisation.

Uncaught Error: cannot call methods on dialog prior to initialization; attempted to call method 'open'jquery-1.10.2.min.js:4 x.extend.errorjquery-ui-1.10.3.custom.min.js:6 (anonymous function)jquery-1.10.2.min.js:4 x.extend.eachjquery-1.10.2.min.js:4 x.fn.x.eachjquery-ui-1.10.3.custom.min.js:6 e.fn.(anonymous function)viewMessage.jsp?call=viewForum&action=1&params=1&forumId=1&addStat=true:2452 addAttachmentVM498:1 (anonymous function)

Actions #3

Updated by Kevin FONTAINE over 7 years ago

  • Status changed from In progress... to Resolved
Actions #4

Updated by Yohann Chastagnier over 7 years ago

  • Status changed from Resolved to Feedback
  • Assignee changed from Kevin FONTAINE to Yohann Chastagnier

Les symptômes du problème ont bien été identifiés.
Cependant les corrections apportées ne peuvent être intégrées.

En effet, ici, le problème majeur est que le source displayAttachedFiles.jsp n'a pas été écrit de manière à ce que le code HTML produit puisse être intégré à plusieurs reprises dans une même page HTML.
Or, dans l'affichage de la liste des messages de l'application Forum, nous sommes dans le cas de l'affichage de plusieurs blocs de fichiers joints. Comme le code HTML produit par displayAttachedFiles.jsp est appelé autant de fois qu'il y a de messages, il y a une multitude de TAG HTML qui ont les mêmes identifiants, les mêmes noms et les routines javascript plantent (qui elles mêmes sont surchargées autant de fois qu'il y a de message).

Le travail qui doit être effectué, pour résoudre ce problème, porte sur un remaniement sévère du code de displayAttachedFiles.jsp afin que ce dernier puisse être appelé dans le contexte de l'affichage d'une liste.

Afin de donner la possibilité d'ajouter ou de mettre à jour des fichiers joints à un message de l'application forum, une solution "temporaire" pourrait être de présenter le bloc des fichiers joints sur la page de modification d'un message.

Je laisse le soin à Nicolas de donner suite à cette anomalie.

Actions #5

Updated by Yohann Chastagnier over 7 years ago

  • Assignee deleted (Yohann Chastagnier)
Actions #6

Updated by Nicolas Eysseric almost 7 years ago

  • Subject changed from Il semble qu'un utilisateur participant ne puisse pas ajouter/modifier de fichier sur ses sujets ou messages to Impossible d'ajouter/modifier un fichier sur un message existant
  • Status changed from Feedback to Qualified

A noter tout de même que grâce au glisser/déposer en HTML5, le problème d'ajout peut être contourné.

Actions #7

Updated by Yohann Chastagnier 23 days ago

  • Status changed from Qualified to In progress...
  • Assignee set to Yohann Chastagnier
  • Target version set to Version 6.3
Actions #8

Updated by Yohann Chastagnier 23 days ago

  • Status changed from In progress... to Resolved
  • % Done changed from 0 to 100

Le problème majeur sur displayAttachedFiles.jsp a été traité et plusieurs blocs de fichiers joints peuvent maintenant être présentés au sein d'une même page HTML.

L'affichage de ce bloc demande d'observer un grand nombre de règles fonctionnelles et repose sur de nombreuses briques de code.
Aussi, la solution choisie pour résoudre cette anomalie n'est pas la plus optimale en termes de code, mais est la plus sûre en termes de non régression fonctionnelle.

PRs:
Actions #9

Updated by Miguel Moquillon 22 days ago

  • Status changed from Resolved to Integration in progress...
Actions #10

Updated by Miguel Moquillon 19 days ago

  • Status changed from Integration in progress... to Closed
Actions

Also available in: Atom PDF