BuildTheEarth/main-bot

Schedule entity abstract class

cAttte opened this issue · 2 comments

So Many Schedules!!!!! we should have an abstract class called something like ScheduleEntity or SchedulableEntity that can be extended by the current AdvancedBuilder, BannerImage*. Task, TimedPunishment, Reminder (and future) entities. so that we don't have to repeat methods and properties like undo() or undoTimeout Every Single Time.

*maybe not this one, though, because it works differently than the rest: it has more of a "global" (static) scheduler that manipulates individual rows as it pleases, instead of requiring to schedule every instance of the entity independently. adding this system to the abstract class would probably not be worth it if it is only used by this sub-class.

@arc25275 @mebrooks01 Do we even want this anymore considering how much the scheduled entities have diverged?

I dont think it is worth the time to implement for how little of an impact it will practically make. It will just create more bugs to solve while only serving to give a slight performance boost.