Placing slider under body field or other CCK fields

miraclegr - March 17, 2009 - 10:05
Project:Slider
Version:6.x-1.4
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

I've found a line of of code that makes the slider always display itself on top of other fields.
It's in slider.module, line 59:

  $node->content['slider']['#weight'] = -10;

We wanted to have the body displayed on top of the slider.
So I commented out the specific line and I was able to reorder the display of the fields as I wanted through CCK.
I see no abvious reason why this should be hard coded there.
Please, if there is something I have misunderstood correct me.
It seems that it works fine without it.

Jean-Paul

#1

Mark Theunissen - March 17, 2009 - 12:58

Jean-Paul, you're right, there's no reason for it to be there. It's no longer an issue in the new version 2 of Slider.

The problem is that I can't remove this for historical reasons - people who's sites have relied on it won't be happy!

So I suggest using a node template file to reorder the fields manually if you want to use version 1 of Slider.

Thanks
Mark

#2

Mark Theunissen - April 24, 2009 - 15:36
Status:active» closed

#3

tian.hack - August 5, 2009 - 15:22

Just to confirm: I'm using 6.x-1.4, so I can comment out this line of code without a problem? And the coming version 2 will not hard-code the weight, correct? Thanks!

#4

Mark Theunissen - August 6, 2009 - 14:29

Yep!

 
 

Drupal is a registered trademark of Dries Buytaert.