Feature: IP Scanner > Pass MAC address as argument to a new profile (#1998)

* Feature: Pass MAC address as argument to a new profile

* Docs: Add #1998

Author: BornToBeRoot

Source/NETworkManager.Models/Network/HostInfo.cs

@@ -9,7 +9,7 @@ public class HostInfo
         public PhysicalAddress MACAddress { get; set; }
         public string Vendor { get; set; }
 
-        public string MACAddressString => MACAddress.ToString();
+        public string MACAddressString => MACAddress?.ToString();
 
         public HostInfo()
         {

Source/NETworkManager/ViewModels/IPScannerViewModel.cs

@@ -326,7 +326,10 @@ private async Task AddProfileSelectedHostAction()
             ProfileInfo profileInfo = new()
             {
                 Name = string.IsNullOrEmpty(SelectedResult.Hostname) ? SelectedResult.PingInfo.IPAddress.ToString() : SelectedResult.Hostname.TrimEnd('.'),
-                Host = SelectedResult.PingInfo.IPAddress.ToString()
+                Host = SelectedResult.PingInfo.IPAddress.ToString(),
+
+                // Additional data
+                WakeOnLAN_MACAddress = SelectedResult.MACAddressString
             };
 
             var customDialog = new CustomDialog
@@ -511,7 +514,7 @@ private void StopScan()
             CancelScan = true;
             _cancellationTokenSource.Cancel();
         }
-        
+
         private async Task DetectIPRange()
         {
             IsSubnetDetectionRunning = true;
@@ -667,15 +670,15 @@ private void DnsResolveFailed(AggregateException e)
         {
             StatusMessage = $"{Localization.Resources.Strings.TheFollowingHostnamesCouldNotBeResolved} {string.Join(", ", e.Flatten().InnerExceptions.Select(x => x.Message))}";
             IsStatusMessageDisplayed = true;
-            
+
             CancelScan = false;
             IsScanRunning = false;
         }
 
         private void UserHasCanceled(object sender, EventArgs e)
         {
             StatusMessage = Localization.Resources.Strings.CanceledByUserMessage;
-            IsStatusMessageDisplayed = true;                        
+            IsStatusMessageDisplayed = true;
         }
 
         private void SettingsManager_PropertyChanged(object sender, PropertyChangedEventArgs e)

docs/Changelog/next-release.md

@@ -52,6 +52,7 @@ The profiles and settings are automatically migrated to the new location when th
   - Max threads changed to to 1024 [#1927](https://github.com/BornToBeRoot/NETworkManager/pull/1927){:target="\_blank"}
   - Event handling when a host is found, user has canceled, etc. [#1969](https://github.com/BornToBeRoot/NETworkManager/pull/1969){:target="\_blank"}
   - Remove option to set the preferred IP protocol for DNS resolution (can now be set globally) [#1950](https://github.com/BornToBeRoot/NETworkManager/pull/1950){:target="\_blank"}
+  - Create profile from IP scanner now passes the MAC address, if available [#1998](https://github.com/BornToBeRoot/NETworkManager/pull/1998){:target="\_blank"}
 - **Port Scanner**
   - Add new port profiles / improve existing ones [#1909](https://github.com/BornToBeRoot/NETworkManager/pull/1909){:target="\_blank"}
   - New Port state "Timed out" if the timelimit is reached. [#1969](https://github.com/BornToBeRoot/NETworkManager/pull/1969){:target="\_blank"}