Add a "Label" field to the Bundle edit form. This patch is against 2.x. It looks to me like the need for this applies to the 3.x branch as well at the moment, but it may end up falling under larger configuration changes.

CommentFileSizeAuthor
#1 eck-edit-bundle-label-2337637-1.patch871 bytesbrad.bulger
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

brad.bulger’s picture

Issue summary: View changes
FileSize
871 bytes
yannickoo’s picture

It should be not possible to edit the machine readable name of a bundle? When not this works fine :)

brad.bulger’s picture

Well that's a separate question, there is precedent elsewhere for and against being able to edit that. I'm talking about the human-language "label" property, though. (Which does also get stored as "name", I guess, granted.) The value you're talking about is the "type" property.

yannickoo’s picture

Status: Active » Needs work

Cool :) Could you add a check_plain to the label?

brad.bulger’s picture

You mean on submit? I was going by how node module handles its "name" value. It applies check_plain() on display. You could make that change here, I suppose, but it seems outside the scope of this issue. Also, compare to how bundle_label is handled when creating a bundle.

yannickoo’s picture

Status: Needs work » Reviewed & tested by the community

Oh sorry, that is totally correct. This is just the saving part, nevermind – RTBC!

fmizzell’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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