Re: Squirrelmail returns blank page after send (PHP 7: Warning: contains logs )

Thought I might as well feedback what I have found so far,in respect
of getting rid of the "Deprecated" messages that comw about simply
because the various classes don;t have an __construct() method.

Clearly the developere will be aware of this info, but the info might
be useful to anyone else, who follows this thread, and who might want
to try and get rid of those messages, whilst the official changes get
added into the codebase.

These files appear to be the ones that supply a class


squirrelmail/class/html.class.php:class html {

squirrelmail/class/deliver/Deliver.class.php:class Deliver {
squirrelmail/class/deliver/Deliver_IMAP.class.php:class Deliver_IMAP
extends Deliver {
squirrelmail/class/deliver/Deliver_SMTP.class.php:class Deliver_SMTP
extends Deliver {
squirrelmail/class/deliver/Deliver_SendMail.class.php:class
Deliver_SendMail extends Deliver {

squirrelmail/class/helper/VCard.class.php:class VCard {

squirrelmail/class/mime/AddressStructure.class.php:class AddressStructure {
squirrelmail/class/mime/ContentType.class.php:class ContentType {
squirrelmail/class/mime/Disposition.class.php:class Disposition {
squirrelmail/class/mime/Language.class.php:class Language {
squirrelmail/class/mime/Message.class.php:class Message {
squirrelmail/class/mime/MessageHeader.class.php:class MessageHeader {
squirrelmail/class/mime/Rfc822Header.class.php:class Rfc822Header {
squirrelmail/class/mime/SMimeMessage.class.php:class SMimeMessage {

squirrelmail/functions/abook_database.php:class abook_database extends
addressbook_backend {
squirrelmail/functions/abook_ldap_server.php:class abook_ldap_server
extends addressbook_backend {
squirrelmail/functions/abook_local_file.php:class abook_local_file extends
addressbook_backend {
squirrelmail/functions/addressbook.php:class AddressBook {
squirrelmail/functions/addressbook.php:class addressbook_backend {
squirrelmail/functions/db_prefs.php:class dbPrefs {
squirrelmail/functions/options.php:class SquirrelOption {

Of those,

these don't even have a constructor named as the Classname, which is the
deprecated
functionality causing the messages.

VCard
AddressStructure
Message
MessageHeader
Rfc822Header
SMimeMessage

For all of the others, a patch, akin to that described in the URL
referenced in my
opening email, that provides an __construct() matching the function
signature of the
original "class name Constructor", and then simply using that original
funtion to
pass through the parameters to the new  __construct() function, seems to
apply
cleanly to the codebase, without so far, affecting functionality.

I'm not sure if attaching such a patch-diff here is the right thing to do ?


------------------------------------------------------------------------------
Developer Access Program for Intel Xeon Phi Processors
Access to Intel Xeon Phi processor-based developer platforms.
With one year of Intel Parallel Studio XE.
Training and support from Colfax.
Order your platform today.http://sdm.link/xeonphi
-----
squirrelmail-devel mailing list
Posting guidelines: http://squirrelmail.org/postingguidelines
List address: [email protected]
List archives: http://news.gmane.org/gmane.mail.squirrelmail.devel
List info (subscribe/unsubscribe/change options): https://lists.sourceforge.net/lists/listinfo/squirrelmail-devel