adherent 0.1.3 → 0.1.4

data/app/helpers/adherent/application_helper.rb

@@ -14,20 +14,35 @@ module Adherent
     end
     
     # Pour transformer un montant selon le format numérique français avec deux décimales
-    def virgule(montant)
+    def two_decimals(montant)
       ActionController::Base.helpers.number_with_precision(montant, precision:2) rescue '0,00'
     end
     
-    # affiche un nombre avec deux décimales
-    # utilisé dans les vues qui demandent un montant
-    def two_decimals(montant)
-      sprintf('%0.02f',montant)
-    rescue
-      '0.00'
-    end 
-    
+        
     def icon_to_users
       icon_to 'users.png', members_path, title:'Liste des membres' 
     end
+    
+    def list_imputations(payment)
+      
+      content_tag(:ul) do
+          payment.reglements.map do |r|
+          content_tag(:li) do
+            imputation_with_actions(payment, r)
+          end
+        end.join.html_safe
+      end
+    end
+    
+    def imputation_with_actions(pay, reglt)
+      html = ''
+      html << "Adhésion #{reglt.adhesion.member.to_s} pour #{number_to_currency(reglt.amount, locale: :fr)}"
+      html << "&nbsp;&nbsp;"
+      html << "#{icon_to 'detail.png', payment_reglement_path(pay, reglt)}"
+#      html << "#{icon_to 'imputation-edit.png', edit_payment_reglement_path(pay, reglt), title:'Modifier l\'imputation'}"
+#      html << "#{icon_to 'imputation-delete.png', payment_reglement_path(pay, reglt), title:'Supprimer l\'imputation'}"
+      html.html_safe
+    end
+    
   end
 end

data/app/helpers/adherent/payments_helper.rb

@@ -1,14 +1,5 @@
 module Adherent
   module PaymentsHelper
-    def list_imputations(payment)
-      
-      content_tag(:ul) do
-          payment.reglements.map do |r|
-          content_tag(:li) do
-            "Adhésion #{r.adhesion.member.to_s} pour #{number_to_currency(r.amount, locale: :fr)}"
-          end
-        end.join.html_safe
-      end
-    end
+    
   end
 end

data/app/models/adherent/adhesion.rb

@@ -32,12 +32,15 @@ module Adherent
            } 
     end
     
-    # liste toutes les adhésions qui ne sont pas payées
-    # TODO voir pour faire une requête SQL ou un scope qui serait surment moins
-    # consommatrice de mémoire
-    def self.unpaid
-      all.reject {|adh| adh.is_paid?}
-    end
+    # liste toutes les adhésions qui ne sont pas payées.
+    # pour rappel un where ne fonctionne pas avec un aggrégat obligeant à utiliser la clause having
+    # reglements_amount.to_i donne le montant des règlements enregistrés
+    # to_i est nécessaire car le retour fait par Rails est un string
+    scope :unpaid, select('adherent_adhesions.*, sum(adherent_reglements.amount) as reglements_amount').
+        joins('left join adherent_reglements on adherent_reglements.adhesion_id = adherent_adhesions.id').
+        group('adherent_adhesions.id').
+        having('adherent_adhesions.amount > 0 AND (adherent_adhesions.amount > sum(adherent_reglements.amount) OR sum(adherent_reglements.amount) IS NULL)')
+ 
     
     # méthode mise en place pour assurer une mise en forme lisible
     # dans la sélection des imputations
@@ -67,6 +70,7 @@ module Adherent
       imputation = [montant, due].min 
       r = reglements.new(amount:imputation)
       r.payment_id = payment_id
+      puts r.errors.messages unless r.valid?
       r.save!
       r
     

data/app/models/adherent/member.rb

@@ -12,9 +12,9 @@ module Adherent
     validates :number, :name, :forname , :organism_id, :presence=>true
     validates_uniqueness_of  :number, :scope=>:organism_id
     
-    # array des adhésions impayées par ordre de date
+    # arel des adhésions impayées par ordre de date
     def unpaid_adhesions
-      adhesions.order(:to_date).reject {|adh| adh.is_paid? }
+      adhesions.order(:to_date).unpaid
     end
     
     # indique s'il y a des adhésions impayées pour ce membre

data/app/models/adherent/payment.rb

@@ -1,5 +1,21 @@
 module Adherent
-  class Payment < ActiveRecord::Base
+  
+  # La classe Payment permet d'enregistrer les payments effectués par les adhérents
+  # 
+  # Un callback after_create tente d'imputer le réglement sur une adhésion qui soit due.
+  # 
+  # Comme il est possible qu'un adhérent fasse un réglement pour plusieurs personnes (les 
+  # membres de sa famille par exemple), il est possible d'éclater des payments sur 
+  # différents réglements. Un payment a donc plusieurs réglements.
+  # 
+  # Des méthodes non_impute et imputation_on_adh(adhesion) permettent de savoir 
+  # si la totalité du payment a été affectée à un réglement et d'imputer le montant
+  # sur une adhésion spécifiée.
+  # 
+  # Un validator spécifique est mis en place pour interdire de diminuer le payment 
+  # en dessous des montants déja imputés.
+  #
+  class Payment < ActiveRecord::Base 
     
     has_many :reglements, :dependent=>:destroy
     belongs_to :member
