Add to "top" of taxonomy instead of bottom

Sounds interesting. Please post a patch so others can have a look.

Although I guess the solution works in practice it doesn't sound very efficient, especially when a taxonomy has a lot of nodes associated to it. Wouldn't it be easier to change the sort order, i.e. DESC vs. ASC? You could probably make this configurable through settings on the taxonomy page and even set this per vocabulary.

Hi there,

I'm looking forward to this update - are you still anticipating a release in June?

Thanks!

nkatt

Hallo,
this thread being quite old, I don't know whether someone will read this, but anyway...

What about just modifying the nodeapi hook in nodeorder.module:

<?php
function nodeorder_nodeapi(&$node, $op, $a3 = NULL, $a4 = NULL) {

[...]

    case 'update':
      // Set the weight_in_tid -- taxonomy probably stomped it because
      // we added the weight_in_tid column to term_node, and taxonomy
      // just wants to delete and re-insert rows when things change...
      
      // Note that we only want to set the weight_in_tid for tids that
      // are in orderable vocabularies...
      if (nodeorder_can_be_ordered($node)) {
        $tids = nodeorder_orderable_tids($node);
  
        if (count($tids) > 0) {
          $sql = "UPDATE {term_node} SET weight_in_tid = %d WHERE tid = %d AND nid = %d";
  
          foreach ($tids as $i => $tid) {
            db_lock_table('term_node');
            $weights = nodeorder_get_term_min_max($tid, FALSE); // get the cached weights
            db_query($sql, $weights["max"] + 1, $tid, $node->nid);
            nodeorder_get_term_min_max($tid, TRUE); // reinitialize the cache
            db_unlock_tables();
          } 
        }

[...]
        
  }
}?>

and replace the sql query as:

db_query($sql, $weights["min"] - 1, $tid, $node->nid);


This way a new post would be the lightest and float up in nodeorder/term/%.

AND

the function nodeorder_orderable_tids is broken, returning nothing!!!
It's because it calls nodeorder_get_tids($key, $value) with $value being a taxonomy object, and the function does not parse objects at all. Just adding get_object_vars in the function call seems to do the trick.

/**
 * Returns an array of the node's tids that are in orderable vocabularies...
 */
function nodeorder_orderable_tids($node) {
  $tids = array();
  $orderable_tids = array();
  $cid = 'nodeorder:orderable_tids:'. $node->type;
  
  if (($cache = cache_get($cid)) && !empty($cache->data)) {
    $orderable_tids = $cache->data;
  } else {
    $sql = "SELECT v.vid AS vid FROM {vocabulary_node_types} vnt JOIN {vocabulary} v ON vnt.vid = v.vid WHERE vnt.type = '%s' AND v.module = 'nodeorder'";
    $result = db_query($sql, $node->type);
  
    while ($row = db_fetch_object($result)) {
      $tree = taxonomy_get_tree($row->vid);
      foreach ($tree as $term) {
        $orderable_tids[] = $term->tid;
      }
    }
	
    //permanently cache the value for easy reuse
    cache_set($cid, $orderable_tids, 'cache');
  }

  // Now select only those tids which are actually assigned to this term
  foreach ($node->taxonomy as $key => $value) {     // $node->taxonomy is an array of objects, 
    $list_of_tids = nodeorder_get_tids($key, get_object_vars($value)); // <-- add this to make it work
    
    $tids = array_merge($tids, array_intersect($list_of_tids, $orderable_tids));
  }

  return $tids;
}

Closing as the Drupal 6 branch of this module (and Drupal 6 in general) has been discontinued.