Deprecated: Methods with the same name as their class will not be constructors in a future version of PHP

Patch applies cleanly to 7.x-3.x-dev. This is PHP4 code!

Here's an updated patch which catches another constructor.

Tested #5 working as expected.

#5
Thanks for the Patch.
Works fine...

I can confirm RTBC. This should be a hot candidate for a new stable release. PHP7 is quite common. :)

+1 on RTBC

BTW, I ran this through php7cc and the only places where this deprecated format is used are fixed in the patch above.

This is a good excuse for a new stable release. It will make updating easier for everyone. 7.0 is a pretty nice version of PHP and we want to encourage it.

+1 for #13 :)

#13++ ;-)
Using auto backups under cron/PHP7 causes cron to vomit this deprecation warning message into my mailbox 12x each day :-( So it's not just on the screen - please merge the fix into stable!

+1 RTBC

+1 for a new release. PHP7 is important :)

Thanks to the committers for getting these fixes in to version 7.10.

--- Oh wait. They rolled the fixes in CTools. My mistake. But please fix them here too. Thanks.

+1 on RTBC ... is there anything you're waiting for?

I start to think that the 7.x branch is not maintained anymore :(

Well... maybe this issue it's a hidden request for co-maintainership ? :)

Just moved a bunch of my sites over to PHP7 and confirmed the patch in #5 fixes both the warnings I am seeing.

Would be great to see this released.

Nice to have this in dev irrespective if there are going to be no more D7 releases

Saves having to track mods / patches :)

I confirm #5 fixes the issue.

confirm #5 fixes issue for me as well

There seems to be no active maintainer?
I tried to contact, but no reply yet.

#5 works for me too.
Thanks.

+1 for a new release

The patch in #5 also worked for me when applied to the latest dev version of the module.

Still no reply from the maintainer... how shell we proceed?

I guess we will have no choice than to use https://www.drupal.org/node/251466

Patch applied. Thanks

Thank you!

Thanks

Thank you very much! That's great!
Would you be so nice to create a new D7 stable release containing this patch?

Yeah. Patch #5 works for me, too

Thank you joelstein!

yeah #5 worked

+1 for a new release. Just upgraded to php 7 and warnings popping up on all sites. Am applying the patch now.

Patch in #5 works. Have applied to many sites that I've upgraded to PHP7.0x - hopefully this patch gets applied to the module.

I am using the latest Dev version (backup_migrate 7.x-3.1+11-dev 2016/12/31) and this patch does not seem to be there.

Patch from #5 works for me as well.

#5 works for me, thanks.

Can someone help me with file.inc? I am still getting this syntax error after the patch:

ParseError: syntax error, unexpected 'if' (T_IF), expecting ';' or '{' in backup_migrate_include() (line 133 of /public_html/sites/all/modules/backup_migrate/includes/files.inc)

Here is the patch:

/**
* Construct a file object given a file path, or create a temp file for writing.
*/
function __construct($params = array())
if (isset($params['filepath'])) && file_exists($params['filepath'])) {
$this->set_filepath($params['filepath']);
}
else {
$this->set_file_info($params);
$this->temporary_file();
}
}

This:

 function __construct($params = array())

Should be this:

 function __construct($params = array()) {

Please just update to 7.x-3.6 and open a new issue if you're still running into problems. Thank you.

Thanks @DamienMcKenna. Updating to Backup and Migrate 7.x-3.6 alleviated all my issues. Good advice. Thank you.

@geogeektress: that's awesome, thanks for letting me know.

thanks, #5 worked.