search

DarkmiraTour / community-event-manager

Web application with a collection of tools helping people organising community events such as multi-days conference with reminders, CRM and such
GNU General Public License v3.0
21 stars 22 forks source link

Remove bind parameter on param name #236

Open tdutrion opened 4 years ago

tdutrion commented 4 years ago

Not a fan of that bind $currency as it prevents name changes within a class...

Binding should not depend on something the developer is free to change (i.e. param name or index)

Originally posted by @tdutrion in https://github.com/DarkmiraTour/community-event-manager/pull/233

tdutrion commented 4 years ago

Create a value object to put the value in, and then inject using the VO type hint