#1734648: Media 2.x branch missing dependencies outlined that the dependency generation used by drupal.org testbots does not yet respect versioned dependencies and will ignore them.

Media should use an unversioned dependency on File Entity to facilitate testing on d.o.

I believe that this blocks all testing issues including #1733960: Cleanup existing tests.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Devin Carlson’s picture

Status: Active » Needs review
FileSize
369 bytes

The attached patch changes the versioned dependency on file_entity to an unversioned dependency.

aaron’s picture

Status: Needs review » Reviewed & tested by the community

maybe we can implement hook_requirements instead. I will leave that for another issue though, as I see no reason to hold up testing further over that.

ParisLiakos’s picture

Status: Reviewed & tested by the community » Fixed

thanks, commited

Dave Reid’s picture

Priority: Normal » Major
Status: Fixed » Needs work

We need to have a hook_requirements check as a critical follow-up. This is one of the major things that bites people on the upgrade path from 1.x to 2.x.

ParisLiakos’s picture

Assigned: Devin Carlson » ParisLiakos
Status: Needs work » Needs review
FileSize
1.16 KB

Something like this

Devin Carlson’s picture

Devin Carlson’s picture

Title: Use unversioned dependency on File Entity to enable automated testing on drupal.org » Add check for File entity 2.x to installation requirements

Updated title.

ParisLiakos’s picture

Status: Needs review » Fixed

swapped t() with get_t() and commited this.
see
http://drupalcode.org/project/media.git/commit/3f4c786

ParisLiakos’s picture

Status: Fixed » Needs work

http://drupal.org/node/1496624#comment-6667974

maybe i should remove this check on install and leave it only on update...it actually makes sense..

jmpease’s picture

I'm trying an install using the 2.x media_dev profile and am getting a "The specified database connection is not defined: default" notification in the "Verify requirements" stage. It appears to be related to http://drupalcode.org/project/media.git/commit/3f4c786 as commenting out the hook_requirements impl allows me to continue with the install.

jhedstrom’s picture

Status: Needs work » Needs review
FileSize
469 bytes

This change also broke installation profiles that attempt to enable media during site installation.

WD php: PDOException: SQLSTATE[42S02]: Base table or view not found: 1146 Table 'hf.system' doesn't exist: SELECT * FROM   [error]
{system} WHERE type = 'theme' OR (type = 'module' AND status = 1) ORDER BY weight ASC, name ASC;

Changing to only check on update fixes the issue for me.

ParisLiakos’s picture

Assigned: ParisLiakos » Unassigned
Status: Needs review » Fixed

yeap:)
committed #11 thanks for the patch
http://drupalcode.org/project/media.git/commit/83c903d

knalstaaf’s picture

Status: Fixed » Needs review

I'm still getting this error:

Failed: PDOException: SQLSTATE[42S02]: Base table or view not found: 1146 Table 'sitename1.file_type' doesn't exist: INSERT INTO {file_type} (type, label, description) VALUES (:db_insert_placeholder_0, :db_insert_placeholder_1, :db_insert_placeholder_2); Array ( [:db_insert_placeholder_0] => [:db_insert_placeholder_1] => [:db_insert_placeholder_2] => ) in file_type_save() (regel 541 van /home/usr1/sitename/public_html/sites/all/modules/file_entity/file_entity.file_api.inc).

I used the patch of #5 (above) and the File Entity patch of #175 in #1553094: Alt and Title support for Images.

I'm also using this patch, but I've got no idea if it's of any relevance in this case (#1244476: Add an "Override image style" setting to the rendered file formatter).

The module versions:

  • Media 7.x-2.x-dev
  • File entity 7.x-2.x-dev

Can someone help me out please?

Edit [10-11-2012]: my apologies for the (unintended) hijack of this thread. The solution to this issue can be found here.

ParisLiakos’s picture

Dont hijack fixed issues for support requests.please open a new issue

Status: Needs review » Closed (fixed)

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