fix here

[spider.git] / perl / Route / Node.pm

diff --git a/perl/Route/Node.pm b/perl/Route/Node.pm
index 5858ddcc0053184f77b2eb7481441a3906c3c660..bd69d3d77290947af5089a73484ebc339cddbfe1 100644 (file)
--- a/perl/Route/Node.pm
+++ b/perl/Route/Node.pm
@@ -27,8 +27,9 @@ use vars qw(%list %valid @ISA $max $filterdef);
                  users => '0,Users,parray',
                  usercount => '0,User Count',
                  version => '0,Version',
+                 build => '0,Build',
+                 sw => '0,Software',
                  np => '0,Using New Prot,yesno',
-                 lid => '0,Last Msgid',
 );
 
 $filterdef = $Route::filterdef;
@@ -57,7 +58,7 @@ sub max
 # object with that callsign. The upper layers are expected to do something
 # sensible with this!
 #
-# called as $parent->add(call, dxchan, version, flags) 
+# called as $parent->add(call, version, flags) 
 #
 
 sub add
@@ -65,13 +66,21 @@ sub add
        my $parent = shift;
        my $call = uc shift;
        confess "Route::add trying to add $call to myself" if $call eq $parent->{call};
+       my $version = shift;
+       my $here = shift;
+       
        my $self = get($call);
        if ($self) {
                $self->_addparent($parent);
                $parent->_addnode($self);
+               if ($self->{version} != $version || $self->{flags} != $here) {
+                       $self->{version} = $version;
+                       $self->{flags} = $here;
+                       return $self;
+               }
                return undef;
        }
-       $self = $parent->new($call, @_);
+       $self = $parent->new($call, $version, $here);
        $parent->_addnode($self);
        return $self;
 }
@@ -140,10 +149,10 @@ sub add_user
        my $uref = Route::User::get($ucall);
        my @out;
        if ($uref) {
-               @out = $uref->addparent($self);
+               push @out, $uref->addparent($self);
        } else {
                $uref = Route::User->new($ucall, $self->{call}, @_);
-               @out = $uref;
+               push @out, $uref;
        }
        $self->_adduser($uref);
        $self->{usercount} = scalar @{$self->{users}};
@@ -208,6 +217,39 @@ sub rnodes
        return @out;
 }
 
+# return the differences in nodes between what we currently have and
+# the list proffered. Returns two refs one to a list of nodes to remove and  
+# the other a list of nodes to add
+# 
+# input is a list of callsigns (not refs)
+sub diff_nodes
+{
+       my $self = shift;
+       my $in = ref $_[0] ? shift : \@_;
+       my %del = map {($_, 1)} nodes($self);
+       my %in = map {($_, 1)} @$in;
+       
+       # remove all the calls that are in both lists
+       for (@$in) {
+               delete $in{$_} if delete $del{$_};
+       }
+       return ([keys %del], [keys %in]);
+}
+
+# same as above but for users
+sub diff_users
+{
+       my $self = shift;
+       my $in = ref $_[0] ? shift : \@_;
+       my %del = map {($_, 1)} users($self);
+       my %in = map {($_, 1)} @$in;
+       
+       # remove all the calls that are in both lists
+       for (@$in) {
+               delete $in{$_} if delete $del{$_};
+       }
+       return ([keys %del], [keys %in]);
+}
 
 sub new
 {
@@ -218,8 +260,8 @@ sub new
        
        my $self = $pkg->SUPER::new($call);
        $self->{parent} = ref $pkg ? [ $pkg->{call} ] : [ ];
-       $self->{version} = shift;
-       $self->{flags} = shift;
+       $self->{version} = 0 || shift;
+       $self->{flags} = 0 || shift;
        $self->{users} = [];
        $self->{nodes} = [];
        $self->{lid} = 0;