@@ -7,11 +23,12 @@ module Adherent
     attr_accessible :amount, :date, :mode
     
     validates :amount, :date, :mode, :member_id, presence:true
+    validates :amount, :over_imputations=>true
     
-    pick_date_for :date
+    pick_date_for :date 
     
-    # le montant ne peut être que positif
-    validates_numericality_of :amount, greater_than: 0
+    
+    validates_numericality_of :amount
     # le mode doit être dans les MODES
     validates_inclusion_of :mode, in: Adherent::MODES
     
@@ -29,7 +46,11 @@ module Adherent
     # calcule le montant du paiement qui n'a pas été imputé, donc qui 
     # ne correspond pas à des réglements
     def non_impute
-      amount - reglements(true).sum(:amount)
+      amount - impute
+    end
+    
+    def impute
+      reglements(true).sum(:amount)
     end
     
     
@@ -50,7 +71,7 @@ module Adherent
       # on crée un réglement du montant dû dans la limité du montant du paiement.
       a_imputer = amount
       member.unpaid_adhesions.each  do |adh|
-        if a_imputer > 0
+        if a_imputer > 0 
           reglemt = adh.add_reglement(id, a_imputer)
           a_imputer -= reglemt.amount
         end

data/app/views/adherent/allpayments/_payment.html.haml

@@ -0,0 +1,14 @@
+%tr
+  %td
+    = payment.member.to_s
+    = icon_to 'payment.png', member_payments_path(payment.member), title:'Liste des paiements de cet adhérent'
+  %td= payment.date
+  %td.numeric= two_decimals payment.amount
+  %td= payment.mode
+  %td= list_imputations(payment)
+  %td.numeric= two_decimals payment.non_impute
+
+  %td
+    - if (payment.non_impute != 0.0)
+      = icon_to 'imputation.png', new_payment_reglement_path(payment), title: 'Imputer le solde sur une adhésion'
+    = icon_to 'detail.png', member_payment_path(payment.member, payment)

data/app/views/adherent/allpayments/index.html.haml

@@ -0,0 +1,22 @@
+= content_for :menu do
+  = icon_to_users
+  
+
+
+.champ
+  %h3 Table de tous les paiements
+  
+  %table.table.display.table-striped.table-bordered.table-condensed#allpayments(cellpadding="0" cellspacing="0" border="0")
+    %thead
+      %tr
+        %th De
+        %th.date-euro Le
+        %th Montant (€)
+        %th Par
+        %th Imputé sur 
+        %th Reste à imputer (€)
+        %th Actions
+    %tbody
+      = render partial:'adherent/allpayments/payment', collection:@payments
+        
+            

data/app/views/adherent/members/index.html.erb

@@ -1,6 +1,7 @@
 <%= content_for :menu do %>
-<%= icon_to 'home.png', main_app.root_path %>
-<%= icon_to 'user-plus.png', new_member_path %>
+
+<%= icon_to 'nouveau.png', new_member_path, title:'Ajouter un membre' %>
+<%= icon_to 'payment.png', payments_path, title:'Table des paiements' %>
 <% end -%>
 
 
@@ -8,7 +9,7 @@
 
 
   <h3>Liste des <%= I18n::t('members') %></h3>
-<table class="table table-striped table-bordered table-condensed" cellpadding="0" cellspacing="0" border="0">
+<table class="table display table-striped table-bordered table-condensed" id='members' cellpadding="0" cellspacing="0" border="0">
 <thead>
   <tr>
   
@@ -18,12 +19,12 @@
   
     <th><%= I18n::t('forname') %></th>
   
-    <th><%= I18n::t('birthdate') %></th>
+    <th class='date-euro'><%= I18n::t('birthdate') %></th>
     
     <th><%= I18n::t('mail') %></th>
     <th><%= I18n::t('tel') %></th>
   
-    <th>Actions</th>
+    <th class='actions'>Actions</th>
     
   </tr>
 </thead>

data/app/views/adherent/payments/_payment.html.haml

@@ -0,0 +1,11 @@
+%tr
+  %td= payment.date
+  %td.numeric= number_to_currency(payment.amount, locale: :fr)
+  %td= payment.mode
+  %td= list_imputations(payment)
+  %td.numeric= number_to_currency payment.non_impute, locale: :fr
+
+  %td
+    - if (payment.non_impute != 0.0)
+      = icon_to 'imputation.png', new_payment_reglement_path(payment), title: 'Imputer le solde sur une adhésion'
+    = icon_to 'detail.png', member_payment_path(payment.member, payment)

data/app/views/adherent/payments/edit.html.erb

@@ -0,0 +1,13 @@
+<%= content_for :menu do %>
+
+<%= icon_to 'payment.png', member_payments_path(@member) %>
+<% end -%>
+
+<div class="champ">
+  <h3>Enregistrement d'un paiement de <%=  @member.to_s %></h3>
+</div>
+<div class="row-fluid inner-champ">
+
+<%= render 'form' %>
+</div>
+

