not working for me- special case with UC Conditional Payments & UC Node Published modules

**BEGIN EDIT: update: it looks like the "Publish node on full payment" predicate comes from the uc_node_checkout_scheduler module which i also have installed. I've posted this issue there as well for advice...

http://drupal.org/node/1180236

it creates the "Action: Set Node Publish and Unpublish Dates"
which is to "Update the node status to Published."
but doesn't for me as per the above issue and conditions i've added.
**END EDIT:

alright, after some time going through the code here's what i did and it seems to be working for me.

I believe this was happening because of the Node Published module actually, and the CA predicate i was trying to modify didn't actually do any publishing of the node upon checkout (thus i believe the name of the action "Update the node status to Published" is misleading, at least not in the context of initial checkout of the node checkout node(?), but i might not be completely understanding the big picture here)

In uc_node_published.module from the Node Published module, i edited the uc_node_published_order function as follows, which accomplishes what i initially wanted to do.

Around line 143

function uc_node_published_order($op, &$arg1, $arg2) {
  switch ($op) {
    case 'update':

/*			if ($uid = 1)  {  //added by SchwebDesign
				dargs(); //added by SchwebDesign
			} //added by SchwebDesign
*/
     // BEFORE SchwebDesign EDIT:  if ($arg2 == 'completed') {
      if (($arg2 == 'completed') || ($arg2 != 'completed' && $arg1->payment_method == 'other')) {    //AFTER SchwebDesign Edit
        foreach ($arg1->products as $item) {
          $node = node_load($item->data['node_checkout_nid']);
          $types = array_keys(uc_node_checkout_product_map());

          // Make sure we only affect uc_node_checkout related items.
          if (in_array($node->type, $types)) {
            // Publish the node.
            $node->status = 1;
            node_save($node);

            watchdog(
              'uc_node_published', 'Set @type %title to published.',
              array(
                '@type' => node_get_types('name', $node),
                '%title' => $node->title,
              )
            );
          }
        }
      }

If anyone has any advice on a better way to do this, that would be greatly appreciated. Otherwise, i hope this helps somebody.