Great find! I'm wondering where else we have the same assumption...
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Advanced Search
Wed, Jul 24
Thu, Jul 18
Cleanup
Separate token scope
Wed, Jul 17
Mon, Jul 15
sync bcc via eas 16.0
Fri, Jul 12
Tue, Jul 9
Mon, Jul 8
Sun, Jul 7
Looks good to me.
Fri, Jul 5
This can be submitted once it is clear that it has the desired effect.
It seems to me we should:
- Adjust the catch in https://git.kolab.org/diffusion/RPK/browse/master/plugins/libcalendaring/lib/libcalendaring_vcalendar.php (to specifically just ignore a broken itip).
- Adjust all the catches in Sync.php to also catch Throwable
- Add a test that triggers this error to the syncroton testsuite
- Remove the catch in this patch again, so we don't end up with 3 layers potentially catching.
..
In D4839#59874, @machniak wrote:Indeed. I just noticed that the actual error wasn't an exception. It was PHP Fatal error: Uncaught Error: __clone method called on non-object in /usr/share/roundcubemail/plugins/libcalendaring/lib/libcalendaring_vcalendar.php:633, so to catch it we'd have to catch Throwable, I suppose, I'm not sure.
Still, it makes sense to not skip the entire message, but sync it as it was a normal message, ignoring the fact it has an invitation.
Thu, Jul 4
I suppose it would be better to just make sure that the logic in Sync.php works instead of ignoring ical errors specifically.
What I find a bit odd is that we already try to catch exceptions in Command/Sync.php on all getEntry calls, so that should already take care of it, no?
You can just push it once you looked over the migration change.
Looks good apart from the migration change (which might be fine but doesn't relate to the diff).
nice cleanup
In D4827#59694, @machniak wrote:Good progress, but I was about to work on DAV, and I already have some conflicting code. Please, don't work on DAV at this moment.
Small imap fixup, let the exporter implement the migration strategy
Wed, Jul 3
dav migration, separate migrators for imap/dav
Tue, Jul 2
Mon, Jul 1
This only implements migrating imap messages but highlights how the current framework doesn't cleanly facilitate not using the job per item strategy.
I think there's a cleaner approach to implementing the two strategies than what I did above, e.g. by moving the processFolder logic entirely to the importer.
I think we can merge it and take it from there.
Thu, Jun 27
Maybe this would be better as a subtab in the finances tab, but it's a start.
Jun 26 2024
Jun 25 2024
Don't break all existing logic related to cost
Sum up charges and fees first, then limit by wallet balance.