From sources.filesource.inc:

  function _restore_from_file_php($file, &$settings) {

      // Older B&M Files format included a base 'files' directory.
      if (file_exists($temp .'/files')) {
        $temp = $temp . '/files';
      }

I see this exists for historical compatibility, but it causes a problem if your files directory contains a directory called 'files', for example, /sites/default/files/files. Instead of restoring from /sites/default/files, BAM will restore from /sites/default/files/files.

This resulted in a bit of confusion on my part until I figured it out. Perhaps it's time to sunset this compatibility layer?

CommentFileSizeAuthor
#2 backup_migrate-2753175.patch702 bytesentendu

Comments

entendu created an issue. See original summary.

entendu’s picture

entendu commented
StatusFileSize
new backup_migrate-2753175.patch702 bytes

Patch in case you agree.

couturier’s picture

couturier commented
Status: Active » Needs work

This patch may need to be rewritten based on 7.x-3.2.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 2: backup_migrate-2753175.patch, failed testing. View results

brankoc’s picture

brankoc commented

I was thinking of rerolling the patch against the most recent 7.x-3.x-dev version, but I ran into two problems:

1) I cannot reproduce the issue (I do not know how to), so I cannot test the patch.

2) There is a second occurrence of the offending code, namely in the method _restore_from_file_cli(). It is not clear to me if that should be included in the reroll. I assume not, because the code _restore_from_file_cli() is older than this issue.

ivnish’s picture

ivnish
he/him
commented
Status: Needs work » Closed (outdated)

Drupal 7 is EOL. This issue will be closed.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.