Author Topic: JSCook bug and partial fix  (Read 21660 times)

DDPaul

  • Beginner
  • *
  • Posts: 28
JSCook bug and partial fix
« on: July 12, 2008, 02:26:29 am »
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

  • Peter
  • Moderator
  • Sr. Member
  • *
  • Posts: 2874
    • VirtueMart Extensions
Re: JSCook bug and partial fix
« Reply #1 on: July 12, 2008, 07:11:20 am »
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

  • Beginner
  • *
  • Posts: 7
Re: JSCook bug and partial fix
« Reply #2 on: August 26, 2008, 06:41:53 am »
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

  • Beginner
  • *
  • Posts: 3
Re: JSCook bug and partial fix
« Reply #3 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 :-)

mulito

  • Beginner
  • *
  • Posts: 5
Re: JSCook bug and partial fix
« Reply #4 on: November 12, 2008, 22:26:55 pm »
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

  • Beginner
  • *
  • Posts: 31
Re: JSCook bug and partial fix
« Reply #5 on: January 11, 2009, 00:55:31 am »
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

  • Jr. Member
  • **
  • Posts: 85
Re: JSCook bug and partial fix
« Reply #6 on: January 15, 2009, 17:47:57 pm »
I think rrommel meant theme.js  8)

herbcap

  • Beginner
  • *
  • Posts: 3
Re: JSCook bug and partial fix
« Reply #7 on: January 27, 2009, 23:53:12 pm »
I have got joomla 1.5.9 and vm 1.13

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

rrommel

  • Beginner
  • *
  • Posts: 31
Re: JSCook bug and partial fix
« Reply #8 on: January 28, 2009, 09:28:17 am »
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

  • Beginner
  • *
  • Posts: 3
Re: JSCook bug and partial fix
« Reply #9 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.

mtvsound

  • Beginner
  • *
  • Posts: 5
Re: JSCook bug and partial fix
« Reply #10 on: February 17, 2009, 12:17:03 pm »
great worked perfect :)

aomegax

  • Beginner
  • *
  • Posts: 9
Re: JSCook bug and partial fix
« Reply #11 on: October 13, 2009, 13:54:08 pm »
Hi...all these don't work for me...
this is my site.

thanks

dorriGa

  • Beginner
  • *
  • Posts: 12
Re: JSCook bug and partial fix
« Reply #12 on: January 27, 2010, 22:10:12 pm »
--->    /includes/js/ThemeOffice/';  work for me


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

ruben-sp

  • Jr. Member
  • **
  • Posts: 64
Re: JSCook bug and partial fix
« Reply #13 on: April 12, 2010, 01:22:24 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

  • Beginner
  • *
  • Posts: 10
Re: JSCook bug and partial fix
« Reply #14 on: April 25, 2010, 20:08:08 pm »
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