converted ITimeProvider to use DateTimeOffset

+semver: minor
nexussays · May 25, 2017 · 2ff7132 · 2ff7132
1 parent 186761d
commit 2ff7132
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 14 deletions.
diff --git a/src/nexus.core/logging/SystemLog.cs b/src/nexus.core/logging/SystemLog.cs
@@ -323,7 +323,7 @@ private void CreateLogEntry( LogLevel severity, Object[] objects, String message
  }
 
  // get the time immediately in case serializing objects takes time
- var time = m_timeProvider.UtcNow;
+ var time = m_timeProvider.UtcNow.UtcDateTime;
 
  var data = new List<Object>();
  if(objects != null)

diff --git a/src/nexus.core/time/DefaultTimeProvider.cs b/src/nexus.core/time/DefaultTimeProvider.cs
@@ -8,12 +8,10 @@
 
 namespace nexus.core.time
 {
- /// <summary>
- /// Use <see cref="DateTime.UtcNow" />
- /// </summary>
+ /// <inheritdoc />
  public class DefaultTimeProvider : ITimeProvider
  {
  /// <inheritdoc />
- public DateTime UtcNow => DateTime.UtcNow;
+ public DateTimeOffset UtcNow => DateTimeOffset.UtcNow;
  }
 }
diff --git a/src/nexus.core/time/ITimeProvider.cs b/src/nexus.core/time/ITimeProvider.cs
@@ -13,17 +13,9 @@ namespace nexus.core.time
  /// </summary>
  public interface ITimeProvider
  {
- /*
- * TODO: Implement, or change UtcNow to DateTimeOffset
- /// <summary>
- /// The current time in the local timezone
- /// </summary>
- DateTimeOffset Now { get; }
- */
-
  /// <summary>
  /// The current time in UTC
  /// </summary>
- DateTime UtcNow { get; }
+ DateTimeOffset UtcNow { get; }
  }
 }