Fix coding style

Author: n-rodriguez

.rubocop.yml

@@ -1,15 +1,15 @@
 AllCops:
-  TargetRubyVersion: 2.5
+  TargetRubyVersion: 2.4
   Exclude:
+    - bin/*
+    - lib/generators/**/*.rb
     - gemfiles/*
-    - spec/**/*.rb
-    - lib/ajax-datatables-rails.rb
-    - lib/generators/rails/templates/*.rb
+    - spec/**/*
 
-Documentation:
+Style/Documentation:
   Enabled: false
 
-Layout/AlignHash:
+Layout/HashAlignment:
   Enabled: false
 
 Layout/EmptyLines:
@@ -45,6 +45,10 @@ Metrics/ClassLength:
 Naming/AccessorMethodName:
   Enabled: false
 
+Naming/FileName:
+  Exclude:
+    - lib/ajax-datatables-rails.rb
+
 Style/TrailingCommaInArrayLiteral:
   EnforcedStyleForMultiline: comma
 

ajax-datatables-rails.gemspec

@@ -31,5 +31,6 @@ Gem::Specification.new do |s|
   s.add_development_dependency 'rails', '>= 5.0'
   s.add_development_dependency 'rake'
   s.add_development_dependency 'rspec'
+  s.add_development_dependency 'rubocop'
   s.add_development_dependency 'simplecov'
 end

bin/rubocop

@@ -0,0 +1,29 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+#
+# This file was generated by Bundler.
+#
+# The application 'rubocop' is installed as part of a gem, and
+# this file is here to facilitate running it.
+#
+
+require "pathname"
+ENV["BUNDLE_GEMFILE"] ||= File.expand_path("../../Gemfile",
+  Pathname.new(__FILE__).realpath)
+
+bundle_binstub = File.expand_path("../bundle", __FILE__)
+
+if File.file?(bundle_binstub)
+  if File.read(bundle_binstub, 300) =~ /This file was generated by Bundler/
+    load(bundle_binstub)
+  else
+    abort("Your `bin/bundle` was not generated by Bundler, so this binstub cannot run.
+Replace `bin/bundle` by running `bundle binstubs bundler --force`, then run this command again.")
+  end
+end
+
+require "rubygems"
+require "bundler/setup"
+
+load Gem.bin_path("rubocop", "rubocop")

lib/ajax-datatables-rails.rb

@@ -8,7 +8,7 @@
 loader.ignore(generators)
 loader.inflector.inflect(
   'orm'                   => 'ORM',
-  'ajax-datatables-rails' => 'AjaxDatatablesRails',
+  'ajax-datatables-rails' => 'AjaxDatatablesRails'
 )
 loader.setup
 

lib/ajax-datatables-rails/datatable/simple_order.rb

@@ -14,7 +14,7 @@ def initialize(datatable, options = {})
       end
 
       def query(sort_column)
-        [sort_column, direction, nulls_last_sql].compact.join(" ")
+        [sort_column, direction, nulls_last_sql].compact.join(' ')
       end
 
       def column
@@ -44,9 +44,9 @@ def nulls_last_sql
 
         case AjaxDatatablesRails.config.db_adapter
         when :pg, :postgresql, :postgres, :oracle
-          "NULLS LAST"
+          'NULLS LAST'
         when :mysql, :mysql2, :sqlite, :sqlite3
-          "IS NULL"
+          'IS NULL'
         else
           raise 'unsupported database adapter'
         end