HEEDLS-374 Centre Configuration - Edit centre details

DigitalLearningSolutions.Data.Tests/DataServices/CentresDataServiceTests.cs

@@ -227,6 +227,38 @@ public void UpdateCentreWebsiteDetails_updates_centre()
             }
         }
 
+        [Test]
+        public void UpdateCentreDetails_updates_centre()
+        {
+            using var transaction = new TransactionScope();
+            try
+            {
+                // Given
+                const string notifyEmail = "test@centre.com";
+                const string bannerText = "Test banner text";
+                var signature = new byte[100];
+                var logo = new byte[200];
+
+
+                // When
+                centresDataService.UpdateCentreDetails(2, notifyEmail, bannerText, signature, logo);
+                var updatedCentre = centresDataService.GetCentreDetailsById(2)!;
+
+                // Then
+                using (new AssertionScope())
+                {
+                    updatedCentre.NotifyEmail.Should().BeEquivalentTo(notifyEmail);
+                    updatedCentre.BannerText.Should().BeEquivalentTo(bannerText);
+                    updatedCentre.SignatureImage.Should().BeEquivalentTo(signature);
+                    updatedCentre.CentreLogo.Should().BeEquivalentTo(logo);
+                }
+            }
+            finally
+            {
+                transaction.Dispose();
+            }
+        }
+
         [Test]
         public void GetCentreAutoRegisterValues_should_return_correct_values()
         {

DigitalLearningSolutions.Data/DataServices/CentresDataService.cs

@@ -37,6 +37,14 @@ void UpdateCentreWebsiteDetails(
             string? otherInformation
         );
 
+        void UpdateCentreDetails(
+            int centreId,
+            string? notifyEmail,
+            string bannerText,
+            byte[]? centreSignature,
+            byte[]? centreLogo
+        );
+
         (string firstName, string lastName, string email) GetCentreManagerDetails(int centreId);
         string[] GetCentreIpPrefixes(int centreId);
         (bool autoRegistered, string? autoRegisterManagerEmail) GetCentreAutoRegisterValues(int centreId);
@@ -215,6 +223,32 @@ public void UpdateCentreWebsiteDetails(
             );
         }
 
+        public void UpdateCentreDetails(
+            int centreId,
+            string? notifyEmail,
+            string bannerText,
+            byte[]? centreSignature,
+            byte[]? centreLogo
+        )
+        {
+            connection.Execute(
+                @"UPDATE Centres SET
+                    NotifyEmail = @notifyEmail,
+                    BannerText = @bannerText,
+                    SignatureImage = @centreSignature,
+                    CentreLogo = @centreLogo
+                WHERE CentreId = @centreId",
+                new
+                {
+                    notifyEmail,
+                    bannerText,
+                    centreSignature,
+                    centreLogo,
+                    centreId
+                }
+            );
+        }
+
         public (string firstName, string lastName, string email) GetCentreManagerDetails(int centreId)
         {
             var info = connection.QueryFirstOrDefault<(string, string, string)>(

DigitalLearningSolutions.Data/Services/ImageResizeService.cs

@@ -6,10 +6,13 @@
     using System.Drawing.Imaging;
     using System.IO;
     using Microsoft.AspNetCore.Http;
+    using Org.BouncyCastle.Crypto.Tls;
 
     public interface IImageResizeService
     {
         public byte[] ResizeProfilePicture(IFormFile formProfileImage);
+
+        public byte[] ResizeCentreImage(IFormFile formCentreImage);
     }
 
     public class ImageResizeService : IImageResizeService
@@ -22,6 +25,14 @@ public byte[] ResizeProfilePicture(IFormFile formProfileImage)
             return SquareImageFromMemoryStream(memoryStream, 300);
         }
 
+        public byte[] ResizeCentreImage(IFormFile formCentreImage)
+        {
+            using var memoryStream = new MemoryStream();
+            formCentreImage.CopyTo(memoryStream);
+
+            return ResizedImageFromMemoryStream(memoryStream, 500);
+        }
+
         private byte[] SquareImageFromMemoryStream(MemoryStream memoryStream, int targetSideLengthPx)
         {
             using var image = Image.FromStream(memoryStream);
@@ -35,6 +46,17 @@ private byte[] SquareImageFromMemoryStream(MemoryStream memoryStream, int target
             return result.ToArray();
         }
 
+        private byte[] ResizedImageFromMemoryStream(MemoryStream memoryStream, int maxSideLengthPx)
+        {
+            using var image = Image.FromStream(memoryStream);
+
+            using var resizedImage = ResizeImageByMaxSideLength(image, maxSideLengthPx);
+
+            using var result = new MemoryStream();
+            resizedImage.Save(result, ImageFormat.Jpeg);
+            return result.ToArray();
+        }
+
         private Image CropImageToCentredSquare(Image image)
         {
             var minSideLength = Math.Min(image.Height, image.Width);
@@ -67,8 +89,25 @@ private Image CropImageToCentredSquare(Image image)
 
         private Image ResizeSquareImage(Image image, int sideLengthPx)
         {
-            var destRect = new Rectangle(0, 0, sideLengthPx, sideLengthPx);
-            var returnImage = new Bitmap(sideLengthPx, sideLengthPx);
+            return ResizeImageToDimensions(image, sideLengthPx, sideLengthPx);
+        }
+
+        private Image ResizeImageByMaxSideLength(Image image, int maxSideLengthPx)
+        {
+            var longestSideLengthPx = Math.Max(image.Width, image.Height);
+            // No need to resize if image is smaller than the max size
+            var ratio = Math.Min((float)maxSideLengthPx / (float)longestSideLengthPx, 1);
+
+            var newWidth = (int)(image.Width * ratio);
+            var newHeight = (int)(image.Height * ratio);
+
+            return ResizeImageToDimensions(image, newWidth, newHeight);
+        }
+
+        private Image ResizeImageToDimensions(Image image, int widthPx, int heightPx)
+        {
+            var destRect = new Rectangle(0, 0, widthPx, heightPx);
+            var returnImage = new Bitmap(widthPx, heightPx);
 
             returnImage.SetResolution(image.HorizontalResolution, image.VerticalResolution);
 

DigitalLearningSolutions.Web.AutomatedUiTests/AccessibilityTests/BasicAccessibilityTests.cs

@@ -31,6 +31,7 @@ public void Page_has_no_accessibility_errors(string url, string pageTitle)
         [InlineData("/TrackingSystem/Centre/Ranking", "Centre ranking")]
         [InlineData("/TrackingSystem/Centre/ContractDetails", "Contract details")]
         [InlineData("/TrackingSystem/CentreConfiguration", "Centre configuration")]
+        [InlineData("/TrackingSystem/CentreConfiguration/EditCentreDetails", "Edit centre details")]
         [InlineData("/TrackingSystem/CentreConfiguration/EditCentreManagerDetails", "Edit centre manager details")]
         [InlineData(
             "/TrackingSystem/CentreConfiguration/EditCentreWebsiteDetails",

DigitalLearningSolutions.Web.Tests/Controllers/MyAccount/MyAccountControllerTests.cs

@@ -124,7 +124,7 @@ public void EditDetailsPostSave_for_admin_user_with_missing_delegate_answers_doe
         }
 
         [Test]
-        public void EditDetailsPostSave_with_profile_image_fails_validation()
+        public void EditDetailsPostSave_without_previewing_profile_image_fails_validation()
         {
             // Given
             var myAccountController = new MyAccountController(