• Fix type hint

-fix

The disabled attribute value could be boolean not string, but it will work either way.

• Cleanup
• Simplify resize() calls
• More tests
• Test updateConnection endpoint
• Media setup dialog
• Fix test regressions
• Add tests for media setup dialog
• Fix connection role propagation to new connections

• Rebase

• Add missing migration file

I've found one issue. The role change is not propagated to new connections. Scenario:

1. User1 joins as a subscriber
2. Moderator promotes him to a publisher
3. User2 joins and sees a UI issue, because the User1 tail is a subscriber-like tile, and there's a video element of User1 "out of the matrix".

• Add tests for media setup dialog

• Fix test regressions

• Media setup dialog

If you think that this internal cache is not really needed then go this way and remove use of self::$typedata. In rare cases when we list folders for all folder types separately in a single request, we'll end up with several requests to the Roundcube imap cache, instead of one. I'm afraid that we might call folder_typedata() or folder_type() methods quite a lot, but I didn't check that, so I'm not sure.

• Cleanup
• Simplify resize() calls
• More tests
• Test updateConnection endpoint

How about we call self::folders_typedata($prefix); to populate the cache, but then user folder_type() anyways to fetch missing entries (it still uses the cache if available)?

With this change METADATA for every folder will be asked separately. This definitely will have performance implications. Maybe a better approach would be to add a method to reset kolab_storage::$typedata (and other cache-like data if needed). And call that method from syncroton's kolab_sync::sleep() or kolab_sync_backend_folder::hasHierarchyChanges().

• Fix typo

• Improve constants handling
• Add unique key on openvidu_connections.session_id
• Fix: session_id should be indexed but not unique
• Remove redundant where clause

Wouldn't global STATUS_FOLDER_HIERARCHY_HAS_CHANGED status be more appropriate? Or this does not work with Outlook?

This code is not supposed to be used by Kolab backend. We're using kolab_sync_backend_state::validate().

• Fix: session_id should be indexed but not unique

• Add unique key on openvidu_connections.session_id

I'm not sure I like this. While fixing the issue you also changed how the list will look like. I.e. flat list instead of a tree-like list. If we do this anyway we should do the same in kolab_delegation plugin.

• Fix phpstan error and flaky tests

• Add tests

• Improve constants handling

• Fix browser tests regression

• Complete restoring/restored events handlers in user and domain observers
• Add test for user:restore command
• Add domain:restore command
• Set proper status in user/domain DeleteJob

• Set proper status in user/domain DeleteJob

• Complete restoring/restored events handlers in user and domain observers
• Add test for user:restore command
• Add domain:restore command

1. Did you check/estimate max number of messages we can handle with default 1MB packet?
2. This requires php-zlib extension which is not a default. We should make sure it's marked as required in packaging, I guess.
3. I'm not sure pushing compressed data into database is safe (without e.g. base64-encoding it).

Also, maybe we should tackle this from a different angle. If the folder has been disabled why do we even attempt to sync anything from that folder. We should tell the client that a hierarchy changed. So, maybe the problem is somewhere else, and could be fixed in a better way. We of course can still keep this solution as it prevents from timeout issues caused by other factors.

• Fix unexpected logon form when accessing the room

• Remove redundant code

• Corrected components lazy-loading + loader element