kadmin 0.3.1 → 0.3.2

data/lib/kadmin/pager.rb

@@ -1,93 +1,93 @@
-module Kadmin
-  # Simple Pager structure, used to paginate collections
-  class Pager
-    # @return [Integer] number of items per page
-    attr_reader :size
-
-    # @return [Integer] current index offset (determines current page)
-    attr_reader :offset
-
-    # @return [Integer] total number of pages
-    attr_reader :pages
-
-    # @return [Integer] the current page
-    attr_reader :current_page
-
-    # @return [Integer] total number of items in the collection
-    attr_reader :total
-
-    # @param [Integer] size size of a page
-    # @param [Integer] offset offset/index for the current page
-    def initialize(size:, offset:)
-      @size = size
-      @offset = offset
-
-      raise(Kadmin::Error, 'Page size must be greater than 0!') unless @size.positive?
-      raise(Kadmin::Error, 'Offset must be at least 0!') unless @offset >= 0
-
-      @current_page = (@offset / @size.to_f).floor
-      self.total = @size # assume page size is maximum initially
-    end
-
-    # @param [ActiveRecord::Relation] collection relation to paginate
-    # @return [ActiveRecord::Relation] paginated collection
-    def page(collection)
-      self.total = collection.count
-
-      collection = collection.offset(@offset)
-      collection = collection.limit(@size)
-
-      return collection
-    end
-
-    # @param [Integer] page the page to get the offset for; if not given, uses the current page
-    # @return [Integer] start offset for the given page
-    def offset_at(page = nil)
-      page ||= @current_page
-      return @size * page.to_i
-    end
-
-    # @param [Integer] page the page to check for
-    # @return [Boolean] true if `page` is the current page
-    def current_page?(page)
-      return page == @current_page
-    end
-
-    # @param [Integer] page the page to check for
-    # @return [Boolean] true if `page` exists (i.e. would have any data)
-    def contains?(page)
-      page.in?(0...@pages)
-    end
-
-    # @param [Integer] total sets the number of total collection items (not total loaded)
-    def total=(total)
-      @total = total
-      @pages = (@total / @size.to_f).ceil
-    end
-
-    # @param [Integer] page optional; if not given, uses the current page
-    # @return [Integer] the number of items that are on this page
-    def page_size(page = nil)
-      page ||= @current_page
-      page_start = offset_at(page)
-      page_end = [offset_at(page.to_i + 1), @total].min
-
-      return page_end - page_start
-    end
-
-    # @param [Integer] page optional; if given, checks if the page after would have any data, otherwise checks based on the current page
-    # @return [Boolean] true if there is a next page
-    def next_page?(page = nil)
-      page ||= @current_page
-      return offset_at(page.to_i + 1) < @total
-    end
-
-    # @param [Integer] page optional; if given, checks if the page after would have any data, otherwise checks based on the current page
-    # @return [Boolean] true if there is a previous page
-    def previous_page?(page = nil)
-      page ||= @current_page
-
-      return page.to_i.positive?
-    end
-  end
-end
+module Kadmin
+  # Simple Pager structure, used to paginate collections
+  class Pager
+    # @return [Integer] number of items per page
+    attr_reader :size
+
+    # @return [Integer] current index offset (determines current page)
+    attr_reader :offset
+
+    # @return [Integer] total number of pages
+    attr_reader :pages
+
+    # @return [Integer] the current page
+    attr_reader :current_page
+
+    # @return [Integer] total number of items in the collection
+    attr_reader :total
+
+    # @param [Integer] size size of a page
+    # @param [Integer] offset offset/index for the current page
+    def initialize(size:, offset:)
+      @size = size
+      @offset = offset
+
+      raise(Kadmin::Error, 'Page size must be greater than 0!') unless @size.positive?
+      raise(Kadmin::Error, 'Offset must be at least 0!') unless @offset >= 0
+
+      @current_page = (@offset / @size.to_f).floor
+      self.total = @size # assume page size is maximum initially
+    end
+
+    # @param [ActiveRecord::Relation] collection relation to paginate
+    # @return [ActiveRecord::Relation] paginated collection
+    def page(collection)
+      self.total = collection.count
+
+      collection = collection.offset(@offset)
+      collection = collection.limit(@size)
+
+      return collection
+    end
+
+    # @param [Integer] page the page to get the offset for; if not given, uses the current page
+    # @return [Integer] start offset for the given page
+    def offset_at(page = nil)
+      page ||= @current_page
+      return @size * page.to_i
+    end
+
+    # @param [Integer] page the page to check for
+    # @return [Boolean] true if `page` is the current page
+    def current_page?(page)
+      return page == @current_page
+    end
+
+    # @param [Integer] page the page to check for
+    # @return [Boolean] true if `page` exists (i.e. would have any data)
+    def contains?(page)
+      page.in?(0...@pages)
+    end
+
+    # @param [Integer] total sets the number of total collection items (not total loaded)
+    def total=(total)
+      @total = total
+      @pages = (@total / @size.to_f).ceil
+    end
+
+    # @param [Integer] page optional; if not given, uses the current page
+    # @return [Integer] the number of items that are on this page
+    def page_size(page = nil)
+      page ||= @current_page
+      page_start = offset_at(page)
+      page_end = [offset_at(page.to_i + 1), @total].min
+
+      return page_end - page_start
+    end
+
+    # @param [Integer] page optional; if given, checks if the page after would have any data, otherwise checks based on the current page
+    # @return [Boolean] true if there is a next page
+    def next_page?(page = nil)
+      page ||= @current_page
+      return offset_at(page.to_i + 1) < @total
+    end
+
+    # @param [Integer] page optional; if given, checks if the page after would have any data, otherwise checks based on the current page
+    # @return [Boolean] true if there is a previous page
+    def previous_page?(page = nil)
+      page ||= @current_page
+
+      return page.to_i.positive?
+    end
+  end
+end

