Bugs and features to improve this plugin

  • Resolved Andrew MAGIK

    (@magikru)


    9 years ago

    While I am working with development of my own woocommerce theme, using this plugin – I found some bugs:

    This code:
    <div class=related products>
    Should be replaces with this (with quotes around the classes:
    <div class="related products">

    In the original templates – block header was created using <h2> tag (up-sells.php and related.php templates from single-product folder), but you used <h3>, so, it would be nice to change this line:
    <h3><?php echo get_option('woorelated_wtitle'); ?></h3>
    To this line:
    <h2><?php echo get_option('woorelated_wtitle'); ?></h2>

    Also I think it would be nice to place plugin’s setting icon not in the root of wordpress admin menu as now, but make it as submenu in woocommerce menu. For this you should change:
    add_menu_page('Related Products', 'Related Products', 'administrator', 'woo-related-options', 'admin_page');
    To this line:
    add_submenu_page( 'woocommerce', __( 'Related Products', 'woocommerce' ), __( 'Related Products', 'woocommerce' ), 'administrator', 'woo-related-options', 'admin_page' );

    Hope it was helpful, and you will fix this issues to improve your greet plugin.

    https://www.ads-software.com/plugins/woo-related-products-refresh-on-reload/

Viewing 2 replies - 1 through 2 (of 2 total)
  • Thread Starter Andrew MAGIK

    (@magikru)

    Also would be nice to change:

    <select name="woorelated_basedon">
  <option selected="<?php echo esc_attr( get_option('woorelated_basedon') ); ?>"><?php echo esc_attr( get_option('woorelated_basedon') ); ?></option>
  <option value="category">Category</option>
  <option value="tag">Tag</option>
</select>

    To something like this:

    <select name="woorelated_basedon">
  <?php if ( esc_attr(get_option('woorelated_basedon')) == 'tag' ){?>
    <option value="category">Category</option>
    <option selected value="tag">Tag</option>
  <?php } ?>
  <?php if ( esc_attr(get_option('woorelated_basedon')) == 'category' ){?>
    <option selected value="category">Category</option>
    <option value="tag">Tag</option>
  <?php } ?>
</select>

    This will remove 3rd unneeded (duplicated) option from related by select.

    Plugin Author Vagelis

    (@eboxnet)

    Thank you for your suggestions,
    will push an update in the next few days.

Viewing 2 replies - 1 through 2 (of 2 total)
  • The topic ‘Bugs and features to improve this plugin’ is closed to new replies.