new hash format

Author: GBH

README.md

@@ -20,7 +20,7 @@ active_link_to 'Users', '/users'
 This is exactly the same as:
 
 ```ruby
-active_link_to 'Users', '/users', :active => :inclusive
+active_link_to 'Users', '/users', active: :inclusive
 # => <a href="/users" class="active">Users</a>
 ```
 
@@ -44,72 +44,72 @@ nodes. Most commonly used for 'home' links.
 
 ```ruby
 # For URL: /users will be active
-active_link_to 'Users', users_path, :active => :exclusive
+active_link_to 'Users', users_path, active: :exclusive
 # => <a href="/users" class="active">Users</a>
 ```
 
 ```ruby
 # But for URL: /users/123 it will not be active
-active_link_to 'Users', users_path, :active => :exclusive
+active_link_to 'Users', users_path, active: :exclusive
 # => <a href="/users">Users</a>
 ```
 
 If we need to set link to be active based on some regular expression, we can do
 that as well. Let's try to activate links urls of which begin with 'use':
 
 ```ruby
-active_link_to 'Users', users_path, :active => /^\/use/
+active_link_to 'Users', users_path, active: /^\/use/
 ```
 
 If we need to set link to be active based on an exact match, we can do
 that as well:
 
 ```ruby
-active_link_to 'Users', users_path, :active => :exact
+active_link_to 'Users', users_path, active: :exact
 ```
 
 What if we need to mark link active for all URLs that match a particular controller,
 or action, or both? Or any number of those at the same time? Sure, why not:
 
 ```ruby
 # For matching multiple controllers and actions:
-active_link_to 'User Edit', edit_user_path(@user), :active => [['people', 'news'], ['show', 'edit']]
+active_link_to 'User Edit', edit_user_path(@user), active: [['people', 'news'], ['show', 'edit']]
 
 # for matching all actions under given controllers:
-active_link_to 'User Edit', edit_user_path(@user), :active => [['people', 'news'], []]
+active_link_to 'User Edit', edit_user_path(@user), active: [['people', 'news'], []]
 
 # for matching all controllers for a particular action
-active_link_to 'User Edit', edit_user_path(@user), :active => [[], ['edit']]
+active_link_to 'User Edit', edit_user_path(@user), active: [[], ['edit']]
 ```
 
 Sometimes it should be as easy as giving link true or false value:
 
 ```ruby
-active_link_to 'Users', users_path, :active => true
+active_link_to 'Users', users_path, active: true
 ```
 
 ## More Options
 You can specify active and inactive css classes for links:
 
 ```ruby
-active_link_to 'Users', users_path, :class_active => 'enabled'
+active_link_to 'Users', users_path, class_active: 'enabled'
 # => <a href="/users" class="enabled">Users</a>
 
-active_link_to 'News', news_path, :class_inactive => 'disabled'
+active_link_to 'News', news_path, class_inactive: 'disabled'
 # => <a href="/news" class="disabled">News</a>
 ```
 
 Sometimes you want to replace link tag with a span if it's active:
 
 ```ruby
-active_link_to 'Users', users_path, :active_disable => true
+active_link_to 'Users', users_path, active_disable: true
 # => <span class="active">Users</span>
 ```
 
 If you are constructing navigation menu it might be helpful to wrap links in another tag, like `<li>`:
 
 ```ruby
-active_link_to 'Users', users_path, :wrap_tag => :li
+active_link_to 'Users', users_path, wrap_tag: :li
 # => <li class="active"><a href="/users" class="active">Users</a></li>
 ```
 
@@ -126,7 +126,7 @@ is_active_link?(users_path, :inclusive)
 `active_link_to_class` will return the css class:
 
 ```
-active_link_to_class(users_path, :active => :inclusive)
+active_link_to_class(users_path, active: :inclusive)
 # => 'active'
 ```
 

Rakefile

@@ -8,4 +8,4 @@ Rake::TestTask.new(:test) do |test|
   test.verbose = true
 end
 
-task :default => :test
+task default: :test

lib/active_link_to/active_link_to.rb

@@ -7,8 +7,8 @@ module ActiveLinkTo
   #   :disable_active => Boolean
   #   :wrap_tag       => Symbol
   # Example usage:
-  #   active_link_to('/users', :class_active => 'enabled')
-  #   active_link_to(users_path, :active => :exclusive, :wrap_tag => :li)
+  #   active_link_to('/users', class_active: 'enabled')
+  #   active_link_to(users_path, active: :exclusive, wrap_tag: :li)
   def active_link_to(*args, &block)
     if block_given?
       name          = capture(&block)
@@ -44,12 +44,12 @@ def active_link_to(*args, &block)
       link_to(name, url, link_options)
     end
 
-    wrap_tag ? content_tag(wrap_tag, link, :class => (css_class if css_class.present?)) : link
+    wrap_tag ? content_tag(wrap_tag, link, class: (css_class if css_class.present?)) : link
   end
 
   # Returns css class name. Takes the link's URL and its params
   # Example usage:
