Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

DataObjects::URI.new arguments to utilise @options[:scheme] #9

Open
wants to merge 1 commit into from

0 participants

@ghost

Hi. I'm not sure if this really matters but I found DataObjectsAdapter#normalized_uri checks @options[:scheme] but doesn't use it at all. I've added it to the :scheme option in DataObjects::URI.new.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Sep 15, 2011
  1. might have to utilise @options[:scheme] as well

    Hirobumi Hama authored
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  lib/dm-do-adapter/adapter.rb
View
2  lib/dm-do-adapter/adapter.rb
@@ -229,7 +229,7 @@ def normalized_uri
port = @options[:port].nil? ? nil : @options[:port].to_int
DataObjects::URI.new(
- :scheme => @options[:adapter],
+ :scheme => @options[:scheme] || @options[:adapter],
:user => @options[:user] || @options[:username],
:password => @options[:password],
:host => @options[:host],
Something went wrong with that request. Please try again.