Just a little heads up: I think on the node/add initial page (content type selection), the content types should be sorted by the 'name' value, not the label. I tried customising the module myself, but I don't understand what a stdClass is and it's making things difficult.

On the Content Type Administration page, they are correctly sorted by name.

Just about everything in the UI is sorted by the displayed name, regardless of the backend label.

So this is somewhere between bug report/feature request.

CommentFileSizeAuthor
#3 208023-3.patch770 bytestraxer
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

aeronox’s picture

aeronox CreditAttribution: aeronox commented

I realise this isn't really an API thing - it's a node module issue. But there's no project for Node.

I found the simple answer:
node.module line 2214

$item[$type->type] = $out;

to

$item[$type->name] = $out;

.. lucky I spotted it because I was about to give up on all the array & object sorting stuff.

ax’s picture

ax CreditAttribution: ax commented
Project: API » Drupal core
Version: 5.x-1.0 » 5.x-dev
Component: Code » node.module

the "project for Node" is Drupal ...

traxer’s picture

traxer CreditAttribution: traxer commented
Status: Active » Needs review
FileSize
208023-3.patch770 bytes

Confirmed. Patch implements solution from aeronox. Drupal 6 is not affected.

drumm’s picture

drumm
he/him
Location NY, US
CreditAttribution: drumm commented
Status: Needs review » Fixed

Committed to 5.x.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Fixed » Closed (fixed)

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