ajax_pagination 0.5.1 → 0.6.0

data/.gitignore

@@ -1,6 +1,6 @@
 *.gem
 .bundle
-/Gemfile.lock
+Gemfile.lock
 pkg/*
 *~
 *.swp

data/.travis.yml

@@ -1,7 +1,6 @@
 language: ruby
 rvm:
   - 1.8.7
-  - 1.9.2
   - 1.9.3
   - rbx-18mode
   - rbx-19mode

data/CHANGELOG.md

@@ -1,3 +1,18 @@
+## v0.6.0
+The :pagination option no longer makes as much sense, now that the sections are called ajax_section. Also, this gem is as much about site navigation as pagination. This is the reason for some of the following changes.
+
+* Further name changes made - :pagination option now called :section_id option in AJAX Pagination methods (with exception of ajax_section method), :pagination still works, but is now deprecated.
+
+**It is suggested to grep :pagination and rename to :section_id**
+
+* ajax_section :pagination option now called :id option, :pagination still works, but deprecated
+* New :section_id, ajax_section :id options now defaults to "global" in all methods - this is backwards incompatible
+* Added ajax_links helper method, to replace the magic container (a div container with class of ajaxpagination).
+* div containers of class pagination are no longer magic containers
+* Added ajax_section method in ActionController - which is set to the only section that should be displayed in an AJAX request. It is nil for normal requests.
+
+**Avoid using data-pagination, or inspecting params[:pagination] directly. These will be changed in the next version. Instead, use the helper methods provided.**
+
 ## v0.5.1
 * Added generator for assets (so asset pipeline is no longer required).
 * Added tests for a Rails 3.0.x application.

data/README.md

@@ -3,9 +3,9 @@
 
 [Wiki](https://github.com/ronalchn/ajax_pagination/wiki) | [RDoc](http://rdoc.info/gems/ajax_pagination/frames) | [Changelog](https://github.com/ronalchn/ajax_pagination/blob/master/CHANGELOG.md)
 
-Handles AJAX pagination for you, by hooking up the links you want to load content with javascript in designated page containers. Each webpage can have multiple page containers, each with a different set of pagination links. The page containers can be nested. Degrades gracefully when javascript is disabled.
+Handles AJAX pagination for you, by hooking up the links you want to load content with javascript in designated page containers. Use this to ajaxify your site navigation. Degrades gracefully when javascript is disabled.
 
-Basically, there is a helper function to use to create a section in your webpage, where content can be changed. Links can reference the section, and thus load new content into it.
+Each webpage can have multiple ajax_section containers, whether they are side by side, or even nested. Links can reference an ajax_section to load new content into the section using AJAX. Watch as the URL in the browser address bar updates, with fully working back/forward buttons.
 
 For more, see [Introduction and Background](https://github.com/ronalchn/ajax_pagination/wiki/Introduction-and-Background).
 
@@ -40,8 +40,8 @@ To use this on Rails 3.0, or without using the asset pipeline, read [Installatio
 In the ActionView, use ajax_section to declare a section, and ajax_link_to to create a link loading content into the section:
 
 ```erb
-<%= ajax_link_to "Link", link_url, :pagination => "section_name" %>
-<%= ajax_section :pagination => "section_name", :render => "mypartial" %>
+<%= ajax_link_to "Link", link_url, :section_id => "section_name" %>
+<%= ajax_section :id => "section_name", :render => "mypartial" %>
 ```
 
 Then, in the ActionController, use ajax_respond to render only the partial when an AJAX request is made:
@@ -52,7 +52,7 @@ Class ObjectsController < ApplicationController
     ...
     respond_to do |format|
       format.html # index.html.erb
-      ajax_respond format, :pagination => "section_name", :render => "_mypartial"
+      ajax_respond format, :section_id => "section_name", :render => "_mypartial"
     end
   end
   ...

data/lib/ajax_pagination/controller_additions.rb

@@ -2,33 +2,34 @@ module AjaxPagination
   # This module is automatically added to all controllers
   module ControllerAdditions
     module ClassMethods
-      # Adds default render behaviour for requests with a pagination parameter matching a certain name. By default, this name is the empty string. However, it can be changed.
+      # Adds default render behaviour for AJAX requests for a section with matching a certain name. By default, this name is the empty string. However, it can be changed.
       # Options:
+      # [:+section_id+]
+      #   The AJAX section name which should be matched to invoke AJAX Pagination response. Defaults to "global".
+      #
       # [:+pagination+]
-      #   The pagination name which should be matched to invoke AJAX Pagination response. Defaults to the empty string "". This is unlike other
-      #   methods which default to "page". However, this is a better default in this case because this affects more than one controller/action.
+      #   Deprecated. Alias for section_id.
       #
       # [:+render+]
-      #   Overrides default render behaviour for AJAX Pagination, which is to render the partial with name matching the pagination option,
+      #   Overrides default render behaviour for AJAX Pagination, which is to render the partial with name matching the section_id option,
       #   or if it does not exist, renders the default template
       #
       def ajax_respond(options = {});
         # instead of defining default render normally, we save an unbound reference to original function in case it was already defined, since we want to retain the original behaviour, and add to it (if the method is redefined after, this new behaviour is lost, but at least we don't remove others' behaviour - note that this also allows multiple invocations of this with different parameters)
         default_render = self.instance_method(:default_render) # get a reference to original method
-        pagination = options[:pagination] || ""
+        section_id = options[:section_id] || options[:pagination] || "global"
         view = options[:render] || nil
         define_method(:default_render) do |*args|
-          paramspagination = request.GET[:pagination] || params[:pagination]
-          if paramspagination && paramspagination == pagination && request.format == "html" # override if calling AJAX Pagination
+          if ajax_section && ajax_section == section_id && request.format == "html" # override if calling AJAX Pagination
             unless view
-              if lookup_context.find_all("#{params[:controller]}/_#{paramspagination}").any?
-                view = { :partial => paramspagination } # render partial, layout is off
+              if lookup_context.find_all("#{params[:controller]}/_#{ajax_section}").any?
+                view = { :partial => ajax_section } # render partial, layout is off
               else
                 view = { :layout => false } # render default view, but turn off layout
               end
             end
             respond_to do |format|
-              ajax_respond format, :pagination => pagination, :render => view
+              ajax_respond format, :section_id => section_id, :render => view
             end
           else # otherwise do what would have been done
             default_render.bind(self).call(*args) # call original method of the same name
@@ -38,8 +39,24 @@ module AjaxPagination
     end
     def self.included(base)
       base.extend ClassMethods
+
+      if AjaxPagination.redirect_after_filter == true
+        base.after_filter :ajax_pagination_redirect
+      end
+
+      base.before_filter do
+        @_ajax_section = request.GET[:pagination] || params[:pagination]
+        params.delete(:pagination) if request.get?
+      end
     end
-    # Registers an ajax response in html format when params [:pagination] matches options [:pagination] ( = "page" by default).
+
+    # Returns the name of the ajax section to be responded to, if present. Otherwise this is not an AJAX Pagination request
+    # (ie. this is otherwise a normal full page request). When no specific ajax section is requested, returns nil.
+    def ajax_section
+      @_ajax_section
+    end
+
+    # Registers an ajax response in html format when a request is made by AJAX Pagination (in which case ajax_section.nil? is false).
     # AJAX Pagination uses this response to render only the content which has changed. When this format is triggered,
     # a partial is passed back, and sent to AJAX Pagination as a function argument in javascript.
     #
@@ -56,31 +73,33 @@ module AjaxPagination
     #   end
     #
     # Options:
+    # [:+section_id+]
+    #   Changes the AJAX section name triggering this response. Triggered when ajax_section == options [:section_id].
+    #   Defaults to "global"
+    #
     # [:+pagination+]
-    #   Changes the pagination name triggering this response. Triggered when params [:pagination] == options [:pagination].
-    #   Defaults to "page"
+    #   Deprecated. Alias for section_id
     #
     # [:+render+]
     #   Changes the default template/partial that is rendered by this response. The value can be any object,
     #   and is rendered directly. The render behaviour is the same as the render method in controllers. If this option is not used,
-    #   then the default is a partial of the same name as :pagination, if it exists, otherwise, if it does not,
+    #   then the default is a partial of the same name as :section_id, if it exists, otherwise, if it does not,
     #   the default template is rendered (ie. the :controller/:action.:format view file). By default, no layout is used
     #   to render the template/partial. It can be set by passing in a layout key.
     #
     #     def welcome
     #       respond_to do |format|
     #         format.html
-    #         ajax_respond format, :pagination => :menu, :render => {:file => "pages/welcome"}
+    #         ajax_respond format, :section_id => :menu, :render => {:file => "pages/welcome"}
     #       end
     #     end
     #
     def ajax_respond(format,options = {})
-      paramspagination = request.GET[:pagination] || params[:pagination]
-      if paramspagination == (options[:pagination] || 'page').to_s
+      if ajax_section == (options[:section_id] || options[:pagination] || 'global').to_s
         if options[:render]
           view = options[:render] # render non partial
-        elsif lookup_context.find_all(params[:controller] + "/_" + paramspagination).any?
-          view = {:partial => paramspagination} # render partial of the same name as pagination
+        elsif lookup_context.find_all(params[:controller] + "/_" + ajax_section).any?
+          view = {:partial => ajax_section} # render partial of the same name as pagination
         else # render usual view
           view = {}
         end
@@ -94,18 +113,18 @@ module AjaxPagination
 
     # Tests whether an AJAX Pagination partial might be displayed in the view. If the response is not directly controlled by
     # AJAX Pagination, it will return true, because the partial might be displayed. The response is handled by AJAX Pagination
-    # if the format is html, and the params [:pagination] parameter is set. If it is set, then it will return whether
-    # params [:pagination] == pagination (the name of the pagination set, which defaults to page).
+    # if the format is html, and the ajax_section parameter is set. If it is set, then it will return whether
+    # ajax_section == section_id (the name of the section, which defaults to page).
     #
     # This method is a convenience function so that the controller does not need to perform heavy computation which might only
-    # be required if a certain pagination partial is displayed.
+    # be required if only a certain section is displayed (for an AJAX request).
     #
-    # For example, suppose an index page contains two sets of pagination partials, one for upcoming posts, and one for published
+    # For example, suppose an index page contains two ajax_section containers, one for upcoming posts, and one for published
     # posts, then you might use:
     # 
     #   class PostsController < ApplicationController
     #     def index
-    #       if ajax_section_displayed? do
+    #       if ajax_section_displayed? :page do
     #         @posts = Post.published
     #         @posts.each do |post|
     #           post.heavycomputation
@@ -119,16 +138,15 @@ module AjaxPagination
     #       end
     #       respond_to do |format|
     #         format.html # index.html.erb
-    #         ajax_respond format
-    #         ajax_respond format, :pagination => 'upcomingpage'
+    #         ajax_respond format, :section_id => 'page'
+    #         ajax_respond format, :section_id => 'upcomingpage'
     #       end
     #     end
     #   end
     #
     # The heavy computation will only be performed on posts which will be displayed when AJAX Pagination only wants a partial.
-    def ajax_section_displayed?(pagination = :page)
-      paramspagination = request.GET[:pagination] || params[:pagination]
-      (!request.format.html?) || (paramspagination.nil?) || (paramspagination == pagination.to_s)
+    def ajax_section_displayed?(section_id = :global)
+      (ajax_section.nil?) || (ajax_section == section_id.to_s)
     end
 
     # This after_filter method is automatically included by AJAX Paginate, and does not need to be included manually. However,
@@ -144,8 +162,7 @@ module AjaxPagination
     # This filter should not affect other uses, because only AJAX calls trigger this. In addition, a ?pagination= parameter is required.
     # Therefore other AJAX libraries or usage otherwise should not be affected.
     def ajax_pagination_redirect
-      paramspagination = request.GET[:pagination] || params[:pagination]
-      if request.xhr? && paramspagination && response.status==302 # alter redirect response so that it can be detected by the client javascript
+      if request.xhr? && ajax_section && response.status==302 # alter redirect response so that it can be detected by the client javascript
         response.status = 200 # change response to OK, location header is preserved, so AJAX can get the new page manually
       end
     end

data/lib/ajax_pagination/helper_additions.rb

@@ -14,10 +14,10 @@ module AjaxPagination
     #   <%= link_to 'New Comment', new_comment_path %>
     #
     # If you prefer to can render yourself, or call another function instead (useful for using this in the
-    # application layout), you can pass in a block. Any content wrapped by the pagination tag will be changed
+    # application layout), you can pass in a block. Any content wrapped by this section will be changed
     # when paginating. A possible way to use this function by passing a block in an application layout is shown:
     # 
-    #   <div class="ajaxpagination menu" data-pagination="menu">
+    #   <%= ajax_links :section_id => "global", :class => "menu" do %>
     #     <ul>
     #       <li><%= link_to "Home", root_url %></li>
     #       <li><%= link_to "Posts", posts_url %></li>
@@ -25,19 +25,22 @@ module AjaxPagination
     #       <li><%= link_to "Readme", pages_readme_url %></li>
     #       <li><%= link_to "About", pages_about_url %></li>
     #     </ul>
-    #   </div>
-    #   <%= ajax_section :pagination => "" do %>
+    #   <% end %>
+    #   <%= ajax_section :id => "global" do %>
     #     <%= yield %>
     #   <% end %>
     #
     # Options:
-    # [:+pagination+]
-    #   Changes the pagination name, which is used for requesting new content, and to uniquely identify the
+    # [:+id+]
+    #   Changes the AJAX section name, which is used for requesting new content, and to uniquely identify the
     #   wrapping div tag. The name passed here should be the same as the pagination name used in the controller
-    #   respond_to block. Defaults to "page".
+    #   respond_to block. Defaults to "global".
+    #
+    # [:+pagination+]
+    #   Deprecated. Alias for name.
     #
     # [:+render+]
-    #   Changes the partial which is rendered. Defaults to +options [:pagination]+. The partial should generally
+    #   Changes the partial which is rendered. Defaults to +options [:name]+. The partial should generally
     #   be the same as that given in the controller respond_to block, unless you are doing something strange. If a
     #   block is passed to the function, this option is ignored. You can also pass options instead to render other
     #   files, in which case, the behaviour is the same as the render method in views.
@@ -83,9 +86,9 @@ module AjaxPagination
     #   the link simply creates a cool AJAX effect on the current page.
     #
     def ajax_section(options = {})
-      pagination = options[:pagination] || 'page' # by default the name of the pagination is 'page'
-      partial = options[:render] || pagination # default partial rendered is the name of the pagination
-      divoptions = { :id => "#{pagination}_paginated_section", :class => "paginated_section" }
+      section_id = options[:id] || options[:pagination] || 'global' # by default the name of the section is 'global'
+      partial = options[:render] || section_id # default partial rendered is the name of the section
+      divoptions = { :id => "#{section_id}", :class => "paginated_section" }
       data = {};
       if options.has_key? :history
         data[:history] = (options[:history] != false)
@@ -116,10 +119,11 @@ module AjaxPagination
     # A loading image is also displayed above the content. Only one loading zone is allowed. The rest are ignored.
     #
     # Use this tag in your partial, wrapped around all the content you want to disable. For example, if you are
-    # displaying pagination links which you do not want to disable, as well as content you wish to disable,
+    # displaying AJAX links which you do not want to disable, as well as content you wish to disable,
     # your partial might contain:
-    #
-    #   <%= will_paginate @objects, :params => { :pagination => nil } %>
+    #   <% ajax_links :section_id => "page" do %>
+    #     <%= will_paginate @objects %>
+    #   <% end %>
     #   <%= ajax_loadzone do %>
     #     All content here is covered by a semi-transparent rectangle.
     #     A loading image is displayed on top, and any links here are unclickable
@@ -131,19 +135,39 @@ module AjaxPagination
       end
     end
 
+    # Used to wrap ordinary links, which will be treated as AJAX links. Only ordinary links are altered.
+    # If the link contains a data-remote, data-method, data-confirm attribute, it will not be ajaxified by this container.
+    #
+    # Instead of using ajax_link_to for every link, the following can be used:
+    #
+    # ajax_links :section_id => "global" do
+    #   link_to "My link", link_url
+    #   link_to "Back", back_url
+    # end
+    #
+    # This allows a :section_id option to apply to all links within the block, instead of specifying the same option
+    # on each link. The default :section_id is "global" if not otherwise specified.
+    #
+    def ajax_links(options = {})
+      section_id = options[:section_id] || 'global'
+      content_tag :div, "data-pagination" => section_id, :class => ((Array(options[:class]) || []) + ["ajaxpagination"]).join(" ") do
+        yield
+      end
+    end
+
     # modifies the html options, so that it calls AJAX Pagination. This method adds the appropriate parameters to make an AJAX call via
     # AJAX Pagination, using jquery-ujs.
     #
-    # More specifically, it ensures the following attributes are defined :remote => true, "data-type" => 'html', :pagination => ?.
-    # The pagination attribute must be defined, or else it defaults to the empty string "".
+    # More specifically, it ensures the following attributes are defined :remote => true, "data-type" => 'html', :section_id => ?.
+    # The section_id attribute must be defined, or else it defaults to the empty string "global".
     # This link always sets data-remote to true - setting to false is not allowed, since AJAX Pagination would not be triggered.
     #
     # Below is an alternative way to create an ajax link instead of ajax_link_to
     #
-    #   <%= link_to "Name", posts_url, ajax_options :pagination => "page" %>
+    #   <%= link_to "Name", posts_url, ajax_options :section_id => "page" %>
     #
     def ajax_options(html_options = {})
-      html_options["data-pagination".to_sym] = html_options.delete(:pagination) || html_options.delete("data-pagination") || "" # renames the option pagination to data-pagination
+      html_options["data-pagination".to_sym] = html_options.delete(:section_id) || html_options.delete("data-section_id") || html_options.delete(:pagination) || html_options.delete("data-pagination") || "global" # renames the option pagination to data-pagination
       html_options[:remote] = true
       html_options["data-type".to_sym] ||= html_options.delete("data-type") || 'html'
       html_options
@@ -156,7 +180,7 @@ module AjaxPagination
     # The example below creates a link "Name", which when clicked, will load posts_url into the section of the page named
     # "page" using AJAX.
     #
-    #   <%= ajax_link_to "Name", posts_url, :pagination => "page" %>
+    #   <%= ajax_link_to "Name", posts_url, :section_id => "page" %>
     #
     def ajax_link_to(*args, &block)
       if block_given? # inject new html_options argument and call link_to
@@ -172,11 +196,11 @@ module AjaxPagination
 
     # Wrapper for form_tag, following are equivalent:
     #
-    #   <%= ajax_form_tag posts_url, :method => "post", :class => "myclass", :pagination => "page" do %>
+    #   <%= ajax_form_tag posts_url, :method => "post", :class => "myclass", :section_id => "page" do %>
     #     ...
     #   <% end %>
     #
-    #   <%= form_tag posts_url, ajax_options :method => "post", :class => "myclass", :pagination => "page" do %>
+    #   <%= form_tag posts_url, ajax_options :method => "post", :class => "myclass", :section_id => "page" do %>
     #     ...
     #   <% end %>
     #
@@ -191,17 +215,17 @@ module AjaxPagination
 
     # Wrapper for form_for. The following are equivalent
     #
-    #   <%= ajax_form_for @post, :method => "post", :html => {:class => "myclass", :pagination => "menu"} do %>
+    #   <%= ajax_form_for @post, :method => "post", :html => {:class => "myclass", :section_id => "menu"} do %>
     #     ...
     #   <% end %>
     #
-    #   <%= form_for @post, :method => "post", :html => ajax_options({:class => "myclass", :pagination => "menu"}) do %>
+    #   <%= form_for @post, :method => "post", :html => ajax_options({:class => "myclass", :section_id => "menu"}) do %>
     #     ...
     #   <% end %>
     #
     # Please be aware that in the second alternative, you should never set :remote => false manually, eg:
     #
-    #   <%= form_tag @post, :remote => false, :html => ajax_options(:pagination => "menu") do %><!-- Never Do This!!! -->
+    #   <%= form_tag @post, :remote => false, :html => ajax_options(:section_id => "menu") do %><!-- Never Do This!!! -->
     #
     # This will prevent AJAX Pagination from being called.
     #

data/lib/ajax_pagination/rails.rb

@@ -15,12 +15,5 @@ module AjaxPagination
       end
     end
 
-    initializer 'ajax_pagination.redirect_filter' do
-      if AjaxPagination.redirect_after_filter == true
-        ActiveSupport.on_load(:action_controller) do
-          after_filter :ajax_pagination_redirect
-        end
-      end
-    end
   end
 end

data/lib/ajax_pagination/version.rb

@@ -1,3 +1,3 @@
 module AjaxPagination
-  VERSION = "0.5.1"
+  VERSION = "0.6.0"
 end

data/lib/assets/javascripts/ajax_pagination.js.erb

@@ -2,7 +2,7 @@
 //= require jquery.url
 
 /* 
- * AJAX Pagination: Ajaxifying your pagination links
+ * AJAX Pagination: Ajaxifying your navigation
  * https://github.com/ronalchn/ajax_pagination
  * 
  * Copyright (c) 2012 Ronald Ping Man Chan
@@ -56,7 +56,7 @@ jQuery(document).ready(function () {
           swapPage(pagination_name,url,options.history);
         }
         this.exists = function() {
-          return $('#' + pagination_name + '_paginated_section').length == 1;
+          return $('#' + pagination_name).length == 1;
         }
       }
       /////////////////////////////
@@ -95,15 +95,13 @@ jQuery(document).ready(function () {
         }
       }
       function getSection(pagination_name) {
-        var id = "#" + pagination_name + "_paginated_section"; // element id we are looking for
+        var id = "#" + pagination_name; // element id we are looking for
         return $(id);
       }
       function getSectionName(section) {
           var id = section.attr("id");
           if (id === undefined) return undefined; // no name
-          var pagination_name = /^(.*)_paginated_section$/.exec(id)[1];
-          if (pagination_name == null || pagination_name === undefined) return undefined; // pagination not set up properly
-          return pagination_name;
+          return id; // id = pagination_name
       }
       function getSectionNames(sections) {
         var names = new Array();
@@ -158,9 +156,9 @@ jQuery(document).ready(function () {
       // this event handler has the same arguments as for jquery and jquery-ujs, except it also takes the name of the section to put the content into as first argument
       // adapter functions will be used to reconcile the differences in arguments, this is required because jquery and jquery-ujs has different ways to get the pagination_name argument
       function beforeSendHandler(pagination_name,jqXHR,settings) {
-        var id = "#" + pagination_name + "_paginated_section"; // element id we are looking for
+        var id = "#" + pagination_name; // element id we are looking for
         var requesturl = settings.url;
-        var countid = $('[id="' + pagination_name + '_paginated_section"]').length;
+        var countid = $('[id="' + pagination_name + '"]').length;
         if (countid != 1) { // something wrong, cannot find unique section to load page into
           <% if AjaxPagination.warnings %>
             alert("AJAX Pagination UNIQUE_SECTION_NOT_FOUND:\nExpected one pagination section called " + pagination_name + ", found " + countid);
@@ -234,7 +232,7 @@ jQuery(document).ready(function () {
         });
       }
       function pushHistory(pagination_name,url) {
-        var data = $("#" + pagination_name + "_paginated_section").data("pagination");
+        var data = $("#" + pagination_name).data("pagination");
         if (data === undefined || data.history === undefined || data.history) { // check that history is not disabled
           // construct visible url
           var data = $.deparam.querystring($.url(url).attr('query'));
@@ -252,21 +250,20 @@ jQuery(document).ready(function () {
         }
       }
       // these special containers are for convenience only, to apply the required data-remote, data-pagination attributes to all links inside
-      $(document).on("click", ".pagination a, .ajaxpagination a, a.ajaxpagination", function(e) {
+      $(document).on("click", ".ajaxpagination a", function(e) {
         // ignore if already selected by jquery-ujs
         if ($(this).filter($.rails.linkClickSelector).length>0) return true; // continue with jquery-ujs - this behaviour is necessary because we do not know if the jquery-ujs handler executes before or after this handler
         // find out what data-pagination should be set to
-        var pagination_container = $(this).closest(".pagination, .ajaxpagination"); // container of links (use to check for data-pagination first)
+        var pagination_container = $(this).closest(".ajaxpagination"); // container of links (use to check for data-pagination first)
         var pagination_name = pagination_container.data('pagination');
         if (pagination_name === undefined) {
-          pagination_name = /^(.*)_paginated_section$/.exec($(this).closest(".paginated_section").attr("id")); // if data-pagination not present, search up the tree for a suitable section
+          pagination_name = $(this).closest(".paginated_section").attr("id"); // if data-pagination not present, search up the tree for a suitable section
           if (pagination_name == null) {
             <% if AjaxPagination.warnings %>
-              alert("AJAX Pagination MISSING_REFERENCE:\nNo pagination section name given for link, and none could be implicitly assigned, AJAX cancelled for this request");
+              alert("AJAX Pagination MISSING_REFERENCE:\nNo pagination section id given for link, and none could be implicitly assigned, AJAX cancelled for this request");
             <% end %>
             return true; // pagination not set up properly
           }
-          pagination_name = pagination_name[1];
         }
         
         // set data-remote, data-pagination