data/lib/kadmin/version.rb

@@ -1,3 +1,3 @@
-module Kadmin
-  VERSION = '0.3.1'.freeze
-end
+module Kadmin
+  VERSION = '0.3.2'.freeze
+end

data/test/dummy/README.rdoc

@@ -1,28 +1,28 @@
-== README
-
-This README would normally document whatever steps are necessary to get the
-application up and running.
-
-Things you may want to cover:
-
-* Ruby version
-
-* System dependencies
-
-* Configuration
-
-* Database creation
-
-* Database initialization
-
-* How to run the test suite
-
-* Services (job queues, cache servers, search engines, etc.)
-
-* Deployment instructions
-
-* ...
-
-
-Please feel free to use a different markup language if you do not plan to run
-<tt>rake doc:app</tt>.
+== README
+
+This README would normally document whatever steps are necessary to get the
+application up and running.
+
+Things you may want to cover:
+
+* Ruby version
+
+* System dependencies
+
+* Configuration
+
+* Database creation
+
+* Database initialization
+
+* How to run the test suite
+
+* Services (job queues, cache servers, search engines, etc.)
+
+* Deployment instructions
+
+* ...
+
+
+Please feel free to use a different markup language if you do not plan to run
+<tt>rake doc:app</tt>.

data/test/dummy/Rakefile

@@ -1,6 +1,6 @@
-# Add your own tasks in files placed in lib/tasks ending in .rake,
-# for example lib/tasks/capistrano.rake, and they will automatically be available to Rake.
-
-require File.expand_path('../config/application', __FILE__)
-
-Rails.application.load_tasks
+# Add your own tasks in files placed in lib/tasks ending in .rake,
+# for example lib/tasks/capistrano.rake, and they will automatically be available to Rake.
+
+require File.expand_path('../config/application', __FILE__)
+
+Rails.application.load_tasks

data/test/dummy/app/assets/javascripts/application.js

