--- nfo/perl/libs/Data/Storage.pm	2003/01/19 03:12:59	1.15
+++ nfo/perl/libs/Data/Storage.pm	2003/01/20 16:52:13	1.16
@@ -1,6 +1,6 @@
 ## ------------------------------------------------------------------------
 ##
-##    $Id: Storage.pm,v 1.15 2003/01/19 03:12:59 joko Exp $
+##    $Id: Storage.pm,v 1.16 2003/01/20 16:52:13 joko Exp $
 ##
 ##    Copyright (c) 2002  Andreas Motl <andreas.motl@ilo.de>
 ##
@@ -9,6 +9,9 @@
 ## ------------------------------------------------------------------------
 ##
 ##  $Log: Storage.pm,v $
+##  Revision 1.16  2003/01/20 16:52:13  joko
+##  + now using 'DesignPattern::Object' to create a new 'Data::Storage::Handler::Xyz' on demand - before we did this in a hand-rolled fashion
+##
 ##  Revision 1.15  2003/01/19 03:12:59  joko
 ##  + modified header
 ##  - removed pod-documentation - now in 'Storage.pod'
@@ -81,12 +84,14 @@
 use strict;
 use warnings;
 
-use Data::Storage::Locator;
 use Data::Dumper;
-
-# TODO: wipe out!
+# FIXME: wipe out!
 use DBI;
 
+use Data::Storage::Locator;
+use DesignPattern::Object;
+
+
 # TODO: actually implement level (integrate with Log::Dispatch)
 my $TRACELEVEL = 0;
 
@@ -133,7 +138,6 @@
 
   # advanced logging of AUTOLOAD calls ...
   # ... nice but do it only when TRACING (TODO) is enabled
-    if ($TRACELEVEL) {
       my $logstring = "";
       $logstring .= __PACKAGE__ . "[$self->{locator}->{type}]" . "->" . $method;
       #print "count: ", $#_, "\n";
@@ -143,15 +147,20 @@
       # TODO: only ok if logstring doesn't contain 
       #            e.g. "Data::Storage[Tangram]->insert(SystemEvent=HASH(0x5c0034c))          (AUTOLOAD)"
       # but that would be _way_ too specific as long as we don't have an abstract handler for this  ;)
+    if ($TRACELEVEL) {
       $logger->debug( $logstring );
       #print join('; ', @_);
     }
     
   # filtering AUTOLOAD calls and first-time-touch of the actual storage impl
   if ($self->_filter_AUTOLOAD($method)) {
-    #print "_accessStorage\n";
     $self->_accessStorage();
-    return $self->{STORAGEHANDLE}->$method(@_);
+    if ($self->{STORAGEHANDLE}) {
+      return $self->{STORAGEHANDLE}->$method(@_);
+    } else {
+      $logger->critical( __PACKAGE__ . "->AUTOLOAD: ERROR: " . $logstring );
+      return;
+    }
   }
   
 }
@@ -184,8 +193,34 @@
     $logger->debug( __PACKAGE__ .  "[$self->{locator}->{type}]" . "->_accessStorage()" );
   }
   if (!$self->{STORAGEHANDLE}) {
-    $self->_createStorageHandle();
+    return $self->_createStorageHandle();
+  }
+}
+
+sub _createStorageHandle1 {
+  my $self = shift;
+  my $type = $self->{locator}->{type};
+  $logger->debug( __PACKAGE__ .  "[$type]" . "->_createStorageHandle()" );
+
+  my $pkg = "Data::Storage::Handler::" . $type . "";
+  
+  # try to load perl module at runtime
+  my $evalstr = "use $pkg;";
+  eval($evalstr);
+  if ($@) {
+    $logger->error( __PACKAGE__ .  "[$type]" . "->_createStorageHandle(): $@" );
+    return;
   }
+  
+  # build up some additional arguments to pass on
+  #my @args = %{$self->{locator}};
+  my @args = ();
+
+  # - create new storage handle object
+  # - propagate arguments to handler
+  # - pass locator by reference to be able to store status- or meta-information in it
+  $self->{STORAGEHANDLE} = $pkg->new( locator => $self->{locator}, @args );
+
 }
 
 sub _createStorageHandle {
@@ -193,9 +228,13 @@
   my $type = $self->{locator}->{type};
   $logger->debug( __PACKAGE__ .  "[$type]" . "->_createStorageHandle()" );
 
+#print Dumper($self);
+#exit;
+
   my $pkg = "Data::Storage::Handler::" . $type . "";
   
   # try to load perl module at runtime
+=pod
   my $evalstr = "use $pkg;";
   eval($evalstr);
   if ($@) {
@@ -211,6 +250,10 @@
   # - propagate arguments to handler
   # - pass locator by reference to be able to store status- or meta-information in it
   $self->{STORAGEHANDLE} = $pkg->new( locator => $self->{locator}, @args );
+=cut
+
+  $self->{STORAGEHANDLE} = DesignPattern::Object->fromPackage( $pkg, locator => $self->{locator} );
+  return 1;
 
 }
 
@@ -249,6 +292,7 @@
   $logger->remove($name);
 }
 
+=pod
 sub getDbName {
   my $self = shift;
   my $dsn = $self->{locator}->{dbi}->{dsn};
@@ -286,6 +330,7 @@
     $logger->warning( __PACKAGE__ .  "[$self->{locator}->{type}]" . "->testDsn(): " . "DBI-error: " . $DBI::errstr );
   }
 }
+=cut
 
 1;
 __END__