developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 13 years ago

Closed 13 years ago

#269 closed bug (fixed)

Bug dans jFormsBase : saveFile() et saveAllFiles()

Reported by: bballizlife Owned by:
Priority: normal Milestone: Jelix 1.0beta3
Component: jelix:forms Version: 1.0 beta2.1
Severity: major Keywords:
Cc: Blocked By:
Blocking: Documentation needed:
Hosting Provider: Php version:

Description (last modified by bballizlife)

Les méthodes saveFile() et saveAllFiles() de la classe jFormsBase ne fonctionnent pas.

  • Dans les 2 méthodes, le test suivant est erroné :

$_FILES[$controlName] != UPLOAD_ERR_OK

A corriger par : $_FILES[$controlName]['error'] != UPLOAD_ERR_OK

  • Dans saveFile(), le nom du fichier de destination n'est pas attribué lors du move_uploaded_file() et ne tiens pas compte ni du nom d'origine du fichier, ni de l'éventuel paramètre $alternateName.
  • Dans saveAllFiles(), le nom du fichier de destination n'est pas attribué non plus lors du move_uploaded_file()

Attachments (2)

ticket-269.diff (2.2 KB) - added by bballizlife 13 years ago.
Patch pour correction du bug
ticket-269-2.diff (2.8 KB) - added by bballizlife 13 years ago.
Nouveau patch intégrant le commentaire de laurentj

Download all attachments as: .zip

Change History (5)

Changed 13 years ago by bballizlife

Patch pour correction du bug

comment:1 Changed 13 years ago by bballizlife

  • Description modified (diff)

comment:2 Changed 13 years ago by laurentj

Patch ok, mais il faut toutefois rajouter un test sur la valeur de $path pour ajouter un / final si il est manquant, sinon le chemin de destination complet sera erronné.

   if($path) {
     ...
   } else if (substr($path,-1,1) != '/') {
       $path.='/';
   }

Review ok si ce test est ajouté.

Changed 13 years ago by bballizlife

Nouveau patch intégrant le commentaire de laurentj

comment:3 Changed 13 years ago by bballizlife

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.