From 2088972db485c57a2d1bea673b5fba2ae2b25ae1 Mon Sep 17 00:00:00 2001
From: reiyua <rayyan.hodges@studytafensw.edu.au>
Date: Fri, 12 Apr 2024 22:13:08 +1000
Subject: [PATCH] IT ALIVE

---
 AutoOUADCreator.ps1 | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/AutoOUADCreator.ps1 b/AutoOUADCreator.ps1
index 8e80f7f..54349c1 100644
--- a/AutoOUADCreator.ps1
+++ b/AutoOUADCreator.ps1
@@ -2,7 +2,7 @@
 # Contact: rayyan.hodges@studytafensw.edu.au
 # Program Name: AutoOUADCreator
 # Purpose of script: Create a batch set of OU' using a CSV file within an existing Active Directory Forest.
-# Extra Notes: Modify line 37 with appropriate domain info. (AlphaDelta.com is used in this scenario)
+# Extra Notes: Modify line 37 and 59 with appropriate domain info. (AlphaDelta.com is used in this scenario)
 
 #Import the Active Directory module to allow modifcations to the forest.
 import-module ActiveDirectory
@@ -29,13 +29,16 @@ echo $fpath
 #Create OU's within the Active Directory forest by looping throughout each row within the CSV file.
 foreach ($row in $fous) {
         # Get the name of the OU from the CSV
-        $ouName = $row.Name
+        $ouName = $row.ouName
+
+        # Debug output - Check if program is actually reading CSV containing OU names.
+        Write-Host "Processing OU name: '$ouName'"
 
         # Ensure $ouName is not null or empty
         if (-not [string]::IsNullOrWhiteSpace($ouName)) {
 # Get all existing OUs in the specified path
             try {
-                $existingOUs = Get-ADOrganizationalUnit -Filter * -SearchBase "DC=PSTest,DC=local" | Select-Object -ExpandProperty Name
+                $existingOUs = Get-ADOrganizationalUnit -Filter * -SearchBase "DC=alphadelta,DC=com" | Select-Object -ExpandProperty Name
             } catch {
                 Write-Host "Error retrieving existing OUs: $_"
                 exit
@@ -50,10 +53,10 @@ foreach ($existingOU in $existingOUs) {
     }
 }
 
-i if (-not $ouExists) {
+if (-not $ouExists) {
                 # Create the OU
                 try {
-                    New-ADOrganizationalUnit -Name $ouName -Path "DC=PSTest,DC=local" -ErrorAction Stop
+                    New-ADOrganizationalUnit -Name $ouName -Path "DC=alphadelta,DC=com" -ErrorAction Stop
                     Write-Host "OU '$ouName' created successfully."
                 } catch {
                     Write-Host "Error creating OU '$ouName': $_"