Closed (fixed)
Project:
Flexinode
Version:
master
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
3 Feb 2006 at 21:03 UTC
Updated:
13 May 2006 at 18:31 UTC
Jump to comment: Most recent file
I post code examples on my site
Sometimes posts are as short as:
------
function MyFunc() example:
<code>
// a lot of code here
</code>
------
When teaser is generated, only opening tag fits into allowed length. Therefore all the front page below looks weird.
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | field_textarea.inc_1.patch | 875 bytes | samc |
| #6 | field_textarea.inc_0.patch | 647 bytes | bryan kennedy |
| #2 | field_textarea.inc.patch | 934 bytes | seanr |
Comments
Comment #1
bryan kennedy commentedI am experiencing this as well. Flexinode is choping any of my code imcluding image tags right in half at what seems to be an arbitrary point. It is not acknowledging the teaser length set by the whole drupal site.
Comment #2
seanrThis patch fixes field_textarea.inc to generate the teaser using the proper node_teaser function. It was prevously hardcoded using the substr. I have tested this patch on my system and it works perfectly as far as I can tell.
Comment #3
seanrforgot to change the status (done).
Comment #4
bryan kennedy commentedThis patch failed for me. I ran:
and got these errors:
Any ideas?
Comment #5
bryan kennedy commentedOoops. I made a mistake in the above post. I assumed the above messages were an error. But I realized that the patch did come through okay. I tested this patched version of the module and it worked for me running 4.7rc3.
It fixed the problem with the teasers not following the overall Drupal settings.
+1
Comment #6
bryan kennedy commentedSo while the patch above did fix the problem it had some errors in it and was technically against 4.6.
Here is a patch made against the current CVS version of flexinode that shouldn't give any errors.
Comment #7
samc commentedCouldn't get the patch to apply, but I did it manually and it tests out great.
A much needed fix for flexinode. Thanks all.
Rerolled patch attached.
Comment #8
Bèr Kessels commentedCommitted. Thanks.
Comment #9
(not verified) commented