Text only | Text with Images

VirtueMart Forum

VirtueMart Dev/Coding Central: VM1 (old version) => Quality & Testing VirtueMart 1.1.x => Virtuemart 1.1 Development (Archiv) => Q&T Resolved => Topic started by: doorknob on July 13, 2008, 03:07:00 AM

Title: [FIXED] Bug in class/pageNavigation.php
Post by: doorknob on July 13, 2008, 03:07:00 AM
It's good to see the SEO wand being waved over browse navigation but there is still a bug

Line 165 of pageNavigation.php.

Code Select
      $html .= "\n<li><span class=\"pagenav\">".$VM_LANG->_('PN_END')." &raquo;&raquo;</span><li>";
 has 2 opening li tags instead of an opening/closing pair. Should be
Code Select
      $html .= "\n<li><span class=\"pagenav\">".$VM_LANG->_('PN_END')." &raquo;&raquo;</span></li>";
Title: Re: [CONFIRMED] Bug in class/pageNavigation.php
Post by: gregdev on July 14, 2008, 19:29:01 PM
Thanks. Fixed in SVN rev. 1469.

Greg
Title: Re: [FIXED] Bug in class/pageNavigation.php
Post by: korb on July 14, 2008, 23:05:23 PM
greg please search pageNavigation.class.php for "LI" (list item) match all words and you will find:
-line 143       $html .= "\n<li><span class=\"pagenav\"> $i </span><li>"; 

here last <li> should be </li>
Title: Re: [FIXED] Bug in class/pageNavigation.php
Post by: gregdev on July 15, 2008, 00:12:38 AM
korb,

Perhaps you are not using the latest from SVN. They are all fine there.

Greg
Title: Re: [FIXED] Bug in class/pageNavigation.php
Post by: korb on July 15, 2008, 11:52:30 AM
yes, thank you
Text only | Text with Images