Node listing is empty for entire vocabulary

If you request a node listing for all terms within a vocabulary using (for example):

• http://localhost/taxonomy_menu/7
• http://localhost/category/3

you end up with a "no postings in this categoy" message.

Changing the logic of line 283 in taxonomy_menu.inc seems to resolve the problem. If you are using module page names (like "category" or "taxonomy_menu") in the node list menu path then the existing logic incorrectly picks up this module page name as the vid instead of the actual vid number (arg(0) instead of arg(1)). The short-circuit boolean logic seems a bit suspect too but I didn't bother proving it. The change I made is:

Index: taxonomy_menu.inc
===================================================================
--- taxonomy_menu.inc	(revision 944)
+++ taxonomy_menu.inc	(working copy)
@@ -280,7 +280,7 @@
  */
 function _taxonomy_menu_page() {
   // Check if the Vocabulary ID is set
-  if (($vid = arg(0)) || ($vid=variable_get('taxonomy_menu_hide_module_page', FALSE)?arg(0):FALSE)) {
+  if (($vid=variable_get('taxonomy_menu_hide_module_page', FALSE)?arg(0):FALSE) || ($vid=arg(1))) {
 
     // Depending on what Output technique is used,
     // show the nodes' list

I've tested this and it seems to work now with or without module page name enabled in the path.

Hope that helps someone or a maintainer.