The 7.x-3.x-dev release of media_theplatform_mpx needs to be using the 7.x-2.x versions of both file_entity and media. The correct version of file_entity is used, but not the correct version of media:

Release node:
https://www.drupal.org/node/2248049

Currently defined dependencies:

dependencies[] = media (2.x)
dependencies[] = media_internet
dependencies[] = file_entity (2.x)
dependencies[] = encrypt (2.x)
dependencies[] = elements
test_dependencies[] = file_admin

Current checkouts from qa.drupal.org:

    'dependencies' => array(
      0 => array(
        'repository' => array(
          'type' => 'git',
          'url' => 'git://git.drupal.org/project/drupal.git',
        ),
        'vcs_identifier' => '7.x',
      ),
      1 => array(
        'repository' => array(
          'type' => 'git',
          'url' => 'git://git.drupal.org/project/file_entity.git',
        ),
        'vcs_identifier' => '7.x-2.0-beta1',
      ),
      2 => array(
        'repository' => array(
          'type' => 'git',
          'url' => 'git://git.drupal.org/project/ctools.git',
        ),
        'vcs_identifier' => '7.x-1.7',
      ),
      3 => array(
        'repository' => array(
          'type' => 'git',
          'url' => 'git://git.drupal.org/project/media.git',
        ),
        'vcs_identifier' => '7.x-1.5',
      ),
      4 => array(
        'repository' => array(
          'type' => 'git',
          'url' => 'git://git.drupal.org/project/elements.git',
        ),
        'vcs_identifier' => '7.x-1.4',
      ),
      5 => array(
        'repository' => array(
          'type' => 'git',
          'url' => 'git://git.drupal.org/project/encrypt.git',
        ),
        'vcs_identifier' => '7.x-2.0',
      ),
      6 => array(
        'repository' => array(
          'type' => 'git',
          'url' => 'git://git.drupal.org/project/file_admin.git',
        ),
        'vcs_identifier' => '7.x-1.0-beta8',
      ),
    ),
  ),

Comments

trobey’s picture

trobey CreditAttribution: trobey commented
Status: Active » Fixed

The problem is the media_internet module depends on the media module. This is getting processed first and it does not have a version requirement. So when the media dependency is processed Project Dependency sees that it already is in the list of dependencies and skips to the next dependency. Since the media_internet module is in the media project the version requirement should also apply to it. So a better way to specify the dependencies is:

dependencies[] = media_internet (2.x)
dependencies[] = file_entity (2.x)
dependencies[] = encrypt (2.x)
dependencies[] = elements
test_dependencies[] = file_admin

There is no need to specify the media module as a dependency since the media_internet module already specifies it.

It looks like the version requirement to media_internet already was added yesterday so this should already be working and I will mark the issue as fixed.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Fixed » Active

Hrm, while I did add the version dependency to media_internet, the test dependencies are still out of date.

trobey’s picture

trobey CreditAttribution: trobey commented

Okay, I will have to update my local copy so I can see why this fix is not taking care of the problem.

  • trobey committed 14c84f0 on 7.x-1.x 
    Issue #2474703: Exclude components during dependency calculation.
trobey’s picture

trobey CreditAttribution: trobey commented

There was an obscure bug. The project for the release was being removed at the end of the dependency calculation but this is one of those rare cases where it needs to be excluded during the calculation. I have committed the changes and will create an updated release.

trobey’s picture

trobey CreditAttribution: trobey commented
Category: Support request » Bug report

Release 7.x-1.0-beta4 needs to be installed on drupal.org.

  • trobey committed 14c84f0 on 7.x-2.x 
    Issue #2474703: Exclude components during dependency calculation.