fix delete file from task tab #32756
Conversation
|
@rycks je crois que c'est surtout un soucis de méthode qu'il faudrait uniformiser !!! la méthode qu'il faut bannir et qui ne fonctionne pas :
et l'appel qu'il faut utiliser :
|
|
@rycks il faut appeler "document.php" et virer "task.php" ici |
|
@hregis hum ? why did you close that PR ? |
|
@rycks à force de mettre des pansements sur des jambes de bois on rajoute des effets de bords... regarde le lien qui fonctionne et fait en sorte d'avoir le même lien ! |
|
ok then maybe i have to do a better fix, you don't have to CLOSE that PR ... just tell me "please do like that" ... with such close before answer you send a bad message to people who just want to contribute ! |
|
@rycks désolé parfois je suis abrupte mais il y a des évidences qui me paraissent évidentes 😄 |
|
@rycks je te kiss la molaire ! 💋 |
|
@rycks le grand bleu ? |
|
@rycks alors je rembobine... Monsieur Eric auriez-vous l'obligeance de faire en sorte que le lien de la poubelle qui ne fonctionne pas soit le même que celui qui fonctionne avant que "ça me casse les couilles" !! 😄 😄 |
|
@hregis c'est pas tant la manière de le qui compte mais c'est le fait de cloturer la pr sans laisser la possibilité au contributeur de réagir qui est "de trop" ... |
|
and about your suggestion to change from remove_file to document.php?action=deletefile that's a PR on 18.0 branch ... are you sure we have to do such code update ? |
|
@rycks tu as raison et je m'en excuse... vieux ! Dolibarrien ! 💋 |
il faut demander au grand maître ! @eldy ? C'est bloquant donc on peut modifier ? |
|
@rycks à mon avis l'appel à task.php au lieu de document.php pour supprimer un fichier devait être historique et n'a jamais été modifié ! |
|
@rycks tu vois de quoi je parle au niveau de l'histoire hein ? vieux développeur ! 😄 |
|
@hregis oeuf corse entre vieux codeurs ! |
|
@eldy |
| //in case of a task filename is like PJ2211-0001%2FTK2211-0001%2FTK2211-0001_task.odt | ||
| //then dol_sanitizeFileName will convert it to PJ2211-0001_FTK2211-0001_FTK2211-0001_task.odt | ||
| //and that file does not exists so can't be deleted | ||
| $sanitizedFileName = dol_sanitizeFileName(GETPOST('file')); |
There was a problem hiding this comment.
This fix is very suspicious. If we have to manipulate the parameter fil to get the correct path of file, the problem is probably in the link where parameter file is wrong.
There was a problem hiding this comment.
of course but tasts are a the only case where files are in a subdirectory ... so please let me know what kind of parameter we could make to do it better ?
.../...file=PJ2211-0001/FTK2211-0001/FTK2211-0001_task.odt.../...
must be replaced by what ?
eldy
added
the
Discussion
eldy
added
PR to fix - See feedback in comments
|
So fix in v18 could be done the sameway it was done in v20. |
|
sorry I didn't see that it was for branch 18 |
|
@rycks tu doutes de mes pouvoirs petit scarabée ?! 😄 |
On task tab there is the list of linked files:
the trash icon have that sort of url
/projet/tasks/task.php?id=1&action=remove_file&token=xxx&file=PJ2211-0001%2FTK2211-0001%2FTK2211-0001_tache_modele-02.odt&entity=1
then the source code will sanitize file name and replace %2 with underscore and the consequence is delete can't be done but the success message is displayed !
this PR will fix that bug