nagimov / agendav-docker

Docker image for agendav
MIT License
30 stars 14 forks source link

Two bugfixes #9

Closed colinleroy closed 2 years ago

colinleroy commented 2 years ago

Hello !

Here is a PR with two fixes. The first one allows long, multi-line descriptions - without php5.6-mbstring installed, a paragraph-long description makes the event unsavable, with the following error :

PHP Fatal error: Call to undefined function Sabre\\VObject\\mb_strcut() in /var/www/agendav/web/vendor/sabre/vobject/lib/Property.php on line 252,

The second one adds an environment variable for the caldav.baseurl.public Agendav variable, with fixes the "Calendar URL for CalDAV clients" link that we can find when clicking the cogs at the right of a calendar. By default, it falls back to the existing variable, so as not to break existing deployments.

Hope this helps,

nagimov commented 2 years ago

@colinleroy merged, thanks!