X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?p=scpubgit%2FObject-Remote.git;a=blobdiff_plain;f=lib%2FObject%2FRemote%2FRole%2FLogForwarder.pm;h=1ff8e5f569b5015006031cd475d1959da5e4e1af;hp=d57a7b8c11b1ea1f57ad7cfa57e04b117117dd9d;hb=d2eadebbdb4416f421ffd5298c782fdc32f388e7;hpb=09130cd025586aa10cb57c33d3ae899a7cc7e185

diff --git a/lib/Object/Remote/Role/LogForwarder.pm b/lib/Object/Remote/Role/LogForwarder.pm
index d57a7b8..1ff8e5f 100644
--- a/lib/Object/Remote/Role/LogForwarder.pm
+++ b/lib/Object/Remote/Role/LogForwarder.pm
@@ -1,64 +1,35 @@
-#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';
+use Moo::Role;
 
-#TODO re-weaken router references when object::remote
-#weak reference operation is figured out
+has enable_forward => ( is => 'rw', default => sub { 1 } );
+has _forward_destination => ( is => 'rw' );
+has _forward_stop => ( is => 'ro', required => 1, default => sub { {} } );
 
-has child_routers => ( is => 'ro', required => 1, default => sub { [] } );
-has parent_router => ( is => 'rw', );#weak_ref => 1 );
+after _deliver_message => sub {
+  my ($self, $level, $generator, $args, $metadata) = @_;
+  my $package = $metadata->{package};
+  my $destination = $self->_forward_destination;
+  our $reentrant;
 
-sub BUILD { }
+  return unless $self->enable_forward;
+  return unless defined $destination;
+  return if $self->_forward_stop->{$package};
 
-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);
+  if (defined $reentrant) {
+    warn "log forwarding went reentrant. bottom: '$reentrant' top: '$package'";
+    return;
   }
-    
-  return $buf; 
-}
+  
+  local $reentrant = $package;
+  
+  $destination->_deliver_message($level, $generator, $args, $metadata);
+};
 
-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; 
+sub exclude_forwarding {
+  my ($self, $package) = @_;
+  $package = caller unless defined $package;
+  $self->_forward_stop->{$package} = 1;
 }
 
-#sub remove_child_router {
-#  my ($self, $description) = @_;
-#  return delete $self->child_routers->{$description};
-#}
-
-after handle_log_message => sub {
-  my ($self, @args) = @_;
-  my $parent = $self->parent_router;
-      
-  return unless defined $parent;
-  $parent->handle_log_message(@args);
-};
-
 1;
-