diff --git a/CHANGELOG.md b/CHANGELOG.md index 47320d4..73a1e90 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -10,6 +10,15 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 _nothing changed yet_ +## [v0.1.1] - 2025-10-24 + +### Changed + +- Property names of the event should be more descriptive: + - From `CallerNumber` to `ExternalNumber` + - From `CalleeNumber` to `InternalNumber` + + ## [v0.1.0] - 2025-08-28 _Inital release_ @@ -27,6 +36,7 @@ _Inital release_ -[Unreleased]: https://github.com/AM-WD/FritzCallMonitor/compare/v0.1.0...HEAD +[Unreleased]: https://github.com/AM-WD/FritzCallMonitor/compare/v0.1.1...HEAD +[v0.1.1]: https://github.com/AM-WD/FritzCallMonitor/compare/v0.1.0...v0.1.1 [v0.1.0]: https://github.com/AM-WD/FritzCallMonitor/commits/v0.1.0 diff --git a/src/FritzCallMonitor.Demo/Program.cs b/src/FritzCallMonitor.Demo/Program.cs index 3c7b2e1..bb43c71 100644 --- a/src/FritzCallMonitor.Demo/Program.cs +++ b/src/FritzCallMonitor.Demo/Program.cs @@ -44,11 +44,11 @@ namespace FritzCallMonitor.Demo switch (e.Event) { case EventType.Ring: - Console.WriteLine($"{e.Timestamp:yyyy-MM-dd HH:mm:ss K} | #{e.ConnectionId} | Incoming Call from {e.CallerNumber} to {e.CalleeNumber}"); + Console.WriteLine($"{e.Timestamp:yyyy-MM-dd HH:mm:ss K} | #{e.ConnectionId} | Incoming Call from {e.ExternalNumber} to {e.InternalNumber}"); break; case EventType.Connect: - Console.WriteLine($"{e.Timestamp:yyyy-MM-dd HH:mm:ss K} | #{e.ConnectionId} | Call connected to {e.CallerNumber}"); + Console.WriteLine($"{e.Timestamp:yyyy-MM-dd HH:mm:ss K} | #{e.ConnectionId} | Call connected to {e.ExternalNumber}"); break; case EventType.Disconnect: @@ -56,7 +56,7 @@ namespace FritzCallMonitor.Demo break; case EventType.Call: - Console.WriteLine($"{e.Timestamp:yyyy-MM-dd HH:mm:ss K} | #{e.ConnectionId} | Outgoing Call from {e.CalleeNumber} to {e.CallerNumber}"); + Console.WriteLine($"{e.Timestamp:yyyy-MM-dd HH:mm:ss K} | #{e.ConnectionId} | Outgoing Call from {e.InternalNumber} to {e.ExternalNumber}"); break; } }; diff --git a/src/FritzCallMonitor/EventArgs/CallMonitorEventArgs.cs b/src/FritzCallMonitor/EventArgs/CallMonitorEventArgs.cs index f33c8ac..946db57 100644 --- a/src/FritzCallMonitor/EventArgs/CallMonitorEventArgs.cs +++ b/src/FritzCallMonitor/EventArgs/CallMonitorEventArgs.cs @@ -31,12 +31,12 @@ namespace AMWD.Net.Api.Fritz.CallMonitor /// /// Gets the external number displayed in the FRITZ!Box. /// - public string? CallerNumber { get; private set; } + public string? ExternalNumber { get; private set; } /// /// Gets the internal number registered in the FRITZ!Box. /// - public string? CalleeNumber { get; private set; } + public string? InternalNumber { get; private set; } /// /// Gets the duration of the call (only on event). @@ -71,13 +71,13 @@ namespace AMWD.Net.Api.Fritz.CallMonitor switch (eventType) { case EventType.Ring: - args.CallerNumber = columns[3]; - args.CalleeNumber = columns[4]; + args.ExternalNumber = columns[3]; + args.InternalNumber = columns[4]; break; case EventType.Connect: args.LinePort = int.TryParse(columns[3], out int connectLinePort) ? connectLinePort : null; - args.CallerNumber = columns[4]; + args.ExternalNumber = columns[4]; break; case EventType.Disconnect: @@ -87,8 +87,8 @@ namespace AMWD.Net.Api.Fritz.CallMonitor case EventType.Call: args.LinePort = int.TryParse(columns[3], out int callLinePort) ? callLinePort : null; - args.CalleeNumber = columns[4]; - args.CallerNumber = columns[5]; + args.InternalNumber = columns[4]; + args.ExternalNumber = columns[5]; break; default: diff --git a/test/FritzCallMonitor.Tests/CallMonitorClientTest.cs b/test/FritzCallMonitor.Tests/CallMonitorClientTest.cs index ca544db..c1a328c 100644 --- a/test/FritzCallMonitor.Tests/CallMonitorClientTest.cs +++ b/test/FritzCallMonitor.Tests/CallMonitorClientTest.cs @@ -166,8 +166,8 @@ namespace FritzCallMonitor.Tests Assert.AreEqual(EventType.Ring, eventArgs.Event); Assert.AreEqual(2, eventArgs.ConnectionId); Assert.IsNull(eventArgs.LinePort); - Assert.AreEqual("012345678901", eventArgs.CallerNumber); - Assert.AreEqual("9876543", eventArgs.CalleeNumber); + Assert.AreEqual("012345678901", eventArgs.ExternalNumber); + Assert.AreEqual("9876543", eventArgs.InternalNumber); Assert.IsNull(eventArgs.Duration); _tcpClientMock.VerifyGet(m => m.IsConnected, Times.Exactly(2)); @@ -209,8 +209,8 @@ namespace FritzCallMonitor.Tests Assert.AreEqual(EventType.Ring, eventArgs.Event); Assert.AreEqual(2, eventArgs.ConnectionId); Assert.IsNull(eventArgs.LinePort); - Assert.AreEqual("012345678901", eventArgs.CallerNumber); - Assert.AreEqual("9876543", eventArgs.CalleeNumber); + Assert.AreEqual("012345678901", eventArgs.ExternalNumber); + Assert.AreEqual("9876543", eventArgs.InternalNumber); Assert.IsNull(eventArgs.Duration); _tcpClientMock.VerifyGet(m => m.IsConnected, Times.Exactly(3)); diff --git a/test/FritzCallMonitor.Tests/CallMonitorEventArgsTest.cs b/test/FritzCallMonitor.Tests/CallMonitorEventArgsTest.cs index 4620d8f..22aa531 100644 --- a/test/FritzCallMonitor.Tests/CallMonitorEventArgsTest.cs +++ b/test/FritzCallMonitor.Tests/CallMonitorEventArgsTest.cs @@ -29,8 +29,8 @@ namespace FritzCallMonitor.Tests Assert.AreEqual(EventType.Ring, result.Event); Assert.AreEqual(2, result.ConnectionId); Assert.IsNull(result.LinePort); - Assert.AreEqual("012345678901", result.CallerNumber); - Assert.AreEqual("9876543", result.CalleeNumber); + Assert.AreEqual("012345678901", result.ExternalNumber); + Assert.AreEqual("9876543", result.InternalNumber); Assert.IsNull(result.Duration); } @@ -45,8 +45,8 @@ namespace FritzCallMonitor.Tests Assert.AreEqual(EventType.Connect, result.Event); Assert.AreEqual(1, result.ConnectionId); Assert.AreEqual(3, result.LinePort); - Assert.AreEqual("012345678901", result.CallerNumber); - Assert.IsNull(result.CalleeNumber); + Assert.AreEqual("012345678901", result.ExternalNumber); + Assert.IsNull(result.InternalNumber); Assert.IsNull(result.Duration); } @@ -61,8 +61,8 @@ namespace FritzCallMonitor.Tests Assert.AreEqual(EventType.Disconnect, result.Event); Assert.AreEqual(2, result.ConnectionId); Assert.IsNull(result.LinePort); - Assert.IsNull(result.CallerNumber); - Assert.IsNull(result.CalleeNumber); + Assert.IsNull(result.ExternalNumber); + Assert.IsNull(result.InternalNumber); Assert.AreEqual(TimeSpan.FromSeconds(42), result.Duration); } @@ -77,8 +77,8 @@ namespace FritzCallMonitor.Tests Assert.AreEqual(EventType.Call, result.Event); Assert.AreEqual(4, result.ConnectionId); Assert.AreEqual(7, result.LinePort); - Assert.AreEqual("012345678901", result.CallerNumber); - Assert.AreEqual("9876543", result.CalleeNumber); + Assert.AreEqual("012345678901", result.ExternalNumber); + Assert.AreEqual("9876543", result.InternalNumber); Assert.IsNull(result.Duration); } @@ -147,8 +147,8 @@ namespace FritzCallMonitor.Tests string line = "25.08.25 20:15:30;RING;2;012345678901;9876543;SIP0;EXTRA;COLUMN;"; var result = CallMonitorEventArgs.Parse(line); Assert.IsNotNull(result); - Assert.AreEqual("012345678901", result.CallerNumber); - Assert.AreEqual("9876543", result.CalleeNumber); + Assert.AreEqual("012345678901", result.ExternalNumber); + Assert.AreEqual("9876543", result.InternalNumber); } } }