oqapi/webgui
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Add back code for throwing exceptions when session is not passed

Browse source 
to class methods.
This commit is contained in:
Colin Kuskie 2009-04-24 03:47:40 +00:00
parent 30547ad310
commit a75cd108b1
2 changed files with 6 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

6
lib/WebGUI/Shop/Tax.pm
View file

@ -128,6 +128,9 @@ A WebGUI::Session object. Required in class context, optional in instance contex
sub getDriver {
my $self = shift;
my $session = shift || $self->session;
unless (defined $session && $session->isa("WebGUI::Session")) {
WebGUI::Error::InvalidObject->throw(expected=>"WebGUI::Session", got=>(ref $session), error=>"Need a session.");
}
my $className = $session->setting->get( 'activeTaxPlugin' );
my $driver = eval {
@ -152,6 +155,9 @@ Constructor for the WebGUI::Shop::Tax. Returns a WebGUI::Shop::Tax object.
sub new {
my $class = shift;
my $session = shift;
unless (defined $session && $session->isa("WebGUI::Session")) {
WebGUI::Error::InvalidObject->throw(expected=>"WebGUI::Session", got=>(ref $session), error=>"Need a session.");
}
my $self = {};
bless $self, $class;
register $self;

2
t/Shop/Tax.t
View file

@ -98,8 +98,6 @@ SKIP: {
isa_ok( $e, 'WebGUI::Error::InvalidParam', 'getDriver throws error when no session object is passed in class context' );
is( $e->error, 'Need a session.', 'getDriver passes correct message for ommitted session object' );
TODO: {
local $TODO = 'test www_ methods';
#######################################################################