Index: [Article Count Order] [Thread]

Date:  Tue, 18 Apr 2006 10:08:19 +0200
From:  Roman Buerkle <buerkle (at mark) stimme.net>
Subject:  [coba-e:04738] Re: bugfix CMU
To:  coba-e (at mark) bluequartz.org
Message-Id:  <1145347699.3505.22.camel@silverbird>
In-Reply-To:  <20060416213906.M16642 (at mark) scargo.nl>
References:  <20060416213906.M16642 (at mark) scargo.nl>
X-Mail-Count: 04738

Hi Taco, 

thank you for that, you made us a nice eastern gift :)

Some more things we "discovered" during our importings:


- while importing larger sites, users with forwarders to other users of
the same domain, are not correctly imported, because the other user was
not already imported. These forwarders had to be corrected manually
afterwards. Woulndt it be good to force the cmuimport to import the
forwarders "as is", without checking the target?

It would be a lot of work to re-sort the cmu.xml just to get the right
order.


- the aliases "admin" was allowed on RaQ4 but not on BQ 


- when importing aliases fails for some reason, none of them are
imported for the user


- the alias "mail" will not be imported, but could be set manually. Why
is not imported then?


- is there a list which shows all the "not allowed" aliases, so we can
check/change them before tryiing to import them?



Thanx  und Greets

Roman






On Sun, 2006-04-16 at 22:39 +0100, Taco Scargo wrote:
> Hisao (and other interested parties),
> 
> After reading some user comments about cmu not importing siteadmins,
> I spent quite some time figuring out why CMU did not do what it should do 
> (when importing a RaQ4 cmu export).
> After adding some additional debugging info cmu suddenly started working.
> After I removed the additional 'warn' statements, it stopped working again.
> 
> I finally found out (by removing lines one by one) that the problem was 
> introduced due to the fact that the 5100Radjust.pl script converts a raq4 
> cmu.xml file to a raq550 compatible cmu.xml file.
> 
> To be precise:
> 
> <admin value = "t"/>
> 
> needs to be converted to:
> 
> <capLevels>
>   <cap>siteAdmin</cap>
> </capLevels>
> 
> The current code forces the value $oldVal to become a string, whereas the 
> data could become an array (<cap> inside of <capLevels>).
> 
> By commenting out the my $oldVal everything works as expected.
> 
> See the diff -u below:
> 
> --- 5100Radjust.pl.orig 2006-04-16 22:14:16.000000000 +0200
> +++ 5100Radjust.pl      2006-04-16 23:15:39.000000000 +0200
> @@ -238,7 +238,7 @@
>         }
>         if(defined $remapData{$class}) {
>                 my $remapHash = $remapData{$class};
> -               my $oldVal;
> +#              my $oldVal;
>                 foreach my $name (keys %{ $remapHash }) {
>                         next if(!defined $hash->{$name});
>                         $oldVal = $hash->{$name};
> 
> 
> Hisao, can you please implement this in the cmu code and push out an updated 
> CMU rpm ?
> 
> If other people have encountered other problems with cmu, please let me know 
> so I can have a look at these problems as well. Due to the debugging I did 
> this weekend I think I am quite 'into cmu' now....
> 
> Thanks,
> 
> Taco
>