Zizaco / confide

Confide is a authentication solution for Laravel 4
1.19k stars 258 forks source link

Letting Ardent do your Validation #453

Closed jmichaelterenin closed 10 years ago

jmichaelterenin commented 10 years ago

Do you think there might be some changes in the near future where Confide would utilize the latest Ardent package to do its validations? It seems like the best way to go in terms of letting the models do the validations as opposed to validating prior to saving.

Laravel Administrator does the same thing, but it seems to be a cleaner solution to let Ardent do the work.

I welcome input/feeback from the developer or anyone using Confide that has an opinion on this, positive or negative : )

ghost commented 10 years ago

this is very interesting suggestion.

Maybe you already knew, it actually had Ardent do the validation before 4.0. Just quote from Changelog

Version 4.0.0 Beta 1

Dropped Ardent dependency.

Personally, I'd want to make my package decouple from other packages too.

jmichaelterenin commented 10 years ago

I didn't know about the recent decoupling, new to Confide, and relatively new to Laravel, thank you for sharing this with me . I can understand the desire to not be dependent on another package, but I also have seen dependent packages work out quite well in other frameworks, notably Wordpress plugins.

I guess the author has made his decsion on the matter, although it's still in beta. I'm sure he had his reasons, so I will close this out.