We introduced in #471212: No content types checked: allow all the ability to have no subtypes (= bundles) set on a flag to mean 'available on all subtypes'. The advantages being that a) you don't have to ticky all the boxes and b) you don't have to edit your flags whenever you add a new node type.

However, this has broken the second param of flag_get_flags() and thus checkboxes on node forms and the settings on node type forms for flags that have no subtypes set.

Comments

joachim’s picture

Status: Active » Needs review
StatusFileSize
new621 bytes

I'm almost disappointed this didn't mean we had to go ahead with #1699660: drop {flag_types} table; move bundle data to $flag->options...

joachim’s picture

I've filed a follow-on for this function I didn't even know existed until tonight: #1784112: move _flag_entity_enabled() to a method of flag base class.

joachim’s picture

Status: Needs review » Fixed

Issue #1784104 by joachim: Fixed flags with no content types set not appearing on node and node type forms.

Status: Fixed » Closed (fixed)

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