Needs review
Project:
Sections
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
19 Aug 2010 at 09:35 UTC
Updated:
27 Aug 2010 at 15:37 UTC
Jump to comment: Most recent file
Comments
Comment #1
sweetchuckThis is a simple solution.
Comment #2
hass commentedThere is no hook_content_extra_fields in core?
Comment #3
sweetchuckThe weight of the theme selector fieldset (which is added in the sections_form_alter() in line 90) is hard coded to 30.
The site administrators cannot change the position of this filedset on the form.
Apply this patch and goto ?q=admin/content/node-type/story/fields page, when CCK is installed and you will see what is the different.
hook_content_extra_fields() called by the CCK module.
Comment #4
hass commentedAnd without cck?
Comment #5
sweetchuckDrupal without CCK is like Dallas without Boby Ewing. :-)
Add a module_exists check.
Comment #6
hass commentedComment #7
sweetchuckComment #8
hass commentedI do not see the reason why we need to change this. No module I'm aware about - does this. All core modules assign a weight of 30, path_redirect and many others, too. I have never ever seen a module that works your way. Therefore won't fix.
Comment #9
sweetchuckThis implementation explain each fields of core modules for the Manage Fields page.
http://drupalcontrib.org/api/function/content_content_extra_fields/6
and here are some other modules
http://drupalcontrib.org/api/search/6/_content_extra_fields
Comment #10
sweetchuckComment #11
hass commentedI was unclear... I do not care much about cck here. Other modules work thecurrent way, too. I see no reason why we must change this as it is very uncommon.