new and edit connection links

17 files changed, 87 insertions, 47 deletions

diff --git a/Gemfile.lock b/Gemfile.lock
index 5e5a435c7..7a355d4ea 100644
--- a/Gemfile.lock
+++ b/Gemfile.lock
@@ -1,6 +1,6 @@
 GIT
   remote: git://chouette.dryade.priv/ninoxe
-  revision: d07cf5dc8b9648e2ef7c27745b7c9b4575a3f676
+  revision: 341789ef66979138d36469284339e3076cb5fe87
   specs:
     ninoxe (0.0.8)
       GeoRuby
@@ -164,8 +164,8 @@ GEM
     jruby-openssl (0.7.4)
       bouncy-castle-java
     jruby-rack (1.1.5)
-    json (1.6.6)
-    json (1.6.6-java)
+    json (1.7.0)
+    json (1.7.0-java)
     launchy (2.0.5)
       addressable (~> 2.2.6)
     launchy (2.0.5-java)
diff --git a/app/views/connection_links/_connection_link.erb b/app/views/connection_links/_connection_link.erb
index 116adafb6..04ed2e833 100644
--- a/app/views/connection_links/_connection_link.erb
+++ b/app/views/connection_links/_connection_link.erb
@@ -1,8 +1,18 @@
 <%= div_for(connection_link) do %>
   <%= link_to truncate(connection_link.name, :length => 30), [@referential, connection_link], :title => "Correspondance #{connection_link.name}" %>
   <div class="info">
-    <%= connection_link.human_attribute_name('departure') %> <%= link_to_if connection_link.departure, connection_link.departure.name, referential_stop_area_path(@referential, connection_link.departure), :title => "#{connection_link.human_attribute_name('departure')} #{connection_link.departure.name}" %> -
-    <%= connection_link.human_attribute_name('arrival') %> <%= link_to_if( connection_link.arrival, connection_link.arrival.name, referential_stop_area_path(@referential, connection_link.arrival), :title => "#{connection_link.human_attribute_name('arrival')} #{connection_link.arrival.name}" )  %>
+  	<%= connection_link.human_attribute_name('departure') %> 
+    <% if connection_link.departure.present? %> 
+      <%= link_to_if connection_link.departure, connection_link.departure.name, referential_stop_area_path(@referential, connection_link.departure), :title => "#{connection_link.human_attribute_name('departure')} #{connection_link.departure.name}" %>
+    <% else %>
+      <%= connection_link.human_attribute_name("undefined") %>
+    <% end %>
+    &nbsp;-&nbsp;<%= connection_link.human_attribute_name('arrival') %> 
+    <% if connection_link.arrival.present? %> 
+      <%= link_to_if( connection_link.arrival, connection_link.arrival.name, referential_stop_area_path(@referential, connection_link.arrival), :title => "#{connection_link.human_attribute_name('arrival')} #{connection_link.arrival.name}" )  %>
+    <% else %>
+      <%= connection_link.human_attribute_name("undefined") %>
+    <% end %>
 
     <div class="actions">  
       <%= link_to t("actions.edit"), edit_referential_connection_link_path(@referential, connection_link), :class => "edit" %> |
diff --git a/app/views/connection_links/_form.erb b/app/views/connection_links/_form.erb
index 16174ec9a..31777c253 100644
--- a/app/views/connection_links/_form.erb
+++ b/app/views/connection_links/_form.erb
@@ -1,12 +1,8 @@
 <!-- <%= @connection_link.errors.inspect %> -->
 <%= semantic_form_for [@referential, @connection_link] do |form| %>
   <%= form.inputs do %> 
-  <!--
-    <%= form.input :departure, :as => :select, :collection => Chouette::Network.all, :include_blank => false %> 
-    <%= form.input :arrival, :as => :select, :collection => Chouette::Company.all, :include_blank => false%>
- -->
     <%= form.input :name %>
-    <%= form.input :connection_link_type, :as => :select, :collection => Chouette::ConnectionLink.connection_link_types, :include_blank => false, :member_label => Proc.new { |mode| t("connection_link_types.label.#{mode}") }  %>
+    <%= form.input :connection_link_type, :as => :select, :collection => Chouette::ConnectionLink.connection_link_types, :include_blank => false, :member_label => Proc.new { |type| t("connection_link_types.label.#{type}") }  %>
     <%= form.input :comment %>              
     <% if @connection_link.new_record? %>
       <%= form.input :objectid %>  
