Mapeo de usuario tiene prioridad sobre interpretación de MP

nando · Apr 29, 2010 · b5fab68 · b5fab68
1 parent d10ba70
commit b5fab68
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 7 deletions.
diff --git a/features/es_ES/page_mapping_priority.feature b/features/es_ES/page_mapping_priority.feature
@@ -0,0 +1,5 @@
+# language: es
+Característica: User page mapping is more important than MP page interpretation
+  Escenario: Tiene prioridad mi mapeo de página sobre la interpretación de página de MP
+    Dado que visito la página de creación de lechugas
+    Entonces debo ver la etiqueta H1 con el valor "Portada de Mundo Pepino"
diff --git a/features/step_definitions/es_ES.rb b/features/step_definitions/es_ES.rb
@@ -95,7 +95,10 @@ def strftime(format)
     },
     /^la página de (?:identificación|login)/i => lambda {
       MundoPepino.world.new_session_path
-    }
+    },
+    # Para testear que tienen prioridad estos mapeos que
+    # la interpretación que MP hace de la expresión
+    /^la página de creación de lechugas$/ => lambda { MundoPepino.world.root_path }
   }
 end
 

diff --git a/lib/mundo_pepino.rb b/lib/mundo_pepino.rb
@@ -68,9 +68,9 @@ def common_mappings
       end
       String.add_mapper(:month) {|month| month.capitalize}
       String.add_mapper(:real_value) {|value| value} # true, false...
-      String.add_mapper(:user_page)
+      String.add_mapper(:automagical_page)
       String.add_mapper(:page) do |str|
-        str.to_user_page || (str =~ /^\/.*$|^https?:\/\//i ? str : nil)
+        str.to_automagical_page || (str =~ /^\/.*$|^https?:\/\//i ? str : nil)
       end
     end
 
@@ -94,8 +94,8 @@ def user_specific_mappings
       config.relation_model_mappings.each {|k,v| String.relation_model_mappings[k] = v}
       config.field_mappings.each {|k,v| String.field_mappings[k] = v}
       # url_mappings are deprecated, use page_mappings instead
-      config.url_mappings.each {|k,v| String.user_page_mappings[k] = v}
+      config.url_mappings.each {|k,v| String.page_mappings[k] = v}
-      config.page_mappings.each {|k,v| String.user_page_mappings[k] = v}
+      config.page_mappings.each {|k,v| String.page_mappings[k] = v}
     end
   end
 end
diff --git a/lib/mundo_pepino/es_ES/mappings.rb b/lib/mundo_pepino/es_ES/mappings.rb
@@ -48,7 +48,7 @@ def language_specific_mappings
         :noviembre       => 'November',
         :diciembre       => 'December'
       }
-      String.page_mappings.merge!({
+      String.automagical_page_mappings.merge!({
         /^la (?:portada|home\s?(?:page)?)$/i => lambda{MundoPepino.world.root_path},
         /^(#{_el_listado_de_}) ([\w]+|['"][\w ]+["'])$/i => 
           lambda{ |el_listado_de, modelo|

diff --git a/lib/mundo_pepino/implementations_api.rb b/lib/mundo_pepino/implementations_api.rb
@@ -214,7 +214,7 @@ def resource_index_or_mapped_page(the_page_of, raw_model)
       if model = unquoted_model.to_model
         pile_up model.new
         MundoPepino.world.send "#{model.table_name}_path"
-      elsif url = "#{the_page_of} #{raw_model}".to_user_page
+      elsif url = "#{the_page_of} #{raw_model}".to_page
         url
       else
         raise MundoPepino::ModelNotMapped.new(unquoted_model)