rondy · April 26, 2012 20:46 · Mar 11, 2012 · Sep 28, 2011
diff --git a/credential.rb b/credential.rb
@@ -7,15 +7,15 @@ class Credential
   validate :user_exists, :unless => :errors?
 
   def initialize(attributes = {})
-    attributes.each {|k, v| set_recognized_attribute(k, v) }
+    attributes.each { |k, v| set_recognized_attribute(k, v) }
   end
 
   def save
     valid? && create_device
   end
 
-  def as_json(options = {})
-    {:api_key => @device.api_key}
+  def api_key
+    @device.api_key
   end
 
   private
@@ -48,5 +48,4 @@ def create_device
   def user_exists
     errors.add(:user, 'not found') unless user.present?
   end
-
 end
diff --git a/credential_serializer.rb b/credential_serializer.rb
@@ -0,0 +1,13 @@
+class CredentialSerializer
+  def initialize(credential)
+    @credential = credential
+  end
+
+  def to_json
+    {:api_key => @credential.api_key}.to_json
+  end
+
+  def errors
+    @credential.errors.map { |k, m| "#{k} #{m}" }
+  end
+end
diff --git a/credentials_controller.rb b/credentials_controller.rb
@@ -1,16 +1,8 @@
 class CredentialsController < ApplicationController
   def create
     credential = Credential.new(params)
-    if credential.save
-      render :json => credential
-    else
-      render :json => {:error => errors_for(credential)}, :status => :unprocessable_entity
-    end
-  end
-
-  private
+    serializer = CredentialSerializer.new(credential)
 
-  def errors_for(object)
-    object.errors.map {|k, m| "#{k} #{m}" }
+    respond_with serializer
   end
 end
diff --git a/credential.rb b/credential.rb
@@ -0,0 +1,52 @@
+class Credential
+  include ActiveModel::Validations
+
+  attr_accessor :screen_name, :oauth_token, :oauth_secret, :token, :description
+
+  validates_presence_of :screen_name, :oauth_token, :oauth_secret, :message => 'required'
+  validate :user_exists, :unless => :errors?
+
+  def initialize(attributes = {})
+    attributes.each {|k, v| set_recognized_attribute(k, v) }
+  end
+
+  def save
+    valid? && create_device
+  end
+
+  def as_json(options = {})
+    {:api_key => @device.api_key}
+  end
+
+  private
+
+  def errors?
+    errors.any?
+  end
+
+  def set_recognized_attribute(name, value)
+    setter_method = "#{name}="
+    self.send(setter_method, value) if respond_to?(setter_method)
+  end
+
+  def user
+    @user ||= User.by_screen_name(screen_name).where({
+      :oauth_token  => oauth_token,
+      :oauth_secret => oauth_secret
+    }).first
+  end
+
+  def create_device
+    @device = Device.find_or_create_by_token!({
+      :token       => token,
+      :description => description,
+      :user_id     => user.id
+    })
+    !@device.nil?
+  end
+
+  def user_exists
+    errors.add(:user, 'not found') unless user.present?
+  end
+
+end
diff --git a/credentials_controller.rb b/credentials_controller.rb
@@ -0,0 +1,16 @@
+class CredentialsController < ApplicationController
+  def create
+    credential = Credential.new(params)
+    if credential.save
+      render :json => credential
+    else
+      render :json => {:error => errors_for(credential)}, :status => :unprocessable_entity
+    end
+  end
+
+  private
+
+  def errors_for(object)
+    object.errors.map {|k, m| "#{k} #{m}" }
+  end
+end