When creating a new CCK content type, the title and body fields do not show up at admin/content/node-type/(content type). Also, if no custom fields are entered, Title and Body do not show up at admin/content/node-type/(content type)/fields. This happens initially and when all custom fields are deleted.

CommentFileSizeAuthor
#12 jamesan_270685-12.patch645 bytesJamesAn
#4 jamesan_270685.patch673 bytesJamesAn

Comments

aren cambre’s picture

aren cambre commented
Title: Title and Body fields do not show up properly » Problems with appearance of Title and Body fields in content type editor
Version: 6.x-2.x-dev » 6.x-2.0-beta

Oops, now I see that they are in the collapsed the Submission form section.

I am leaving this as a bug because, in light of the fact that title and body are so essential, I don't understand why Submission form section is collapsed by default. I just double-checked my Drupal 5 system, and that section is expanded by default.

Also, it seems that, regardless of the presence of CCK-generated fields, title and body should always show up at admin/content/node-type/(content type)/fields.

yched’s picture

yched commented
Project: Content Construction Kit (CCK) » Drupal core
Version: 6.x-2.0-beta » 7.x-dev
Component: content.module » node system

Those pages are handled by core's node.module

karschsp’s picture

karschsp commented
Issue tags: +Novice

tagging for the novice queue. just need to change '#collapsed' from TRUE to FALSE in function node_type_form in content_types.inc and roll the patch.

JamesAn’s picture

JamesAn commented
StatusFileSize
new jamesan_270685.patch673 bytes

Voila!

JamesAn’s picture

JamesAn commented
Status: Active » Needs review

Oops. Forgot to flip the status. ^^"

Status: Needs review » Needs work

The last submitted patch failed testing.

JamesAn’s picture

JamesAn commented
Status: Needs work » Needs review

It passed before. Retesting..

Someone care to review this for submission?

xmacinfo’s picture

xmacinfo
he/him
Primary language French
Location Canada
commented
Status: Needs review » Reviewed & tested by the community

That's was an easy one. Thanks. RTBC

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

There's a lot of work going on right now in #372743: Body and teaser as fields to pull these out and make them actual fields which could be managed in CCK along with everything else. In the meantime though, I agree that this is a nice improvement since it's nearly impossible for anyone to find these settings otherwise.

Committed to HEAD. Thanks!

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

'#collapsed' => FALSE is the default value for fieldsets, so we could have as well just removed the now superfluous line. Most of core relies on this default.

xmacinfo’s picture

xmacinfo
he/him
Primary language French
Location Canada
commented

@sun: Can you create a patch to remove the superfluous? :-)

JamesAn’s picture

JamesAn commented
Status: Fixed » Needs review
StatusFileSize
new jamesan_270685-12.patch645 bytes

Thanks for catching that, sun.

I've created a patch to remove it altogether.

catch’s picture

catch
he/him
Primary language English
commented
Status: Needs review » Reviewed & tested by the community
webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Committed to HEAD. Thanks!

Status: Fixed » Closed (fixed)
Issue tags: -Novice

Automatically closed -- issue fixed for 2 weeks with no activity.

Issue tags: +Novice

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.