Automatic Folder Migration

[ldoddrell]

Afternoon all,

I have the following script that is designed to discover old AD accounts, filter the results to a certain user group (accounts starting 'P4'), export to CSV, forcefully give my security group ownership of the directories and then use RoboCopy to migrate the folders to a server share.

RoboCopy is where I'm facing a silly issue I can't find a straight forward answer to. Not all the users have HomeDirectories in AD and so when RoboCopy comes across a null value, it takes places it into my command but unwittingly thinks it's the sourcedir.

Here is the code:

# Import AD into Quest

import-module activedirectory

# Find the date 365 days ago as a variable

	$old_p45_profile_path = "\\server\folder\profiles"

	$old_p45_home_path = "\\server\folder\home"

	$StartDate = [DateTime]::Now.AddDays(-365)

# Search for users that have not logged in for 365 days or more AND their account was created more than 365 days ago

	Search-ADAccount -UsersOnly -AccountInactive -TimeSpan 365.00:00:00 | %{Get-QADUser $_.ObjectGuid -CreatedBefore $StartDate} | select samaccountname, name, whenCreated, lastLogonTimeStamp, profilePath, homeDirectory | where-object {$_.samaccountname -like "P4*"}| export-csv c:\output\inactiveusers.csv -NoTypeInformation

# Read in List of Folders

	$rows=Import-Csv "inactiveusers.csv"

		foreach ($row in $rows)

			$profilePath = $row.ProfilePath

				takeown /F $profilePath /R /A

		foreach ($row in $rows)

			$homeDirectory = $row.HomeDirectory

				takeown /F $homeDirectory /R /A

# Move Profiles

	foreach ($row in $rows) {

		if ($profilePath -EQ $NULL) {}

		else {robocopy $profilePath $old_p45_profile_path /MIR /MOVE /EFSRAW

                             }

# Move Home Directories

	foreach ($row in $rows) {

		if ($homeDirectory -EQ $NULL) {}

		else {robocopy $homeDirectory $old_p45_home_path /MIR /MOVE /EFSRAW }

				}

}

How do I get RoboCopy to ignore a null cell in the CSV and move to the next cell?

Cheers!

 

[58sniper]

Which cell?

$rows = Import-Csv "inactiveusers.csv" | ? {$_.HomeDirectory -ne $null}

might work


Do you have your Tek-Tips.com Swag? I've got mine!

Stop by the new Tek-Tips group at LinkedIn.

 

[58sniper]

I meant to add that that code you posted is pretty ugly. Lots of room for improvement there, including parameter validation, error checking, and using only a single ForEach loop.


Do you have your Tek-Tips.com Swag? I've got mine!

Stop by the new Tek-Tips group at LinkedIn.