Closed (fixed)
Project:
Project
Version:
x.y.z
Component:
Projects
Priority:
Critical
Category:
Bug report
Assigned:
Reporter:
Created:
3 Aug 2005 at 05:02 UTC
Updated:
10 Aug 2006 at 07:13 UTC
Jump to comment: Most recent file
Problem: When creating new project issues or projects, the extra form fields do not appear. (This appears to be a recent CVS issue.)
Diagnosis: node_form no longer looks for _form, but looks for _form. This breaks project.module, and may break many other modules.
Attached patch modifies node.module to look for both forms of the _form hook.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | node-project_1.patch | 1.6 KB | Eric Scouten |
| #1 | node-project_0.patch | 589 bytes | Eric Scouten |
| node-project.patch | 540 bytes | Eric Scouten |
Comments
Comment #1
Eric Scouten commentedErm... should have checked to see if the function exists. (Corrected in this version of the patch.)
Diagnosis line should read (< and > got stripped on me):
Diagnosis: node_form no longer looks for (node type name)_form, but only looks for (module name)_form. This breaks project.module, and may break other modules.
Comment #2
Eric Scouten commentedThere's more to this than I initially thought. More extensive patch coming soon...
Comment #3
Eric Scouten commentedThird time is charm? Now remapping node_form, node_invoke, and node_hook to support either (module name)_hook or (node type name)_hook.
Note: I considered patching project.module to be compatible with the new APIs. This worked until I got to hook_type_name, which doesn't pass either a node or a type string. That basically forced me into creating a backwards-compatibilty mode for node.module which is provided by this updated patch.
Comment #4
dwwthis is clearly a stale issue that never got resolved when project was ported to 4.6 or 4.7 or something. ;)