Closed (fixed)
Project:
Drupal core
Version:
4.6.7
Component:
node.module
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
8 Apr 2006 at 17:35 UTC
Updated:
14 Nov 2007 at 14:25 UTC
Jump to comment: Most recent
Comments
Comment #1
edrex commented+1
This is still broken in 4.7. It affects me strongly because most of my sites custom theme nodes. All I get is the teaser in the node preview, because $page==0 on preview pages.
Comment #2
robertgarrigos commentedThis have been solved for the last version of head (4.7! nor 4.6.6): // $Id: node.module,v 1.630 2006/04/11 11:33:15 killes Exp $
4.6.6 is still having this problem. We might have to wait until 4.7 will be released as all the forces are concentrated on it. Until then use this patch.
Comment #3
FlemmingLeer commentedBut adding the post with the change above the page blanks. Hitting F5 or equivelant reposts the node content and still results in blank page.
It happens for both blog and article nodes.
But title did show up now after all.
I manually upgraded from 4.6.6 to 4.6.7.
I got this error message:
Cannot modify header information - headers already sent by (output started at /customers/denmarkonline.dk/denmarkonline.dk/httpd.www/modules/node.module:1) i /customers/denmarkonline.dk/denmarkonline.dk/httpd.www/includes/common.inc på linje 192.Comment #4
gregglesAs you can see in http://drupal.org/node/52586 it is recommended to use drupal_clone instead of clone
Comment #5
gregglesI take it back - drupal_clone is not available in 4.6 http://api.drupal.org/api/4.6/function/drupal_clone (gives a 404)
Looking at the common.inc in 4.6 it implements it's own clone function if running on php< 5.0 so I believe the original patch is RTC. It didn't apply for me using "-p0" and I had to find the file manually, robertgarrigos the advice in the first comment on http://drupal.org/diffandpatch is handy on that to make the files quick to apply for reviewers.
Comment #6
gregglesSorry, my use of "php > 5" messed up my post. The missing part says how I tested this patch, it worked fine but uses a very long path structure that is not standard. I feel this is RTC since it's the same solution that made into 4.7.
Comment #7
FlemmingLeer commentednp, greggles
I have to add that I did not patch it. I use windows. But the hosted server is running php5, phpMyAdmin 2.5.7-pl1 and MySQL 4.0.24.
I added the patch to line 1514, which is the correct line in 4.6.7 node.module
Thanks for looking in to it.
Comment #8
dries commentedIsn't clone() a PHP 5 only function? (I.e. not available in PHP 4.)
Comment #9
gregglesDries - from the drupal-4-6 revision of common.inc http://cvs.drupal.org/viewcvs/drupal/drupal/includes/common.inc?annotate...
I believe that should make this patch work for all 4.6.x installations.
Comment #10
FlemmingLeer commentedno it does not.
It blanks and gives this error message:
Also i disabled Htmlarea module but even after that I get this error message. (Plus updated the html area module)
I´m going back to original 4.6.7. node.module
Any ideas ?
Comment #11
FlemmingLeer commentedCan banner.module affect this ?
And is there anyway to dump all the modules I use for your information ?
Comment #12
greggles-FlemmingLeer - I don't think your problems are related to this particular issue. Please open a new forum topic or issue to cover the error messages that you are seeing.
I tried this patch on a fresh installation of Drupal4.6.7 using both PHP4.4.2 and PHP5.1.2. In both cases it worked as expected and did not create any errors on the screen or in the Drupal log. I believe this is RTC.
Comment #13
dries commentedCommitted to DRUPAL-4-6. Thanks.
Comment #14
(not verified) commentedComment #15
Anonymous (not verified) commentedStrange, i still get this bug in Drupal 5.3, when the text goes over the teaser limit, it wont show... and it's not even in the database too..
I had a look at the node.module and noticed it is, in fact, using drupal_clone() but only in one of them. I tried applying it to all but still didn't work
Comment #16
greggles@xushi - your problem sounds like a new issue. Please open a new bug for it and mark the version as 5.3.