Skip to content

Improve scheduling objects deletion#249

Open
tchapi wants to merge 6 commits intomainfrom
cc/fix-scheduling-objects-deletion
Open

Improve scheduling objects deletion#249
tchapi wants to merge 6 commits intomainfrom
cc/fix-scheduling-objects-deletion

Conversation

@tchapi
Copy link
Owner

@tchapi tchapi commented Mar 7, 2026

No description provided.

@tchapi tchapi force-pushed the cc/fix-scheduling-objects-deletion branch from 713ef0f to 3388090 Compare March 7, 2026 21:27
tchapi added 2 commits March 7, 2026 22:28
@tchapi tchapi force-pushed the cc/fix-scheduling-objects-deletion branch from 3388090 to bfe718b Compare March 7, 2026 21:34
Comment on lines -40 to +47
} else {
// Returns CalendarInstances as objects
return $query->getQuery()
->getResult();
}

// Returns CalendarInstances as objects
return $query->getQuery()
->getResult();
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just linting

Comment on lines -86 to +102
$objectRepository = $this->getEntityManager()->getRepository(\App\Entity\CalendarObject::class);
$objectRepository = $this->getEntityManager()->getRepository(CalendarObject::class);
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Linting

Comment on lines -98 to +116
\App\Entity\Calendar::COMPONENT_EVENTS => 'events',
\App\Entity\Calendar::COMPONENT_NOTES => 'notes',
\App\Entity\Calendar::COMPONENT_TODOS => 'tasks',
Calendar::COMPONENT_EVENTS => 'events',
Calendar::COMPONENT_NOTES => 'notes',
Calendar::COMPONENT_TODOS => 'tasks',
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Linting

@tchapi tchapi mentioned this pull request Mar 7, 2026
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

Successfully merging this pull request may close these issues.

1 participant