Skip to content

Instantly share code, notes, and snippets.

@gillien
Created April 30, 2012 12:21
Show Gist options
  • Save gillien/2557864 to your computer and use it in GitHub Desktop.
Save gillien/2557864 to your computer and use it in GitHub Desktop.
- filter_description = [ { :value => :approved, :traduction => t('approved'), :condition => true }, { :value => :pending, :traduction => t('pending'), :condition => true }, { :value => :canceled, :traduction => t('canceled'), :condition => true } ]
- sort_by_description = { :created_at => t('created_at') }
#clients_my_credits{ :'data-module' => 'Groups.Clients.MyCredits' }
= render :partial => "shared/tm_table", :locals => { :controller => 'groups/clients', :action => 'my_credits', :elements => @wire_requests, :empty_msg => "empty_table_bottom_my_invoices", :partial_for_element => "/groups/clients/my_credits/my_wire_transfer_row_table", :partial_for_action => '/groups/clients/my_credits/buy_bundles', :sort_by_description => sort_by_description, :filter_description => filter_description }
@apneadiving
Copy link

Deux petites remarques:

  • c'est une très mauvaise habitude de déclarer des variables dans la vue, j'espère que c'est juste à titre d'illustration
  • il serait idéal d'harmoniser les notations au fur et à mesure et d'utiliser le 1.9.2 style pour les Hash: key: value au lieu de :key => value. Ca te va?

@gillien
Copy link
Author

gillien commented May 2, 2012

Merci pour tes feedbacks. C'etait effecitvement une mauvaise habitude pour les variables dans la vue. Comme ce sont des valeurs uniquement dediees a la vue je les avais mises dedans. Je vais voir pour les sortir de la vue,
C'est note pour l'harmonisation ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment