I'm getting following errors when saving a node:

Notice: Undefined variable: tid in _menu_to_taxonomy_save_term_without_access_control() (line 431 of mysite\sites\all\modules\menu_to_taxonomy\menu_to_taxonomy.module).
Notice: Undefined variable: tid in _menu_to_taxonomy_save_term_without_access_control() (line 442 of mysite\sites\all\modules\menu_to_taxonomy\menu_to_taxonomy.module).

I guess that $term->tid should be passed in instead of $tid, on both lines.

CommentFileSizeAuthor
#2 menu_to_taxonomy-2558653-2.patch942 bytesludo.r

Comments

dolu created an issue. See original summary.

ludo.r’s picture

Version: 7.x-1.0-beta7 » 7.x-1.x-dev
Status: Active » Needs review
StatusFileSize
new942 bytes

Here is a patch that fixes that small issue.

stefan.r’s picture

Status: Needs review » Reviewed & tested by the community

  • stefan.r committed 96d1cb0 on 7.x-1.x authored by dolu
    Issue #2558653 by dolu: Undefined variable: tid in...
stefan.r’s picture

Status: Reviewed & tested by the community » Closed (fixed)
bernardopaulino’s picture

I still have the notice "Notice: Undefined property: stdClass::$tid in _menu_to_taxonomy_save_term_without_access_control() (line 431 of /var/www/html/docroot/profiles/openfed/modules/contrib/menu_to_taxonomy/menu_to_taxonomy.module)." when I create a menu link through the menu administration page or through a node.

I already did some debug on this and found that the there is a missing condition to check if the $term->tid is not empty because if it is empty is doesn't make any sense resetting the cache for a term that does not exist.