Skip to content

Commit 9c830dc

Browse files
author
Matt Lewis
committed
refactor: remove redundant check
1 parent fd30c39 commit 9c830dc

File tree

2 files changed

+11
-17
lines changed

2 files changed

+11
-17
lines changed

src/modules/day/calendar-day-view.component.ts

+5-8
Original file line numberDiff line numberDiff line change
@@ -429,15 +429,12 @@ export class CalendarDayViewComponent implements OnChanges, OnInit, OnDestroy {
429429
const pixelAmountInMinutes: number =
430430
MINUTES_IN_HOUR / (this.hourSegments * this.hourSegmentHeight);
431431
const minutesMoved: number = draggedInPixels * pixelAmountInMinutes;
432-
// TODO - remove this check once https://github.com/mattlewis92/angular-draggable-droppable/issues/21 is fixed
433-
if (minutesMoved !== 0) {
434-
const newStart: Date = addMinutes(dayEvent.event.start, minutesMoved);
435-
let newEnd: Date;
436-
if (dayEvent.event.end) {
437-
newEnd = addMinutes(dayEvent.event.end, minutesMoved);
438-
}
439-
this.eventTimesChanged.emit({ newStart, newEnd, event: dayEvent.event });
432+
const newStart: Date = addMinutes(dayEvent.event.start, minutesMoved);
433+
let newEnd: Date;
434+
if (dayEvent.event.end) {
435+
newEnd = addMinutes(dayEvent.event.end, minutesMoved);
440436
}
437+
this.eventTimesChanged.emit({ newStart, newEnd, event: dayEvent.event });
441438
}
442439

443440
private refreshHourGrid(): void {

src/modules/week/calendar-week-view.component.ts

+6-9
Original file line numberDiff line numberDiff line change
@@ -364,16 +364,13 @@ export class CalendarWeekViewComponent implements OnChanges, OnInit, OnDestroy {
364364
dayWidth: number
365365
): void {
366366
const daysDragged: number = draggedByPx / dayWidth;
367-
// TODO - remove this check once https://github.com/mattlewis92/angular-draggable-droppable/issues/21 is fixed
368-
if (daysDragged !== 0) {
369-
const newStart: Date = addDays(weekEvent.event.start, daysDragged);
370-
let newEnd: Date;
371-
if (weekEvent.event.end) {
372-
newEnd = addDays(weekEvent.event.end, daysDragged);
373-
}
374-
375-
this.eventTimesChanged.emit({ newStart, newEnd, event: weekEvent.event });
367+
const newStart: Date = addDays(weekEvent.event.start, daysDragged);
368+
let newEnd: Date;
369+
if (weekEvent.event.end) {
370+
newEnd = addDays(weekEvent.event.end, daysDragged);
376371
}
372+
373+
this.eventTimesChanged.emit({ newStart, newEnd, event: weekEvent.event });
377374
}
378375

379376
/**

0 commit comments

Comments
 (0)