BlogMVC in Laravel4

[Bionik6]

I begin the blog with the Laravel Framework :)

[broucz]

👍
j'attends le premier dépôt pour pull (pas vraiment le temps de faire plus)

[Bionik6]

d'ici quelques heures @broucz ;)

[thujohn]

Voilà ma contribution : https://github.com/thujohn/blogMVC
Je ferai le README demain.

Très bonne idée @Grafikart au passage :)

[Grafikart]

Super tu pourrais faire un fork de ce dépôt, y inclure ton code dans un dossier Laravel et ensuite faire un pull request ? C'est pour centraliser les infos et simplifier la participation (met ton nom/site dans le readme.md).

[Grafikart]

Sinon j'ai regardé le code j'ai compris un peu le fonctionnement faudra que je test :). En revanche tu as bien configurer ton git on dirait que tu rajoute des espaces/lignes en trop :(

[thujohn]

Je m'occupe du fork/readme dès que possible.

Tu entends quoi par "espaces/lignes" en trop ?

[Grafikart]

Tu pourrais faire en sorte que le title change sur la page d'un article ? Tu ferais ça via un @yield ou {{tittle}} ?

[thujohn]

Pas de souci je fais ça ce soir, avec @yield

[Grafikart]

On a actuellement 2 dépôt concernant Laravel sur le système. N'étant pas en mesure de juger quel méthode est meilleur que l'autre j'ai laissé les 2 en place. Si vous vous sentez de faire un merge :D

[mykiwi]

@Bionik6 @Grafikart : ce serait mieux de renommer le dossier en Laravel4. Il n'y a aucun intérêt de laisser le pseudo même si 2 personnes font le projet avec le même framework.

[Grafikart]

C'est le temps de fusionner les dépôts

[thujohn]

Il faudrait se mettre d'accord sur la manière de travailler parce que là l'approche n'est pas la même. Personnellement je préfère avoir l'admin séparée du reste mais les deux approches sont discutables.

[Bionik6]

@thujohn oui je te l'accorde, les deux approches sont différentes. Moi j'ai mis dans mon PostsController tout ce qui concerne les posts avec ses différentes actions (create, update, destroy). Même si c'est l'Admin qui gère ces posts, il suffit juste d'ajouter le filtre "admin" sur ce contrôleur, I believe it makes more sense.... le reste y'a pas trop de différences...

[thujohn]

@Bionik6 Cela a plus de sens sur une grosse appli et sans doute moins sur une app d'exemple. Après je sais que je préfère recréer la structure des dossiers dans le dossier admin pour retrouver plus rapidement ce que je cherche.

[Bionik6]

@thujohn chacun a sa vision des choses et sa façon de coder, je n'irai pas créér un dossier Admin pour, comme tu dis, une app d'exemple, mais bon let's work together and build just a suitable repo for the Laravel Framework, j'accepte toutes les critiques et suggestions :)

[thujohn]

@Bionik6 Tout pareil, à chacun sa façon de faire :) Je pense que ce serait pas mal d'avoir des avis supplémentaires.

[Bionik6]

@thujohn 👍 ouais, ça serait pas mal :)

[Golgotha29]

Je vais donner mon avis :) Bionik6 ayant ouvert en disant qu'il travaillait sur la version Laravel4, c'est sont dépôt qui devrait être prioritaire. Techniquement... chacun a ça façon de faire, et heureusement j'ai envie de dire.

[Grafikart]

Les 2 approches sont intéréssantes je vais placer la version de @Bionik6 dans un sous dossier

[JonathanDaSilva]

Personnelement je trouve qu'il y a du bon dans les deux dépots.
@Bionik6
Se sert du compact pour envoyer ses variables.
Fusionne les actions par table et les filtres (Je trouve ça plus clair surtout sur de large projet).
@thujohn
Se sert du route binding.
Utilise le helper HTML pour ses liens.

Je n'ai pas encore tout regardé, mais les deux sont assez complémentaire je trouve. Une fusion permettrait d'avoir un bon dépôt.

P.S.: Vous avez tous les deux crée une vue de pagination alors qu'elle existe déjà.
Il y a trois vu de base Simple, Slider et Slider-3 (pourquoi 3 allez savoir).
On les trouves dans vendor\laravel\framework\src\Illuminate\Pagination\views.
Pour changer de vue par default il suffit de modifier la variable qui est dans \app\config\view.php par pagination::LeNomsDeLaVue.

[thujohn]

Cette vue n'existe que depuis 3 jours :) laravel/framework@424dd76

slider-3 pour Bootstrap 3 mais il aurait été mieux de l'appeler slider-bs3 à mon avis...

[JonathanDaSilva]

x) Ah effectivement. Je n'avais pas remarqué la date. Je trouvais ça bizarre et j'avais simplement remarqué qu'elle existait déjà.

[Bahhous]

failed to open stream: No such file or directory in C:\wamp\www\Mesframework\CakePHP\webroot\index.php on line 99