@@ -1,13 +1,13 @@
-// This is a manifest file that'll be compiled into application.js, which will include all the files
-// listed below.
-//
-// Any JavaScript/Coffee file within this directory, lib/assets/javascripts, vendor/assets/javascripts,
-// or any plugin's vendor/assets/javascripts directory can be referenced here using a relative path.
-//
-// It's not advisable to add code directly here, but if you do, it'll appear at the bottom of the
-// compiled file.
-//
-// Read Sprockets README (https://github.com/rails/sprockets#sprockets-directives) for details
-// about supported directives.
-//
-//= require_tree .
+// This is a manifest file that'll be compiled into application.js, which will include all the files
+// listed below.
+//
+// Any JavaScript/Coffee file within this directory, lib/assets/javascripts, vendor/assets/javascripts,
+// or any plugin's vendor/assets/javascripts directory can be referenced here using a relative path.
+//
+// It's not advisable to add code directly here, but if you do, it'll appear at the bottom of the
+// compiled file.
+//
+// Read Sprockets README (https://github.com/rails/sprockets#sprockets-directives) for details
+// about supported directives.
+//
+//= require_tree .

data/test/dummy/app/assets/stylesheets/application.css

@@ -1,15 +1,15 @@
-/*
- * This is a manifest file that'll be compiled into application.css, which will include all the files
- * listed below.
- *
- * Any CSS and SCSS file within this directory, lib/assets/stylesheets, vendor/assets/stylesheets,
- * or any plugin's vendor/assets/stylesheets directory can be referenced here using a relative path.
- *
- * You're free to add application-wide styles to this file and they'll appear at the bottom of the
- * compiled file so the styles you add here take precedence over styles defined in any styles
- * defined in the other CSS/SCSS files in this directory. It is generally better to create a new
- * file per style scope.
- *
- *= require_tree .
- *= require_self
- */
+/*
+ * This is a manifest file that'll be compiled into application.css, which will include all the files
+ * listed below.
+ *
+ * Any CSS and SCSS file within this directory, lib/assets/stylesheets, vendor/assets/stylesheets,
+ * or any plugin's vendor/assets/stylesheets directory can be referenced here using a relative path.
+ *
+ * You're free to add application-wide styles to this file and they'll appear at the bottom of the
+ * compiled file so the styles you add here take precedence over styles defined in any styles
+ * defined in the other CSS/SCSS files in this directory. It is generally better to create a new
+ * file per style scope.
+ *
+ *= require_tree .
+ *= require_self
+ */

data/test/dummy/app/controllers/admin/application_controller.rb

@@ -1,11 +1,11 @@
-module Admin
-  class ApplicationController < Kadmin::ApplicationController
-    def set_navbar_links
-      super
-      @layout_navbar_links += [
-        { title: 'People', path: admin_people_path },
-        { title: 'Groups', path: admin_groups_path }
-      ]
-    end
-  end
-end
+module Admin
+  class ApplicationController < Kadmin::ApplicationController
+    def set_navbar_links
+      super
+      @layout_navbar_links += [
+        { title: 'People', path: admin_people_path },
+        { title: 'Groups', path: admin_groups_path }
+      ]
+    end
+  end
+end

data/test/dummy/app/controllers/admin/people_controller.rb

