orcid 0.0.1 → 0.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 148909130b9d10ade4c7e2e6b32d07321dcac06f
-  data.tar.gz: 0fb6c0784ef0a0000bda7e737fe7604519c7afe3
+  metadata.gz: a6c934af18b7c7e4406bd1b4205e71dc44377838
+  data.tar.gz: c9c4b765a0ba649e91e1c79d6891158622c770ef
 SHA512:
-  metadata.gz: 7d8b914e980f40cb79f6d19942c56ba5f3a5d27330b05d3ef3cb3b3f2c809b829e5c566803b1b5ac1b5025a52da4163e1ee495c232142e2ee1a63fa69575d34c
-  data.tar.gz: 9bc94eba6a209de41c0fefa4719d511eb9528ff9c2fc02882bb3be065a0e80a1af06cef5d4afc7f2a23b77a731e76ce4460652536dd4eae7c9c2ebde71db0cd6
+  metadata.gz: c614efa5c8b7290fe4faf5ce20ed9f87a622dba3b2d54579c207aee34489985a2169c1d1e888217ded354d685842bb97512ddf621a008e6ddc824be82f9413c2
+  data.tar.gz: e95c8d1d8911a135eb334267b9796b9290a04af92a3ce96f3942af55df208c3ce1f6bf333d642872e39a4962cb8f28148547a664ff3b039d7b3cf7051f4578a0

data/README.md

@@ -1,7 +1,18 @@
-# Orcid
+# Orcid [![Version](https://badge.fury.io/rb/orcid.png)](http://badge.fury.io/rb/orcid) [![Build Status](https://travis-ci.org/jeremyf/orcid.png?branch=master)](https://travis-ci.org/jeremyf/orcid)
+
 
 A Rails Engine for integrating with Orcid
 
-## TODO
+## Installation
+
+Add this line to your application's Gemfile:
+
+    gem 'orcid'
+
+And then execute:
+
+    $ bundle
+
+And then install:
 
-* Create a generator for installing configuration options into your Rails application.
+    $ rails generate orcid:install

data/Rakefile

@@ -6,11 +6,21 @@ end
 
 Bundler::GemHelper.install_tasks
 
-require 'engine_cart/rake_task'
 
+begin
+  APP_RAKEFILE = File.expand_path("../spec/internal/Rakefile", __FILE__)
+  load 'rails/tasks/engine.rake'
+rescue LoadError
+  puts "Unable to load all app tasks for #{APP_RAKEFILE}"
+end
+
+require 'engine_cart/rake_task'
 require 'rspec/core/rake_task'
 
 namespace :spec do
+  RSpec::Core::RakeTask.new(:all) do |t|
+    ENV['COVERAGE'] = 'true'
+  end
   desc 'Only run specs that do not require net connect'
   RSpec::Core::RakeTask.new(:offline) do |t|
     t.rspec_opts = "--tag ~requires_net_connect"
@@ -34,3 +44,4 @@ Rake::Task["default"].clear rescue nil
 Rake::Task["spec"].clear
 
 task :spec => 'spec:offline'
+task :default => :spec

data/app/models/orcid/profile.rb

@@ -5,8 +5,8 @@ module Orcid
     private :mapper
     def initialize(orcid_profile_id, config = {})
       @orcid_profile_id = orcid_profile_id
-      @mapper = config.fetch(:mapper) { ::Mappy }
-      @remote_service = config.fetch(:remote_service) { Orcid::RemoteWorkService }
+      @mapper = config.fetch(:mapper) { Orcid.mapper }
+      @remote_service = config.fetch(:remote_service) { Orcid::Remote::WorkService }
       @xml_renderer = config.fetch(:xml_renderer) { Orcid::Work::XmlRenderer }
       @xml_parser = config.fetch(:xml_parser) { Orcid::Work::XmlParser }
     end

data/app/models/orcid/profile_connection.rb

@@ -12,29 +12,44 @@ module Orcid
     validates :user, presence: true
     validates :orcid_profile_id, presence: true
 
-    def persisted?; false; end
 
     def save(config = {})
       persister = config.fetch(:persister) { Orcid.method(:connect_user_and_orcid_profile) }
       valid? ? persister.call(user, orcid_profile_id) : false
     end
 
