un-publishing nodes loses block settings

I need the ability to place nodeblocks but another user with only edit and publish permission will be editing nodeblocks and change the status.
Currently if you un-publish a nodeblock all its block settings are lost and when your re-publish the nodeblock you have to replace it in a block region and re-set its visibility settings.

To achieve what I want I made the following small changes.

Hope somebody else finds it useful.

Change 1:
nodeblock_nodeapi() – move message to only be on insert

    case 'insert':
    case 'update':
      drupal_set_message(t('The block you just created is now available on the <a href="!url">block configuration page</a>.', array('!url' => url('admin/build/block'))));

TO

    case 'insert':
      drupal_set_message(t('The block you just created is now available on the <a href="!url">block configuration page</a>.', array('!url' => url('admin/build/block'))));
      
    case 'update':

Change 2:
nodeblock_block() – changed SQL to not filter by status

  if ($op == 'list') {
    foreach ($types as $type) {
      if (nodeblock_type_enabled($type)) {
        // Fetch all nodes of this type, excluding translations.
        $result = db_query("SELECT nid, title FROM {node} WHERE type = '%s' AND status = 1 AND (nid = tnid OR tnid = 0)", $type->type);
        while ($node = db_fetch_object($result)) {
          $blocks[$node->nid] = array('info' => $node->title .' (nodeblock)');
        }
      }
    }
    return $blocks;
  }

TO

  if ($op == 'list') {
    foreach ($types as $type) {
      if (nodeblock_type_enabled($type)) {
        // Fetch all nodes of this type, excluding translations.
        $result = db_query("SELECT nid, title FROM {node} WHERE type = '%s' AND (nid = tnid OR tnid = 0)", $type->type);
        while ($node = db_fetch_object($result)) {
          $blocks[$node->nid] = array('info' => $node->title .' (nodeblock)');
        }
      }
    }
    return $blocks;
  }

Change 3:
nodeblock_block() – changed view to check status of node

  elseif ($op == 'view') {
    $node = node_load($delta);
    if (!node_access('view', $node)) {
      return;
    }

TO

  elseif ($op == 'view') {
    $node = node_load($delta);
    if (!node_access('view', $node) || !$node->status) {
      return;
    }