feat: delete recent contact #55247
feat: delete recent contact #55247
Conversation
SebastianKrupinski
force-pushed
the
fix/issue-55169-delete-recent-contact
branch
from
afbdf07 to
13d96ed
Compare
| [ | ||
| 'privilege' => '{DAV:}unbind', | ||
| 'principal' => $this->getOwner(), | ||
| 'protected' => true, | ||
| ], |
There was a problem hiding this comment.
| [ | |
| 'privilege' => '{DAV:}unbind', | |
| 'principal' => $this->getOwner(), | |
| 'protected' => true, | |
| ], |
Unbind is ignored for resources that are not collections.
There was a problem hiding this comment.
Done. Removed
| 'protected' => true, | ||
| ], | ||
| [ | ||
| 'privilege' => '{DAV:}unbind', |
There was a problem hiding this comment.
for my understanding, why is the unbind ACL granted for the collection but not the card? doesn't that mean the collection is allowed to be deleted? 🤔
There was a problem hiding this comment.
Yeah, I thought the same thing at first the ACL's should technically be on the Object, not the collection, but SabreDAV ignores the ACL on the Object. (Dose not send them) So in the context of Address books, unbind on the collection means "Delete an object", I'm guessing this is because it's a flat list instead of a collection/object tree. I can look more in to it, if you want.
Signed-off-by: SebastianKrupinski <krupinskis05@gmail.com>
SebastianKrupinski
force-pushed
the
fix/issue-55169-delete-recent-contact
branch
from
fbf0456 to
bb78130
Compare
|
@ChristophWurst I was thinking this should be back ported to at least 31? |
|
It's a feature and there is currently no requirement to have this in 32. Don't backport |
github-project-automation
bot
moved this from 🏗️ In progress
to ☑️ Done
in 💌 📅 👥 Groupware team
Summary
Checklist
3. to review, feature component)stable32)