Skip to content

Commit

Permalink
Config::JSON stopped providing a BUILDARGS method, so we can't "aroun…
Browse files Browse the repository at this point in the history
…d" it. subclass new() the old fashioned way and bump the Config::JSON version dep so we know we're getting the non-Moose one.
  • Loading branch information
scottwalters committed Jul 10, 2012
1 parent f4aacb9 commit 55aead8
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 6 deletions.
9 changes: 4 additions & 5 deletions lib/WebGUI/Config.pm
Expand Up @@ -100,13 +100,12 @@ The filename of the config file to read.
=cut

around BUILDARGS => sub {
my $orig = shift;
my $class = shift;
sub new {
my $package = shift;
my $filename = shift;
$filename = Cwd::realpath(File::Spec->rel2abs($filename, WebGUI::Paths->configBase));
return $class->$orig($filename);
};
return $package->SUPER::new($filename);
}

1;

2 changes: 1 addition & 1 deletion sbin/testEnvironment.pl
Expand Up @@ -111,7 +111,7 @@ BEGIN
checkModule("JSON", 2.12 );
checkModule("JSON::Any", 1.22 );
checkModule("JSON::PP", 0.00 );
checkModule("Config::JSON", "1.3.1" );
checkModule("Config::JSON", 1.5100 );
checkModule("Text::CSV_XS", "0.64" );
checkModule("Net::CIDR::Lite", 0.20 );
checkModule("Finance::Quote", 1.15 );
Expand Down

0 comments on commit 55aead8

Please sign in to comment.