Closed (fixed)
Project:
Recipe
Version:
6.x-0.1
Component:
Code
Priority:
Critical
Category:
Support request
Assigned:
Unassigned
Reporter:
Created:
6 Mar 2008 at 23:38 UTC
Updated:
4 May 2008 at 20:12 UTC
I tried the experimental code available at your external site, and found that it would not create new ingredients. I do not profess to be much of a Drupal programmer.
However, it looks like the problem is in recipe_ingredient_id_from_name($name), at the point where it tries to set $node_link. This is the point in the code where we've determined that the ingredient is not in the database. I do not understand why $node_link needs to be set, or why it needs to go into the link field, but it is clear that it needs to be set to something real. A null value fails.
Comments
Comment #1
N.Levy commentedIt has been reccommended that users add the following line:
if (empty($node_link)) { $node_link=0; }
on line 559
Comment #2
sunfish62 commentedThat patch does fix the problem. Is it actually the *right* fix?
Comment #3
marble commentedI believe so. I think we probably need to get this version into cvs so that we can keep track of stuff like this.
Comment #4
brdwor commentedchanged version issue is marked against.
Comment #5
brdwor commentedcommitted to 6.x-1.x-dev.
Comment #6
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.