data/app/views/adherent/payments/index.html.haml

@@ -16,15 +16,4 @@
         %th Reste à imputer
         %th Actions
     %tbody
-      - @payments.each do |payment|
-        %tr
-          %td= payment.date
-          %td= number_to_currency(payment.amount, locale: :fr)
-          %td= payment.mode
-          %td= list_imputations(payment)
-          %td= number_to_currency payment.non_impute, locale: :fr
-
-          %td
-            - if (payment.non_impute != 0.0)
-              = icon_to 'imputation.png', new_payment_reglement_path(payment), title: 'Imputer le solde sur une autre adhésion'
-            = icon_to 'supprimer.png', [@member,payment], method: :delete, confirm: 'Etes vous sûr ?'
+      = render @payments

data/app/views/adherent/payments/new.html.erb

@@ -4,7 +4,7 @@
 <% end -%>
 
 <div class="champ">
-  <h3>Enregistrement d'un paiement de <%=  @member.to_s %></h3>
+  <h3>Enregistrement d'un paiement de <%= @member.to_s %></h3>
 </div>
 <div class="row-fluid inner-champ">
 

data/app/views/adherent/payments/show.html.haml

@@ -0,0 +1,33 @@
+= content_for :menu do
+  = icon_to 'payment.png', member_payments_path(@member), title:"Liste des paiements de #{@member.to_s}" 
+  = icon_to 'money-plus.png', new_member_payment_path(@member), title:'Nouveau paiement'
+  - if (@payment.non_impute != 0.0)
+    = icon_to 'imputation.png', new_payment_reglement_path(@payment), title: 'Imputer le solde sur une autre adhésion'
+  = icon_to 'modifier.png', edit_member_payment_path(@payment.member, @payment)
+  = icon_to 'supprimer.png', [@member,@payment], method: :delete, confirm: 'Etes vous sûr ?'
+
+
+.champ
+  %h3 Détail d'un paiement
+
+.inner-champ
+  %p
+    %b Effectué par :
+    = @member.to_s
+  %p
+    %b Le :
+    = @payment.date
+  %p
+    %b Montant :
+    = number_to_currency(@payment.amount, locale: :fr)
+  %p
+    %b Par :
+    = @payment.mode
+  %p
+    %b Imputé sur :
+    = list_imputations(@payment)
+  %p
+    %b Reste à imputer :
+    = number_to_currency @payment.non_impute, locale: :fr
+  
+            

data/app/views/adherent/reglements/show.html.haml

@@ -0,0 +1,31 @@
+= content_for :menu do
+  = icon_to 'supprimer.png', [@payment, @reglement], method: :delete, confirm: 'Etes vous sûr ?'
+  = icon_to 'retour.png', :back
+  
+
+
+.champ
+  %h3 Détail d'un réglement
+
+.inner-champ
+  %p
+    %b Imputé sur un payment effectué par :
+    = @payment.member.to_s
+  %p
+    %b Le :
+    = @payment.date
+  %p
+    %b d'un montant total de :
+    = number_to_currency(@payment.amount, locale: :fr)
+  %p
+    %b dont sur ce réglement :
+    = number_to_currency(@reglement.amount, locale: :fr)
+  %p
+    %b enregistré le :
+    = l @reglement.created_at
+  - if @reglement.created_at != @reglement.updated_at
+    %p
+      %b modifié le :
+      = l @reglement.updated_at
+  
+            

data/config/initializers/validators.rb

@@ -0,0 +1,12 @@
+# ce validator est utilisé par Payment pour s'assurer que 
+# le montant du Payment est effectivement supérieur aux réglements
+# qui ont déjà été imputés sur ce paiement.
+#
+class OverImputationsValidator < ActiveModel::EachValidator
+
+  def validate_each(record, attribute, value)
+    if (record.reglements.any? && value < record.impute)
+      record.errors.add(attribute, :under_imputations)
+    end
+  end
+end

data/config/locales/fr.yml

@@ -210,7 +210,7 @@ fr:
       too_long: Trop long
       greater_than_or_equal_to: Valeur trop basse
       greater_than: Valeur trop faible
-   
+      under_imputations: Montant inférieur aux montants déjà imputés
       
       
   coord: coordonnée

data/config/routes.rb

@@ -5,7 +5,10 @@ Adherent::Engine.routes.draw do
   
   
 
+  get "allpayments/index"
+
   resources :coords
+  resources :payments, :controller=>:allpayments
 
 
   resources :members do

data/lib/adherent/version.rb

@@ -1,3 +1,3 @@
 module Adherent
-  VERSION = "0.1.3"
+  VERSION = "0.1.4"
 end

data/test/dummy/app/models/organism.rb

@@ -2,4 +2,5 @@ class Organism < ActiveRecord::Base
   attr_accessible :status, :title
   
   has_many :members, class_name:'Adherent::Member'
+  has_many :payments, :through=>:members, class_name:'Adherent::Payment'
 end