Hi there,
Thanks for a great module.
I updated to the latest version of devel today and found a warning for your module.

notice: Undefined variable: output in recursive_link_creator() (line 286 of /vpfa-dev/hr_drupal6_template/sites/all/modules/jquerymenu/jquerymenu.module). =>

I asked about it in devel issues and found out that

"notice: Undefined variable" used to be ignored, but it's good practice to always give your variables a value before referencing them; these notices often point to bugs, and that's why we're not suppressing them anymore.

ref: http://drupal.org/node/1000490#comment-3837610

Comments

_vid’s picture

I've got a site to roll out but I'll multi-task and try to look into submitting a patch.

In the interim, I gather from the error msg. that the function recursive_link_creator function is missing $output = '';
So that could be inserted on line 205 in
function recursive_link_creator($items = array(), $trail) {
$output = '';

On my PC winmerge generates this output for a patch:

--- //jquerymenu/jquerymenu.module	Mon Dec 20 12:59:27 2010
+++ //jquerymenu/jquerymenu_patch_test.module	Mon Dec 20 13:05:50 2010
@@ -202,6 +202,7 @@
   
   }
 function recursive_link_creator($items = array(), $trail) {
+	$output='';
   $url_array = $trail;
   $i = 0;
   if (!empty($items)) {
pat redmond’s picture

Status: Active » Closed (fixed)

Will be fixed in the upcoming release