Question

What am I doing wrong. I can’t seem to pull based upon content type fields in drupal 7.

function ycs_list($number) {
  $query = db_select('field_data_field_active_image', 'a')
  ->fields('a', array('field_active_image_value', 'entity_id'))
  ->condition('a.field_active_image_value', 1);
  $query->join('node', 'n', 'n.nid = a.entity_id');
  $query
  ->fields('n', array('nid', 'title', 'uid'))
  ->range(0, $number)
  ->addTag('node_access')
  ->execute();
  print $query;
  return $query;
}

This is how the query prints:

SELECT a.field_active_image_value AS field_active_image_value, a.entity_id AS entity_id, n.nid AS nid, n.title AS title, n.uid AS uid FROM {field_data_field_active_image} a INNER JOIN {node} n ON n.nid = a.entity_id WHERE (a.field_active_image_value = :db_condition_placeholder_0) LIMIT 3 OFFSET 0

It looks right and does work directly in mysql. I have to change :db_conditon_placehoder_0 to 1 and it works doing a direct sql query. I want to pull an array of nodes based upon a condition in the active_image field. Any help would be much appreciated.

Answer 1

Without knowing the actual error you’re getting it’s a bit tricky to debug your existing code, but I’d advise using the EntityFieldQuery class to do this instead:

function ycs_list($number) {
  $query = new EntityFieldQuery;
  $query->entityCondition('entity_type', 'node')
    ->entityCondition('bundle', 'name_of_node_type') // Remove this line if you don't want to specify a particular node type for the query.
    ->fieldCondition('field_active_image', 'value', 1)
    ->range(0, $number)
    ->addTag('node_access');

  // Execute the query and check for results.
  $results = $query->execute();
  $nodes = array();
  if ($results && isset($results['node'])) {
    // Load the node objects based on the node ids returned from the query.
    $nodes = node_load_multiple(array_keys($results['node']));
  }

  // If nodes matching your conditions were found you now have an array of fully loaded node objects in $nodes.
}