Skip Menu |

This queue is for tickets about the Net-Google-Drive-Simple CPAN distribution.

Report information
The Basics
Id: 101655
Status: resolved
Priority: 0/
Queue: Net-Google-Drive-Simple
People
Owner: Nobody in particular
Requestors: emmanuel.rodriguez [...] gmail.com
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Wed Jan 21 04:06:52 2015 emmanuel.rodriguez [...] gmail.com - Ticket created
Subject: Constructor ignores config_file
Date: Wed, 21 Jan 2015 10:06:40 +0100
To: bug-Net-Google-Drive-Simple [...] rt.cpan.org
From: Emmanuel Rodriguez <emmanuel.rodriguez [...] gmail.com>
While the documentation shows that we can use config_file, the constructor fails to do anything with that parameter and the oath class completely ignores the setting and always defaults to lookup the config file in the home folder. -- Emmanuel Rodriguez
  Mon Jan 26 23:46:09 2015 MSCHILLI [...] cpan.org - Correspondence added
On Wed Jan 21 04:06:52 2015, emmanuel.rodriguez@gmail.com wrote: Show quoted text
> While the documentation shows that we can use config_file, the > constructor fails to do anything with that parameter and the oath > class completely ignores the setting and always defaults to lookup the > config file in the home folder.
I've fixed the documentation, it no longer talks about the unsupported parameter config_file. Since the file gets created by a different module (OAuth::Cmdline::GoogleDrive), which also defaults to ~/.google-drive.yml, it probably confuse a lot of users to offer the option to change the default, because to do so, you'd have to change it in both places. Hope that works for you!
  Mon Jan 26 23:46:09 2015 The RT System itself - Status changed from 'new' to 'open'
  Mon Jan 26 23:46:15 2015 MSCHILLI [...] cpan.org - Status changed from 'open' to 'resolved'
  Mon Jan 26 23:47:14 2015 MSCHILLI [...] cpan.org - Correspondence added
Fix: https://github.com/mschilli/net-google-drive-simple/commit/1844420bb5d674e369ad2d60c73f1d363dae49ae 0.12 will be available on CPAN soon.