--- nfo/perl/libs/Data/Storage/Handler/Tangram.pm 2002/12/16 20:49:17 1.20 +++ nfo/perl/libs/Data/Storage/Handler/Tangram.pm 2002/12/18 22:28:16 1.22 @@ -1,8 +1,14 @@ ############################################ # -# $Id: Tangram.pm,v 1.20 2002/12/16 20:49:17 jonen Exp $ +# $Id: Tangram.pm,v 1.22 2002/12/18 22:28:16 jonen Exp $ # # $Log: Tangram.pm,v $ +# Revision 1.22 2002/12/18 22:28:16 jonen +# + added extended logging at 'getObjectByGuid()' +# +# Revision 1.21 2002/12/16 22:20:49 jonen +# + fixed bug at 'getObjectByGuid()' +# # Revision 1.20 2002/12/16 20:49:17 jonen # + added sub 'getObjectByGuid()' # + added functionality to use 'getObjectByGuid' at 'getObjectAsHash()' @@ -560,6 +566,7 @@ # Guid and Classname is needed if(!$guid || !$options->{classname}) { + $logger->error( __PACKAGE__ . "->getObjectByGuid: No 'guid' OR no Classname in options hash was given but needed!" ); return; } @@ -568,15 +575,16 @@ $self->{_COREHANDLE}->unload() if ($options->{destroy}); # search for object with given Classname and Guid - my $obj_tmp = $self->{_COREHANDLE}->remote($classname); + my $obj_tmp = $self->{_COREHANDLE}->remote($options->{classname}); my @result = $self->{_COREHANDLE}->select($obj_tmp, $obj_tmp->{guid} eq $guid); # we searched for global unique identifer of some object, - # so it think we can trust there would be only one result + # so I think we can trust there would be only one result if($result[0]) { return $result[0]; } else { - return "No Object with Classname $classname and GUID $options->{guid} found!"; + $logger->error( __PACKAGE__ . "->getObjectByGuid: No Object with Classname $options->{classname} and GUID $guid found!" ); + return; } }