bring link_tag generation logic back into the main logic

app/views/kaminari/_paginator.html.erb

@@ -12,7 +12,13 @@
       <% if page.current? -%>
         <%= current_page_tag %>
       <% elsif page.left_outer? || page.right_outer? || page.inside_window? -%>
-        <%= page_link_tag %>
+        <% if page.first? -%>
+          <%= first_page_link_tag %>
+        <% elsif page.last? -%>
+          <%= last_page_link_tag %>
+        <% else -%>
+          <%= page_link_tag %>
+        <% end -%>
       <% elsif !page.was_truncated? -%>
         <%= truncated_span_tag %>
       <% end -%>

app/views/kaminari/_paginator.html.haml

@@ -11,7 +11,12 @@
       - if page.current?
         = current_page_tag
       - elsif page.left_outer? || page.right_outer? || page.inside_window?
-        = page_link_tag
+        - if page.first?
+          = first_page_link_tag
+        - elsif page.last?
+          = last_page_link_tag
+        - else
+          = page_link_tag
       - elsif !page.was_truncated?
         = truncated_span_tag
     = num_pages > current_page ? next_link_tag : next_span_tag

lib/kaminari/helpers.rb

@@ -13,19 +13,12 @@ module Kaminari
         @left, @window, @right = (options[:left] || options[:outer_window] || 1), (options[:window] || options[:inner_window] || 4), (options[:right] || options[:outer_window] || 1)
       end
 
-      def current_page_tag
+      %w[current_page first_page_link last_page_link page_link].each do |tag|
-        @last = CurrentPage.new self, :page => @page
+        eval <<-DEF
-      end
+          def #{tag}_tag
-
+            @last = #{tag.classify}.new self, :page => @page
-      def page_link_tag
+          end
-        @last = case @page
+        DEF
-        when 1
-          FirstPageLink
-        when @options[:num_pages]
-          LastPageLink
-        else
-          PageLink
-        end.new self, :page => @page
       end
 
       %w[prev_link prev_span next_link next_span truncated_span].each do |tag|
@@ -109,6 +102,14 @@ module Kaminari
           @page == @renderer.options[:current_page]
         end
 
+        def first?
+          @page == 1
+        end
+
+        def last?
+          @page == @renderer.options[:num_pages]
+        end
+
         def left_outer?
           @page <= @renderer.left + 1
         end