diff --git a/app/views/connection_links/index.html.erb b/app/views/connection_links/index.html.erb
index 2b8bdfe5b..8b9fc04fa 100644
--- a/app/views/connection_links/index.html.erb
+++ b/app/views/connection_links/index.html.erb
@@ -4,10 +4,29 @@
   <%= f.label :name_cont, "#{t('.name')} :" %>
   <%= f.text_field :name_cont %>
 
+<!--
+  <%= f.hidden_field :departure_id_eq %>
+  <%= f.hidden_field :arrival_id_eq %>
+-->
+
   <%= f.submit t('actions.search') %> <%= t("or") %>
   <%= link_to t("cancel"), referential_connection_links_path(@referential) %>
 <% end %>
 
+<!--
+<% if @q.departure_id_eq.present? %>
+<p>
+  <%= Chouette::ConnectionLink.human_attribute_name('departure') %> : <%= @referential.stop_areas.find(@q.departure_id_eq).name %>
+</p>
+<% end %>
+
+<% if @q.arrival_id_eq.present? %>
+<p>
+  <%= Chouette::ConnectionLink.human_attribute_name('arrival') %> : <%= @referential.stop_areas.find(@q.arrival_id_eq).name %>
+</p>
+<% end %>
+-->
+
 <div class="pagination">
   <div class="page_info">
     <%= page_entries_info @connection_links %>
diff --git a/app/views/connection_links/show.html.erb b/app/views/connection_links/show.html.erb
index cd564ce9b..69aad91ea 100644
--- a/app/views/connection_links/show.html.erb
+++ b/app/views/connection_links/show.html.erb
@@ -5,19 +5,27 @@
   <div class="summary">
     <p>
       <label><%= @connection_link.human_attribute_name(:departure) %>: </label>
-      <%= link_to @connection_link.departure.name, [@referential, @connection_link.departure] %>
+      <% if @connection_link.departure.present? %> 
+        <%= link_to @connection_link.departure.name, [@referential, @connection_link.departure] %>
+      <% else %>
+         <%= @connection_link.human_attribute_name("undefined") %>  
+      <% end %>
     </p>
     <p>
       <label><%= @connection_link.human_attribute_name(:arrival) %>: </label>
-      <%= link_to @connection_link.arrival.name, [@referential, @connection_link.arrival] %>
+      <% if @connection_link.arrival.present? %> 
+         <%= link_to @connection_link.arrival.name, [@referential, @connection_link.arrival] %>
+      <% else %>
+         <%= @connection_link.human_attribute_name("undefined") %>  
+      <% end %>
     </p>
     <p>
       <label><%= @connection_link.human_attribute_name("name") %>: </label>
       <%= @connection_link.name %>
     </p>
     <p>
-      <label><%= @connection_link.human_attribute_name("link_type") %>: </label>
-      <%= t("connection_link_types.label.#{@connection_link.link_type}") %>
+      <label><%= @connection_link.human_attribute_name("connection_link_type") %>: </label>
+      <%= t("connection_link_types.label.#{@connection_link.connection_link_type}") %>
     </p>
     <p>
       <label><%= @connection_link.human_attribute_name("comment") %>: </label>
diff --git a/app/views/help/companies.textile b/app/views/help/companies.textile
index 0b89d9796..07d7789c5 100644
--- a/app/views/help/companies.textile
+++ b/app/views/help/companies.textile
@@ -32,7 +32,7 @@ cet identifiant est compos&eacute; de 3 parties : pr&eacute;fixe:type:id_techniq
 * type = Company : cl&eacute; identifiant le type d'objet (valeur impos&eacute;e)
 * id_technique : valeur identifiant un seul object d'un type donn&eacute; pour un m&ecirc;me producteur
 ce champ obligatoire est automatiquement g&eacute;n&eacute;r&eacute; s'il n'est pas renseign&eacute; 
-il est possible aussi de ne renseigner que le pr&eacute;fixe auquel cas la valeur sera compl&eacute;t&eacute;e automatiquement.
+il est possible aussi de ne renseigner que l'identifiant technique auquel cas la valeur sera compl&eacute;t&eacute;e automatiquement.
 =:
 - Date de cr&eacute;ation := date &agrave; laquelle l'objet a &eacute;t&eacute; cr&eacute;&eacute; ou modifi&eacute; pour la derni&egrave;re fois
 - Version := version de l'objet (auto incr&eacute;ment&eacute; &agrave; chaque modification)
