From f0a2319bee33b5719c7b2e30a395a125c20481c4 Mon Sep 17 00:00:00 2001
From: "KFS\\guym" <guym@kingdomfirstsolutions.com>
Date: Wed, 22 Jun 2022 10:05:01 -0400
Subject: [PATCH 1/3] Fix bug with buidling director groups due to non-unique
 ids being generated

---
 .../Utilities/Translators/CcbGroup.cs         | 22 +++++++++++++++----
 1 file changed, 18 insertions(+), 4 deletions(-)

diff --git a/Slingshot.CCB/Slingshot.CCB/Utilities/Translators/CcbGroup.cs b/Slingshot.CCB/Slingshot.CCB/Utilities/Translators/CcbGroup.cs
index f5c5d3c..660a91d 100644
--- a/Slingshot.CCB/Slingshot.CCB/Utilities/Translators/CcbGroup.cs
+++ b/Slingshot.CCB/Slingshot.CCB/Utilities/Translators/CcbGroup.cs
@@ -1,6 +1,7 @@
 using System;
 using System.Collections.Generic;
 using System.Linq;
+using System.Security.Cryptography;
 using System.Text;
 using System.Threading.Tasks;
 using System.Xml.Linq;
@@ -40,10 +41,17 @@ public static List<Group> Translate( XElement inputGroup )
 
             groups.Add( group );
 
-            // add the department as a group with an id of 9999 + its id to create a unique group id for it
             if ( inputGroup.Element( "department" ) != null && inputGroup.Element( "department" ).Attribute( "id" ) != null && inputGroup.Element( "department" ).Attribute( "id" ).Value.IsNotNullOrWhitespace() )
