Hi

7.x-2.0-unstable7 and 7.x-2.x-dev are both giving me the following errors, even with Media module disabled:

Reports - Status report:
Fatal error: Unsupported operand types in .../modules/file_entity/file_entity.module on line 455

Admin pages:
Notice: Undefined index: view modes in file_entity_entity_info_alter() (line 442 of .../modules/file_entity/file_entity.module)

I have tried upgrading Ctools to latest dev too.

Thanks

CommentFileSizeAuthor
#5 file_entity-view_modes_fatal_error-1885936-5.patch1.13 KBParisLiakos

Comments

Jason Dean’s picture

Jason Dean commented

I downgraded to 7.x-2.0-unstable6 and the problem goes away :)

gmclelland’s picture

gmclelland commented
Version: 7.x-2.0-unstable7 » 7.x-2.x-dev

I'm getting the same problems with the latest dev versions of media-2.x and file_entity-2.x.

Notice: Undefined index: view modes in file_entity_entity_info_alter() (line 442 of /Users/glenn/websites/7c9d097f-0bc1-457d-8314-1d9e37f102bf/profiles/cmf/modules/contrib/file_entity/file_entity.module). (x2)
ParisLiakos’s picture

ParisLiakos commented
Priority: Normal » Critical

we need to check view modes if is set before using += on it.
this bug was introduced here http://drupalcode.org/project/file_entity.git/blobdiff/70204d56f49d97efc...

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

Or just define them instead of using a merge (like the previous version just goes and defines 'full', use the same syntax for the three view modes).

ParisLiakos’s picture

ParisLiakos commented
Status: Active » Needs review
StatusFileSize
new file_entity-view_modes_fatal_error-1885936-5.patch1.13 KB

patch

gmclelland’s picture

gmclelland commented

Patch in #5 applied cleanly and I don't see any errors.

ParisLiakos’s picture

ParisLiakos commented
Status: Needs review » Fixed

I committed this
http://drupalcode.org/project/file_entity.git/commit/4a82014
thanks for testing gmclelland

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

(not verified) commented
Issue summary: View changes

Make clearer