Closed (fixed)
Project:
Flexinode
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Anonymous (not verified)
Created:
9 Mar 2004 at 17:08 UTC
Updated:
5 May 2004 at 23:20 UTC
Jump to comment: Most recent file
A recent change to the node module (see http://drupal.org/node/view/6140) has broken 'read more' links in flexinodes . Attached patch fixes the problem.
Great work by the way.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | read_more_forced2.patch | 992 bytes | Anonymous (not verified) |
| #4 | read_more_forced.patch | 659 bytes | Anonymous (not verified) |
| #3 | read_more.patch | 608 bytes | Anonymous (not verified) |
| #1 | drupal_4 | 517 bytes | Anonymous (not verified) |
| drupal_3 | 517 bytes | Anonymous (not verified) |
Comments
Comment #1
(not verified) commentedSorry, patch didn't attach properly, so here it is...
Comment #2
jonbob commentedDoesn't this patch end up filtering multiple times? I'll look into it some more...
Comment #3
(not verified) commentedI've attached a revised patch which doesn't filter multiple times.
Comment #4
(not verified) commentedFollowing on from the previous post...
Are flexinodes supposed to always have a 'read more' link (see lines 520-521 in the 4.4.0 version of the module) or is based solely on the description field?
If they are supposed to always have a 'read more' link, the attached patch is replacement for the previous one with and additional line which forces a 'read more' link.
Comment #5
jonbob commentedThe forced "read more" link is a band-aid for the problem that without forced links, if the teaser matches the description, there is no way to get to the additional defined fields. The correct long-term solution is probably to have some system for customizing how teasers are generated from all the fields, not just the description fields. I will look at the submitted patch.
Comment #6
(not verified) commentedSorry, previous patch wasn't filtering the node's description, attached patch does...
Comment #7
killes@www.drop.org commentedApplied.
Comment #8
(not verified) commented