+            MD5 md5Hasher = MD5.Create();
+            
+            // add the department as a group with id set to hash of 9999 + department id to create a unique group id for it
             {
-                departmentId = ( "9999" + inputGroup.Element( "department" ).Attribute( "id" ).Value ).AsInteger();
+                var hashedDepartmentId = md5Hasher.ComputeHash( Encoding.UTF8.GetBytes( $@"
+                     {9999}
+                     {inputGroup.Element( "department" ).Attribute( "id" ).Value}
+                    " ) );
+                departmentId = Math.Abs( BitConverter.ToInt32( hashedDepartmentId, 0 ) ); // used abs to ensure positive number
+
                 var departmentName = inputGroup.Element( "department" ).Value;
                 if ( departmentName.IsNullOrWhiteSpace() )
                 {
@@ -52,10 +60,16 @@ public static List<Group> Translate( XElement inputGroup )
                 groups.Add( new Group { Id = departmentId.Value, Name = inputGroup.Element( "department" ).Value, IsActive = true, GroupTypeId = 9999 } );
             }
 
-            // add the director as a group with an id of 9998 + its id to create a unique group id for it
             if ( inputGroup.Element( "director" ) != null && inputGroup.Element( "director" ).Attribute( "id" ) != null && inputGroup.Element( "director" ).Attribute( "id" ).Value.IsNotNullOrWhitespace() )
             {
-                directorId = ( "9998" + inputGroup.Element( "director" ).Attribute( "id" ).Value ).AsInteger();
+                // add the director as a group with an id of 9998 + its id + its department id (if any) to create a unique group id for it
+                var departmentIdString = departmentId.HasValue ? departmentId.Value.ToString() : string.Empty;
+                var hashedDirectorId = md5Hasher.ComputeHash( Encoding.UTF8.GetBytes( $@"
+                     {9998}
+                     {inputGroup.Element( "director" ).Attribute( "id" ).Value}
+                     {departmentIdString}
+                    " ) );
+                directorId = Math.Abs( BitConverter.ToInt32( hashedDirectorId, 0 ) ); // used abs to ensure positive number
 
                 var directorGroup = new Group();
                 directorGroup.Id = directorId.Value;

From 3fd844a776cacf0cea9c575e235a7842db0aa70e Mon Sep 17 00:00:00 2001
From: "KFS\\guym" <guym@kingdomfirstsolutions.com>
Date: Wed, 22 Jun 2022 10:07:27 -0400
Subject: [PATCH 2/3] Add ability to make director group breakout to be
 optional

---
 Slingshot.CCB/Slingshot.CCB/MainWindow.xaml   |  1 +
 .../Slingshot.CCB/MainWindow.xaml.cs          |  3 +++
 .../Slingshot.CCB/Utilities/CcbApi.cs         | 25 ++++++++++++++++---
 .../Utilities/Translators/CcbGroup.cs         | 23 +++++++++++++----
 4 files changed, 43 insertions(+), 9 deletions(-)

diff --git a/Slingshot.CCB/Slingshot.CCB/MainWindow.xaml b/Slingshot.CCB/Slingshot.CCB/MainWindow.xaml
index 107b74e..b682c70 100644
--- a/Slingshot.CCB/Slingshot.CCB/MainWindow.xaml
+++ b/Slingshot.CCB/Slingshot.CCB/MainWindow.xaml
@@ -89,6 +89,7 @@
                         <Label Content="Advanced Options:" FontWeight="Bold" Margin="0,0,0,0"/>
                         <CheckBox Name="cbDumpResponseToXmlFile" Content="Dump response to XML files" Margin="5,10,0,10"  Padding="3,0,0,0"  />
                         <CheckBox Name="cbConsolidateSchedules" IsChecked="True" Content="Consolidate schedules to Day of Week and Time" Margin="5,0,0,10"  Padding="3,0,0,0"  />
+                        <CheckBox Name="cbDirectorsAsGroups" IsChecked="True" Content="Export Directors as Groups" Margin="5,0,0,10"  Padding="3,0,0,0"  />
                         <StackPanel Orientation="Horizontal" Margin="5,0,0,0">
                             <Label Name="lblItemsPerPage" Content="Items Per Page:" HorizontalAlignment="Left" Padding="0,5,5,5" Width="100" />
                             <TextBox x:Name="txtItemsPerPage"
diff --git a/Slingshot.CCB/Slingshot.CCB/MainWindow.xaml.cs b/Slingshot.CCB/Slingshot.CCB/MainWindow.xaml.cs
index 37d1471..10a025f 100644
--- a/Slingshot.CCB/Slingshot.CCB/MainWindow.xaml.cs
+++ b/Slingshot.CCB/Slingshot.CCB/MainWindow.xaml.cs
@@ -257,6 +257,9 @@ private void btnDownloadPackage_Click( object sender, RoutedEventArgs e )
             // Set ConsolidateScheduleNames to true to consolidate schedules names as 'Sunday at 11:00 AM'
             CcbApi.ConsolidateScheduleNames = cbConsolidateSchedules.IsChecked ?? false;
 
+            // Set ExportDepartmentDirectorsAsGroups to true to export Directors as groups
+            CcbApi.ExportDirectorsAsGroups = cbDirectorsAsGroups.IsChecked ?? false;
+
             // Reset API Request Counter.
             CcbApi.ApiRequestCount = 0;
 
diff --git a/Slingshot.CCB/Slingshot.CCB/Utilities/CcbApi.cs b/Slingshot.CCB/Slingshot.CCB/Utilities/CcbApi.cs
index 7382026..fdfdafc 100644
--- a/Slingshot.CCB/Slingshot.CCB/Utilities/CcbApi.cs
+++ b/Slingshot.CCB/Slingshot.CCB/Utilities/CcbApi.cs
@@ -200,6 +200,15 @@ public static int ApiThrottleRate
         /// </value>
         public static bool ConsolidateScheduleNames { get; set; } = false;
 
+        /// <summary>
+        /// Gets or sets a value indicating whether [export directors as groups].
+        /// Set ExportDirectorsAsGroups to true to export Directors as Groups in the hierarchy.
+        /// </summary>
+        /// <value>
+        ///   <c>true</c> if [export directors as groups]; otherwise, <c>false</c>.
+        /// </value>
+        public static bool ExportDirectorsAsGroups { get; set; } = false;
+
         /// <summary>
         /// Gets or sets the daily limit.
         /// </summary>
@@ -474,7 +483,7 @@ private static bool GetGroupProfiles( List<int> selectedGroupTypes, DateTime? mo
 
                         if ( selectedGroupTypes.Contains( groupTypeId ) )
                         {
-                            var importGroups = CcbGroup.Translate( groupNode );
+                            var importGroups = CcbGroup.Translate( groupNode, CcbApi.ExportDirectorsAsGroups );
 
                             if ( importGroups != null )
                             {
@@ -562,7 +571,7 @@ private static bool GetGroupProfilesWorkaround( List<int> selectedGroupTypes, Da
                                 // any participants.
                                 IncompleteGroups.Add( groupId );
 
-                                var importGroups = CcbGroup.Translate( groupNode );
+                                var importGroups = CcbGroup.Translate( groupNode, CcbApi.ExportDirectorsAsGroups );
 
                                 if ( importGroups != null )
                                 {
@@ -615,7 +624,7 @@ private static void GetGroupProfileById( int groupId )
                 return;
             }
 
-            var importGroups = CcbGroup.Translate( groupNode );
+            var importGroups = CcbGroup.Translate( groupNode, CcbApi.ExportDirectorsAsGroups );
             if ( importGroups == null )
             {
                 return;
@@ -645,11 +654,19 @@ private static void ExportDepartments()
                 }
 
                 var sourceDepartments = xdocCustomFields.Element( "ccb_api" )?.Element( "response" )?.Elements( "items" );
+                MD5 md5Hasher = MD5.Create();
 
                 foreach ( var sourceDepartment in sourceDepartments.Elements( "item" ) )
                 {
+                    // Use hash of 9999 + department id to create a unique group id
+                    var hashedDepartmentId = md5Hasher.ComputeHash( Encoding.UTF8.GetBytes( $@"
+                     {9999}
+                     {sourceDepartment.Element( "id" ).Value}
+                    " ) );
+                    var groupId = Math.Abs( BitConverter.ToInt32( hashedDepartmentId, 0 ) ); // used abs to ensure positive number
+
                     var group = new Group();
-                    group.Id = ( "9999" + sourceDepartment.Element( "id" ).Value ).AsInteger();
+                    group.Id = groupId;
                     group.Name = sourceDepartment.Element( "name" )?.Value;
                     group.Order = sourceDepartment.Element( "order" ).Value.AsInteger();
                     group.IsActive = true;
diff --git a/Slingshot.CCB/Slingshot.CCB/Utilities/Translators/CcbGroup.cs b/Slingshot.CCB/Slingshot.CCB/Utilities/Translators/CcbGroup.cs
index 660a91d..3d0ab30 100644
--- a/Slingshot.CCB/Slingshot.CCB/Utilities/Translators/CcbGroup.cs
+++ b/Slingshot.CCB/Slingshot.CCB/Utilities/Translators/CcbGroup.cs
@@ -12,7 +12,7 @@ namespace Slingshot.CCB.Utilities.Translators
 {
     public static class CcbGroup
     {
-        public static List<Group> Translate( XElement inputGroup )
+        public static List<Group> Translate( XElement inputGroup, bool exportDirectorsAsGroups = true )
         {
             List<Group> groups = new List<Group>();
 
@@ -41,10 +41,13 @@ public static List<Group> Translate( XElement inputGroup )
 
             groups.Add( group );
 
-            if ( inputGroup.Element( "department" ) != null && inputGroup.Element( "department" ).Attribute( "id" ) != null && inputGroup.Element( "department" ).Attribute( "id" ).Value.IsNotNullOrWhitespace() )
+            var hasDepartment = inputGroup.Element( "department" ) != null && inputGroup.Element( "department" ).Attribute( "id" ) != null && inputGroup.Element( "department" ).Attribute( "id" ).Value.IsNotNullOrWhitespace();
+            var hasDirector = inputGroup.Element( "director" ) != null && inputGroup.Element( "director" ).Attribute( "id" ) != null && inputGroup.Element( "director" ).Attribute( "id" ).Value.IsNotNullOrWhitespace();
+            
             MD5 md5Hasher = MD5.Create();
             
             // add the department as a group with id set to hash of 9999 + department id to create a unique group id for it
+            if ( hasDepartment )
             {
                 var hashedDepartmentId = md5Hasher.ComputeHash( Encoding.UTF8.GetBytes( $@"
                      {9999}
@@ -57,10 +60,20 @@ public static List<Group> Translate( XElement inputGroup )
                 {
                     departmentName = "No Department Name";
                 }
-                groups.Add( new Group { Id = departmentId.Value, Name = inputGroup.Element( "department" ).Value, IsActive = true, GroupTypeId = 9999 } );
+                var departmentGroup = new Group();
+                departmentGroup.Id = departmentId.Value;
+                departmentGroup.IsActive = true;
+                departmentGroup.Name = departmentName;
+                departmentGroup.GroupTypeId = 9999;
+
+                if ( !exportDirectorsAsGroups && hasDirector )
+                {
+                    departmentGroup.GroupMembers.Add( new GroupMember { PersonId = inputGroup.Element( "director" ).Attribute( "id" ).Value.AsInteger(), Role = "Director", GroupId = departmentGroup.Id } );
+                }
+                groups.Add( departmentGroup );
             }
 
-            if ( inputGroup.Element( "director" ) != null && inputGroup.Element( "director" ).Attribute( "id" ) != null && inputGroup.Element( "director" ).Attribute( "id" ).Value.IsNotNullOrWhitespace() )
+            if ( exportDirectorsAsGroups && hasDirector )
             {
                 // add the director as a group with an id of 9998 + its id + its department id (if any) to create a unique group id for it
                 var departmentIdString = departmentId.HasValue ? departmentId.Value.ToString() : string.Empty;
@@ -159,7 +172,7 @@ public static List<Group> Translate( XElement inputGroup )
             }
 
             // determine the parent group
-            if ( directorId.HasValue )
+            if ( exportDirectorsAsGroups && directorId.HasValue )
             {
                 group.ParentGroupId = directorId.Value;
             }

From 0a8b60028a7e260ae84494b9c474b9098ad0b549 Mon Sep 17 00:00:00 2001
From: Guy Maness <guy.maness@gmail.com>
Date: Thu, 23 Jun 2022 11:58:36 -0400
Subject: [PATCH 3/3] Update
 Slingshot.CCB/Slingshot.CCB/Utilities/Translators/CcbGroup.cs

---
 Slingshot.CCB/Slingshot.CCB/Utilities/Translators/CcbGroup.cs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Slingshot.CCB/Slingshot.CCB/Utilities/Translators/CcbGroup.cs b/Slingshot.CCB/Slingshot.CCB/Utilities/Translators/CcbGroup.cs
index 3d0ab30..19dcdb3 100644
--- a/Slingshot.CCB/Slingshot.CCB/Utilities/Translators/CcbGroup.cs
+++ b/Slingshot.CCB/Slingshot.CCB/Utilities/Translators/CcbGroup.cs
@@ -75,7 +75,7 @@ public static List<Group> Translate( XElement inputGroup, bool exportDirectorsAs
 
             if ( exportDirectorsAsGroups && hasDirector )
             {
-                // add the director as a group with an id of 9998 + its id + its department id (if any) to create a unique group id for it
+                // add the director as a group with id set to hash of 9998 + its id + its department id (if any) to create a unique group id for it
                 var departmentIdString = departmentId.HasValue ? departmentId.Value.ToString() : string.Empty;
                 var hashedDirectorId = md5Hasher.ComputeHash( Encoding.UTF8.GetBytes( $@"
                      {9998}