Scheduler does not publish nodes

Thanks. Issued fixed.

Let me give you more details. I assigned a taxonomy term (e.g. HR) to this node and added a published and unpublished time.

I encountered 2 issues:

1) The node was published successfully. But it did not show up on HR page. The node status was "published" under "Content Management > Content", but since the taxonomy term was lost after running cron, it did not show up on the HR page. Is this a bug?

2) After #1, the same node failed to unpublish itself. If I re-enter a "unpublished on", the node will unpublish. It looks to me that Scheduler module works only when either "published on" or "unpublished on" is entered. NOT BOTH. Please clarify.

Thanks.

Thank you. I have modified some of the codings in "scheduler.module" to fix these issues:

1) Issue: scheduler does not work when both "published on" and "unpublished on" are filled.

Solution: Scheduler delete the whole record from "Scheduler" table once the node was published even though "unpublished has values. In function scheduler_cron(), i changed from "== 0" to "<0":
//if this node is not to be unpublished, then we can delete the record
if ($node->unpublish_on < 0) {
db_query('DELETE FROM {scheduler} WHERE nid = %d', $node->nid);
}

2) Issue: Taxonomy term disappeared after the node was published / unpublished.

Solution: In function scheduler_cron(), it is better to update the "node" table directly instead of using node_save(). When node_save() is used, other nodeapi, (category / taxonomy for example) will also be called (http://api.drupal.org/api/HEAD/function/node_save). Since category / taxonomy does have any values under this circumstances, original value will be deleted from the term_node table.

while ($node = db_fetch_object($nodes)) {
$node = node_load($node->nid);
// $node->changed = $node->utc_publish_on;
// $node->status = 1;
// node_save($node);
db_query('UPDATE {node} SET status=1, changed=%d WHERE nid=%d', time(), $node->nid);
.
.
.
// node_save($node);
db_query('UPDATE {node} SET status=0, changed=%d WHERE nid=%d', time(), $node->nid);

Since Category / Taxonomy is now the core of Drupal 5, more and more people will have the same issue. Scheduler module might need to be modified. I am very new to Drupal and understand it is NOT GOOD change the module directly. If I can make it themeable on my template.php, please let me know. Thanks.