diff --git a/app/views/help/connection_links.textile b/app/views/help/connection_links.textile
index 4969db77d..941918e2d 100644
--- a/app/views/help/connection_links.textile
+++ b/app/views/help/connection_links.textile
@@ -34,7 +34,7 @@ cet identifiant est compos&eacute; de 3 parties : pr&eacute;fixe:type:id_techniq
 * type = ConnectionLink : cl&eacute; identifiant le type d'objet (valeur impos&eacute;e)
 * id_technique : valeur identifiant un seul object d'un type donn&eacute; pour un m&ecirc;me producteur
 ce champ obligatoire est automatiquement g&eacute;n&eacute;r&eacute; s'il n'est pas renseign&eacute; 
-il est possible aussi de ne renseigner que le pr&eacute;fixe auquel cas la valeur sera compl&eacute;t&eacute;e automatiquement.
+il est possible aussi de ne renseigner que l'identifiant technique auquel cas la valeur sera compl&eacute;t&eacute;e automatiquement.
 =:
 - Date de cr&eacute;ation := date &agrave; laquelle l'objet a &eacute;t&eacute; cr&eacute;&eacute; ou modifi&eacute; pour la derni&egrave;re fois
 - Version := version de l'objet (auto incr&eacute;ment&eacute; &agrave; chaque modification)
diff --git a/app/views/help/index.textile b/app/views/help/index.textile
index 73c0f04c1..a5e488cca 100644
--- a/app/views/help/index.textile
+++ b/app/views/help/index.textile
@@ -19,12 +19,13 @@ l'avancement actuel permet de :
 
 * cr&eacute;er, modifier et supprimer les objets
 * consulter les cartes