-  #   active_link_to_class('/root', :class_active => 'on', :class_inactive => 'off')
+  #   active_link_to_class('/root', class_active: 'on', class_inactive: 'off')
   #
   def active_link_to_class(url, options = {})
     if is_active_link?(url, options[:active])

test/active_link_to_test.rb

@@ -105,19 +105,19 @@ def test_is_active_link_array
   def test_is_active_link_hash
     params[:a] = 1
 
-    assert is_active_link?('/', {:a => 1})
-    assert is_active_link?('/', {:a => 1, :b => nil})
+    assert is_active_link?('/', {a: 1})
+    assert is_active_link?('/', {a: 1, b: nil})
 
-    refute is_active_link?('/', {:a => 1, :b => 2})
-    refute is_active_link?('/', {:a => 2})
+    refute is_active_link?('/', {a: 1, b: 2})
+    refute is_active_link?('/', {a: 2})
 
     params[:b] = 2
 
-    assert is_active_link?('/', {:a => 1, :b => 2})
-    assert is_active_link?('/', {:a => 1, :b => 2, :c => nil})
+    assert is_active_link?('/', {a: 1, b: 2})
+    assert is_active_link?('/', {a: 1, b: 2, c: nil})
 
-    assert is_active_link?('/', {:a => 1})
-    assert is_active_link?('/', {:b => 2})
+    assert is_active_link?('/', {a: 1})
+    assert is_active_link?('/', {b: 2})
   end
 
   def test_is_active_link_with_memoization
@@ -131,10 +131,10 @@ def test_is_active_link_with_memoization
   def test_active_link_to_class
     set_fullpath('/root')
     assert_equal 'active', active_link_to_class('/root')
-    assert_equal 'on', active_link_to_class('/root', :class_active => 'on')
+    assert_equal 'on', active_link_to_class('/root', class_active: 'on')
 
     assert_equal '', active_link_to_class('/other')
-    assert_equal 'off', active_link_to_class('/other', :class_inactive => 'off')
+    assert_equal 'off', active_link_to_class('/other', class_inactive: 'off')
   end
 
   def test_active_link_to
@@ -148,54 +148,54 @@ def test_active_link_to
 
   def test_active_link_to_with_existing_class
     set_fullpath('/root')
-    link = active_link_to('label', '/root', :class => 'current')
+    link = active_link_to('label', '/root', class: 'current')
     assert_html link, 'a.current.active[href="/root"]', 'label'
 
-    link = active_link_to('label', '/other', :class => 'current')
+    link = active_link_to('label', '/other', class: 'current')
     assert_html link, 'a.current[href="/other"]', 'label'
   end
 
   def test_active_link_to_with_custom_classes
     set_fullpath('/root')
-    link = active_link_to('label', '/root', :class_active => 'on')
+    link = active_link_to('label', '/root', class_active: 'on')
     assert_html link, 'a.on[href="/root"]', 'label'
 
-    link = active_link_to('label', '/other', :class_inactive => 'off')
+    link = active_link_to('label', '/other', class_inactive: 'off')
     assert_html link, 'a.off[href="/other"]', 'label'
   end
 
   def test_active_link_to_with_wrap_tag
     set_fullpath('/root')
-    link = active_link_to('label', '/root', :wrap_tag => :li)
+    link = active_link_to('label', '/root', wrap_tag: :li)
     assert_html link, 'li.active a.active[href="/root"]', 'label'
 
-    link = active_link_to('label', '/root', :wrap_tag => :li, :active_disable => true)
+    link = active_link_to('label', '/root', wrap_tag: :li, active_disable: true)
     assert_html link, 'li.active span.active', 'label'
 
-    link = active_link_to('label', '/root', :wrap_tag => :li, :class => 'testing')
+    link = active_link_to('label', '/root', wrap_tag: :li, class: 'testing')
     assert_html link, 'li.testing.active a.testing.active[href="/root"]', 'label'
   end
 
   def test_active_link_to_with_active_disable
     set_fullpath('/root')
-    link = active_link_to('label', '/root', :active_disable => true)
+    link = active_link_to('label', '/root', active_disable: true)
     assert_html link, 'span.active', 'label'
   end
 
   def test_should_not_modify_passed_params
     set_fullpath('/root')
-    params = { :class => 'testing', :active => :inclusive }
+    params = {class: 'testing', active: :inclusive}
     out = active_link_to 'label', '/root', params
     assert_html out, 'a.testing.active[href="/root"]', 'label'
-    assert_equal ({:class => 'testing', :active => :inclusive }), params
+    assert_equal ({class: 'testing', active: :inclusive }), params
   end
 
   def test_no_empty_class_attribute
     set_fullpath('/root')
-    link = active_link_to('label', '/root', :wrap_tag => :li)
+    link = active_link_to('label', '/root', wrap_tag: :li)
     assert_html link, 'li.active a.active[href="/root"]', 'label'
 
-    link = active_link_to('label', '/other', :wrap_tag => :li)
+    link = active_link_to('label', '/other', wrap_tag: :li)
     assert_html link, 'li a[href="/other"]', 'label'
   end