coupon vouchers treeview problem

[wkdwich]

I've got an issue with the treeview module with the coupon codes. 2 of the stores I manage have extensive category trees (250 categories in the larger store), 1500 products in one store, over 8000 in the other. When creating or editing a coupon code, viewing the products or categories with the new treeview tool, it takes forever, the slow script popup comes for my store 3 times, for the other one countless times.

Since the files for the old method of simply viewing a list of the products or categories was still there (validcategories.php & validproducts.php), I made the following changes to not use the javascript tree method, but go back to the old method.

This worked fine with no apparent repercussions..


catalog\admin\coupon_admin.php
FIND (Line 754)

        <td align="left"><?php echo tep_draw_input_field('coupon_products', $coupon_products); ?> <input type=button name=open_popup ONCLICK="window.open('treeview.php', 'popuppage', 'scrollbars=yes,resizable=yes,menubar=yes,width=400,height=600'); " value=" View "></td>

REPLACE WITH

        <td align="left"><?php echo tep_draw_input_field('coupon_products', $coupon_products); ?> <A HREF="validproducts.php" TARGET="_blank" ONCLICK="window.open('validproducts.php', 'Valid_Products', 'scrollbars=yes,resizable=yes,menubar=yes,width=600,height=600'); return false">View</A></td>

FIND (line 759)

        <td align="left"><?php echo tep_draw_input_field('coupon_categories', $coupon_categories); ?> <input type=button name=open_popup ONCLICK="window.open('treeview.php', 'popuppage', 'scrollbars=yes,resizable=yes,menubar=yes,width=400,height=600'); " value=" View "></td>

REPLACE WITH:

        <td align="left"><?php echo tep_draw_input_field('coupon_categories', $coupon_categories); ?> <A HREF="validcategories.php" TARGET="_blank" ONCLICK="window.open('validcategories.php', 'Valid_Categories', 'scrollbars=yes,resizable=yes,menubar=yes,width=600,height=600'); return false">View</A></td>

[ridexbuilder]

Just FYI: treeview has been in the core for some time, just not utilised.
For people who run larger stores on underpowered shared servers (), it could be useful to have an Admin option, in Configuration, to use the old method.

[wkdwich]

the server is not shared underpowered maybe - but dedicated If you can find an extra 10 hrs in a day for me I would do something about the underpowered

Thanks.. I was hoping the change wouldn't make a mess

[pgmarshall]

Add it to the bugTracker as a feature request for v2.6 - that way it will get done ...

Regards,

[wkdwich]

Thanks PG, will do tonight