Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

InvertTimeRange property in DailyCalendar #1507

Closed
qzavyer opened this issue Feb 3, 2022 Discussed in #1506 · 4 comments
Closed

InvertTimeRange property in DailyCalendar #1507

qzavyer opened this issue Feb 3, 2022 Discussed in #1506 · 4 comments
Milestone

Comments

@qzavyer
Copy link
Contributor

qzavyer commented Feb 3, 2022

Discussed in #1506

Originally posted by qzavyer February 3, 2022
Hi, I try get the DailyCalendar from my scheduler, but in this instance InvertTimeRange property always false. In DailyCalendar.Clone I don't see where this property copying in new instance of Calendar:

public override ICalendar Clone()
{
    DailyCalendar dailyCalendar = new DailyCalendar(base.CalendarBase, RangeStartingTime, RangeEndingTime);
    CloneFields(dailyCalendar);
    return dailyCalendar;
}

and

protected BaseCalendar CloneFields(BaseCalendar clone)
{
    clone.Description = Description;
    clone.TimeZone = TimeZone;
    clone.CalendarBase = CalendarBase?.Clone();
    return clone;
}
```</div>
@lahma
Copy link
Member

lahma commented Feb 3, 2022

Seems like a bug. Would you like to offer a PR with a test case and a fix?

@qzavyer
Copy link
Contributor Author

qzavyer commented Feb 4, 2022

I can try but I never participated in opensource projects. I don't how I can do it

@lahma
Copy link
Member

lahma commented Feb 4, 2022

OK, all good. Here's a some guidance https://www.digitalocean.com/community/tutorials/how-to-create-a-pull-request-on-github how to get started, but if you feel that it's something your don't want to do at this point, we can put into TODO queue.

It's a bit easier when community members participate, but of course the barrier of entry is always a bit higher for the first contribution.

qzavyer added a commit to qzavyer/quartznet that referenced this issue Feb 12, 2022
@lahma lahma added this to the vNext milestone Feb 12, 2022
@lahma
Copy link
Member

lahma commented Feb 12, 2022

Great work, I hope you've found the contribution process painless and nice. All help is always appreciated!

@lahma lahma closed this as completed Feb 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants