This may be a minor issue but ran through the diff from alpha2 to alpha3 and ran accross this:

--- a/html/sites/all/modules/contrib/file_entity/file_entity.module
+++ b/html/sites/all/modules/contrib/file_entity/file_entity.module
@@ -38,8 +38,8 @@ require_once dirname(__FILE__) . '/file_entity.field.inc';
  */
 function file_entity_hook_info() {
   $hooks = array(
-    'file_operation_info',
-    'file_operation_info_alter',
+    'file_operations',
+    'file_operations',
     'file_type_info',
     'file_type_info_alter',
     'file_formatter_info',

Comments

dave reid’s picture

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

Wait...what happened to hook_file_operation_info() and hook_file_operation_info_alter() and why was it changed??

dave reid’s picture

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

Strange! It looks like the API docs were removed in https://drupal.org/node/1227706#comment-5934038 but we never noticed it went missing. hook_file_operations was re-added in #1651126: Add additional filters to the File List page which also removed the "deprecated" file_entity_get_file_operation_info(). I'm really shocked we missed this.

  • Commit 20f3070 on 7.x-2.x by Dave Reid: 
    Issue #2283147 by joelpittet: Fixedduplicate hook in...
devin carlson’s picture

devin carlson commented
Version: 7.x-2.0-alpha3 » 7.x-2.x-dev
Status: Active » Fixed

The new updated hook names are now declared in file_entity_hook_info().

Marking this as fixed after checking a number of media integration modules which all used the updated hook name. Unless we want to add support for both file_operation_info() and file_operations().

Status: Fixed » Closed (fixed)

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