Fixed bug to logging by day of week

[RussianDragon]

Hi!
We found bug in logging by day of week.
If you set option "archiveEvery" as any day of week, nlog will create new archive file every day.

Мы нашли ошибку в создании файлов логов-архивов, если установлено архивирование по дням недели. Файлы создаются каждый день, а не в определенные дни

---

next code сontains fix bug

private static DateTime TruncateArchiveTime(DateTime input, FileArchivePeriod resolution)
        {
            switch (resolution)
            {
                case FileArchivePeriod.Year:
                    return new DateTime(input.Year, 1, 1, 0, 0, 0, 0, input.Kind);
                case FileArchivePeriod.Month:
                    return new DateTime(input.Year, input.Month, 1, 0, 0, 0, input.Kind);
                case FileArchivePeriod.Day:
                    return input.Date;
                case FileArchivePeriod.Hour:
                    return input.AddTicks(-(input.Ticks % TimeSpan.TicksPerHour));
                case FileArchivePeriod.Minute:
                    return input.AddTicks(-(input.Ticks % TimeSpan.TicksPerMinute));
                case FileArchivePeriod.Sunday:
                    return CalculateNextWeekday(input.Date, DayOfWeek.Sunday);
                case FileArchivePeriod.Monday:
                    return CalculateNextWeekday(input.Date, DayOfWeek.Monday);
                case FileArchivePeriod.Tuesday:
                    return CalculateNextWeekday(input.Date, DayOfWeek.Tuesday);
                case FileArchivePeriod.Wednesday:
                    return CalculateNextWeekday(input.Date, DayOfWeek.Wednesday);
                case FileArchivePeriod.Thursday:
                    return CalculateNextWeekday(input.Date, DayOfWeek.Thursday);
                case FileArchivePeriod.Friday:
                    return CalculateNextWeekday(input.Date, DayOfWeek.Friday);
                case FileArchivePeriod.Saturday:
                    return CalculateNextWeekday(input.Date, DayOfWeek.Saturday);
                default:
                    return input;   // Unknown time-resolution-truncate, leave unchanged
            }
        }

[304NotModified]

Hi, please use the issue template

[RussianDragon]

We writing about of bug, and writing fix code.
What's not clear?

Мы написали об ошибке и написали как исправить.
Что тут непонятного?

[304NotModified]

which NLog version? is this NLog 4.4.8? NLog 5 beta?

[RussianDragon]

It is master branch.
Commit 3954157

[snakefoot]

@RussianDragon Sounds like you are very close to make a unit-test and a github pull-request :). This will increase the chance of it making it into next NLog-release.

[RussianDragon]

I have not enough permissions to push my fix-branch...

[304NotModified]

Steps:

Create a fork (the button in github)
Add it as remote (or clone and copy changes)
Push
Create PR in github

[RussianDragon]

@304NotModified Thank you. :)
I created new pull requests.

[304NotModified]

You fixed it!
(in #2093)

thanks!

[304NotModified]

NLog 4.4.9 is in the deploy pipeline! Will be online in an hour!