add some most logged in report

and some infrastructure for reports in general
app/controllers/reports_controller.rb
  def index, and some private methods
app/models/book.rb
    tidy
app/models/report.rb
    class Report
app/views/reports/index.html.haml
    table for most logins
config/routes.rb
    get 'reports' => 'reports#index'
spec/factories.rb
    tidy
spec/features/reports_spec.rb
    describe 'Report Features:' do
      describe 'Index' do
        describe 'as a user' do
          it "can't be seen" do
        describe 'as an admin' do
          describe 'before an impersonation' do
            it 'shows the most logged in user' do
          describe 'after an impersonation' do
            it 'still has not changed the most logged in user' do
Merge pull request #125 from Mattsface/master

updated readme
remove deprecation warning in a test

change should to expect
add true_user_id to books and reviews, some reports

rails g migration add-true_user_id-to-books true_user_id:integer
rails g migration add-true_user_id-to-reviews true_user_id:integer
app/controllers/reports_controller.rb
+  def all_reviews
+  def all_books
+  def most_books
+  def all_ratings
+  def most_activity
+  def highest_ratings  #empty
+  def lowest_ratings   #empty
+  def rated
+  def rating_averages
app/controllers/reviews_controller.rb
+    @review.true_user_id = true_user.id
app/views/reports/index.html.haml
+%h2 Most Reviews
+%h2 Most Books
+%h2 Most Activity
db/seeds.rb
   add true_user_id to books and reviews
spec/factories.rb
+    after(:build) { |book| book.true_user_id = book.owner.id }
+    rating 3
+    after(:build) { |review| review.true_user_id = review.user_id }
spec/features/reports_spec.rb
       describe 'before an impersonation' do
         it 'shows the most logged in user' do
+          expect(page).to have_text("logins #{@user.id} #{@user.email} 3")
+        it 'shows the user with the most reviews' do
+        it 'shows the user with the most books' do
+        it 'shows the user with the lowest ratings' do
+        it 'shows the user with the highest ratings' do
+        it 'shows the most active user' do
+        it 'still has not changed the most active user' do
+          pending
complete reports. closes (#24) and (#29)

reports controller is disgusting,
but CC still gives us a 3.34 so I am not refactoring for now

app/controllers/reports_controller.rb
   def highest_ratings
   def lowest_ratings
app/views/reports/index.html.haml
+%h2 Highest Ratings
+%h2 Lowest Ratings
spec/features/reports_spec.rb
         it 'shows the user with the lowest ratings' do
         it 'shows the user with the highest ratings' do
         it 'still has not changed the most active user' do
           fix some capybara clicking
Merge branch 'log-by-hand' of https://github.com/nbirnel/Library-Application into nbirnel-log-by-hand

Conflicts:
	db/schema.rb
	spec/factories.rb
Merge branch 'nbirnel-log-by-hand'
add report link to user_admin, update README.
make btn-default in books index
nuke ratyrate migrations
Merge pull request #127 from nbirnel/kill-old-migrations

nuke ratyrate migrations
changed pg to production db
changed production db to pg
config.assets.compile = true
add a few empty methods for searching
added default rating value to book
added default rating to reviews
make search work, closing (#23)

Gemfile
 nuke cancan and rolify, because they suck and we weren't using them anyway
 gem 'ransack'                             #search
app/controllers/authors_controller.rb
  remove cancan stuff
app/controllers/books_controller.rb
  remove cancan stuff
  add set_book to more actions
  remove abortive search_by funcs

  ransack stuff:
    @q = Book.search(params[:q])
    def search
app/views/layouts/application.html.haml
    nuke the stupid sidebar
app/views/shared/_search_form.html.erb
    ransack search_form_for
config/routes.rb
   ransack route stuff
spec/features/search_spec.rb
 describe 'Search Features:' do
   describe 'Index' do
     it 'shows some search fields' do
     it 'searches' do
Merge pull request #128 from Mattsface/fix_seeds

added default rating value to book
Merge branch 'ransack' into ransack-merge