feat(meetings): Add endpoint to get writable calendars and the defaul… #13964
Conversation
…t calendar Signed-off-by: Joas Schilling <coding@schilljs.com>
nickvergessen
added
3. to review
enhancement
feature: api 🛠️
… default calendar
|
Noticed that default calendar has a color 'null'. Is it also the case for you, and is it on our side? In calendar app i see it as purple, but maybe that's a fallback |
Color can be null, yes (as defined on the API). In that case you should use the primary color. |
| protected function getSchedulingCalendarFromPropertiesTable(string $userId) { | ||
| $propertyPath = 'principals/users/' . $userId; | ||
| $propertyName = '{urn:ietf:params:xml:ns:caldav}schedule-default-calendar-URL'; | ||
|
|
||
| $query = $this->db->getQueryBuilder(); | ||
| $query->select('propertyvalue') | ||
| ->from('properties') | ||
| ->where($query->expr()->eq('userid', $query->createNamedParameter($userId))) | ||
| ->andWhere($query->expr()->eq('propertypath', $query->createNamedParameter($propertyPath))) | ||
| ->andWhere($query->expr()->eq('propertyname', $query->createNamedParameter($propertyName))) | ||
| ->setMaxResults(1); | ||
|
|
||
| $result = $query->executeQuery(); | ||
| $property = $result->fetchOne(); | ||
| $result->closeCursor(); | ||
|
|
||
| return $property; |
There was a problem hiding this comment.
You should go through the DAV app to get the default calendar URI, or you can use the user config:
$uri = $this->appConfig->getUserValue($userId, 'dav', 'defaultCalendar', CalDavBackend::PERSONAL_CALENDAR_URI);
There was a problem hiding this comment.
right, should use that preference,
but going through the dav app is to risky as it's all private api
There was a problem hiding this comment.
So I toggled this:
But there is no value in oc_preferences:
There was a problem hiding this comment.
I also only find delete and get for that preference, no set anywhere:
…t calendar
☑️ Resolves
🛠️ API Checklist
🚧 Tasks
schedule-meetingwill be added once all APIs are implemented🏁 Checklist
docs/has been updated or is not required