Like stated in the documentation:

Return value
This varies depending on the operation.

The "presave", "insert", "update", "delete", "print" and "view" operations have no return value.
The "load" operation should return an array containing pairs of fields => values to be merged into the node object.

That is not what the module does, which uses code like:

  case 'load':
      $node->nodewords = _nodewords_load('node', $node->nid);
      break;

Comments

vladimir.dolgopolov’s picture

vladimir.dolgopolov commented

btw, path.module (a Core module!) has the same behaviour:

      case 'load':
        $path = 'node/'. $node->nid;
        $alias = drupal_get_path_alias($path, $language);
        if ($path != $alias) {
          $node->path = $alias;
        }
        break;
avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented

Yes, but it's a core module; if there would be a change on the way that hook is handled, the core module would be updated as well.
That is different from the case where a third party module has code that could be incompatible with future versions of Drupal; as the author of the module doesn't make new versions of the Drupal core, there would be some incompatibilities with new versions of Drupal. That is what the Drupal documentation warns about.

Robrecht Jacques’s picture

Robrecht Jacques commented
Status: Active » Fixed

Fixed in CVS. Thanks!

Status: Fixed » Closed (fixed)

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