Posted by dman on October 24, 2010 at 12:52pm
1 follower
| Project: | Node reference field |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs review |
Issue Summary
I was trialling this out.
Got a load of PHP complaints (I run strict on dev) so I had to eliminate them before I could evaluate the rest.
Turns out that as it doesn't do secondary nodereferences or taxonomy, it won't fit what I'm building. I guess I'll look at one of the iframe solutions instead.
Anyway, here are a few improvements to stop the PHP issues in the case of "rendering fields for a node that hasn't been created yet" - empty data assumptions caused some issues. You'll only see it with E_ALL on, or a CVS checkout of core I guess.
| Attachment | Size |
|---|---|
| nodereference_field-strict_php_notices.patch | 2.64 KB |