Merge pull request #2 from propertybase/fix_nil_error

Fix undefined method `length' for nil:NilClass"

CHANGELOG.md

@@ -1,3 +1,9 @@
+## 0.1.1 (June 17th, 2013)
+
+BUG FIXES:
+
+  - Just return `nil` for nil input
+
 ## 0.1.0 (June 17th, 2013)
 
 - initial release

lib/comfan.rb

@@ -4,7 +4,7 @@ module Comfan
   extend self
 
   def api_id input_id
-    return input_id if input_id.length >= 18
+    return input_id if input_id.nil? || input_id.length >= 18
     suffix = ''
 
     3.times do |i|
@@ -21,7 +21,7 @@ def api_id input_id
   end
 
   def ui_id input_id
-    return input_id if input_id.length <= 15
+    return input_id if input_id.nil? || input_id.length <= 15
 
     input_id[0..-4]
   end

lib/comfan/version.rb

@@ -1,3 +1,3 @@
 module Comfan
-  VERSION = '0.1.0'
+  VERSION = '0.1.1'
 end

spec/lib/comfan_spec.rb

@@ -15,6 +15,10 @@
       output = '752S00000000KtkIAE'
       expect(subject.api_id(input)).to eq(output)
     end
+
+    it 'returns nil for nil input' do
+      expect(subject.api_id(nil)).to be_nil
+    end
   end
 
   describe '.ui_id' do
@@ -28,5 +32,9 @@
       output = '752S00000000Ktk'
       expect(subject.ui_id(input)).to eq(output)
     end
+
+    it 'returns nil for nil input' do
+      expect(subject.ui_id(nil)).to be_nil
+    end
   end
 end