Re: About BLOCK variables

First off: Assigning '' to $uid here is both mildly confusing of the issue, and redundant. The reason is that my $uid; will implicitly set $uid to undef, which is false.

However, that's not the problem, as far as I can see. There's two possiblities. The first is that $entry->get($_) might not return the same thing all three times you call it. Presumably, you'd know if this were a possiblity, so I won't talk much about it. The second possiblity is that it returns somthing that isn't an array reference, but is true.

I'd do this somthing like this:

for my $entry ($searchresult->entries) {
   my $uid = $entry->get('uid');
   if (!$uid || !UNIVERSAL::isa($uid, 'ARRAY')) {
     print "*** No uid for the following  entry:\n";
     print Dumper(\$entry);
   }
   $uid = ${$uid}[0];
   if (!$uid) {
     print "*** No uid for the following  entry:\n";
     print Dumper(\$entry);
   }
   for (@vars) {
         $uids{$uid}{$_} = '';
         $uids{$uid}{$_} = ${$uid}[0];
   }
}
[download]

Note that UNIVERSAL::isa($foo, 'ARRAY'); will tell you if $foo is a reference to an array (or if it is a reference to somthing else blessed into a class named ARRAY, which I find terribly unlikely).

Warning: Unless otherwise stated, code is untested. Do not use without understanding. Code is posted in the hopes it is useful, but without warranty. All copyrights are relinquished into the public domain unless otherwise stated. I am not an angel. I am capable of error, and err on a fairly regular basis. If I made a mistake, please let me know (such as by replying to this node).

Comment on Re: About BLOCK variables Select or Download Code

Replies are listed 'Best First'.
Re: Re: About BLOCK variables by Anonymous Monk on May 24, 2003 at 06:48 UTC
Actually I don't think 'isa' is an issue. The structures are generated by the CPAN LDAP modules (as are the methods for accessing it) and the snippet is pretty close to the Pod examples. But since I didn't include the actual setup calls to the LDAP query, it probably made it difficult to tell what's going on. Sorry, it didn't seem relevant to me at the time the post was formulated. I assume you made a cut/paste error for the second for loop. The first line is redundant since the second assignment doesn't include the required conditional test for definedness. It also omits the get method so you're actually assigning the same value(s) to each of the different @var values of each $uid instance. E.g.: `$uids{$uid}{mail} = $uids{$uid}{cn} = ${$uid}[0]` [download] which shouldn't be the case.	[reply] [d/l]

Replies are listed 'Best First'.

Re: Re: About BLOCK variables
by Anonymous Monk on May 24, 2003 at 06:48 UTC

I assume you made a cut/paste error for the second for loop. The first line is redundant since the second assignment doesn't include the required conditional test for definedness. It also omits the get method so you're actually assigning the same value(s) to each of the different @var values of each $uid instance. E.g.:

$uids{$uid}{mail} = $uids{$uid}{cn} = ${$uid}[0]
[download]

[reply]
[d/l]