Closed (fixed)
Project:
eC Live Subproducts
Version:
5.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
29 Aug 2007 at 10:41 UTC
Updated:
14 Sep 2007 at 12:11 UTC
After installing the 5.x-2.09 version of this module, an empty link started to appear on all my pages (not just product pages):
<div class="links">» <ul class="links inline"><li class="first last add_to_cart"></li>
</ul></div>
Turns out this was caused by a faulty E_ALL check in ec_live_subproducts_link(). The 5.x-2.x-dev code “fixes” the issue by removing the E_ALL check, but that check really should be there.
The attached patch corrects the E_ALL check in 5.x-2.x-dev.
On a related note, why does the ec_live_subproducts_link() function even exist? All it does is add an empty link to product pages.
| Comment | File | Size | Author |
|---|---|---|---|
| emptylink.patch | 897 bytes | johnalbin |
Comments
Comment #1
brmassa commentedJohn,
you are right. its fixed on CVS an soon i will publish a new version.
regards,
massa
Comment #2
(not verified) commented