Issue #2659262 by tusharbodke, Denchev, slashrsm, Berdir, zerolab, gaurav.bajpai: Undefined function drupal_get_schema() in file_entity.module

Author: git

file_entity.module

@@ -282,226 +282,6 @@ function file_entity_view_mode_label($view_mode, $default = FALSE) {
   return isset($labels[$view_mode]) ? $labels[$view_mode] : $default;
 }
 
-/**
- * @defgroup file_entity_access File access rights
- * @{
- * The file access system determines who can do what to which files.
- *
- * In determining access rights for a file, file_entity_access() first checks
- * whether the user has the "bypass file access" permission. Such users have
- * unrestricted access to all files. user 1 will always pass this check.
- *
- * Next, all implementations of hook_file_entity_access() will be called. Each
- * implementation may explicitly allow, explicitly deny, or ignore the access
- * request. If at least one module says to deny the request, it will be rejected.
- * If no modules deny the request and at least one says to allow it, the request
- * will be permitted.
- *
- * There is no access grant system for files.
- *
- * In file listings, the process above is followed except that
- * hook_file_entity_access() is not called on each file for performance reasons
- * and for proper functioning of the pager system. When adding a filelisting to
- * your module, be sure to use a dynamic query created by db_select()
- * and add a tag of "file_entity_access". This will allow modules dealing
- * with file access to ensure only files to which the user has access
- * are retrieved, through the use of hook_query_TAG_alter().
- *
- * Note: Even a single module returning FILE_ENTITY_ACCESS_DENY from
- * hook_file_entity_access() will block access to the file. Therefore,
- * implementers should take care to not deny access unless they really intend to.
- * Unless a module wishes to actively deny access it should return
- * FILE_ENTITY_ACCESS_IGNORE (or simply return nothing)
- * to allow other modules to control access.
- *
- * Stream wrappers that are considered private should implement a 'private'
- * flag equal to TRUE in hook_stream_wrappers().
- */
-
-/**
- * Implements hook_query_TAG_alter().
- *
- * This is the hook_query_alter() for queries tagged with 'file_access'. It adds
- * file access checks for the user account given by the 'account' meta-data (or
- * global $user if not provided).
- */
-function file_entity_query_file_access_alter(SelectInterface $query) {
-  _file_entity_query_file_entity_access_alter($query, 'file');
-}
-
-/**
- * Helper for file entity access functions.
- *
- * @param $query
- *   The query to add conditions to.
- * @param $type
- *   Either 'file' or 'entity' depending on what sort of query it is. See
- *   file_entity_query_file_entity_access_alter() and
- *   file_entity_query_entity_field_access_alter() for more.
- */
-function _file_entity_query_file_entity_access_alter($query, $type) {
-  $user = \Drupal::currentUser();
-
-  // Read meta-data from query, if provided.
-  if (!$account = $query->getMetaData('account')) {
-    $account = $user;
-  }
-
-  // If $account can bypass file access, we don't need to alter the query.
-  if ($account->hasPermission('bypass file access')) {
-    return;
-  }
-
-  $tables = $query->getTables();
-  $base_table = $query->getMetaData('base_table');
-  // If no base table is specified explicitly, search for one.
-  if (!$base_table) {
-    $fallback = '';
-    foreach ($tables as $alias => $table_info) {
-      if (!($table_info instanceof SelectInterface)) {
-        $table = $table_info['table'];
-        // If the file_managed table is in the query, it wins immediately.
-        if ($table == 'file_managed') {
-          $base_table = $table;
-          break;
-        }
-        // Check whether the table has a foreign key to file_managed.fid. If it
-        // does, do not run this check again as we found a base table and only
-        // file_managed can triumph that.
-        if (!$base_table) {
-          // The schema is cached.
-          $schema = drupal_get_schema($table);
-          if (isset($schema['fields']['fid'])) {
-            if (isset($schema['foreign keys'])) {
-              foreach ($schema['foreign keys'] as $relation) {
-                if ($relation['table'] === 'file_managed' && $relation['columns'] === array('fid' => 'fid')) {
-                  $base_table = $table;
-                }
-              }
-            }
-            else {
-              // At least it's a fid. A table with a field called fid is very
-              // very likely to be a file_managed.fid in a file access query.
-              $fallback = $table;
-            }
-          }
-        }
-      }
-    }
-    // If there is nothing else, use the fallback.
-    if (!$base_table) {
-      if ($fallback) {
-        watchdog('security', 'Your file listing query is using @fallback as a base table in a query tagged for file access. This might not be secure and might not even work. Specify foreign keys in your schema to file_managed.fid ', array('@fallback' => $fallback), WATCHDOG_WARNING);
-        $base_table = $fallback;
-      }
-      else {
-        throw new Exception(t('Query tagged for file access but there is no fid. Add foreign keys to file_managed.fid in schema to fix.'));
-      }
-    }
-  }
-
-  if ($type == 'entity') {
-    // The original query looked something like:
-    // @code
-    //  SELECT fid FROM sometable s
-    //  WHERE ($file_access_conditions)
-    // @endcode
-    //
-    // Our query will look like:
-    // @code
-    //  SELECT entity_type, entity_id
-    //  FROM field_data_something s
-    //  WHERE (entity_type = 'file' AND $file_access_conditions) OR (entity_type <> 'file')
-    // @endcode
-    //
-    // So instead of directly adding to the query object, we need to collect
-    // all of the file access conditions in a separate db_and() object and
-    // then add it to the query at the end.
-    $file_conditions = db_and();
-  }
-  foreach ($tables as $falias => $tableinfo) {
-    $table = $tableinfo['table'];
-    if (!($table instanceof SelectInterface) && $table == $base_table) {
-      $subquery = db_select('file_managed', 'fm_access')->fields('fm_access', array('fid'));
-      $subquery_conditions = db_or();
-
-      $wrappers = file_entity_get_public_and_private_stream_wrapper_names();
-      if (!empty($wrappers['public'])) {
-        if ($account->hasPermission('view files')) {
-          foreach (array_keys($wrappers['public']) as $wrapper) {
-            $subquery_conditions->condition('fm_access.uri', $wrapper . '%', 'LIKE');
-          }
-        }
-        elseif ($account->hasPermission('view own files')) {
-          foreach (array_keys($wrappers['public']) as $wrapper) {
-            $subquery_conditions->condition(db_and()
-              ->condition('fm_access.uri', $wrapper . '%', 'LIKE')
-              ->condition('fm_access.uid', $account->id())
-            );
-          }
-        }
-      }
-      if (!empty($wrappers['private'])) {
-        if ($account->hasPermission('view private files')) {
-          foreach (array_keys($wrappers['private']) as $wrapper) {
-            $subquery_conditions->condition('fm_access.uri', $wrapper . '%', 'LIKE');
-          }
-        }
-        elseif ($account->hasPermission('view own private files')) {
-          foreach (array_keys($wrappers['private']) as $wrapper) {
-            $subquery_conditions->condition(db_and()
-              ->condition('fm_access.uri', $wrapper . '%', 'LIKE')
-              ->condition('fm_access.uid', $account->id())
-            );
-          }
-        }
-      }
-
-      if ($subquery_conditions->count()) {
-        $subquery->condition($subquery_conditions);
-
-        $field = 'fid';
-        // Now handle entities.
-        if ($type == 'entity') {
-          // Set a common alias for entities.
-          $base_alias = $falias;
-          $field = 'entity_id';
-        }
-        $subquery->where("$falias.$field = fm_access.fid");
-
-        // For an entity query, attach the subquery to entity conditions.
-        if ($type == 'entity') {
-          $file_conditions->exists($subquery);
-        }
-        // Otherwise attach it to the node query itself.
-        elseif ($table == 'file_managed') {
-          // Fix for https://drupal.org/node/2073085
-          $db_or = db_or();
-          $db_or->exists($subquery);
-          $db_or->isNull($falias . '.' . $field);
-          $query->condition($db_or);
-        }
-        else {
-          $query->exists($subquery);
-        }
-      }
-    }
-  }
-
-  if ($type == 'entity' && $file_conditions->count()) {
-    // All the file access conditions are only for field values belonging to
-    // files.
-    $file_conditions->condition("$base_alias.entity_type", 'file');
-    $or = db_or();
-    $or->condition($file_conditions);
-    // If the field value belongs to a non-file entity type then this function
-    // does not do anything with it.
-    $or->condition("$base_alias.entity_type", 'file', '<>');
-    // Add the compiled set of rules to the query.
-    $query->condition($or);
-  }
-}
-
 /**
  * Implements hook_file_download().
  */
@@ -523,10 +303,6 @@ function file_entity_file_download($uri) {
   return -1;
 }
 
-/**
- * @} End of "defgroup file_entity_access".
- */
-
 /**
  * @name pathauto_file Pathauto integration for the core file module.
  * @{