[Pkg-giraffe-discuss] zarafa-7.2.1 RC1 released
Jelle van der Waa
j.vanderwaa at zarafa.com
Fri Aug 21 20:05:17 UTC 2015
Hi,
Just a quick reply.
> But wait, I remember a patch from Jelle about dropping the JSON.php
> complete from the source inside the PHP files...
Yes I can rebase it and send it to you, if you still want it :)
> > -rw-r--r-- root/root /usr/share/zarafa-webapp/LICENSE.txt
> > -rw-r--r-- root/root /usr/share/zarafa-webapp/client/tinymce/jquery.tinymce.min.js
> > -rw-r--r-- root/root /usr/share/zarafa-webapp/client/tinymce/license.txt
> > -rw-r--r-- root/root /usr/share/zarafa-webapp/client/tinymce/plugins/media/moxieplayer.swf
> > -rw-r--r-- root/root /usr/share/zarafa-webapp/plugins/titlecounter/config.php
> > -rw-r--r-- root/root /usr/share/zarafa-webapp/plugins/titlecounter/js/titlecounter.js
> > -rw-r--r-- root/root /usr/share/zarafa-webapp/plugins/titlecounter/manifest.xml
> > -rw-r--r-- root/root /usr/share/zarafa-webapp/plugins/titlecounter/php/plugin.titlecounter.php
Hmm, the titlecounter plugin is a valid plugin and should be fully AGPLv3. It modifies your tab title to show
the amount of unread emails.
> Yes, the PHP logging isn't clear to me, the htaccess file is using a
> relative path for the output file. Better using here a absolute path to
> /var/log/zarafa/ for example.
>
> And some small more documentation inside .htaccess
About error reporting, we explicitly disable it for some unknown reason in config.php see this line:
"error_reporting(0);"
I'd say that's wrong and we should at least log errors to the Apache log. I will be investigating this, but I don't think the change will end up in 2.1.0
The last line of the htcacess requires some investigation I am not sure where or what should be logged there.
The option to enable short tags should be unnecessary, because we converted everything to non short open tags.
Thanks for the hard work!
Jelle
More information about the Pkg-giraffe-discuss
mailing list