I've seen some code, like

if (!$node->status) { /* warn the user to save data */ }

I think the acl module should do this because it's unclear that the form needs to be submitted twice (Add User + Submit).

Thanks,
Yonas

Comments

salvis’s picture

salvis
he/his
commented
Status: Active » Closed (won't fix)

The module that embeds the ACL form can do that if it wants. ACL is a utility module and it must avoid making choices for its clients. What may suit you may not suit others.

I see that you're a relatively new user of Drupal, and node access is one of the more complex areas. I'm not sure you're ready to tackle that yet... IAC, tread lightly and assume that things are working correctly if you're using them in the way they're intended to be used. Look at modules that do similar things and learn from them.