search

piotrmurach / loaf

Manages and displays breadcrumb trails in Rails app - lean & mean.
MIT License
406 stars 24 forks source link

Proc was removed? #7

Closed philipgiuliani closed 9 years ago

philipgiuliani commented 9 years ago

Since this method isn't used anymore, procs are not working: https://github.com/peter-murach/loaf/blob/master/lib/loaf/controller_extensions.rb#L28

The I18n Scope is also not configurable in the current version.

piotrmurach commented 9 years ago

Hi Philip, thanks for reporting this. On the issue of procs, in my recent work I didn't have requirement to use procs and thus deemed it unnecessary for the release. However, if you could provide a use case or failing test I would be happy to look into it. It terms of translations, would you have time to investigate? There is already a test app to help you out testing this feature.

piotrmurach commented 9 years ago

@philipgiuliani Please see readme for details, both scope and procs should work. Please get 0.5.0 version if you still need this functionality.

philipgiuliani commented 9 years ago

Oh sorry i forgot to answer you. I need it for Model.model_name.human for example. So i dont have to duplicate the model names in the translations.

Thanks i will start with the implementation on sunday.