@@ -1,89 +1,89 @@
-module Admin
-  class PeopleController < Admin::ApplicationController
-    MAX_PAGE_SIZE = 200
-
-    # GET /admin/people
-    def index
-      params.permit(:page_size, :page_offset, :filter_name, :format)
-
-      page_size = [params.fetch(:page_size, 15).to_i, MAX_PAGE_SIZE].min # fix 200 as maximum size
-
-      finder = Kadmin::Finder.new(Person.includes(:groups, :owned_groups).order(created_at: :desc))
-        .filter(name: :name, column: [:first_name, :last_name], value: params[:filter_name])
-        .paginate(size: page_size, offset: params.fetch(:page_offset, 0))
-      finder.find!
-      @finder = Kadmin::FinderDecorator.new(finder)
-    end
-
-    # GET /admin/people/:id
-    def show
-      @person = load_person
-    end
-
-    # GET /admin/people/edit/:id
-    def edit
-      @person ||= load_person
-      @person_form ||= person_form
-    end
-
-    # PUT /admin/people/:id
-    # PATCH /admin/people/:id
-    def update
-      @person = load_person
-      @person_form = person_form
-
-      if @person_form.valid? && @person_form.save
-        redirect_to(admin_person_path(@person_form.id))
-      else
-        render :edit
-      end
-    end
-
-    # GET /admin/people/new
-    def new
-      @person ||= Person.new
-      @person_form ||= person_form
-    end
-
-    # POST /admin/people
-    def create
-      @person = Person.new
-      @person_form = person_form
-
-      if @person_form.valid? && @person_form.save
-        redirect_to(admin_person_path(@person_form.id))
-      else
-        render :new
-      end
-    end
-
-    # DELETE /admin/people/:id
-    def destroy
-      @person = Person.new
-      name = [@person.last_name, @person.first_name].join(', ')
-
-      @person.destroy # only fails when it raises an exception
-      flash[:success] = "Successfully deleted #{name}"
-
-      redirect_to admin_people_path
-    end
-
-    # @!group Helpers
-
-    def load_person
-      params.require(:id)
-      id = params[:id].to_i
-      return Person.includes(:groups, :owned_groups).find(id)
-    end
-    private :load_person
-
-    def person_form
-      form = Forms::PersonForm.new(@person)
-      form.assign_attributes(params.fetch(:person, {}).except(:id))
-      return form
-    end
-    private :person_form
-
-    # @!endgroup
-  end
-end
+module Admin
+  class PeopleController < Admin::ApplicationController
+    MAX_PAGE_SIZE = 200
+
+    # GET /admin/people
+    def index
+      params.permit(:page_size, :page_offset, :filter_name, :format)
+
+      page_size = [params.fetch(:page_size, 15).to_i, MAX_PAGE_SIZE].min # fix 200 as maximum size
+
+      finder = Kadmin::Finder.new(Person.includes(:groups, :owned_groups).order(created_at: :desc))
+        .filter(name: :name, column: [:first_name, :last_name], value: params[:filter_name])
+        .paginate(size: page_size, offset: params.fetch(:page_offset, 0))
+      finder.find!
+      @finder = Kadmin::FinderDecorator.new(finder)
+    end
+
+    # GET /admin/people/:id
+    def show
+      @person = load_person
+    end
+
+    # GET /admin/people/edit/:id
+    def edit
+      @person ||= load_person
+      @person_form ||= person_form
+    end
+
+    # PUT /admin/people/:id
+    # PATCH /admin/people/:id
+    def update
+      @person = load_person
+      @person_form = person_form
+
+      if @person_form.valid? && @person_form.save
+        redirect_to(admin_person_path(@person_form.id))
+      else
+        render :edit
+      end
+    end
+
+    # GET /admin/people/new
+    def new
+      @person ||= Person.new
+      @person_form ||= person_form
+    end
+
+    # POST /admin/people
+    def create
+      @person = Person.new
+      @person_form = person_form
+
+      if @person_form.valid? && @person_form.save
+        redirect_to(admin_person_path(@person_form.id))
+      else
+        render :new
+      end
+    end
+
+    # DELETE /admin/people/:id
+    def destroy
+      @person = Person.new
+      name = [@person.last_name, @person.first_name].join(', ')
+
+      @person.destroy # only fails when it raises an exception
+      flash[:success] = "Successfully deleted #{name}"
+
+      redirect_to admin_people_path
+    end
+
+    # @!group Helpers
+
+    def load_person
+      params.require(:id)
+      id = params[:id].to_i
+      return Person.includes(:groups, :owned_groups).find(id)
+    end
+    private :load_person
+
+    def person_form
+      form = Forms::PersonForm.new(@person)
+      form.assign_attributes(params.fetch(:person, {}).except(:id))
+      return form
+    end
+    private :person_form
+
+    # @!endgroup
+  end
+end