Needs review
Project:
NodeReferrer
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
29 Aug 2007 at 18:30 UTC
Updated:
28 Nov 2007 at 05:02 UTC
"full" and "teaser" formatters have recently been added to nodereference. Rather than copy them, I created a patch that simply reuses nodereference's formatters. This is one step towards merging nodereferrer into nodereference, which I think should be our eventual goal.
Depends on http://drupal.org/node/139283
| Comment | File | Size | Author |
|---|---|---|---|
| noderef_formatters.patch | 1.1 KB | edrex |
Comments
Comment #1
dopry commentedCan you explain the reasoning behind not copying the code? Are formatters for both receiving the same data structures? Will it remain that way? Can you think of any reason it may change?
Comment #2
david lesieur commentedJust tested the patch. Works for me!
It seems to make sense to use nodereference's formatters. Actually, the only difference between nodereference and nodereferrers is that only the former owns the relationship. The rest is about handling the display of a node at one end of that relationship. At both ends of the relationship, what the formatter deals with is a node, and I don't see any reason for this to change.
Extra benefit from this patch: Consistent naming of the display options for both modules.