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

Closed 11 years ago

Last modified 11 years ago

#937 closed enhancement (fixed)

Give permission has jelix-scripts to create applications

Reported by: Lipki Owned by:
Priority: lowest Milestone: Jelix 1.2 beta
Component: jelix-scripts Version: 1.1
Severity: minor Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

Actuellement seul createapp peut créer des applications a cause de :

if($commandName !='createapp' && $commandName !='help'){
    echo("Error: the given application doesn't exist (".JELIXS_APPTPL_PATH." )\n");
    exit(1);
}

Je propose d'autoriser les jelix-scripts a ne pas spécifier de nom d'application, Dans le cas de createapp on pourrait écrire :

php jelix.php createapp appname (c'est juste un exemple, je me rend bien compte qu'il faut garder la forme actuelle pour la compatibilités)

Je ne connais pas assez cette partie pour savoir si c'est une possibilité ou pas.

Change History (9)

comment:1 Changed 11 years ago by laurentj

  • Priority changed from normal to lowest
  • review review? deleted
  • Severity changed from normal to minor

Et comment les commandes savent sur quelle application faire leur process, si on n'indique pas le nom de l'application ?

Et je ne comprend pas l'origine de la demande.

comment:2 Changed 11 years ago by Lipki

Pour l'origine de la demande je cherche a faire un commande qu'en plus de créer l'application va faire d'autres tâche, exécuter d'autres commandes, et au final créer une application différente de celle créer par createapp.

Comment les commandes savent sur quelle application faire leur process,

Comment fait createapp ? on l'exécute alors que l'appli n'existe pas encore.

alors peut être que c'est jelix-scripts qui créer l'application et appel createapp ensuite, mais justement ce fonctionnement est limité a createapp. J'aimerais que d'autres commandes puisse avoir ce droit.

Autre choses pourquoi une commande devrait s'exécuter sur une application, on pourrait imaginer une commande qui installe un module a coté de la lib, ou dans la lib elle même, une commande qui modifie certain paramètre de la lib, ou je ne sait quoi d'autres.

C'est commandes nous offre bcp de possibilité, mais finalement on est vite bloquer, par cette simple condition.

if($commandName !='createapp' && $commandName !='help')

comment:3 Changed 11 years ago by Lipki

  • review set to review?

comment:4 Changed 11 years ago by bballizlife

  • review changed from review? to review-

Je ne comprends pas le contexte ni le pourquoi/comment de la demande donc pas de review+ pour le moment. Et en l'état je review- même car nous devons obliger les jelix-scripts à fournir un nom d'application pour savoir où créer le module, la zone, le crud, ...

Attention ce n'est pas un rejet, c'est une demande de précision sur ce que tu veux faire et pourquoi et nous sommes au moins 2 dans ce cas j'ai l'impression.

comment:5 Changed 11 years ago by Lipki

Le pourquoi je viens de le dire

1) Autoriser les développeur a faire des scripts équivalent a createapp, donc capable des créer une application a partir de rien

2) Permettre au développeur de créer des scripts n'étant pas lier a une application.

Je l'ai fait pour mes propres besoin, mais c'est toujours mieux les modifs officiel.

comment:6 Changed 11 years ago by laurentj

  • review review- deleted

Merci d'arreter de jouer avec le champs review. Il n'est là que pour les patchs. Et je ne vois pas de patch ici.

comment:7 Changed 11 years ago by laurentj

lipki, j'ai compris ta demande. pourquoi pas.

comment:8 Changed 11 years ago by laurentj

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

J'ai rajouté une propriété applicationMustExist sur l'objet command. Par defaut, c'est à true.

comment:9 Changed 11 years ago by Lipki

Ok cool.

Note: See TracTickets for help on using tickets.