+    def persisted?; false; end
+
+    attr_writer :profile_lookup_service
+    def profile_lookup_service
+      @profile_lookup_service ||= default_profile_lookup_service
+    end
+    private :profile_lookup_service
+
+    def default_profile_lookup_service
+      ProfileLookupRunner.new {|on|
+        on.found {|results| self.orcid_profile_candidates = results }
+        on.not_found { self.orcid_profile_candidates = [] }
+      }
+    end
+    private :default_profile_lookup_service
+
     def with_orcid_profile_candidates
-      if query_for_orcid_profile_candidates?
-        yield(orcid_profile_candidates)
-      end
+      yield(orcid_profile_candidates) if email.present?
     end
-    attr_writer :orcid_profile_querier
-    protected
+
+    attr_writer :orcid_profile_candidates
+    private :orcid_profile_candidates=
     def orcid_profile_candidates
-      @orcid_profile_candidates ||= orcid_profile_querier.call({q: "email:#{email}"})
-    end
-    def query_for_orcid_profile_candidates?
-      email.present?
+      @orcid_profile_candidates || lookup_profile_candidates
     end
 
-    def orcid_profile_querier
-      @orcid_profile_querier ||= ProfileLookupService
+    def lookup_profile_candidates
+      if email.present?
+        profile_lookup_service.call(email: email)
+      end
     end
+    private :lookup_profile_candidates
+
   end
 end

data/app/models/orcid/profile_request.rb

@@ -11,6 +11,7 @@ module Orcid
 
     self.table_name = :orcid_profile_requests
 
+    alias_attribute :email, :primary_email
     validates :user_id, presence: true, uniqueness: true
     validates :given_names, presence: true
     validates :family_name, presence: true
@@ -21,15 +22,18 @@ module Orcid
     def run(options = {})
       # Why dependency injection? Because this is going to be a plugin, and things
       # can't possibly be simple.
-      before_run_validator = options.fetch(:before_run_validator) { method(:validate_before_run) }
-      return false unless before_run_validator.call(self)
+      validator = options.fetch(:validator) { method(:validate_before_run) }
+      return false unless validator.call(self)
 
       payload_xml_builder = options.fetch(:payload_xml_builder) { method(:xml_payload) }
-      profile_creation_service = options.fetch(:profile_creation_service) { Orcid::ProfileCreationService }
-      profile_creation_responder = options.fetch(:profile_creation_responder) { method(:handle_profile_creation_response) }
+      profile_creation_service = options.fetch(:profile_creation_service) { default_profile_creation_service }
+      profile_creation_service.call(payload_xml_builder.call(attributes))
+    end
 
-      orcid_profile_id = profile_creation_service.call(payload_xml_builder.call(attributes))
-      profile_creation_responder.call(orcid_profile_id)
+    def default_profile_creation_service
+      @default_profile_creation_service ||= Orcid::Remote::ProfileCreationService.new do |on|
+        on.success {|orcid_profile_id| handle_profile_creation_response(orcid_profile_id) }
+      end
     end
 
     def validate_before_run(context = self)

data/app/runners/orcid/profile_lookup_runner.rb

@@ -0,0 +1,29 @@
+require_dependency './app/runners/orcid/runner'
+module Orcid
+  class ProfileLookupRunner < Runner
+
+    def initialize(config = {}, &block)
+      super(&block)
+      @query_service = config.fetch(:query_service) { Remote::ProfileLookupService }
+    end
+    attr_reader :query_service
+    private :query_service
+
+    def call(parameters)
+      email = parameters.fetch(:email)
+      response = query_service.call({q: "email:#{email}"})
+      handle(response)
+    end
+
+    private
+    def handle(response)
+      if response.any?
+        callback(:found, response)
+      else
+        callback(:not_found)
+      end
+      response
+    end
+
+  end
+end

data/app/runners/orcid/runner.rb

@@ -0,0 +1,15 @@
+require_dependency './lib/orcid/named_callbacks'
+module Orcid
+  class Runner
+    def initialize
+      @callbacks = NamedCallbacks.new
+      yield(@callbacks) if block_given?
+    end
+
+    def callback(name, *args)
+      @callbacks.call(name, *args)
+      args
+    end
+
+  end
+end

data/app/services/orcid/remote.rb

