Inserting "nid = 0" into db during node add

Ran into a problem on node form add: there is no nid assigned, so when you save the node the restrictions don't get saved along with it. The node doesn't exist, so at that point in the form it has no nid -- a value of nid=0 was being added to {restricted content}.

What the user would see is the access restriction just not being saved. They'd have to go back in and edit the saved node, and then set the access again before it would "take." I don't have time to fix it properly, so all I did was insert this:

function restricted_content_form_alter(&$form, $form_state, $form_id) {
+ $nid = $form['nid']['#value'];

// ...and then I added isset($nid)

- elseif ($form['#id'] == 'node-form') {
+ elseif ($form['#id'] == 'node-form' && isset($nid)) {

Basically what this means is that the Restricted Access form just doesn't show up AT ALL on node/add, only on node/edit. In terms of usability it at least stops the confusion and gets rid of the creation of dud db entries, but it means you have to tell users to go back in. You'd have to tell them that anyway to get it to work, but at least this way they won't somehow think they've already done it.

(I didn't roll a patch because of the weird patch problems earlier in the queue to do with creating defaults per content type. But this should be enough to help those who are going wha?? :)