adherent 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: f46b619c95046614d77348c16f1722c1c0685689
-  data.tar.gz: 53af5fc158eeefd2ff4de7aa9accb91fc8a6f58e
+  metadata.gz: cc38f0e8374b387fb78018d2256a35d72eaa42a0
+  data.tar.gz: 6da830b9cba2dfb5172b2187667515a232e913c4
 SHA512:
-  metadata.gz: 5d92233c98a0957408105cc3f7dd0aad60a1a93b0c26211c42085192b2a43b0940797164cb2eae3d80659d56e1824107534efb6bab193049461d78b1f6cee8ac
-  data.tar.gz: a1f744d115859f1993c9bdb6afd62c11fc9f0d10b9d410edd02ab85b83dbb21e64ec26816a56fadd8e5efcfd529dafee12f7a7fa452eecf2cef791ed778a047c
+  metadata.gz: 8929f5da67a218679c1cb133780e9c64e1643c2eb638f9d4d19e5aefcdfc7f3d972fe1809fa7b8a6350eb98480dc05e92dc98ec916ef2e8ecaff7967e4b8c787
+  data.tar.gz: 050ca3b73234556f4b891a1e9827f68478ce33386d2622970e479bfdd59e281f3fb285200ea4bc8e47da2e8d6c1a1570bd6f7e4776495572704c9711935696af

data/app/controllers/adherent/adhesions_controller.rb

@@ -21,7 +21,7 @@ module Adherent
     # PUT /adhesions/1
     # PUT /adhesions/1.json
     def update
-      @adhesion = Adhesion.find(params[:id])
+      @adhesion = @member.adhesions.find(params[:id])
   
       respond_to do |format|
         if @adhesion.update_attributes(params[:adhesion])
@@ -57,11 +57,7 @@ module Adherent
       end
     end
   
-    def show
-      
-      @adhesion= @member.adhesions.find(params[:id])
-      
-    end
+  
     
     # DELETE /adhesion/1
     # DELETE /coords/1.json

data/app/controllers/adherent/coords_controller.rb

@@ -6,17 +6,7 @@ module Adherent
   class CoordsController < ApplicationController
     
     before_filter :find_member, :except=>[:index]
-    # GET /coords
-    # GET /coords.json
-    def index
-      @coords = @organism.members.collect {|m| m.coord }
-  
-      respond_to do |format|
-        format.html # index.html.erb
-        format.json { render json: @coords }
-      end
-    end
-  
+   
     # GET /coords/1
     # GET /coords/1.json
     def show
@@ -79,17 +69,7 @@ module Adherent
       end
     end
   
-    # DELETE /coords/1
-    # DELETE /coords/1.json
-    def destroy
-      @coord = Coord.find(params[:id])
-      @coord.destroy
-  
-      respond_to do |format|
-        format.html { redirect_to coords_url }
-        format.json { head :no_content }
-      end
-    end
+    
     
     protected 
     

data/app/controllers/adherent/payments_controller.rb

@@ -24,14 +24,8 @@ module Adherent
         render 'new'
       end
     end
-  
-    def edit
-      @payment = @member.payments.find(params[:id])
-    end
-  
-    def show
-      @payment = @member.payments.find(params[:id])
-    end
+    
+       
     
     # DELETE /payment/1
     # DELETE /coords/1.json

data/app/controllers/adherent/reglements_controller.rb

@@ -14,6 +14,7 @@ module Adherent
       @unpaid_adhesions = Adhesion.unpaid
     end
     
+    # TODO voir pour faire les deux approches (succès et échec)
     def create
       @payment = Payment.find(params[:payment_id])
       @payment.imputation_on_adh(params[:reglement][:adhesion_id])

data/app/helpers/adherent/application_helper.rb

@@ -17,5 +17,9 @@ module Adherent
     def virgule(montant)
       ActionController::Base.helpers.number_with_precision(montant, precision:2) rescue '0,00'
     end
+    
+    def icon_to_users
+      icon_to 'users.png', members_path, title:'Liste des membres' 
+    end
   end
 end