-* d&eacute;placer un arr&ecirc;t depuis la carte
+* d&eacute;placer un arr&ecirc;t depuis la carte (voir mode op&eacute;ratoire dans l'aide des arr&ecirc;ts)
 * associer les arr&ecirc;ts &agrave; leurs parents
 
 les action suivantes ne sont pas encore disponible : 
 
 * afficher le fond G&eacute;oportail
 * ajouter ou supprimer des dates et des p&eacute;riodes aux calendriers
+* associer des arr&ecirc;ts à une s&eacute;quences d'arr&ecirc;ts
 * associer une ITL &agrave; une ligne
 
diff --git a/app/views/help/lines.textile b/app/views/help/lines.textile
index 9997c0d50..78348896f 100644
--- a/app/views/help/lines.textile
+++ b/app/views/help/lines.textile
@@ -34,7 +34,7 @@ cet identifiant est compos&eacute; de 3 parties : pr&eacute;fixe:type:id_techniq
 * type = Line : cl&eacute; identifiant le type d'objet (valeur impos&eacute;e)
 * id_technique : valeur identifiant un seul object d'un type donn&eacute; pour un m&ecirc;me producteur
 ce champ obligatoire est automatiquement g&eacute;n&eacute;r&eacute; s'il n'est pas renseign&eacute; 
-il est possible aussi de ne renseigner que le pr&eacute;fixe auquel cas la valeur sera compl&eacute;t&eacute;e automatiquement.
+il est possible aussi de ne renseigner que l'identifiant technique auquel cas la valeur sera compl&eacute;t&eacute;e automatiquement.
 =:
 - Date de cr&eacute;ation := date &agrave; laquelle l'objet a &eacute;t&eacute; cr&eacute;&eacute; ou modifi&eacute; pour la derni&egrave;re fois
 - Version := version de l'objet (auto incr&eacute;ment&eacute; &agrave; chaque modification)
diff --git a/app/views/help/networks.textile b/app/views/help/networks.textile
index 424808f48..2a2de2309 100644
--- a/app/views/help/networks.textile
+++ b/app/views/help/networks.textile
@@ -29,7 +29,7 @@ cet identifiant est compos&eacute; de 3 parties : pr&eacute;fixe:type:id_techniq
 * type = GroupOfLine : cl&eacute; identifiant le type d'objet (valeur impos&eacute;e)
 * id_technique : valeur identifiant un seul object d'un type donn&eacute; pour un m&ecirc;me producteur
 ce champ obligatoire est automatiquement g&eacute;n&eacute;r&eacute; s'il n'est pas renseign&eacute; 
-il est possible aussi de ne renseigner que le pr&eacute;fixe auquel cas la valeur sera compl&eacute;t&eacute;e automatiquement.
+il est possible aussi de ne renseigner que l'identifiant technique auquel cas la valeur sera compl&eacute;t&eacute;e automatiquement.
 =:
 - Date de cr&eacute;ation := date &agrave; laquelle l'objet a &eacute;t&eacute; cr&eacute;&eacute; ou modifi&eacute; pour la derni&egrave;re fois
 - Version := version de l'objet (auto incr&eacute;ment&eacute; &agrave; chaque modification)
diff --git a/app/views/help/routes.textile b/app/views/help/routes.textile
index f02946b41..30234c492 100644
--- a/app/views/help/routes.textile
+++ b/app/views/help/routes.textile
@@ -26,7 +26,7 @@ cet identifiant est compos&eacute; de 3 parties : pr&eacute;fixe:type:id_techniq
 * type = Route : cl&eacute; identifiant le type d'objet (valeur impos&eacute;e)
 * id_technique : valeur identifiant un seul object d'un type donn&eacute; pour un m&ecirc;me producteur
 ce champ obligatoire est automatiquement g&eacute;n&eacute;r&eacute; s'il n'est pas renseign&eacute; 
-il est possible aussi de ne renseigner que le pr&eacute;fixe auquel cas la valeur sera compl&eacute;t&eacute;e automatiquement.
+il est possible aussi de ne renseigner que l'identifiant technique auquel cas la valeur sera compl&eacute;t&eacute;e automatiquement.
 =:
 - Date de cr&eacute;ation := date &agrave; laquelle l'objet a &eacute;t&eacute; cr&eacute;&eacute; ou modifi&eacute; pour la derni&egrave;re fois
 - Version := version de l'objet (auto incr&eacute;ment&eacute; &agrave; chaque modification)
diff --git a/app/views/help/stop_areas.textile b/app/views/help/stop_areas.textile
index 8c468e9f1..94b73c574 100644
--- a/app/views/help/stop_areas.textile
+++ b/app/views/help/stop_areas.textile
@@ -68,7 +68,7 @@ cet identifiant est compos&eacute; de 3 parties : pr&eacute;fixe:type:id_techniq
 * type = Line : cl&eacute; identifiant le type d'objet (valeur impos&eacute;e)
 * id_technique : valeur identifiant un seul object d'un type donn&eacute; pour un m&ecirc;me producteur
 ce champ obligatoire est automatiquement g&eacute;n&eacute;r&eacute; s'il n'est pas renseign&eacute; 
-il est possible aussi de ne renseigner que le pr&eacute;fixe auquel cas la valeur sera compl&eacute;t&eacute;e automatiquement.
+il est possible aussi de ne renseigner que l'identifiant technique auquel cas la valeur sera compl&eacute;t&eacute;e automatiquement.
 =:
 - Date de cr&eacute;ation := date &agrave; laquelle l'objet a &eacute;t&eacute; cr&eacute;&eacute; ou modifi&eacute; pour la derni&egrave;re fois
 - Version := version de l'objet (auto incr&eacute;ment&eacute; &agrave; chaque modification)
diff --git a/app/views/help/timetables.textile b/app/views/help/timetables.textile
index 57fe0d50b..6c55e8440 100644
--- a/app/views/help/timetables.textile
+++ b/app/views/help/timetables.textile
@@ -43,7 +43,7 @@ cet identifiant est compos&eacute; de 3 parties : pr&eacute;fixe:type:id_techniq
 * type = Timetable : cl&eacute; identifiant le type d'objet (valeur impos&eacute;e)
 * id_technique : valeur identifiant un seul object d'un type donn&eacute; pour un m&ecirc;me producteur
 ce champ obligatoire est automatiquement g&eacute;n&eacute;r&eacute; s'il n'est pas renseign&eacute; 
-il est possible aussi de ne renseigner que le pr&eacute;fixe auquel cas la valeur sera compl&eacute;t&eacute;e automatiquement.
+il est possible aussi de ne renseigner que l'identifiant technique auquel cas la valeur sera compl&eacute;t&eacute;e automatiquement.
 =:
 - Date de cr&eacute;ation := date &agrave; laquelle l'objet a &eacute;t&eacute; cr&eacute;&eacute; ou modifi&eacute; pour la derni&egrave;re fois
 - Version := version de l'objet (auto incr&eacute;ment&eacute; &agrave; chaque modification)
diff --git a/config/locales/connection_link_types.yml b/config/locales/connection_link_types.yml
index f896e1883..2404aeeff 100644
--- a/config/locales/connection_link_types.yml
+++ b/config/locales/connection_link_types.yml
@@ -1,13 +1,13 @@
 en:
   connection_link_types:
     label:
-      Underground: Underground
-      Mixed: Mixed
-      Overground: Overground
+      underground: Underground
+      mixed: Mixed
+      overground: Overground
       
 fr:
   connection_link_types:
     label:
-      Underground: Souterrain
-      Mixed: Mixte
-      Overground: Aérien
+      underground: Souterrain
+      mixed: Mixte
+      overground: Aérien
diff --git a/config/locales/connection_links.yml b/config/locales/connection_links.yml
index 5eb2bbacf..15463e47c 100644
--- a/config/locales/connection_links.yml
+++ b/config/locales/connection_links.yml
@@ -25,6 +25,7 @@ en:
       connection_link:
         departure: Start of link
         arrival: End of link
+        undefined: not yet set
         name: Name
         comment: Comment
         linkdistance: Distance
@@ -59,7 +60,7 @@ fr:
     show:
       title: Correspondance %{connection_link}
     index:
-      name: Nom ou indice
+      name: Nom
       title: Correspondances
       departure: Arrêt de départ
       arrival: "Arrêt d'arrivée"
@@ -72,6 +73,7 @@ fr:
       connection_link:
         departure: Arrêt de départ
         arrival: "Arrêt d'arrivée"
+        undefined: non défini
         name: Nom
         comment: Commentaire
         linkdistance: Distance
diff --git a/config/routes.rb b/config/routes.rb
index cadfef970..4e7bebfc2 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -26,7 +26,13 @@ ChouetteIhm::Application.routes.draw do
       resources :routes
     end
 
-    resources :companies, :time_tables, :connection_links
+    resources :companies, :stop_areas,  :connection_links
+    
+    resources :time_tables do
+      resources :time_table_dates
+      resources :time_table_periods
+    end
+
     resources :stop_areas do
       resources :stop_area_parents 
       member do
diff --git a/spec/requests/connection_links_spec.rb b/spec/requests/connection_links_spec.rb
index c40e81970..e137a330b 100644
--- a/spec/requests/connection_links_spec.rb
+++ b/spec/requests/connection_links_spec.rb
@@ -35,26 +35,24 @@ describe "ConnectionLinks" do
   end
 
   describe "new" do     
-    pending ": TODO see how to affect departure and arrival" 
-#    it "creates connection_link and return to show" do
-#      visit referential_connection_links_path(referential)
-#      click_link "Ajouter une correspondance"
-#      fill_in "Nom", :with => "ConnectionLink 1"
-#      fill_in "Identifiant Neptune", :with => "test:Timetable:1"        
-#      click_button("Créer Correspondance")
-#      page.should have_content("ConnectionLink 1")
-#    end
+    it "creates connection_link and return to show" do
+      visit referential_connection_links_path(referential)
+      click_link "Ajouter une correspondance"
+      fill_in "Nom", :with => "ConnectionLink 1"
+      fill_in "Identifiant Neptune", :with => "test:ConnectionLink:1"        
+      click_button("Créer Correspondance")
+      page.should have_content("ConnectionLink 1")
+    end
   end
 
   describe "edit and return to show" do      
-    pending ": TODO see how to affect departure and arrival" 
-#    it "edit connection_link" do
-#      visit referential_connection_link_path(referential, subject)
-#      click_link "Modifier cette correspondance"
-#      fill_in "Nom", :with => "ConnectionLink Modified"
-#      click_button("Modifier Correspondance")
-#      page.should have_content("ConnectionLink Modified")
-#    end
+    it "edit connection_link" do
+      visit referential_connection_link_path(referential, subject)
+      click_link "Modifier cette correspondance"
+      fill_in "Nom", :with => "ConnectionLink Modified"
+      click_button("Modifier Correspondance")
+      page.should have_content("ConnectionLink Modified")
+    end
   end
 
 end