In path_nodeapi() there's this code:

      case 'validate':
        $node->path = trim($node->path);
        if (db_result(db_query("SELECT COUNT(dst) FROM {url_alias} WHERE dst = '%s' AND src != '%s' AND language = '%s'", $node->path, "node/$node->nid", $language))) {
          form_set_error('path', t('The path is already in use.'));
        }
        break;

With E_ALL error reporting, the first line in the case throws an error:

notice: Undefined property: stdClass::$path in /drupal/modules/path/path.module on line 137.

This is going to be a problem (in 6.x, not 7.x at the moment) for the project module port, because project.module handles paths for project nodes on its own and calls unset($form['path']);.

The attached patch checks to see if $node->path is set first, which takes care of the notice.

CommentFileSizeAuthor
path_nodeapi_notice_0.patch1.17 KBaclight
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

killes@www.drop.org’s picture

Status: Needs review » Reviewed & tested by the community

Good to go

Dries’s picture

Status: Reviewed & tested by the community » Fixed

I've committed this patch to DRUPAL-6 and CVS HEAD. Thanks.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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