When I try to migrate a site having media_mover_api 6.x-1.0-beta4 (6101) to 6.x-1.0-beta6 (6101), provision fails with the following error:

The version of the media_mover_api module found on this platform has a lower Schema version than the one the site has installed
CommentFileSizeAuthor
#4 migrate-01.png33.89 KBj0nathan
#4 migrate-02.png77.09 KBj0nathan

Comments

j0nathan’s picture

j0nathan commented

subscribing...

adrian’s picture

adrian commented

both of those have the same tag on the file.

http://drupalcode.org/viewvc/drupal/contributions/modules/media_mover/me...

The test looks correct, perhaps there needs to be a cast to integer? 

              dt("The version of the !name module found on this platform (!versionB) has a lower Schema version than the one the site has installed (!versionA)",

The message also doesn't match what you pasted. Has this changed ?

anarcat’s picture

anarcat commented
Status: Active » Postponed (maintainer needs more info)

I'm not sure we're still having the issue, I'll postpone this until we can reproduce.

The message has changed while trying to diagnose this exact issue: I added the explicit versions in the output to help clarify this. When running with the above code, the version *was* higher up than on the target platform, so maybe we had installed a local more recent version at some point.

Until we can reproduce, feel free to close this.

j0nathan’s picture

j0nathan commented
Status: Postponed (maintainer needs more info) » Active
StatusFileSize
new migrate-02.png77.09 KB
new migrate-01.png33.89 KB

Hi,

Last time I deleted the module media_mover_api from the drupal site to be able to migrate it to a new platform and enabling it after. This is a test site with all our modules enabled, so I didn't have to reconfigure it after.

This time the issue is happening again, but with the module schema I've got more information.

Note that I verified both platforms and the site through aegir and I run a drush updatedb on the site before trying to migrate it.

Going through the process, I noted this below and 2 screenshots...

$ drush updatedb
No database updates required                                         [success]
Finished performing updates.                                         [ok]

admin/reports/status

Database schema	Inconsistent
The Schema comparison report shows:

    * 89 modules with matching tables
    * 1 extra tables
    * 3 warnings
    * 3 module with mis-matching tables

The database schema comparison report provides more details.

admin/build/schema/compare

# media_mover_files.mmfid is part of the primary key but is not specified to be 'not null'.
# media_mover_config_list.cid is part of the primary key but is not specified to be 'not null'.

media_mover_api

    *
      media_mover_files
          o column mmfid - difference on: not null
            declared: array('description' => t('TODO: please describe this field!'), 'type' => 'serial')
            actual: array('description' => t('TODO: please describe this field!'), 'type' => 'serial', 'not null' => TRUE)
    *
      media_mover_config_list
          o column cid - difference on: not null
            declared: array('description' => t('TODO: please describe this field!'), 'type' => 'serial')
            actual: array('description' => t('TODO: please describe this field!'), 'type' => 'serial', 'not null' => TRUE)

I am not able to migrate the site.

j0nathan’s picture

j0nathan commented
Status: Active » Postponed (maintainer needs more info)

I was able to migrate the site with media mover from a platform on Drupal 6.15 to another platform on Drupal 6.16. I don't know why it worked this time.

Until we can reproduce, feel free to close this.

anarcat’s picture

anarcat commented
Status: Postponed (maintainer needs more info) » Fixed

The problem was with some of the hook_update() in the .install file of media_mover_api that weren't following the coding standards:

function media_mover_api_update_6101() {
[...]
function media_mover_api_update_6110 () {

Notice the extra space before the parenthesis. I fixed the regex so it's more tolerant.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

  • Commit 9c6ddea on debian, dev-dns, dev-envobject, dev-koumbit, dev-log_directory, dev-migrate_aliases, dev-multiserver-install, dev-newhooks, dev-nginx, dev-ports, dev-purgebackup, dev-restore, dev-services, dev-simplerinstaller, dev-site_rename, dev-ssl, dev_server_verify, prod-koumbit, dev-ssl-ip-allocation-refactor, dev-1205458-move_sites_out_of_platforms, 7.x-3.x, dev-subdir-multiserver, 6.x-2.x-backports, dev-helmo-3.x by anarcat: 
    #685076 - be more tolerant on hook_update() function names so that...