Proposta para before_filter (hooks)

2 views
Skip to first unread message

dani...@gmail.com

unread,
Feb 24, 2011, 10:21:27 PM2/24/11
to Drumon Framework
Estava afim de criar um before_filter mais robusto o melhor que consegui até agora foi isso:

https://gist.github.com/843321/d0b0005364a66d298d371559511fa0f23f867223

Preciso de sugestões =)

Podem alterar o gist, só criar um novo arquivo nele ou criar um novo gist.


--
-----------------------------------------
Danillo César de O. Melo - blog
Sook - http://www.sook.com.br

Danillo César

unread,
Feb 25, 2011, 9:05:51 AM2/25/11
to Drumon Framework

dani...@gmail.com

unread,
Mar 2, 2011, 10:33:17 AM3/2/11
to Drumon Framework
Hoje descobri um danado que me deu boa ideia para o hooks do controlador, o nome dele é get_class_vars.
Ele obtem as variáveis de uma classe em um array, com isso eu posso fazer um merge das variaveis do app_controller e do controller atual, coisa que não da para ser feito normalmente.

Aqui está a mágica: (exemplo tirado do código do drumon framework)

        $app_controller_vars = get_class_vars('AppController');
        $this->before_action = array_merge($app_controller_vars['before_action'], $this->before_action);


O resultado, vocês podem ver aqui https://gist.github.com/843321 gostei muito.

Também já enviei para o repositório do drumon framework aqui está o commit https://github.com/sook/drumon_framework/commit/a382c7fce6d86c82819f403507dc86ae49c990cd


Sem pressa sempre achamos uma boa solução. (nem sempre é a melhor para cada pessoa)

Agora vou planejar como implementar isso também para os helpers.

2011/2/25 Danillo César <dani...@gmail.com>
Sook - http://www.sook.com.br

Jairo Junior

unread,
Mar 3, 2011, 7:59:30 PM3/3/11
to drumon-fra...@googlegroups.com, dani...@gmail.com
É essa sua proposta fica bem parecida com alguns frameworks que existem no mercado, principalmente com a estrutura do Ruby. Parabéns, também gostei.
--
Atenciosamente,
Jairo Junior
Sook - Desenvolvendo Inovações.
http://www.sook.com.br
Reply all
Reply to author
Forward
0 new messages