@@ -0,0 +1,4 @@
+module Orcid
+  module Remote
+  end
+end

data/app/services/orcid/{profile_creation_service.rb → remote/profile_creation_service.rb}

@@ -1,13 +1,15 @@
-# Responsible for minting a new ORCID for the given payload.
-module Orcid
-  class ProfileCreationService
+require_dependency './app/services/orcid/remote/service'
+module Orcid::Remote
+  # Responsible for minting a new ORCID for the given payload.
+  class ProfileCreationService < Orcid::Remote::Service
 
-    def self.call(payload, config = {})
-      new(config).call(payload)
+    def self.call(payload, config = {}, &callback_config)
+      new(config, &callback_config).call(payload)
     end
 
     attr_reader :token, :path, :headers
-    def initialize(config = {})
+    def initialize(config = {}, &callback_config)
+      super(&callback_config)
       @token = config.fetch(:token) { Orcid.client_credentials_token('/orcid-profile/create') }
       @path = config.fetch(:path) { "v1.1/orcid-profile" }
       @headers = config.fetch(:headers) { default_headers }
@@ -25,7 +27,13 @@ module Orcid
 
     def parse(response)
       uri = URI.parse(response.headers.fetch(:location))
-      uri.path.sub(/\A\//, "").split("/").first
+      if orcid_profile_id = uri.path.sub(/\A\//, "").split("/").first
+        callback(:success, orcid_profile_id)
+        orcid_profile_id
+      else
+        callback(:failure)
+        false
+      end
     end
 
     def default_headers

data/app/services/orcid/{profile_lookup_service.rb → remote/profile_lookup_service.rb}

@@ -1,18 +1,6 @@
-module Orcid
+require_dependency 'json'
+module Orcid::Remote
   class ProfileLookupService
-    class SearchResponse
-      delegate :fetch, :has_key?, :[], to: :@attributes
-      def initialize(attributes = {})
-        @attributes = attributes.with_indifferent_access
-      end
-      def id
-        @attributes.fetch(:id)
-      end
-
-      def label
-        @attributes.fetch(:label)
-      end
-    end
 
     def self.call(query, config = {})
       new(config).call(query)
@@ -54,7 +42,7 @@ module Orcid
         family_name = orcid_bio.fetch('personal-details').fetch('family-name').fetch('value')
         emails = orcid_bio.fetch('contact-details').fetch('email').collect {|email| email.fetch('value') }
         label = "#{given_names} #{family_name}"
-        label << " (" << emails.join(",") << ")" if emails.present?
+        label << " (" << emails.join(",") << ")" if emails.any?
         label << " [ORCID: #{identifier}]"
 
         returning_value << response_builder.new("id" => identifier, "label" => label)

data/app/services/orcid/remote/profile_lookup_service/search_response.rb

@@ -0,0 +1,23 @@
+
+module Orcid::Remote
+  class ProfileLookupService
+    class SearchResponse
+      delegate :[], :has_key?, :fetch, to: :@records
+      def initialize(attributes = {})
+        @attributes = attributes.with_indifferent_access
+      end
+
+      def id
+        @attributes.fetch(:id)
+      end
+
+      def orcid_profile_id
+        @attributes.fetch(:id)
+      end
+
+      def label
+        @attributes.fetch(:label)
+      end
+    end
+  end
+end

data/app/services/orcid/remote/service.rb

@@ -0,0 +1,19 @@
+require_dependency './lib/orcid/named_callbacks'
+module Orcid::Remote
+  class Service
+    def initialize
+      @callbacks = Orcid::NamedCallbacks.new
+      yield(@callbacks) if block_given?
+    end
+
+    def call
+      raise NotImplementedError.new("Define #{self.class}#call")
+    end
+
+    def callback(name, *args)
+      @callbacks.call(name, *args)
+      args
+    end
+
+  end
+end

data/app/services/orcid/{remote_work_service.rb → remote/work_service.rb}

@@ -1,5 +1,6 @@
-module Orcid
-  class RemoteWorkService
+require_dependency './lib/orcid/exceptions'
+module Orcid::Remote
+  class WorkService
     def self.call(orcid_profile_id, options = {})
       new(orcid_profile_id, options).call
     end
@@ -27,7 +28,7 @@ module Orcid
     def deliver
       token.request(request_method, path, body: body, headers: headers)
     rescue OAuth2::Error => e
-      raise RemoteServiceError.new(
+      raise Orcid::RemoteServiceError.new(
         response_body: e.response.body,
         response_status: e.response.status,
         client: token.client,

data/lib/generators/orcid/install/install_generator.rb

@@ -0,0 +1,47 @@
+require 'rails/generators'
+
+module Orcid
+  class InstallGenerator < Rails::Generators::Base
+    source_root File.expand_path('../templates', __FILE__)
+
+    class_option :devise, default: false, type: :boolean
+
+    def install_devise_multi_auth
+      if options[:devise]
+        generate 'devise:multi_auth:install --install_devise'
+      else
+        generate 'devise:multi_auth:install'
+      end
+    end
+
+    def install_migrations
+      rake "orcid:install:migrations"
+    end
+
+    def install_omniauth_strategies
+      config_code = ", :omniauthable, :omniauth_providers => [:orcid]"
+      insert_into_file 'app/models/user.rb', config_code, { :after => /:validatable/, :verbose => false }
+
+      init_code = %(
+        config.omniauth(:orcid, Orcid.provider.id, Orcid.provider.secret,
+                        scope: Orcid.provider.authentication_scope,
+                        client_options: {
+                          site: Orcid.provider.site_url,
+                          authorize_url: Orcid.provider.authorize_url,
+                          token_url: Orcid.provider.token_url
+                        }
+                        )
+      )
+      insert_into_file 'config/initializers/devise.rb', init_code, {after: /Devise\.setup.*$/, verbose: true}
+    end
+
+    def mount_orcid_engine
+      route 'mount Orcid::Engine => "/orcid"'
+    end
+
+    def install_initializer
+      template 'orcid_initializer.rb.erb', 'config/orcid_initializer.rb'
+    end
+
+  end
+end

data/lib/generators/orcid/install/templates/orcid_initializer.rb.erb

@@ -0,0 +1,18 @@
+Orcid.configure do |config|
+  # # Configure your Orcid Client Application. See URL below for more
+  # # information
+  # # http://support.orcid.org/knowledgebase/articles/116739-register-a-client-application
+  # config.provider.id = "Your app's Orcid Client ID"
+  # config.provider.secret = "Your app's Orcid Client Secret"
+
+
+  # # Configure how your applications models will be mapped to an Orcid Work so
+  # # that your application can append the work to an Orcid Profile.
+  # config.register_mapping_to_orcid_work(
+  #   :article,
+  #   [
+  #     [:title, :title],
+  #     [lambda{|article| article.publishers.join("; ")}, :publishers]
+  #   ]
+  # )
+end

data/lib/orcid.rb

@@ -12,40 +12,42 @@ require 'simple_form'
 module Orcid
 
   class << self
-    attr_accessor :configuration
+    attr_writer :configuration
+
+    def configuration
+      @configuration ||= Configuration.new
+    end
   end
 
   module_function
   def configure
-    self.configuration ||= Configuration.new
     yield(configuration)
   end
 
-  def provider
-    @provider ||= Configuration::Provider.new
+  def mapper
+    configuration.mapper
   end
 
-  def provider_name
-    configuration.provider_name
+  def provider
+    configuration.provider
   end
 
   def authentication_model
     configuration.authentication_model
   end
 
-
   def connect_user_and_orcid_profile(user, orcid_profile_id, options = {})
-    authentication_model.create!(provider: provider_name, uid: orcid_profile_id, user: user)
+    authentication_model.create!(provider: 'orcid', uid: orcid_profile_id, user: user)
   end
 
   def access_token_for(orcid_profile_id, options = {})
     client = options.fetch(:client) { oauth_client }
     tokenizer = options.fetch(:tokenizer) { authentication_model }
-    tokenizer.to_access_token(uid: orcid_profile_id, provider: provider_name, client: client)
+    tokenizer.to_access_token(uid: orcid_profile_id, provider: 'orcid', client: client)
   end
 
   def profile_for(user)
-    if auth = authentication_model.where(provider: provider_name, user: user).first
+    if auth = authentication_model.where(provider: 'orcid', user: user).first
       Orcid::Profile.new(auth.uid)
     else
       nil
@@ -60,7 +62,7 @@ module Orcid
     # passing the site: option as Orcid's Sandbox has an invalid certificate
     # for the api.sandbox-1.orcid.org
     @oauth_client ||= Devise::MultiAuth.oauth_client_for(
-      provider_name, options: { site: provider.site_url }
+      'orcid', options: { site: provider.site_url }
     )
   end
 

data/lib/orcid/configuration.rb

@@ -1,20 +1,23 @@
 module Orcid
   class Configuration
-    attr_reader :store
-    def initialize(store = ::ENV)
-      @store = store
+    attr_reader :mapper
+    def initialize(options = {})
+      @mapper = options.fetch(:mapper) { ::Mappy }
+      @provider = Configuration::Provider.new
     end
 
-    attr_writer :provider_name
-    def provider_name
-      @provider_name ||= 'orcid'
-    end
+    attr_reader :provider
 
     attr_writer :authentication_model
     def authentication_model
       @authentication_model ||= Devise::MultiAuth::Authentication
     end
 
+    def register_mapping_to_orcid_work(source_type, legend)
+      mapper.configure do |config|
+        config.register(source: source_type, target: 'orcid/work', legend: legend)
+      end
+    end
   end
 end
 require 'orcid/configuration/provider'

data/lib/orcid/configuration/provider.rb

@@ -1,45 +1,51 @@
 module Orcid
-  class Configuration::Provider
-    attr_reader :store
-    def initialize(store = ::ENV)
-      @store = store
-    end
+  class Configuration
+    class Provider
+      attr_reader :store
+      def initialize(store = ::ENV)
+        @store = store
+      end
 
-    attr_writer :authentication_scope
-    def authentication_scope
-      @authentication_scope ||= store.fetch('ORCID_APP_AUTHENTICATION_SCOPE') {
-        "/authenticate,/orcid-works/create,/orcid-works/update,/read-public,/orcid-grants/create"
-      }
-    end
+      attr_writer :authentication_scope
+      def authentication_scope
+        @authentication_scope ||= store.fetch('ORCID_APP_AUTHENTICATION_SCOPE') {
+          "/authenticate,/orcid-works/create,/orcid-works/update,/read-public"
+        }
+      end
 
-    attr_writer :site_url
-    def site_url
-      @site_url ||= store.fetch('ORCID_SITE_URL') { "http://api.sandbox-1.orcid.org" }
-    end
+      attr_writer :site_url
+      def site_url
+        @site_url ||= store.fetch('ORCID_SITE_URL') { "http://api.sandbox-1.orcid.org" }
+      end
 
-    attr_writer :token_url
-    def token_url
-      @token_url ||= store.fetch('ORCID_TOKEN_URL') { "https://api.sandbox-1.orcid.org/oauth/token" }
-    end
+      attr_writer :token_url
+      def token_url
+        @token_url ||= store.fetch('ORCID_TOKEN_URL') { "https://api.sandbox-1.orcid.org/oauth/token" }
+      end
 
-    attr_writer :signin_via_json_url
-    def signin_via_json_url
-      @signin_via_json_url ||= store.fetch('ORCID_REMOTE_SIGNIN_URL') { "https://sandbox-1.orcid.org/signin/auth.json" }
-    end
+      attr_writer :signin_via_json_url
+      def signin_via_json_url
+        @signin_via_json_url ||= store.fetch('ORCID_REMOTE_SIGNIN_URL') { "https://sandbox-1.orcid.org/signin/auth.json" }
+      end
 
-    attr_writer :authorize_url
-    def authorize_url
-      @authorize_url ||= store.fetch('ORCID_AUTHORIZE_URL') { "https://sandbox-1.orcid.org/oauth/authorize" }
-    end
+      attr_writer :authorize_url
+      def authorize_url
+        @authorize_url ||= store.fetch('ORCID_AUTHORIZE_URL') { "https://sandbox-1.orcid.org/oauth/authorize" }
+      end
 
-    attr_writer :id
-    def id
-      @id ||= store.fetch('ORCID_APP_ID')
-    end
+      attr_writer :id
+      def id
+        @id ||= store.fetch('ORCID_APP_ID') {
+          Rails.logger.error('Set your Orcid.provider.id')
+        }
+      end
 
-    attr_writer :secret
-    def secret
-      @secret ||= store.fetch('ORCID_APP_SECRET')
+      attr_writer :secret
+      def secret
+        @secret ||= store.fetch('ORCID_APP_SECRET') {
+          Rails.logger.error('Set your Orcid.provider.secret')
+        }
+      end
     end
   end
 end

data/lib/orcid/engine.rb

@@ -4,16 +4,12 @@ module Orcid
 
     initializer 'orcid.initializers' do |app|
       app.config.paths.add 'app/services', eager_load: true
+      app.config.paths.add 'app/runners', eager_load: true
       app.config.autoload_paths += %W(
         #{config.root}/app/services
+        #{config.root}/app/runners
       )
     end
 
-    config.before_initialize do |app|
-      Orcid.configure do |config|
-        config.provider_name = 'orcid'
-      end
-    end
-
   end
 end

data/lib/orcid/exceptions.rb

@@ -9,6 +9,9 @@ module Orcid
         text << "id:\n\t#{client.id.inspect}"
         text << "site:\n\t#{client.site.inspect}"
         text << "options:\n\t#{client.options.inspect}"
+        if defined?(Orcid.provider)
+          text << "scopes:\n\t#{Orcid.provider.authentication_scope}"
+        end
       end
       text << "\n-- Token --"
       if token = options[:token]

data/lib/orcid/named_callbacks.rb

@@ -0,0 +1,19 @@
+module Orcid
+  # Inspired by Jim Weirich's NamedCallbacks
+  # https://github.com/jimweirich/wyriki/blob/master/spec/runners/named_callbacks_spec.rb#L1-L28
+  class NamedCallbacks
+    def initialize
+      @callbacks = {}
+    end
+
+    def method_missing(callback_name, *args, &block)
+      @callbacks[callback_name] = block
+    end
+
+    def call(callback_name, *args)
+      name = callback_name.to_sym
+      cb = @callbacks[name]
+      cb ? cb.call(*args) : true
+    end
+  end
+end

data/lib/orcid/version.rb

@@ -1,3 +1,3 @@
 module Orcid
-  VERSION = "0.0.1"
+  VERSION = "0.0.2"
 end

data/lib/tasks/orcid_tasks.rake

@@ -1,4 +1,111 @@
-# desc "Explaining what the task does"
-# task :orcid do
-#   # Task goes here
-# end
+namespace :orcid do
+  namespace :batch do
+    desc 'Given the input: CSV query for existing Orcids and report to output: CSV'
+    task :query => [:environment, :init, :query_runner, :run]
+
+    task :query_runner do
+      @runner = lambda { |person|
+        puts "Processing: #{person.email}"
+        Orcid::ProfileLookupRunner.new {|on|
+          on.found {|results|
+            person.found(results)
+            puts "\tfound" if verbose
+          }
+          on.not_found {
+            person.not_found
+            puts "\tnot found" if verbose
+          }
+        }.call(email: person.email)
+      }
+    end
+
+    desc "Given the input: CSV query for existing Orcids and if not found create new ones recording output: CSV"
+    task :create => [:environment, :init, :create_runner, :run]
+
+    task :create_runner do
+      @creation_service = lambda {|person|
+        puts "Creating Profile for: #{person.email}"
+        profile_creation_service = Orcid::Remote::ProfileCreationService.new do |on|
+          on.success {|orcid_profile_id|
+            person.orcid_profile_id = orcid_profile_id
+            puts "\tcreated #{orcid_profile_id}" if verbose
+          }
+        end
+        profile_request = Orcid::ProfileRequest.new(person.attributes)
+        profile_request.run(
+          validator: lambda{|*| true},
+          profile_creation_service: profile_creation_service
+        )
+      }
+      @runner = lambda { |person|
+        puts "Processing: #{person.email}"
+        Orcid::ProfileLookupRunner.new {|on|
+          on.found {|results|
+            person.found(results)
+            puts "\tfound" if verbose
+          }
+          on.not_found {
+            person.not_found
+            puts "\tnot found" if verbose
+            @creation_service.call(person)
+          }
+        }.call(email: person.email)
+      }
+    end
+
+
+    task :run do
+      if defined?(WebMock)
+        WebMock.allow_net_connect!
+      end
+      input_file = ENV.fetch('input') { './tmp/orcid_input.csv' }
+      output_file = ENV.fetch('output') { './tmp/orcid_output.csv' }
+
+      require 'csv'
+      CSV.open(output_file, 'wb+') do |output|
+        output << @person_builder.to_header_row
+        CSV.foreach(input_file, headers: true, header_converters: [lambda{|col| col.strip }]) do |input|
+          person = @person_builder.new(input.to_hash)
+          @runner.call(person)
+          output << person.to_output_row
+        end
+      end
+    end
+
+    task :init do
+      module Orcid::Batch
+        class PersonRecord
+          def self.to_header_row
+            ['email', 'given_names', 'family_name', 'existing_orcids', 'created_orcid', 'queried_at']
+          end
+          attr_reader :email, :given_names, :family_name, :existing_orcids
+          attr_accessor :created_orcid
+          def initialize(row)
+            @email = row.fetch('email')
+            @given_names = row['given_names']
+            @family_name = row['family_name']
+            @existing_orcids = nil
+          end
+
+          def attributes
+            { email: email, given_names: given_names, family_name: family_name }
+          end
+
+          def found(existing_orcids)
+            @existing_orcids = Array(existing_orcids).collect(&:orcid_profile_id).join("; ")
+          end
+
+          def to_output_row
+            [email, given_names, family_name, existing_orcids, created_orcid, Time.now]
+          end
+
+          def not_found
+            @existing_orcids = 'null'
+          end
+
+        end
+      end
+      @person_builder = Orcid::Batch::PersonRecord
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: orcid
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
 platform: ruby
 authors:
 - Jeremy Friesen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-02-21 00:00:00.000000000 Z
+date: 2014-02-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -262,6 +262,20 @@ dependencies:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rest_client
   requirement: !ruby/object:Gem::Requirement
@@ -276,6 +290,20 @@ dependencies:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: rspec-given
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 description: A Rails engine for orcid.org integration.
 email:
 - jeremy.n.friesen@gmail.com
@@ -289,14 +317,18 @@ files:
 - app/controllers/orcid/profile_connections_controller.rb
 - app/controllers/orcid/profile_requests_controller.rb
 - app/helpers/orcid/application_helper.rb
-- app/models/orcid/configuration.rb
 - app/models/orcid/profile.rb
 - app/models/orcid/profile_connection.rb
 - app/models/orcid/profile_request.rb
 - app/models/orcid/work.rb
-- app/services/orcid/profile_creation_service.rb
-- app/services/orcid/profile_lookup_service.rb
-- app/services/orcid/remote_work_service.rb
+- app/runners/orcid/profile_lookup_runner.rb
+- app/runners/orcid/runner.rb
+- app/services/orcid/remote/profile_creation_service.rb
+- app/services/orcid/remote/profile_lookup_service/search_response.rb
+- app/services/orcid/remote/profile_lookup_service.rb
+- app/services/orcid/remote/service.rb
+- app/services/orcid/remote/work_service.rb
+- app/services/orcid/remote.rb
 - app/templates/orcid/work.template.v1.1.xml.erb
 - app/views/layouts/orcid/application.html.erb
 - app/views/orcid/profile_connections/new.html.erb
@@ -305,10 +337,13 @@ files:
 - config/locales/orcid.en.yml
 - config/routes.rb
 - db/migrate/20140205185338_create_orcid_profile_requests.rb
+- lib/generators/orcid/install/install_generator.rb
+- lib/generators/orcid/install/templates/orcid_initializer.rb.erb
 - lib/orcid/configuration/provider.rb
 - lib/orcid/configuration.rb
 - lib/orcid/engine.rb
 - lib/orcid/exceptions.rb
+- lib/orcid/named_callbacks.rb
 - lib/orcid/spec_support.rb
 - lib/orcid/version.rb
 - lib/orcid.rb

data/app/models/orcid/configuration.rb

@@ -1,19 +0,0 @@
-module Orcid
-  class Configuration
-    attr_reader :store
-    def initialize(store = ::ENV)
-      @store = store
-    end
-
-    attr_writer :provider_name
-    def provider_name
-      @provider_name ||= 'orcid'
-    end
-
-    attr_writer :authentication_model
-    def authentication_model
-      @authentication_model ||= Devise::MultiAuth::Authentication
-    end
-
-  end
-end