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

#418 closed enhancement (fixed)

Pas de controleur cmdline par defaut dans une application ligne de commande

Reported by: doubleface Owned by: doubleface
Priority: low Milestone: Jelix 1.0.3
Component: jelix-scripts Version: 1.0
Severity: trivial Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

Lorsqu'on cree une application jelix avec par exemple la commande suivante :

jelix --testapp createapp -withcmdline

l'application est bien creee, mais avec un module qui ne contient qu'un controleur "classic" et pas un controleur "cmdline".

Je pense que ca serait mieux d'ajouter ce controleur cmdline pour permettre de faire :

php cmdline.php

sans avoir une erreur en retour

Voir le patch joint.

Par contre j'ai du modifier un tout petit peu le loader de commandes jelix (jelix-scripts/includes/utils.lib.php) pour pouvoir charger plusieurs fois une meme commande. Il me semble que c'est normal pour une telle commande, meme si c'est moint optimise.

Attachments (2)

patch.diff (1.7 KB) - added by doubleface 13 years ago.
Corresponding patch
newpatch.diff (1.8 KB) - added by doubleface 13 years ago.

Download all attachments as: .zip

Change History (13)

Changed 13 years ago by doubleface

Corresponding patch

comment:1 Changed 13 years ago by laurentj

  • Milestone set to Jelix 1.1

comment:2 Changed 13 years ago by laurentj

  • Documentation needed unset
  • review set to review?

comment:3 Changed 13 years ago by laurentj

  • review changed from review? to review-

Le corps du test est bon. Par contre, ce serait mieux d'inclure ces lignes de code directement dans le if qui suit, puisqu'il s'agit du même test ;-). À part ça donc, tout est ok.

(fait "accept ticket" quand tu travailles sur un ticket ;-) )

comment:4 Changed 13 years ago by doubleface

  • Owner set to doubleface
  • Status changed from new to assigned

Changed 13 years ago by doubleface

comment:5 Changed 13 years ago by doubleface

OK, je ferai accept le prochain coup, mais j'ai pas encore beaucoup l'habitude de trac...

Patch modifie

comment:6 Changed 13 years ago by bastnic

  • review changed from review- to review+

Autre truc doubleface : quand tu proposes un nouveau patch, il faut mettre à review+ pour que Laurent (ou autre reviewer) puisse voir qu'il y a un patch à valider.

Je l'ai fait pour toi là :p.

comment:7 Changed 13 years ago by laurentj

  • review changed from review+ to review?

hop hop hop bastnic :-) Pour demander une review, il faut mettre review? pour demander une review par un reviewer, et pas review+. review+ signifie qu'un reviewer a validé le patch ;-)

comment:8 Changed 13 years ago by laurentj

  • review changed from review? to review+

patch ok.

comment:9 Changed 13 years ago by laurentj

  • Milestone changed from Jelix 1.1 to Jelix 1.0.3

Merci doubleface ! (le patch sera inclus plus tard dans le trunk et la branche 1.0.x, pas le temps de le faire maintenant)

comment:10 Changed 13 years ago by bastnic

Faux sur toute la ligne... Ca se sent que je suis fatigué !

comment:11 Changed 13 years ago by laurentj

  • Resolution set to fixed
  • Status changed from assigned to closed

Patch inclus dans la branche 1.0.x et le trunk. Merci pour le patch !

@bastnic : moi aussi j'étais fatigué : ma phrase est un peu bancale :-)

Note: See TracTickets for help on using tickets.