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 10 years ago

Closed 10 years ago

#873 closed bug (fixed)

Documentation Cmdline + jForms Upload

Reported by: Nigoki Owned by:
Priority: low Milestone: Jelix 1.1.2
Component: web site: documentation - do not use - Version: 1.1
Severity: minor Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

Le code source de la création du fichier cmdline.php manuellement ne fonctionne pas dans tous les cas. Je ne sais pas ce qu'il en ai quand le fichier est généré par jelix-scripts/jelix.php mais avec les includes posent problèmes lorsqu'on lance le script en absolue, principalement pour les cron.

Exemple :

Actuellement dans la donc on a

require ('../../lib/jelix/init.php');
require_once ('.././application-cli.init.php');

Cela fonctionne quand on fait

php cmdline.php module~action:fonction

Mais on obtient dans les inclusions quand on lance le script comme ceci :

/usr/bin/php /mon/chemin/en/absolue/cmdline.php module~action:fonction

Pour résoudre ce problème j'ai modifié les deux premières lignes du fichier cmdline.php comme ceci :

require_once (dirname(__FILE__).'/../../lib/jelix/init.php');
require_once (dirname(__FILE__).'/../application-cli.init.php');

Une autre remarque qui pourrait être pertinent de rajouter dans la documentation au niveau de jForms dans la partie <upload>
Pour les fichiers .jpg et .jpeg, avec IE7, dans le tableau renvoyé par $_FILES[ref], on a 'type' qui est égale a image/pjpeg et non pas simplement image/jpeg.

Alors est il utile d'ajouter un warning dans la doc car si on met simplement mimetype="image/jpeg" dans le fichier xml, on aura la fonction check() de jFormsControlUpload qui renverra false avec IE et true avec firefox et les autres je pense.

Change History (3)

comment:1 Changed 10 years ago by laurentj

  • Priority changed from normal to low
  • Severity changed from normal to minor

Actuellement dans la doc on a require ('../../lib/jelix/init.php');

L'inclusion de ce fichier n'est pas necessaire, car depuis jelix 1.1, il est normalement inclus par le application.init.php. je viens de corriger la doc.

Mais on obtient dans les inclusions quand on lance le script comme ceci : (...)

Merci de créer un ticket pour qu'on corrige ça.

Alors est il utile d'ajouter un warning dans la doc

bah oui, et si tu pouvais le faire, çe serait cool. (en anglais et en français, et dans le manuel 1.0 et 1.1)

comment:2 Changed 10 years ago by laurentj

  • Type changed from new feature to bug

comment:3 Changed 10 years ago by laurentj

  • Milestone set to Jelix 1.1.2
  • Resolution set to fixed
  • Status changed from new to closed

ok, ticket #873. et doc ok. on peut fermer.

Note: See TracTickets for help on using tickets.