swarrot / SwarrotBundle

A symfony bundle for swarrot integration
MIT License
89 stars 59 forks source link

Merged ProcessorConfiguratorEnablableInterface into ProcessorConfigur… #88

Closed odolbeau closed 8 years ago

odolbeau commented 8 years ago

…atorInterface

Linked to #86. @jderusse : I personally prefer this solution to the other one (check isEnabled only for configurator which implements this interface.) for 2 reasons:

What's your opinion?

jderusse commented 8 years ago

:+1:

jderusse commented 8 years ago

I preferred to follow the interface segregation principle. But you're right, it was overkill...