Fix export logic: ApplicationName mismatches and CSV escaping

[Copilot]

Changes proposed in this pull request

• Fixed 3 XML exports using wrong ApplicationName enum values (IPNetworkInfo, SNTPLookupResultInfo, WiFiNetworkInfo)
• Added RFC 4180-compliant CSV escaping via EscapeCsvValue() helper method
• Applied proper CSV escaping to all 20 export managers, replacing hardcoded quotes and incorrect .Replace(",", "") hacks

Related issue(s)

Based on analysis requested in PRs #3290 and #3291

Copilot generated summary

Copilot summary

ApplicationName Corrections

• IPNetworkInfo: ApplicationName.SNMP → ApplicationName.SubnetCalculator
• SNTPLookupResultInfo: ApplicationName.SNMP → ApplicationName.SNTPLookup
• WiFiNetworkInfo: ApplicationName.IPScanner → ApplicationName.WiFi

CSV Escaping Implementation

Added EscapeCsvValue() helper that wraps values in quotes when they contain commas, quotes, or newlines, and escapes internal quotes by doubling them:

internal static string EscapeCsvValue(string value)
{
    if (string.IsNullOrEmpty(value))
        return value;

    if (value.Contains(',') || value.Contains('"') || value.Contains('\n') || value.Contains('\r'))
    {
        return $"\"{value.Replace("\"", "\"\"")}\"";
    }

    return value;
}

Applied to all string fields across ConnectionInfo, DiscoveryProtocolPackageInfo, IPGeolocationInfo, TracerouteHopInfo, DNSLookupRecordInfo, HostsFileEntry, PingInfo, PortScannerInfo, OUIInfo, PortLookupInfo, IPScannerHostInfo, SNMPReceivedInfo, SNTPLookupResultInfo, NetworkInterface, and WiFiNetworkInfo export managers.

To-Do

• Update documentation to reflect this changes
• Update changelog to reflect this changes

Contributing

By submitting this pull request, I confirm the following:

• I have read and understood the contributing guidelines and the code of conduct.
• I have have added my name, username or email to the contributors list or don't want to.
• The code or resource is compatible with the GNU General Public License v3.0.

Original prompt

/task based on this PR #3290 and #3291 can you anaylze the export logic https://github.com/BornToBeRoot/NETworkManager/tree/main/Source/NETworkManager.Models/Export and find potential bugs, or logic issues in the code. Fix them as needed.

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.