Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[8.1] Deprecated function: explode(): Passing null to parameter #2 ($string) #131

Closed
argiepiano opened this issue Dec 10, 2023 · 1 comment · Fixed by #132
Closed

[8.1] Deprecated function: explode(): Passing null to parameter #2 ($string) #131

argiepiano opened this issue Dec 10, 2023 · 1 comment · Fixed by #132

Comments

@argiepiano
Copy link
Collaborator

This is thrown when backing up the entire site and db.

Deprecated function: explode(): Passing null to parameter #2 ($string) of type string is deprecated in backup_file->set_file_info() (line 521 of /Users/XXX/Sites/localhost/bdtesting/modules/backup_migrate/includes/files.inc).

The problem is that backup_file::set_file_info() expects a file name as part of the received array. But backup_migrate_files_destination_archivesource::get_db() doesn't pass a filename, since the file has already been initialized previously when backing up the site files. The solution is to avoid doing the explode if the array is empty.

@argiepiano
Copy link
Collaborator Author

PR #132

argiepiano added a commit that referenced this issue Dec 12, 2023
Issue #131. Avoid using an empty string in explode.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant