diff --git a/includes/abstracts/abstract-wc-shipping-method.php b/includes/abstracts/abstract-wc-shipping-method.php index 6edad5f1659..07cb519961a 100644 --- a/includes/abstracts/abstract-wc-shipping-method.php +++ b/includes/abstracts/abstract-wc-shipping-method.php @@ -157,11 +157,9 @@ abstract class WC_Shipping_Method extends WC_Settings_API { * * @access public * @return bool - * @todo Must return false if does not */ function has_settings() { - if ( $this->has_settings ) - return true; + return ( $this->has_settings ); } /** @@ -171,9 +169,6 @@ abstract class WC_Shipping_Method extends WC_Settings_API { * @return bool */ public function is_available( $package ) { - /** @todo Delete this */ - global $woocommerce; - if ( $this->enabled == "no" ) return false; @@ -234,4 +229,4 @@ abstract class WC_Shipping_Method extends WC_Settings_API { function supports( $feature ) { return apply_filters( 'woocommerce_shipping_method_supports', in_array( $feature, $this->supports ) ? true : false, $feature, $this ); } -} \ No newline at end of file +} diff --git a/includes/admin/class-wc-admin-assets.php b/includes/admin/class-wc-admin-assets.php index fee2994ddf4..7e54a0b2317 100644 --- a/includes/admin/class-wc-admin-assets.php +++ b/includes/admin/class-wc-admin-assets.php @@ -262,8 +262,7 @@ class WC_Admin_Assets { */ public function product_taxonomy_styles() { - /** @todo This should be just 'return', without false */ - if ( ! current_user_can( 'manage_woocommerce' ) ) return false; + if ( ! current_user_can( 'manage_woocommerce' ) ) return; ?>