IfnotFr / eloquent-vuex-php

Realtime model synchronization between Vuex (VueJs) and Eloquent (Laravel).

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Naming

mathieutu opened this issue · comments

@ifnot je trouve que les noms de tes méthodes et classes sont pas fous.
Exemple: pour moi broadcastWith() c'est une action, qui prend des arguments qui correspondent au with.
là tu retournes juste un array, c'est bizare.
idem pour les autres methodes.

Et pour ton trait, pour moi un trait c'est qqchose qui te rajoute des actions, comme un service que tu rajoute à ton model donc il doit avoir un nom d'action. isStore pour moi c'est un truc qui retourne un booléen, comme tous les is* ou has*. Exemple pour mon package: JsonExportable pour l'interface, JsonExporter pour le trait