data/app/models/adherent/adhesion.rb

@@ -3,7 +3,7 @@
 module Adherent
   class Adhesion < ActiveRecord::Base
     
-    attr_accessible :from_date, :member, :to_date, :amount
+    attr_accessible :from_date, :to_date, :amount
     
     belongs_to :member
     has_many :reglements
@@ -15,23 +15,25 @@ module Adherent
     pick_date_for :from_date, :to_date
     
     # partant d'une adhésion, retourne un hash avec les attributs correspondants à  
-    # un renouvellement d'adhésion
-    def self.next_adh_values(adh = nil)
-      if adh
-         {:from_date =>I18n.l(adh.read_attribute(:to_date)+1),
-           :to_date=>I18n.l(adh.read_attribute(:to_date).years_since(1)),
-           amount: adh.amount
+    # un renouvellement d'adhésion. 
+    # S'il n'y pas d'adhésion, fonctionne avec des valeurs par défaut
+    def self.next_adh_values(montant = 0)
+       {:from_date =>I18n.l(Date.today),
+           :to_date=>I18n.l(Date.today.years_since(1) -1 ),
+           amount: montant
+           } 
+    end
+    
+    def next_adh_values(montant = 0)
+         montant = amount if montant == 0
+        {:from_date =>I18n.l(read_attribute(:to_date)+1),
+           :to_date=>I18n.l(read_attribute(:to_date).years_since(1)),
+           amount: montant
            } 
-      else
-         {:from_date=>I18n.l(Date.today),
-           :to_date=>I18n.l(Date.today.years_since(1) - 1),
-           amount:0
-           }
-      end
     end
     
     # liste toutes les adhésions qui ne sont pas payées
-    # TODO voir pour faire une requête SQL qui serait surment moins
+    # 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?}

data/app/models/adherent/coord.rb

@@ -3,5 +3,9 @@ module Adherent
     belongs_to :member
     
     attr_accessible :address, :city, :gsm, :mail, :office, :references, :tel, :zip
+    
+    validates :member_id, :presence=>true
+    
+    
   end
 end

data/app/models/adherent/member.rb

@@ -5,33 +5,45 @@ module Adherent
     pick_date_for :birthdate
     
     belongs_to :organism, class_name: 'Organism'
-    has_one :coord
+    has_one :coord, dependent: :destroy
     has_many :adhesions
     has_many :payments
     
-    validates :number, :name, :forname , :presence=>true
+    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
     def unpaid_adhesions
-      adhesions.reject {|adh| adh.is_paid? }
+      adhesions.order(:to_date).reject {|adh| adh.is_paid? }
     end
     
+    # indique s'il y a des adhésions impayées pour ce membre
     def unpaid_adhesions?
       unpaid_adhesions.any?
     end
     
+    # donne le montant total des adhésions impayées
     def unpaid_amount
       unpaid_adhesions.sum(&:due)
     end
     
+    # renvoie le prenom NOM
     def to_s
-      [forname, name.capitalize].join(' ')
+      [forname, name.upcase].join(' ')
     end
     
-    def next_adhesion
+    # renvoie une nouvelle adhésion préremplie avec les éléments issus de la
+    # dernière adhésion.
+    # il est possible d'imposer le montant si nécessaire
+    def next_adhesion(amount = nil)
+      amount ||= 0
       adh = adhesions(true).order('to_date').last
-      n_ad =  Adhesion.next_adh_values(adh)
-      adhesions.new(n_ad)
+      if adh
+        vals =  adh.next_adh_values(amount)
+      else
+        vals = Adhesion::next_adh_values(amount)
+      end
+      adhesions.new(vals)
     end
     
   end

data/app/models/adherent/payment.rb

@@ -6,7 +6,7 @@ module Adherent
     
     attr_accessible :amount, :date, :mode
     
-    validates :amount, :date, :mode, presence:true
+    validates :amount, :date, :mode, :member_id, presence:true
     
     pick_date_for :date
     
@@ -17,44 +17,47 @@ module Adherent
     
     after_create :imputation
     
+    # Pour faire une imputation sur une adhesion dont l'id est transmise en argument.
+    # Utile lorsque le payment est fait par un membre pour payer l'adhésion d'un tiers
+    # (par exemple un membre de sa famille).
+    # TODO faire que la valeur de retour soit true ou false pour 
+    # que la méthode créate du controller puisse tester et rediriger en conséquence
+    def imputation_on_adh(adh_id)
+      Adhesion.find(adh_id).add_reglement(id, non_impute)
+    end
+    
+    # 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)
+    end
+    
     
+    protected
     # quand on reçoit un paiement, il faut en réaliser l'imputation, 
     # plusieurs cas de figure sont à envisager.
     # 
-    # Le cas le plus simple est lorsque le montant est inférieur ou égal ou 
+    # Le cas le plus simple est lorsque le montant est inférieur ou égal aux  
     # montant dus par le membre pour les adhésions en cours.
     # 
-    # Donc on impute sur les adhésions du membre non soldées et on regarde s'il reste
-    # quelque chose.
-    # 
+    # Donc on impute sur les adhésions du membre non soldées en commençant par les 
+    # plus anciennes. 
     # 
     # 
     def imputation
       # liste les adhésions non soldées et le montant total dû
       # pour chaque adhésion non soldée (prise dans l'ordre chrono)
       # on crée un réglement du montant dû dans la limité du montant du paiement.
-      un_paid_adhs = member.adhesions.order(:to_date).reject {|adh| adh.is_paid?}
       a_imputer = amount
-      un_paid_adhs.each  do |adh|
+      member.unpaid_adhesions.each  do |adh|
         if a_imputer > 0
-          impute = adh.add_reglement(id, a_imputer)
-          a_imputer -= impute
+          reglemt = adh.add_reglement(id, a_imputer)
+          a_imputer -= reglemt.amount
         end
       end
     end
     
-    # pour faire une imputation sur une adhesion dont l'id est transmise en argument
-    # TODO faire que la valeur de retour soit true ou false pour 
-    # que la méthode créate du controller puisse tester et rediriger en conséquence
-    def imputation_on_adh(adh_id)
-      Adhesion.find(adh_id).add_reglement(id, amount)
-    end
     
-    # 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)
-    end
     
   end
 end

data/app/models/adherent/reglement.rb

@@ -3,5 +3,8 @@ module Adherent
     belongs_to :adhesion
     belongs_to :payment
     attr_accessible :amount
+    
+    validates :adhesion_id, :payment_id, :amount, presence:true
+    validates_numericality_of :amount, greater_than:0.0
   end
 end

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

@@ -1,6 +1,6 @@
 <%= content_for :menu do %>
 
-<%= icon_to 'navigation90.png', member_adhesions_path(@member) %>
+<%= icon_to 'bandaid.png', member_adhesions_path(@member) %>
 <% end -%>
 
 <div class="champ">

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

@@ -1,5 +1,5 @@
 = content_for :menu do
-  = icon_to 'navigation90.png', members_path, title:'Liste des membres'
+  = icon_to_users 
   = icon_to 'nouveau.png', new_member_adhesion_path(@member), title:'Nouvelle adhésion ou renouvellement'
 
 

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

@@ -1,6 +1,6 @@
 <%= content_for :menu do %>
 
-<%= icon_to 'navigation90.png', member_adhesions_path(@member) %>
+<%= icon_to_users %>
 <% end -%>
 
 <div class="champ">

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

@@ -1,8 +1,6 @@
 <%= content_for :menu do %>
+<%= icon_to_users %>
 
-<%= icon_to 'nouveau.png', new_coord_path %>
-<%= icon_to 'detail.png', @coord %>
-<%= icon_to 'retour.png', :back %>
 <% end -%>
 
 <div class="champ">

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

@@ -1,6 +1,6 @@
 <%= content_for :menu do %>
 
-<%= icon_to 'navigation90.png', members_path %>
+<%= icon_to_users %>
 <% end -%>
 
 <div class="champ">

data/app/views/adherent/coords/show.html.erb

@@ -1,12 +1,12 @@
 
 <%= content_for :menu do %>
-<%= icon_to 'navigation90.png', members_path %>
+<%= icon_to_users %>
 <%= icon_to 'modifier.png', edit_member_coord_path(@member) %>
 
 <% end -%>
 
 <div class="champ">
-  <h3>Fiche <%=  I18n::t('coord').pluralize %> <%= "#{@member.forname} #{@member.name}" %>  </h3>
+  <h3>Fiche <%=  I18n::t('coord').pluralize %> <%= "#{@member.to_s}" %></h3>
 </div>
 <div class="row-fluid inner-champ">
 

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

@@ -1,8 +1,7 @@
 <%= content_for :menu do %>
+<%= icon_to 'users.png', members_path, title:'Liste des membres' %>
+<%= icon_to 'card-address.png', member_coord_path(@member), title:'Coordonnées' %>
 
-<%= icon_to 'nouveau.png', new_member_path %>
-<%= icon_to 'detail.png', @member %>
-<%= icon_to 'retour.png', :back %>
 <% end -%>
 
 <div class="champ">

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

@@ -1,13 +1,13 @@
 <%= content_for :menu do %>
-<%= icon_to 'navigation90.png', main_app.root_path %>
-<%= icon_to 'nouveau.png', new_member_path %>
+<%= icon_to 'home.png', main_app.root_path %>
+<%= icon_to 'user-plus.png', new_member_path %>
 <% end -%>
 
 
 <div class="champ">
 
 
-  <h3> Liste des  <%= I18n::t('members') %></h3>
+  <h3>Liste des <%= I18n::t('members') %></h3>
 <table class="table table-striped table-bordered table-condensed" cellpadding="0" cellspacing="0" border="0">
 <thead>
   <tr>
@@ -36,12 +36,12 @@
     <td><%= member.birthdate %></td>
     <td><%= member.coord ? member.coord.mail : '' %>
     <td><%= member.coord ? member.coord.tel : '' %>
-    <td><%= icon_to 'detail.png', member_coord_path(member), title:'Coordonnées' %>
-      <%= icon_to 'afficher.png', member_adhesions_path(member), title:'Liste des adhésions' %>
+    <td><%= icon_to 'card-address.png', member_coord_path(member), title:'Coordonnées', id:"coord_member_#{member.id}" %>
+      <%= icon_to 'bandaid.png', member_adhesions_path(member), title:'Liste des adhésions', id:"adhesion_member_#{member.id}"%>
     
-    <%= icon_to 'payment.png', member_payments_path(member), title: 'Liste des paiements' %>
-      <%= icon_to 'modifier.png', edit_member_path(member) %>
-    <%= icon_to 'supprimer.png', member, :confirm => 'Etes vous sur ?', :method => :delete %></td>
+    <%= icon_to 'payment.png', member_payments_path(member), title: 'Liste des paiements' , id:"payment_member_#{member.id}"%>
+      <%= icon_to 'modifier.png', edit_member_path(member) , id:"edit_member_#{member.id}" %>
+    <%= icon_to 'supprimer.png', member, id:"delete_member_#{member.id}", :confirm => 'Etes vous sur ?', :method => :delete %></td>
   </tr>
 <% end %>
 </tbody>

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

@@ -1,6 +1,6 @@
 <%= content_for :menu do %>
 
-<%= icon_to 'navigation90.png', members_path %>
+<%= icon_to 'users.png', members_path %>
 <% end -%>
 
 <div class="champ">

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

@@ -1,9 +1,10 @@
 
 <%= content_for :menu do %>
-
+<%= icon_to 'users.png', members_path, title:'Liste des membres' %>
+<%= icon_to 'card-address.png', member_coord_path(@member) , title:'Coordonées' %>
 <%= icon_to 'modifier.png', edit_member_path(@member) %>
-<%= icon_to 'detail.png', member_coord_path(@member) %>
-<%= icon_to 'navigation90.png', members_path %>
+
+
 <% end -%>
 
 <div class="champ">