You've already forked wc-licensed-product
Fix stock indicator on licensed variable products (v0.5.12)
- Fixed stock indicator appearing in cart for licensed variable products - Override get_children() with direct SQL query to bypass WooCommerce type check - Override get_variation_attributes() for proper taxonomy attribute loading - Override get_variation_prices() to prevent null array errors - Override get_available_variations() with empty availability_html - Added is_type() override to pass variable type checks - Added multiple stock-related filters for comprehensive coverage - Improved isLicensedProductOrVariation() with DB-level parent type check Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
@@ -55,6 +55,14 @@ class LicensedProduct extends WC_Product
|
||||
return $this->exists() && $this->get_price() !== '';
|
||||
}
|
||||
|
||||
/**
|
||||
* Licensed products are always in stock (virtual, no inventory)
|
||||
*/
|
||||
public function is_in_stock(): bool
|
||||
{
|
||||
return true;
|
||||
}
|
||||
|
||||
/**
|
||||
* Get max activations for this product
|
||||
* Falls back to default settings if not set on product
|
||||
|
||||
@@ -46,9 +46,19 @@ final class LicensedProductType
|
||||
add_action('woocommerce_licensed_add_to_cart', [$this, 'addToCartTemplate']);
|
||||
add_action('woocommerce_licensed-variable_add_to_cart', [$this, 'variableAddToCartTemplate']);
|
||||
|
||||
// Use variable product add-to-cart handler for licensed-variable products
|
||||
add_filter('woocommerce_add_to_cart_handler', [$this, 'addToCartHandler'], 10, 2);
|
||||
|
||||
// Make product virtual by default
|
||||
add_filter('woocommerce_product_is_virtual', [$this, 'isVirtual'], 10, 2);
|
||||
|
||||
// Hide stock HTML for licensed products
|
||||
add_filter('woocommerce_get_stock_html', [$this, 'hideStockHtml'], 10, 2);
|
||||
add_filter('woocommerce_get_availability', [$this, 'hideAvailability'], 10, 2);
|
||||
add_filter('woocommerce_get_availability_text', [$this, 'hideAvailabilityText'], 10, 2);
|
||||
add_filter('woocommerce_product_get_stock_quantity', [$this, 'hideStockQuantity'], 10, 2);
|
||||
add_filter('woocommerce_product_variation_get_stock_quantity', [$this, 'hideStockQuantity'], 10, 2);
|
||||
|
||||
// Display current version under product title on single product page
|
||||
add_action('woocommerce_single_product_summary', [$this, 'displayCurrentVersion'], 6);
|
||||
|
||||
@@ -80,9 +90,9 @@ final class LicensedProductType
|
||||
* @param string $className Default class name
|
||||
* @param string $productType Product type
|
||||
* @param string $postType Post type (usually 'product' or 'product_variation')
|
||||
* @param int $productId Product ID
|
||||
* @param mixed $productId Product ID (can be int or string)
|
||||
*/
|
||||
public function getProductClass(string $className, string $productType, string $postType = '', int $productId = 0): string
|
||||
public function getProductClass(string $className, string $productType, string $postType = '', $productId = 0): string
|
||||
{
|
||||
if ($productType === 'licensed') {
|
||||
return LicensedProduct::class;
|
||||
@@ -91,17 +101,19 @@ final class LicensedProductType
|
||||
return LicensedVariableProduct::class;
|
||||
}
|
||||
// Handle variations of licensed-variable products
|
||||
if ($productType === 'variation') {
|
||||
// Check both by product type and by post type for variations
|
||||
if ($productType === 'variation' || $postType === 'product_variation') {
|
||||
// Get parent ID from the product post
|
||||
$parentId = 0;
|
||||
if ($productId > 0) {
|
||||
$parentId = wp_get_post_parent_id($productId);
|
||||
$productIdInt = (int) $productId;
|
||||
if ($productIdInt > 0) {
|
||||
$parentId = wp_get_post_parent_id($productIdInt);
|
||||
}
|
||||
// Fallback to global $post if product ID not available
|
||||
if (!$parentId) {
|
||||
global $post;
|
||||
if ($post && $post->post_parent) {
|
||||
$parentId = $post->post_parent;
|
||||
$parentId = (int) $post->post_parent;
|
||||
}
|
||||
}
|
||||
|
||||
@@ -281,26 +293,111 @@ final class LicensedProductType
|
||||
wc_get_template('single-product/add-to-cart/simple.php');
|
||||
}
|
||||
|
||||
/**
|
||||
* Use the variable product add-to-cart handler for licensed-variable products
|
||||
* WooCommerce uses product type to determine which handler to use
|
||||
*/
|
||||
public function addToCartHandler(string $handler, \WC_Product $product): string
|
||||
{
|
||||
if ($product->is_type('licensed-variable')) {
|
||||
return 'variable';
|
||||
}
|
||||
return $handler;
|
||||
}
|
||||
|
||||
/**
|
||||
* Hide stock HTML for licensed products (they're always virtual/in-stock)
|
||||
*/
|
||||
public function hideStockHtml(string $html, \WC_Product $product): string
|
||||
{
|
||||
if ($this->isLicensedProductOrVariation($product)) {
|
||||
return '';
|
||||
}
|
||||
return $html;
|
||||
}
|
||||
|
||||
/**
|
||||
* Hide availability data for licensed products (they're always virtual/in-stock)
|
||||
*/
|
||||
public function hideAvailability(array $availability, \WC_Product $product): array
|
||||
{
|
||||
if ($this->isLicensedProductOrVariation($product)) {
|
||||
return [
|
||||
'availability' => '',
|
||||
'class' => '',
|
||||
];
|
||||
}
|
||||
return $availability;
|
||||
}
|
||||
|
||||
/**
|
||||
* Hide availability text for licensed products
|
||||
*/
|
||||
public function hideAvailabilityText(string $availability, \WC_Product $product): string
|
||||
{
|
||||
if ($this->isLicensedProductOrVariation($product)) {
|
||||
return '';
|
||||
}
|
||||
return $availability;
|
||||
}
|
||||
|
||||
/**
|
||||
* Hide stock quantity for licensed products (return null = no stock display)
|
||||
*
|
||||
* @param int|null $quantity
|
||||
* @param \WC_Product $product
|
||||
* @return int|null
|
||||
*/
|
||||
public function hideStockQuantity($quantity, \WC_Product $product)
|
||||
{
|
||||
if ($this->isLicensedProductOrVariation($product)) {
|
||||
return null;
|
||||
}
|
||||
return $quantity;
|
||||
}
|
||||
|
||||
/**
|
||||
* Check if product is a licensed product or variation of one
|
||||
*/
|
||||
private function isLicensedProductOrVariation(\WC_Product $product): bool
|
||||
{
|
||||
// Direct licensed products
|
||||
if ($product->is_type('licensed') || $product->is_type('licensed-variable')) {
|
||||
return true;
|
||||
}
|
||||
|
||||
// Check by class name for our custom variation class
|
||||
if ($product instanceof LicensedProductVariation) {
|
||||
return true;
|
||||
}
|
||||
|
||||
// Check if this is a variation with a licensed-variable parent
|
||||
// Use WC_Product_Factory::get_product_type() to get parent type directly from DB
|
||||
// This is more reliable than loading the full product object
|
||||
$parentId = $product->get_parent_id();
|
||||
if ($parentId) {
|
||||
$parentType = \WC_Product_Factory::get_product_type($parentId);
|
||||
if ($parentType === 'licensed-variable') {
|
||||
return true;
|
||||
}
|
||||
}
|
||||
|
||||
return false;
|
||||
}
|
||||
|
||||
/**
|
||||
* Make licensed products virtual by default
|
||||
*/
|
||||
public function isVirtual(bool $isVirtual, \WC_Product $product): bool
|
||||
{
|
||||
if ($product->is_type('licensed') || $product->is_type('licensed-variable')) {
|
||||
if ($this->isLicensedProductOrVariation($product)) {
|
||||
return true;
|
||||
}
|
||||
// Also handle variations of licensed-variable products
|
||||
if ($product->is_type('variation') && $product->get_parent_id()) {
|
||||
$parent = wc_get_product($product->get_parent_id());
|
||||
if ($parent && $parent->is_type('licensed-variable')) {
|
||||
return true;
|
||||
}
|
||||
}
|
||||
return $isVirtual;
|
||||
}
|
||||
|
||||
/**
|
||||
* Enqueue frontend styles for licensed products on single product pages
|
||||
* Enqueue frontend styles and scripts for licensed products on single product pages
|
||||
*/
|
||||
public function enqueueFrontendStyles(): void
|
||||
{
|
||||
@@ -320,6 +417,11 @@ final class LicensedProductType
|
||||
[],
|
||||
WC_LICENSED_PRODUCT_VERSION
|
||||
);
|
||||
|
||||
// For licensed-variable products, enqueue WooCommerce variation scripts
|
||||
if ($product->is_type('licensed-variable')) {
|
||||
wp_enqueue_script('wc-add-to-cart-variation');
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
@@ -372,8 +474,13 @@ final class LicensedProductType
|
||||
return;
|
||||
}
|
||||
|
||||
// Update global $product to use the correctly loaded instance
|
||||
// This ensures the template has the right product type
|
||||
$product = $variableProduct;
|
||||
|
||||
// Get variations count to determine if we should load them via AJAX
|
||||
$getVariations = count($variableProduct->get_children()) <= apply_filters('woocommerce_ajax_variation_threshold', 30, $variableProduct);
|
||||
$children = $variableProduct->get_children();
|
||||
$getVariations = count($children) <= apply_filters('woocommerce_ajax_variation_threshold', 30, $variableProduct);
|
||||
|
||||
// Get template variables - WooCommerce expects these to be set
|
||||
$availableVariations = $getVariations ? $variableProduct->get_available_variations() : false;
|
||||
|
||||
@@ -35,6 +35,61 @@ class LicensedProductVariation extends WC_Product_Variation
|
||||
return true;
|
||||
}
|
||||
|
||||
/**
|
||||
* Licensed products are always in stock (virtual, no inventory)
|
||||
*/
|
||||
public function is_in_stock(): bool
|
||||
{
|
||||
return true;
|
||||
}
|
||||
|
||||
/**
|
||||
* Get availability - empty for licensed products (no stock indicator)
|
||||
*/
|
||||
public function get_availability(): array
|
||||
{
|
||||
return [
|
||||
'availability' => '',
|
||||
'class' => '',
|
||||
];
|
||||
}
|
||||
|
||||
/**
|
||||
* Don't manage stock for licensed products
|
||||
*/
|
||||
public function managing_stock(): bool
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
/**
|
||||
* Check if variation is purchasable
|
||||
* Override to handle custom parent product type
|
||||
*/
|
||||
public function is_purchasable(): bool
|
||||
{
|
||||
// Check if variation exists
|
||||
if (!$this->exists()) {
|
||||
return false;
|
||||
}
|
||||
|
||||
// Check parent product status
|
||||
$parentId = $this->get_parent_id();
|
||||
$parentStatus = get_post_status($parentId);
|
||||
|
||||
if ($parentStatus !== 'publish' && !current_user_can('edit_post', $parentId)) {
|
||||
return false;
|
||||
}
|
||||
|
||||
// Check if variation has a price
|
||||
$price = $this->get_price();
|
||||
if ($price === '' || $price === null) {
|
||||
return false;
|
||||
}
|
||||
|
||||
return apply_filters('woocommerce_variation_is_purchasable', true, $this);
|
||||
}
|
||||
|
||||
/**
|
||||
* Get max activations for this variation
|
||||
* Falls back to parent product, then to default settings
|
||||
|
||||
@@ -41,6 +41,19 @@ class LicensedVariableProduct extends WC_Product_Variable
|
||||
return 'licensed-variable';
|
||||
}
|
||||
|
||||
/**
|
||||
* Check if product is of a certain type
|
||||
* Override to return true for 'variable' as well, so WooCommerce internal
|
||||
* checks pass (many methods in WC_Product_Variable check is_type('variable'))
|
||||
*/
|
||||
public function is_type($type): bool
|
||||
{
|
||||
if (is_array($type)) {
|
||||
return in_array($this->get_type(), $type, true) || in_array('variable', $type, true);
|
||||
}
|
||||
return $this->get_type() === $type || 'variable' === $type;
|
||||
}
|
||||
|
||||
/**
|
||||
* Licensed products are always virtual
|
||||
*/
|
||||
@@ -60,6 +73,189 @@ class LicensedVariableProduct extends WC_Product_Variable
|
||||
return parent::is_purchasable();
|
||||
}
|
||||
|
||||
/**
|
||||
* Licensed products are always in stock (virtual, no inventory)
|
||||
*/
|
||||
public function is_in_stock(): bool
|
||||
{
|
||||
return true;
|
||||
}
|
||||
|
||||
/**
|
||||
* Get children (variations) for this product
|
||||
* Override because WC_Product_Variable::get_children() checks is_type('variable')
|
||||
* which fails for our 'licensed-variable' type
|
||||
*/
|
||||
public function get_children($context = 'view'): array
|
||||
{
|
||||
if (!$this->get_id()) {
|
||||
return [];
|
||||
}
|
||||
|
||||
// Query variations directly from database since WooCommerce's data store
|
||||
// doesn't work properly with custom variable product types
|
||||
global $wpdb;
|
||||
$children = $wpdb->get_col($wpdb->prepare(
|
||||
"SELECT ID FROM {$wpdb->posts}
|
||||
WHERE post_parent = %d
|
||||
AND post_type = 'product_variation'
|
||||
AND post_status IN ('publish', 'private')
|
||||
ORDER BY menu_order ASC, ID ASC",
|
||||
$this->get_id()
|
||||
));
|
||||
|
||||
$children = array_map('intval', $children);
|
||||
|
||||
if ('view' === $context) {
|
||||
$children = apply_filters('woocommerce_get_children', $children, $this, false);
|
||||
}
|
||||
|
||||
return is_array($children) ? $children : [];
|
||||
}
|
||||
|
||||
/**
|
||||
* Get variation attributes for this product
|
||||
* Override because WC_Product_Variable uses data_store which doesn't work
|
||||
* properly with custom variable product types
|
||||
*/
|
||||
public function get_variation_attributes(): array
|
||||
{
|
||||
$attributes = $this->get_attributes();
|
||||
|
||||
if (!$attributes || !is_array($attributes)) {
|
||||
return [];
|
||||
}
|
||||
|
||||
$variation_attributes = [];
|
||||
|
||||
foreach ($attributes as $attribute) {
|
||||
// For WC_Product_Attribute objects
|
||||
if ($attribute instanceof \WC_Product_Attribute) {
|
||||
if ($attribute->get_variation()) {
|
||||
$attribute_name = $attribute->get_name();
|
||||
|
||||
// For taxonomy attributes, get term slugs
|
||||
if ($attribute->is_taxonomy()) {
|
||||
$attribute_terms = wc_get_product_terms(
|
||||
$this->get_id(),
|
||||
$attribute_name,
|
||||
['fields' => 'slugs']
|
||||
);
|
||||
$variation_attributes[$attribute_name] = $attribute_terms;
|
||||
} else {
|
||||
// For custom attributes, get options directly
|
||||
$variation_attributes[$attribute_name] = $attribute->get_options();
|
||||
}
|
||||
}
|
||||
}
|
||||
// For array-based attributes (older format)
|
||||
elseif (is_array($attribute) && !empty($attribute['is_variation'])) {
|
||||
$attribute_name = $attribute['name'];
|
||||
$values = isset($attribute['value']) ? explode('|', $attribute['value']) : [];
|
||||
$variation_attributes[$attribute_name] = array_map('trim', $values);
|
||||
}
|
||||
}
|
||||
|
||||
return $variation_attributes;
|
||||
}
|
||||
|
||||
/**
|
||||
* Get variation prices (regular, sale, and final prices)
|
||||
* Override because WC_Product_Variable uses data_store which doesn't work
|
||||
* properly with custom variable product types
|
||||
*/
|
||||
public function get_variation_prices($for_display = false): array
|
||||
{
|
||||
$children = $this->get_children();
|
||||
|
||||
if (empty($children)) {
|
||||
return [
|
||||
'price' => [],
|
||||
'regular_price' => [],
|
||||
'sale_price' => [],
|
||||
];
|
||||
}
|
||||
|
||||
$prices = [
|
||||
'price' => [],
|
||||
'regular_price' => [],
|
||||
'sale_price' => [],
|
||||
];
|
||||
|
||||
foreach ($children as $child_id) {
|
||||
$variation = wc_get_product($child_id);
|
||||
if ($variation) {
|
||||
$price = $variation->get_price();
|
||||
$regular_price = $variation->get_regular_price();
|
||||
$sale_price = $variation->get_sale_price();
|
||||
|
||||
if ('' !== $price) {
|
||||
$prices['price'][$child_id] = $price;
|
||||
}
|
||||
if ('' !== $regular_price) {
|
||||
$prices['regular_price'][$child_id] = $regular_price;
|
||||
}
|
||||
if ('' !== $sale_price) {
|
||||
$prices['sale_price'][$child_id] = $sale_price;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
// Sort prices
|
||||
asort($prices['price']);
|
||||
asort($prices['regular_price']);
|
||||
asort($prices['sale_price']);
|
||||
|
||||
$this->prices_array = $prices;
|
||||
|
||||
return $this->prices_array;
|
||||
}
|
||||
|
||||
/**
|
||||
* Get available variations for this product
|
||||
* Override because WC_Product_Variable uses data_store which doesn't work
|
||||
* properly with custom variable product types
|
||||
*/
|
||||
public function get_available_variations($return = 'array')
|
||||
{
|
||||
$children = $this->get_children();
|
||||
$available_variations = [];
|
||||
|
||||
foreach ($children as $child_id) {
|
||||
$variation = wc_get_product($child_id);
|
||||
|
||||
if (!$variation) {
|
||||
continue;
|
||||
}
|
||||
|
||||
// Check if variation should be available
|
||||
if (!$variation->exists()) {
|
||||
continue;
|
||||
}
|
||||
|
||||
// Check if purchasable (has price)
|
||||
if (!$variation->is_purchasable()) {
|
||||
continue;
|
||||
}
|
||||
|
||||
// Build variation data
|
||||
if ($return === 'array') {
|
||||
$variationData = $this->get_available_variation($variation);
|
||||
// Override availability_html to be empty for licensed products
|
||||
$variationData['availability_html'] = '';
|
||||
$available_variations[] = $variationData;
|
||||
} else {
|
||||
$available_variations[] = $variation;
|
||||
}
|
||||
}
|
||||
|
||||
if ($return === 'array') {
|
||||
$available_variations = array_values(array_filter($available_variations));
|
||||
}
|
||||
|
||||
return $available_variations;
|
||||
}
|
||||
|
||||
/**
|
||||
* Get max activations for this product (parent default)
|
||||
* Falls back to default settings if not set on product
|
||||
|
||||
Reference in New Issue
Block a user