revise log router api to match log::contextual router api change

[scpubgit/Object-Remote.git] / lib / Object / Remote / Role / LogForwarder.pm

diff --git a/lib/Object/Remote/Role/LogForwarder.pm b/lib/Object/Remote/Role/LogForwarder.pm
index 495a9c2..5be6851 100644 (file)
--- a/lib/Object/Remote/Role/LogForwarder.pm
+++ b/lib/Object/Remote/Role/LogForwarder.pm
@@ -1,36 +1,64 @@
+#This is an experimental method for working with
+#Log::Contextual crossing Object::Remote connections
+#transparently 
+
 package Object::Remote::Role::LogForwarder;
 
 use Moo::Role; 
+use Scalar::Util qw(weaken);
+use Carp qw(cluck);
 
 with 'Log::Contextual::Role::Router';
 
 #TODO re-weaken router references when object::remote
 #weak reference operation is figured out
 
-has child_routers => ( is => 'ro', required => 1, default => sub { {} } );
+has child_routers => ( is => 'ro', required => 1, default => sub { [] } );
 has parent_router => ( is => 'rw', );#weak_ref => 1 );
 
-#adds a child router to this router and gives it
-#a friendly display name
-sub add_child_router {
-   my ($self, $description, $router) = @_;
-   $self->child_routers->{$description} = $router;
-   #weaken($self->child_routers->{$class});
-   $router->parent_router($self);
-   return; 
+sub BUILD { }
+
+after BUILD => sub {
+  my ($self) = @_; 
+#  my $parent = $self->parent_router; 
+#  return unless defined $parent ; 
+#  $parent->add_child_router($self);
+};
+
+sub describe {
+  my ($self, $depth) = @_; 
+  $depth = -1 unless defined $depth; 
+  $depth++;
+  my $buf = "\t" x $depth . $self->description . "\n";
+  foreach my $child (@{$self->child_routers}) {
+    next unless defined $child; 
+    $buf .= $child->describe($depth);
+  }
+    
+  return $buf; 
 }
 
-sub remove_child_router {
-   my ($self, $description) = @_;
-   return delete $self->child_routers->{$description};
+sub add_child_router {
+  my ($self, $router) = @_;
+  push(@{ $self->child_routers }, $router);
+  #TODO re-weaken when object::remote proxied
+  #weak references is figured out
+#   weaken(${ $self->child_routers }[-1]);
+  return; 
 }
 
-after handle_log_message => sub {
-   my ($self, @args) = @_;
-   my $parent = $self->parent_router;
+#sub remove_child_router {
+#  my ($self, $description) = @_;
+#  return delete $self->child_routers->{$description};
+#}
+
+after get_loggers => sub {
+  my ($self, @args) = @_;
+  my $parent = $self->parent_router;
       
-   return unless defined $parent;
-   $parent->handle_log_message(@args);
+  return unless defined $parent;
+  $parent->handle_log_message(@args);
 };
 
 1;
+