Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Superglobals $_POST, $_GET, etc. #81

Open
guiled opened this issue Jul 21, 2014 · 6 comments
Open

Superglobals $_POST, $_GET, etc. #81

guiled opened this issue Jul 21, 2014 · 6 comments

Comments

@guiled
Copy link
Contributor

guiled commented Jul 21, 2014

Il serait intéressant de rendre plus abstrait l'accès aux super globales.
Personnellement je ne me vois plus utiliser $_POST et $_GET.

Qu'en pensez-vous, dans quelle direction devons-nous aller?

@osaris
Copy link
Contributor

osaris commented Jul 22, 2014

Entièrement d'accord, il faut juste réfléchir à l'API à proposer. Voici quelques idées :

  • accéder à $_POST, $_GET
  • pouvoir contrôler que les input ($_POST, $_GET) comportent bien une variable ? -> has?
  • pouvoir exclure certaines variables des inputs pour passer le reste directement plus loin (à un modèle par exemple) -> except
  • pouvoir ne prendre que certaines variables des inputs pour les passer directement plus loin -> only

@osaris
Copy link
Contributor

osaris commented Jul 22, 2014

Idée supplémentaire pour l'API, avoir la possibilité de récupérer une variable avec une valeur par défaut :

$this->request->get('page', '1');

retournerait le contenu de $_GET['page'] s'il existe ou 1 le cas échéant.

@maitrepylos
Copy link
Contributor

Bonsoir,

Est-il possible d'aliaser les méthodes :

$this->request->get() en $this->get() ou $this->post()

de même avoir une méthode all, qui pourais être simplement $this->post().

idéalement une méthode qui retourne le type de request pour pouvoir faire
ce genre de truc if('POST' === $this->request->method())

Le 22 juillet 2014 22:47, Raphaël Emourgeon [email protected] a
écrit :

Idée supplémentaire pour l'API, avoir la possibilité de récupérer une
valeur avec une valeur par défaut :

$this->request->get('page', '1');

retournerait le contenu de $_GET['page'] s'il existe ou 1 le cas échéant.


Reply to this email directly or view it on GitHub
#81 (comment).

Gérard Ernaelsten
[email protected]

[email protected]

www.formatux.be

Afin de contribuer au respect de l'environnement, merci de n'imprimer ce
courriel qu'en cas de nécessité.

@thehawk970
Copy link

J ai commencer a travailler dans ce sens dans mojo, au moyen d un kit qui a une api suivante ´$this->post->all()´ je sais pas si cela conviendrais :-) http://github.com/camael24/mojo

Envoyé de mon iPad

mardi 22 juillet 2014 23:10 +0200 de Maitrepylos [email protected]:
Bonsoir,

Est-il possible d'aliaser les méthodes :

$this->request->get() en $this->get() ou $this->post()

de même avoir une méthode all, qui pourais être simplement $this->post().

idéalement une méthode qui retourne le type de request pour pouvoir faire
ce genre de truc if('POST' === $this->request->method())

Le 22 juillet 2014 22:47, Raphaël Emourgeon [email protected] a
écrit :

Idée supplémentaire pour l'API, avoir la possibilité de récupérer une
valeur avec une valeur par défaut :

$this->request->get('page', '1');

retournerait le contenu de $_GET['page'] s'il existe ou 1 le cas échéant.


Reply to this email directly or view it on GitHub
< #81 (comment) >.

Gérard Ernaelsten
[email protected]

[email protected]

www.formatux.be

Afin de contribuer au respect de l'environnement, merci de n'imprimer ce
courriel qu'en cas de nécessité.

Reply to this email directly or view it on GitHub .

@guiled
Copy link
Contributor Author

guiled commented Jul 23, 2014

Moi je suis pour un Kit !
J'aime cette idée qui est "on vous propose un framework avec des fonctionnalités qui utilisent les mêmes outils que vous"

@thehawk970
Copy link

Pour completer mon commentaire : https://github.com/camael24/Mojo/blob/master/Application/Controller/Kit/Post.php voici le kit en question il est tout bête mais a le mérite de marcher 👍

Ca ajouter a l'autoload des kit apporter par une précédente PR :D et hop ca s'imbrique parfaitement

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants