Codestin Search App

npotier · 2013-08-08T16:17:53Z

Hi,

After few talks about it, here is a pull request to add yml file configuration and a command to generate theses file from an entity.

YML file configuration

Like doctrine, YML file configuration for an entity must be placed in a special directory and have a special name MyBundle/Resources/config/grid/MyEntity.myGroup.grid.yml
The YML file structure is the following:

# Resources/config/grid/MyEntity.myGroup.grid.yml
namespace\of\MyEntity
    Source:
        columns: 
            id: ~
            myColumn1: ~
        groupBy: ~
        filterable: ~
    Columns:
        id:
            filterable: true
            visible: true
            id: id
            title: columnTitle
        myColumn1: ....
    ...

Annotation and YML Driver are both loaded in services.xml

YML Generation command

The command will allow you to generate a YML config file for an entity and a group:

php app/console apydatagrid:generate:grid --entity="MyBundle:MyEntity" --group="default"

The file will be generated and written in src/MyBundle/Resources/config/grid/MyEntity.default.grid.yml.
The generation of the file is based on a skeleton template stored in APYDataGrid/Resourses/skeleton/grid/grid.yml.twig.

Feel free to make any comments on this PR !

Possibilité d'utiliser un fichier yml à la place des annotations. Pour l'instant, le fichier MyEntity.grid.yml est recherché dans le même répertoire que MyEntity.php Le fichier MyEntity.yml a cette structure : Namespace\of\MyEntity: Source: columns: id : groups : [group1, group2] myColumn: ~ Columns: id: filterable: true visible: true id: id title: MonId myColumn: id : idColumn title : title filterable: true visible: true TODO : - Trouver une stratégie pour déterminer l'emplacement du fichier yml associé à l'entité - Valider le format du fichier YML en amont (utilisation de TreeBuilder ?) - Ecrire des tests

…es/config/grid/MyEntity.MyGroup.yml

…t un groupe

Fait cohabiter les Drivers Annotation et YAML

Abhoryo · 2013-08-08T16:44:34Z

C'est une option.

Abhoryo · 2013-08-08T16:52:50Z

Ça m'a l'air top. Vous êtes bien motivés.
Au final la version xml ne devrait pas être trop longue à faire aussi et on pourrait préciser le format (yml ou xml) dans la ligne de commande.

J'aimais bien les annotations mais maintenant il y a trop d'options.
Sinon j'ai jamais testé mais les annotations et ce nouveau driver mélangent les options, si oui c'est les annotations qui ont le dessus, non ?

npotier · 2013-08-08T17:35:50Z

Merci pour le feedback :)

En effet le XML ne devrait pas être compliqué à mettre en place, et une
option permettant de définir le format ferait l'affaire.

Les deux Drivers peuvent cohabiter. Le driver qui prend le dessus dépend
des priorités configurées dans services.xml

Nicolas

Le 8 août 2013 à 18:54, Petit Yoann [email protected] a écrit :

Ça m'a l'air top. Vous êtes bien motivés.
Au final la version xml ne devrait pas être trop longue à faire aussi et on
pourrait préciser le format (yml ou xml) dans la ligne de commande.

J'aimais bien les annotations mais maintenant il y a trop d'options.
Sinon j'ai jamais testé mais les annotations et ce nouveau driver mélangent
les options, si oui c'est les annotations qui ont le dessus, non ?

—
Reply to this email directly or view it on
GitHubhttps://github.com//pull/449#issuecomment-22337707
.

Abhoryo · 2013-08-08T20:14:37Z

Les priorité sont définies dans le bundle et non par l'utilisateur si je ne m'abuse. Il faudra voir comment résoudre ce problème.

npotier · 2013-08-08T20:56:39Z

Tout a fait. C'est le cas... Pour l'instant. Peut être faut il donner la
possibilité de choisir par configuration quels Drivers activer, et dans
quel ordre.

Nicolas.

Le 8 août 2013 à 22:16, Petit Yoann [email protected] a écrit :

Les priorité sont définies dans le bundle et non par l'utilisateur si je ne
m'abuse. Il faudra voir comment résoudre ce problème.

—
Reply to this email directly or view it on
GitHubhttps://github.com//pull/449#issuecomment-22352103
.

Abhoryo · 2013-08-08T22:08:20Z

Oui, peut être du style:
drivers: ["annotation","yml","xml"]
par défaut.
Mais pour cela, il faudrait leur affecter un nom.

Sinon, je viens de voir, le générateur que tu as fait ne génère que le nom des colonnes, envisages-tu de rajouter automatiquement les types vis à vis des propriétés Doctrine?
D'ailleurs, je pense qu'il faudrait filtrer ces propriétés pour ne faire apparaître que les propriétés qui sont des champs doctrine sinon ça ne va pas marcher.

Abhoryo · 2013-08-08T22:09:42Z

Si je comprends bien, columns sera un tableau de clés correspondants aux noms des colonnes avec des valeurs null, non ?

Oui c'est ça. Cela vient du fait que pour une raison qui m'échape, la génération du code

columns: [id, myColumn1]

entraine une erreur, alors que :

columns : id: ~ myColumn1: ~

fonctionne

In fact you have to write this array like this:
columns: ["id", "myColumn1"]
But I think a simple string is enough like in annotation:
columns: "id, myColumn1"

Abhoryo · 2013-09-05T14:58:53Z

Bonjour,
On en était où sur ce post ?

npotier · 2013-09-05T15:17:58Z

Salut,

Il reste à rendre configurable au niveau des parsers et de leur ordre, et aussi vérifier que les relations ManyToOne sont bien prises en compte

Abhoryo · 2013-09-05T15:34:58Z

Ah ok. Par contre elle n'est plus mergeable.

…ation has been found

L0rD59 · 2013-12-18T12:32:06Z

ping @npotier @Abhoryo cette feature m’intéresse :)

Abhoryo · 2013-12-18T13:34:01Z

On ne peut pas encore tout à fait la merger. Mais ca va venir. tu peux toujours utiliser le code de ce PR.

Abhoryo · 2013-12-18T13:34:34Z

C'est les caractères BOM non ? Pourquoi les rajouter ?

Bonjour,

Cela n'a en effet rien à voir avec la gestion XML de la configuration.

Les caractères BOM ont été ajoutés de façon à ce que Excel prenne en compte les accents lors de l'ouverture directe des fichiers CSV.

Ca pourrait être enlevé ou rendu paramétrable en effet.

Oui plutôt un paramètre mais sur le coup ce n'est pas vraiment le même PR.
Donc ici faudrait le supprimer, et ouvrir un nouveau PR ou au moins une issue pour s'en rappeler.

plfort · 2014-07-17T15:19:24Z

Des nouvelles concernant cette PR ? Je suis très intéressé !

Abhoryo · 2014-07-19T15:48:52Z

Il manque la documentation :D

clubdesarrolladores · 2014-11-18T15:41:01Z

Hi, this is ready for merge in master? we need yml support. Thanks.

Piquinikis · 2014-11-18T15:45:09Z

Hi, I need this pull request is accepted, thanks!

espinozaguillermo · 2014-11-18T15:50:43Z

Hi, I need yml support. Merge this pull request, thanks!

npotier · 2014-11-21T15:43:01Z

Hi guys,

Thanks for your interest for this PR :)

I'll write some doc in order to make this PR mergeable by @Abhoryo

clubdesarrolladores · 2014-11-22T01:07:21Z

Thanks, this is a great PR!!

Janest · 2014-12-02T13:45:02Z

Please merge and tag, I'm very interested in this PR too.

npotier · 2015-01-22T09:33:36Z

Hello / Salut @Abhoryo. Avant tout, bonne année ;)

Peux tu me dire ce qu'il manquerait à cette PR pour être mergée ?

Merci.

Nicolas.

Abhoryo · 2015-02-20T21:25:13Z

Salut, @npotier , Pourrais-tu corriger le fichier summary qui a plein de tabulations.
Je mergerai quand ce sera fait, tant pis pour la régression.

phpetra · 2015-03-07T19:52:31Z

Hi guys,
Can I ask about the status of this PR?
My french is not so good, what exactly needs to be done?

Abhoryo · 2015-03-07T20:46:22Z

Wait for a modification from @npotier and this is good for merge.

npotier · 2015-03-22T12:33:47Z

Salut @Abhoryo, c'est fait. A ta dispo pour avancer.

Nicolas.

YAML file configuration and Command to generate YML from an entity

npotier · 2015-05-10T11:57:55Z

Hourra :)

Merci.

phpetra · 2015-05-11T09:14:46Z

Nice indeed! :)

qferr · 2015-05-11T09:28:58Z

+1

Abhoryo · 2015-05-11T20:33:27Z

@npotier can you do something about exceptions that everyone encounter ?

This reverts commit 08f109c, reversing changes made to 7541ae2.

npotier added 7 commits August 7, 2013 09:43

WORK IN PROGESS : emplacement des fichiers YML dans @MyBundle/Resourc…

75998d8

…es/config/grid/MyEntity.MyGroup.yml

commande permettant de generer le fichier Grid associe a une entite e…

3dc0a0b

…t un groupe

Rend le Driver YAML silencieux si le fichier est absent

5fffa22

Fait cohabiter les Drivers Annotation et YAML

code cleanup

36581ea

code cleanup

a8b009b

code cleanup

663d2f4

Abhoryo reviewed Aug 8, 2013
View reviewed changes

Comment thread Command/GenerateGridCommand.php Outdated

Abhoryo Aug 8, 2013

Copy link
Copy Markdown

Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est une option.

Abhoryo reviewed Aug 8, 2013
View reviewed changes

Matthieu Auger added 2 commits August 19, 2013 18:06

Arguments instead of options

0a38ff4

Fix endlines

441ef3c

npotier and others added 8 commits October 3, 2013 10:39

fix merge

d4c21ae

configuration propre

618ec52

Merge remote-tracking branch 'upstream/master'

e29f676

fix service config

803f2e5

fix

7f9202a

remove config constraints

8b04830

annotation : return empty array instead of array_keys where no inform…

628c7c0

…ation has been found

fix export UTF8 Excel CSV

8e01fa8

Abhoryo reviewed Dec 18, 2013
View reviewed changes

Merge remote-tracking branch 'upstream/master'

eedbdf2

npotier added 4 commits November 21, 2014 16:44

Merge remote-tracking branch 'upstream/master'

43a6406

YML documentation

b3872b6

YML documentation

40c3c88

YML documentation

2280cd3

Dexter Morgan added 2 commits December 8, 2014 15:03

generate exception if no metadata found from all drivers

de94083

generate exception if no metadata found from all drivers

fce3369

npotier added 2 commits March 22, 2015 13:24

Merge remote-tracking branch 'upstream/master'

5cdfedf

remove tabs in doc

f3f2541

Abhoryo pushed a commit that referenced this pull request May 10, 2015

Merge pull request #449 from acseo/master

08f109c

YAML file configuration and Command to generate YML from an entity

Abhoryo merged commit 08f109c into APY:master May 10, 2015

Abhoryo pushed a commit that referenced this pull request May 11, 2015

Revert "Merge pull request #449 from acseo/master"

312cdf9

This reverts commit 08f109c, reversing changes made to 7541ae2.

farzadsbakhtiar mentioned this pull request Aug 18, 2015

Revert of yml support merge #748

Closed

Uh oh!

Conversation

npotier commented Aug 8, 2013

YML file configuration

YML Generation command

Uh oh!

Abhoryo Aug 8, 2013

Choose a reason for hiding this comment

Uh oh!

Abhoryo commented Aug 8, 2013

Uh oh!

npotier commented Aug 8, 2013

Uh oh!

Abhoryo commented Aug 8, 2013

Uh oh!

npotier commented Aug 8, 2013

Uh oh!

Abhoryo commented Aug 8, 2013

Uh oh!

Abhoryo Aug 8, 2013

Choose a reason for hiding this comment

Uh oh!

npotier Aug 9, 2013

Choose a reason for hiding this comment

Uh oh!

Abhoryo Aug 9, 2013

Choose a reason for hiding this comment

Uh oh!

Abhoryo commented Sep 5, 2013

Uh oh!

npotier commented Sep 5, 2013

Uh oh!

Abhoryo commented Sep 5, 2013

Uh oh!

L0rD59 commented Dec 18, 2013

Uh oh!

Abhoryo commented Dec 18, 2013

Uh oh!

Abhoryo Dec 18, 2013

Choose a reason for hiding this comment

Uh oh!

npotier Dec 19, 2013

Choose a reason for hiding this comment

Uh oh!

Abhoryo Dec 19, 2013

Choose a reason for hiding this comment

Uh oh!

plfort commented Jul 17, 2014

Uh oh!

Abhoryo commented Jul 19, 2014

Uh oh!

clubdesarrolladores commented Nov 18, 2014

Uh oh!

Piquinikis commented Nov 18, 2014

Uh oh!

espinozaguillermo commented Nov 18, 2014

Uh oh!

npotier commented Nov 21, 2014

Uh oh!

clubdesarrolladores commented Nov 22, 2014

Uh oh!

Janest commented Dec 2, 2014

Uh oh!

npotier commented Jan 22, 2015

Uh oh!

Abhoryo commented Feb 20, 2015

Uh oh!

phpetra commented Mar 7, 2015

Uh oh!

Abhoryo commented Mar 7, 2015

Uh oh!

npotier commented Mar 22, 2015

Uh oh!

npotier commented May 10, 2015

Uh oh!

phpetra commented May 11, 2015

Uh oh!

qferr commented May 11, 2015

Uh oh!

Abhoryo commented May 11, 2015