On Wed, Sep 17, 2014 at 02:27:57PM -0400, Mitch Kyser wrote:
> Thanks for the response. I took a look at /opt/rt4/local/plugins/
> RT-Extension-Assets-Import-CSV/lib/RT/Extension/Assets/Import/[1]CSV.PM and
> removed the check you suggested by commenting out lines 64 thru 67;
>
> #if ( not RT::Asset->Role($fieldname)->{Single}) {
> # RT->Logger->warning( "Role name $fieldname must be single-value for
> "._column($field2csv->{$fieldname}).", skipping");
> # delete $field2csv->{$fieldname};
> #}
>
> Instead of the single warning I got one for each asset I tried to update that
> had a username to be placed in the HeldBy field;
>
> [error]: Failed to set HeldBy to mkyser for row 5408: No valid Type specified
> (/opt/rt4/local/plugins/RT-Extension-Assets-Import-CSV/lib/RT/Extension/Assets/
> Import/CSV.pm:178)
>
> When I tried using 'Owner' instead of 'HeldBy' I got the same results.Try applying the change from the attached patch > I will see what it takes to submit a bug report like you suggested, then > submit > one. I moved your bug report into the assets queue (rather than the general RT queue). > I am seeing similar things when I try to use the Bulk Update feature with > HeldBy, no error message but no change in HeldBy either. This should be a separate bug report. -kevin
diff --git a/lib/RT/Extension/Assets/Import/CSV.pm
b/lib/RT/Extension/Assets/Import/CSV.pm
index 250b7e4..5f71a92 100644
--- a/lib/RT/Extension/Assets/Import/CSV.pm
+++ b/lib/RT/Extension/Assets/Import/CSV.pm
@@ -174,7 +174,7 @@ sub run {
next if $asset->RoleGroup($field)->HasMember(
$user->PrincipalId );
$changes++;
- my ($ok, $msg) = $asset->AddRoleMember( PrincipalId =>
$user->PrincipalId );
+ my ($ok, $msg) = $asset->AddRoleMember( PrincipalId =>
$user->PrincipalId, Type => $field );
unless ($ok) {
RT->Logger->error("Failed to set $field to $value for
row $i: $msg");
}
pgpHUmUACFfjG.pgp
Description: PGP signature
-- RT Training - Boston, September 9-10 http://bestpractical.com/training
