News:

Support the VirtueMart project and become a member

Main Menu

JSCook bug and partial fix

Started by DDPaul, July 12, 2008, 02:26:29 AM

Previous topic - Next topic

DDPaul

Found a bug in root/modules/mod_virtuemart/vm__JSCook.php at about line 80. The line currently is

$mymenu_content.= "\n['<img src=\"$mosConfig_live_site/components/com_virtuemart/js/ThemeXP/darrow.png\">','".$db->f("category_name",false)."','".sefRelToAbs('index.php?option=com_virtuemart&page=shop.browse&category_id='.$db->f("category_id").$itemid)."',null,'".$db->f("category_name",false)."'\n ";

and it should be

$mymenu_content.= "\n['<img src=\"$mosConfig_live_site/modules/mod_virtuemart/ThemeXP/darrow.png\">','".$db->f("category_name",false)."','".sefRelToAbs('index.php?option=com_virtuemart&page=shop.browse&category_id='.$db->f("category_id").$itemid)."',null,'".$db->f("category_name",false)."'\n ";

Alternately you could add the ThemeXP folder to the js folder. As it stands now the image path points to a  non-existent image.

The JSCook menu still doesn't work right but IE7 no longer display that annoying non-image image.

   DDPaul

I believe after further testing that the above modification completely fixes the problem.

aravot

When reporting bug please specify what the bug is, if the bug is missing arrow images than it has been fixed in v1.1.2

jirwan

Good thing I found out this post. I posted on another thread:

http://forum.virtuemart.net/index.php?topic=44252.0

By changing:

$mymenu_content.= "\n[ '<img src=\"' + ctThemeXPBase + 'darrow.png\" alt=\"arr\" />','".$db->f("category_name",false)."','".sefRelToAbs('index.php?option=com_virtuemart&page=shop.browse&category_id='.$db->f("category_id").$itemid)."',null,'".$db->f("category_name",false)."'\n ";

TO:

         $mymenu_content.= "\n[ '<img src=\"$mosConfig_live_site/components/com_virtuemart/js/ThemeXP/darrow.png\" alt=\"arr\" />','".$db->f("category_name",false)."','".sefRelToAbs('index.php?option=com_virtuemart&page=shop.browse&category_id='.$db->f("category_id").$itemid)."',null,'".$db->f("category_name",false)."'\n ";

fixes the "arr" problem. Hope this helps



LuckyTom

#3
I use VM 1.1.2
The problem exists only when SEO is activated...

I solved the problem with the following replacement in line 99

Original:
   $js_src = 'modules/mod_virtuemart';

My Changes:
   $js_src = '/modules/mod_virtuemart';

Hope this helps :-)

mulito

Hey,
I have tryed these two solutions and they worked partially.
Still in IE7 its showing an X on the right side of the subcategory menu item.
Anyone ?

Regards,
Manolo

rrommel

#5
Quote from: LuckyTom on September 01, 2008, 23:54:41 PM
I use VM 1.1.2
The problem exists only when SEO is activated...
I solved the problem with the following replacement in line 99
Original:
$js_src = 'modules/mod_virtuemart';
My Changes:
$js_src = '/modules/mod_virtuemart';
Hope this helps :-)

This solved my mising arrow but as Mulito I also got a trailing "x" (missing image??!) in rightside of submenu when using IE.

Has anyone solved this?

--------------------
I have found the source of the "X" in IE. This is cause by a missing "blank.png" because the base directory is incorrect. I hav solved this problem by modifying the theme.js in: \includes\js\ThemeOffice

Change the second line as follows:

original:
// directory of where all the images are
var cmThemeOfficeBase = 'includes/js/ThemeOffice/';

to

// directory of where all the images are
var cmThemeOfficeBase = '/includes/js/ThemeOffice/';


(yes, I mean theme.js not theme.css)

cinos


herbcap

I have got joomla 1.5.9 and vm 1.13

The solutions don't work for me. Anybody with a suggestion?

rrommel

In configuration.php (jl 1.5.9) change:

var $live_site = 'http://www.yourdomain.xx';

to

$ var live_site = 'http://www.yourdomain.xx/';

It's working for me after upgrade to 1.5.9

herbcap

I found a simple solution:

hack modules/mod_virtuemart/vm_JSCook.php

replace
80: '<img src=\"' + ctThemeXPBase + 'darrow.png\" alt=\"arr\" />'

with
80: '<img src=\"modules/mod_virtuemart/ThemeXP/darrow.png\" alt=\"arr\" />'

does the job for me.

mtvsound


aomegax

#11
Hi...all these don't work for me...
this is my site.

thanks

dorriGa

#12
--->    /includes/js/ThemeOffice/';  work for me


work in IE and Chrome also I get a JSCook compatible with SEO.

ruben-sp

Quote from: herbcap on January 28, 2009, 10:40:44 AM
I found a simple solution:

hack modules/mod_virtuemart/vm_JSCook.php

replace
80: '<img src=\"' + ctThemeXPBase + 'darrow.png\" alt=\"arr\" />'

with
80: '<img src=\"modules/mod_virtuemart/ThemeXP/darrow.png\" alt=\"arr\" />'

does the job for me.

This one works great for me!!

THANKS!!!

frankie n

i tried all these methods and still got the red x to the right so went into includes/js/ThemeOffice/theme.js and changed line 34 from

// 3, HTML code to the right of the regular item
   itemRight: '<img alt="" src="' + cmThemeOfficeBase + 'blank.png">',

to

// 3, HTML code to the right of the regular item
   itemRight: '&nbsp;',

this took away the blank image