Filter roles #2

puck · 2022-07-23T12:00:15Z

This pull request effectively adds support for loading back in the output of Bulk Download of assets. You still need to create a mapping, but this might be enough to allow an attempt at automatic mapping.

Adds support for:

Add multiple records to Owner (if AssetMultipleOwner enabled), HeldBy, and Contact
Loading of principal details included in the CSV (remove fullnames, handle Nobody)
Remove principals from a role if not present in the CSV
The use of Catalogue as a heading
Add group(s) to a Role
Skips any usernames not found - previously they default to Nobody which was mighty annoying.

This allows loading a CSV generated from the Download Spreadsheet link from the Assets search results pages in RT. It: - Handles multiple principals. - Strips off the fullname. - Handles - ignores - usernames which don't exist (previously Nobody account was used).

This adds: - handling groups - removing users/groups which aren't in the CSV file - handling Owner (with AssetMultipleOwner on and off)

Strictly speaking only the first line needed to be changed, but I figure for consistency it is clearer if we just use $principal. ($user and $principal are pointing to the same object for the second line.)

I noticed that RT inserts "group:$group_name" in the role fields when creating an asset in the web interface. However, Download Spreadsheet uses "Group: $group_name" (this is what I initially selected to support).

puck added 6 commits July 24, 2022 00:28

Move handling of Role fields to a function

f679248

Flesh out managing Roles.

92a85c6

This adds: - handling groups - removing users/groups which aren't in the CSV file - handling Owner (with AssetMultipleOwner on and off)

Download Spreadsheet uses 'Catalogue', support that.

ec3d2e0

Fix the selection of the Nobody user.

dbcd11e

Strictly speaking only the first line needed to be changed, but I figure for consistency it is clearer if we just use $principal. ($user and $principal are pointing to the same object for the second line.)

Support group for signalling a group as well, make space optional.

621fec2

I noticed that RT inserts "group:$group_name" in the role fields when creating an asset in the web interface. However, Download Spreadsheet uses "Group: $group_name" (this is what I initially selected to support).

puck force-pushed the filter-roles branch from 8abc1f4 to 621fec2 Compare July 24, 2022 00:30

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Filter roles #2

Filter roles #2

puck commented Jul 23, 2022

Filter roles #2

Are you sure you want to change the base?

Filter roles #2

Conversation

puck commented Jul 23, 2022