You've already forked wc-licensed-product
Compare commits
5 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| 4a90e6b18b | |||
| 502a8c7cd7 | |||
| 6b83fce8b2 | |||
| 8c33eaff29 | |||
| 98002ae3d7 |
21
CHANGELOG.md
21
CHANGELOG.md
@@ -7,6 +7,27 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|||||||
|
|
||||||
## [Unreleased]
|
## [Unreleased]
|
||||||
|
|
||||||
|
## [0.3.9] - 2026-01-24
|
||||||
|
|
||||||
|
### Added
|
||||||
|
|
||||||
|
- "Generate Licenses" button in order meta box for admin-created orders
|
||||||
|
- "Generate Missing Licenses" button when some products in an order are missing licenses
|
||||||
|
- AJAX handler `ajaxGenerateOrderLicenses()` for manual license generation from admin
|
||||||
|
- Warning message when order domain is not set before generating licenses
|
||||||
|
|
||||||
|
### Fixed
|
||||||
|
|
||||||
|
- **Critical:** Licenses are now generated for orders created manually in admin area
|
||||||
|
- Previously, licenses were only generated via checkout hooks, leaving admin-created orders without licenses
|
||||||
|
|
||||||
|
### Technical Details
|
||||||
|
|
||||||
|
- Added `wclp_generate_order_licenses` AJAX action to `OrderLicenseController`
|
||||||
|
- Updated `order-licenses.js` with generate button handler and page reload on success
|
||||||
|
- Added CSS styles for generate status messages
|
||||||
|
- Updated translations (365 strings)
|
||||||
|
|
||||||
## [0.3.8] - 2026-01-24
|
## [0.3.8] - 2026-01-24
|
||||||
|
|
||||||
### Fixed
|
### Fixed
|
||||||
|
|||||||
36
CLAUDE.md
36
CLAUDE.md
@@ -36,7 +36,7 @@ This project is proudly **"vibe-coded"** using Claude.AI - the entire codebase w
|
|||||||
|
|
||||||
No known bugs at the moment.
|
No known bugs at the moment.
|
||||||
|
|
||||||
### Version 0.3.8
|
### Version 0.3.9
|
||||||
|
|
||||||
No changes at the moment.
|
No changes at the moment.
|
||||||
|
|
||||||
@@ -1091,3 +1091,37 @@ Fixed dashboard widget bugs, improved UI consistency, and added download trackin
|
|||||||
- Created release package: `releases/wc-licensed-product-0.3.7.zip` (827 KB)
|
- Created release package: `releases/wc-licensed-product-0.3.7.zip` (827 KB)
|
||||||
- SHA256: `e93b2ab06f6d43c2179167090e07eda5db6809df6e391baece4ceba321cf33f6`
|
- SHA256: `e93b2ab06f6d43c2179167090e07eda5db6809df6e391baece4ceba321cf33f6`
|
||||||
- Tagged as `v0.3.7` and pushed to `main` branch
|
- Tagged as `v0.3.7` and pushed to `main` branch
|
||||||
|
|
||||||
|
### 2026-01-24 - Version 0.3.8 - Translation Bug Fix
|
||||||
|
|
||||||
|
**Overview:**
|
||||||
|
|
||||||
|
Fixed a critical translation bug that caused the settings page to crash with an `ArgumentCountError`.
|
||||||
|
|
||||||
|
**Bug Fix:**
|
||||||
|
|
||||||
|
- Fixed: Duplicate German translation string in `wc-licensed-product-de_CH.po` causing `ArgumentCountError` in settings page
|
||||||
|
- Root cause: The notification settings description was duplicated in the translation, resulting in two `%s` placeholders when only one argument was passed to `sprintf()`
|
||||||
|
- Location: [wc-licensed-product-de_CH.po:322-328](languages/wc-licensed-product-de_CH.po#L322-L328)
|
||||||
|
|
||||||
|
**Modified files:**
|
||||||
|
|
||||||
|
- `languages/wc-licensed-product-de_CH.po` - Removed duplicated translation string
|
||||||
|
- `languages/wc-licensed-product-de_CH.mo` - Recompiled binary translation
|
||||||
|
|
||||||
|
**Technical notes:**
|
||||||
|
|
||||||
|
- Error was logged to `tmp/fatal-errors-2026-01-24.log`
|
||||||
|
- The German `msgstr` contained the same text twice, each with a `%s` placeholder
|
||||||
|
- `sprintf()` at `SettingsController.php:221` only provided one argument for the single `%s` in the English source
|
||||||
|
- Translation strings with `%s` placeholders must have exactly matching placeholder counts between source and translation
|
||||||
|
|
||||||
|
**Dependency Updates:**
|
||||||
|
|
||||||
|
- Updated `magdev/wc-licensed-product-client` from `9f513a8` to `64d215c`
|
||||||
|
|
||||||
|
**Release v0.3.8:**
|
||||||
|
|
||||||
|
- Created release package: `releases/wc-licensed-product-0.3.8.zip` (829 KB)
|
||||||
|
- SHA256: `50ad6966c5ab8db2257572084d2d8a820448df62615678e1576696f2c0cb383d`
|
||||||
|
- Tagged as `v0.3.8` and pushed to `main` branch
|
||||||
|
|||||||
@@ -16,6 +16,9 @@
|
|||||||
// Order domain save
|
// Order domain save
|
||||||
$('#wclp-save-order-domain').on('click', this.saveOrderDomain.bind(this));
|
$('#wclp-save-order-domain').on('click', this.saveOrderDomain.bind(this));
|
||||||
|
|
||||||
|
// Generate licenses button
|
||||||
|
$(document).on('click', '#wclp-generate-licenses', this.generateLicenses.bind(this));
|
||||||
|
|
||||||
// License domain edit/save/cancel
|
// License domain edit/save/cancel
|
||||||
$(document).on('click', '.wclp-edit-domain-btn', this.startEditDomain);
|
$(document).on('click', '.wclp-edit-domain-btn', this.startEditDomain);
|
||||||
$(document).on('click', '.wclp-save-domain-btn', this.saveLicenseDomain.bind(this));
|
$(document).on('click', '.wclp-save-domain-btn', this.saveLicenseDomain.bind(this));
|
||||||
@@ -135,6 +138,54 @@
|
|||||||
$editBtn.show();
|
$editBtn.show();
|
||||||
},
|
},
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Generate licenses for order
|
||||||
|
*/
|
||||||
|
generateLicenses: function(e) {
|
||||||
|
e.preventDefault();
|
||||||
|
|
||||||
|
var $btn = $(e.currentTarget);
|
||||||
|
var $spinner = $btn.siblings('.spinner');
|
||||||
|
var $status = $btn.siblings('.wclp-generate-status');
|
||||||
|
|
||||||
|
var orderId = $btn.data('order-id');
|
||||||
|
|
||||||
|
$btn.prop('disabled', true);
|
||||||
|
$spinner.addClass('is-active');
|
||||||
|
$status.text('').removeClass('success error');
|
||||||
|
|
||||||
|
$.ajax({
|
||||||
|
url: wclpOrderLicenses.ajaxUrl,
|
||||||
|
type: 'POST',
|
||||||
|
data: {
|
||||||
|
action: 'wclp_generate_order_licenses',
|
||||||
|
nonce: wclpOrderLicenses.nonce,
|
||||||
|
order_id: orderId
|
||||||
|
},
|
||||||
|
success: function(response) {
|
||||||
|
if (response.success) {
|
||||||
|
$status.text(response.data.message).addClass('success');
|
||||||
|
if (response.data.reload) {
|
||||||
|
// Reload the page after a short delay to show the new licenses
|
||||||
|
setTimeout(function() {
|
||||||
|
window.location.reload();
|
||||||
|
}, 1500);
|
||||||
|
}
|
||||||
|
} else {
|
||||||
|
$status.text(response.data.message || wclpOrderLicenses.strings.error).addClass('error');
|
||||||
|
$btn.prop('disabled', false);
|
||||||
|
}
|
||||||
|
},
|
||||||
|
error: function() {
|
||||||
|
$status.text(wclpOrderLicenses.strings.error).addClass('error');
|
||||||
|
$btn.prop('disabled', false);
|
||||||
|
},
|
||||||
|
complete: function() {
|
||||||
|
$spinner.removeClass('is-active');
|
||||||
|
}
|
||||||
|
});
|
||||||
|
},
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Save license domain
|
* Save license domain
|
||||||
*/
|
*/
|
||||||
|
|||||||
Binary file not shown.
File diff suppressed because it is too large
Load Diff
File diff suppressed because it is too large
Load Diff
@@ -36,6 +36,7 @@ final class OrderLicenseController
|
|||||||
// Handle AJAX actions
|
// Handle AJAX actions
|
||||||
add_action('wp_ajax_wclp_update_order_domain', [$this, 'ajaxUpdateOrderDomain']);
|
add_action('wp_ajax_wclp_update_order_domain', [$this, 'ajaxUpdateOrderDomain']);
|
||||||
add_action('wp_ajax_wclp_update_license_domain', [$this, 'ajaxUpdateLicenseDomain']);
|
add_action('wp_ajax_wclp_update_license_domain', [$this, 'ajaxUpdateLicenseDomain']);
|
||||||
|
add_action('wp_ajax_wclp_generate_order_licenses', [$this, 'ajaxGenerateOrderLicenses']);
|
||||||
|
|
||||||
// Enqueue admin scripts
|
// Enqueue admin scripts
|
||||||
add_action('admin_enqueue_scripts', [$this, 'enqueueScripts']);
|
add_action('admin_enqueue_scripts', [$this, 'enqueueScripts']);
|
||||||
@@ -126,6 +127,18 @@ final class OrderLicenseController
|
|||||||
|
|
||||||
<h4><?php esc_html_e('Licenses', 'wc-licensed-product'); ?></h4>
|
<h4><?php esc_html_e('Licenses', 'wc-licensed-product'); ?></h4>
|
||||||
|
|
||||||
|
<?php
|
||||||
|
// Count licensed products to check if all have licenses
|
||||||
|
$licensedProductCount = 0;
|
||||||
|
foreach ($order->get_items() as $item) {
|
||||||
|
$product = $item->get_product();
|
||||||
|
if ($product && $product->is_type('licensed')) {
|
||||||
|
$licensedProductCount++;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
$missingLicenses = $licensedProductCount - count($licenses);
|
||||||
|
?>
|
||||||
|
|
||||||
<?php if (empty($licenses)): ?>
|
<?php if (empty($licenses)): ?>
|
||||||
<p class="description">
|
<p class="description">
|
||||||
<?php esc_html_e('No licenses have been generated for this order yet.', 'wc-licensed-product'); ?>
|
<?php esc_html_e('No licenses have been generated for this order yet.', 'wc-licensed-product'); ?>
|
||||||
@@ -137,6 +150,20 @@ final class OrderLicenseController
|
|||||||
<em><?php esc_html_e('Licenses will be generated when the order is marked as paid/completed.', 'wc-licensed-product'); ?></em>
|
<em><?php esc_html_e('Licenses will be generated when the order is marked as paid/completed.', 'wc-licensed-product'); ?></em>
|
||||||
<?php endif; ?>
|
<?php endif; ?>
|
||||||
</p>
|
</p>
|
||||||
|
<?php if ($orderDomain && $order->is_paid()): ?>
|
||||||
|
<p style="margin-top: 10px;">
|
||||||
|
<button type="button" class="button button-primary" id="wclp-generate-licenses" data-order-id="<?php echo esc_attr($order->get_id()); ?>">
|
||||||
|
<?php esc_html_e('Generate Licenses', 'wc-licensed-product'); ?>
|
||||||
|
</button>
|
||||||
|
<span class="spinner" style="float: none; margin-top: 4px;"></span>
|
||||||
|
<span class="wclp-generate-status"></span>
|
||||||
|
</p>
|
||||||
|
<?php elseif (!$orderDomain): ?>
|
||||||
|
<p class="description" style="margin-top: 10px; color: #d63638;">
|
||||||
|
<span class="dashicons dashicons-warning"></span>
|
||||||
|
<?php esc_html_e('Please set the order domain above before generating licenses.', 'wc-licensed-product'); ?>
|
||||||
|
</p>
|
||||||
|
<?php endif; ?>
|
||||||
<?php else: ?>
|
<?php else: ?>
|
||||||
<table class="widefat striped wclp-licenses-table">
|
<table class="widefat striped wclp-licenses-table">
|
||||||
<thead>
|
<thead>
|
||||||
@@ -223,6 +250,29 @@ final class OrderLicenseController
|
|||||||
);
|
);
|
||||||
?>
|
?>
|
||||||
</p>
|
</p>
|
||||||
|
|
||||||
|
<?php if ($missingLicenses > 0 && $orderDomain && $order->is_paid()): ?>
|
||||||
|
<p style="margin-top: 10px;">
|
||||||
|
<span class="dashicons dashicons-warning" style="color: #dba617;"></span>
|
||||||
|
<?php
|
||||||
|
printf(
|
||||||
|
/* translators: %d: Number of missing licenses */
|
||||||
|
esc_html(_n(
|
||||||
|
'%d licensed product is missing a license.',
|
||||||
|
'%d licensed products are missing licenses.',
|
||||||
|
$missingLicenses,
|
||||||
|
'wc-licensed-product'
|
||||||
|
)),
|
||||||
|
$missingLicenses
|
||||||
|
);
|
||||||
|
?>
|
||||||
|
<button type="button" class="button" id="wclp-generate-licenses" data-order-id="<?php echo esc_attr($order->get_id()); ?>">
|
||||||
|
<?php esc_html_e('Generate Missing Licenses', 'wc-licensed-product'); ?>
|
||||||
|
</button>
|
||||||
|
<span class="spinner" style="float: none; margin-top: 4px;"></span>
|
||||||
|
<span class="wclp-generate-status"></span>
|
||||||
|
</p>
|
||||||
|
<?php endif; ?>
|
||||||
<?php endif; ?>
|
<?php endif; ?>
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
@@ -248,6 +298,9 @@ final class OrderLicenseController
|
|||||||
.wclp-lifetime { color: #0073aa; font-weight: 500; }
|
.wclp-lifetime { color: #0073aa; font-weight: 500; }
|
||||||
.wclp-edit-domain-btn { color: #0073aa; text-decoration: none; }
|
.wclp-edit-domain-btn { color: #0073aa; text-decoration: none; }
|
||||||
.wclp-edit-domain-btn .dashicons { font-size: 16px; width: 16px; height: 16px; }
|
.wclp-edit-domain-btn .dashicons { font-size: 16px; width: 16px; height: 16px; }
|
||||||
|
.wclp-generate-status { font-style: italic; margin-left: 8px; }
|
||||||
|
.wclp-generate-status.success { color: #46b450; }
|
||||||
|
.wclp-generate-status.error { color: #dc3232; }
|
||||||
</style>
|
</style>
|
||||||
<?php
|
<?php
|
||||||
}
|
}
|
||||||
@@ -284,8 +337,9 @@ final class OrderLicenseController
|
|||||||
'strings' => [
|
'strings' => [
|
||||||
'saving' => __('Saving...', 'wc-licensed-product'),
|
'saving' => __('Saving...', 'wc-licensed-product'),
|
||||||
'saved' => __('Saved!', 'wc-licensed-product'),
|
'saved' => __('Saved!', 'wc-licensed-product'),
|
||||||
'error' => __('Error saving. Please try again.', 'wc-licensed-product'),
|
'error' => __('Error. Please try again.', 'wc-licensed-product'),
|
||||||
'invalidDomain' => __('Please enter a valid domain.', 'wc-licensed-product'),
|
'invalidDomain' => __('Please enter a valid domain.', 'wc-licensed-product'),
|
||||||
|
'generating' => __('Generating...', 'wc-licensed-product'),
|
||||||
],
|
],
|
||||||
]);
|
]);
|
||||||
}
|
}
|
||||||
@@ -392,4 +446,96 @@ final class OrderLicenseController
|
|||||||
$pattern = '/^([a-zA-Z0-9]([a-zA-Z0-9\-]{0,61}[a-zA-Z0-9])?\.)+[a-zA-Z]{2,}$/';
|
$pattern = '/^([a-zA-Z0-9]([a-zA-Z0-9\-]{0,61}[a-zA-Z0-9])?\.)+[a-zA-Z]{2,}$/';
|
||||||
return (bool) preg_match($pattern, $domain);
|
return (bool) preg_match($pattern, $domain);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* AJAX handler for generating order licenses
|
||||||
|
*/
|
||||||
|
public function ajaxGenerateOrderLicenses(): void
|
||||||
|
{
|
||||||
|
check_ajax_referer('wclp_order_license_actions', 'nonce');
|
||||||
|
|
||||||
|
if (!current_user_can('manage_woocommerce')) {
|
||||||
|
wp_send_json_error(['message' => __('Permission denied.', 'wc-licensed-product')]);
|
||||||
|
}
|
||||||
|
|
||||||
|
$orderId = absint($_POST['order_id'] ?? 0);
|
||||||
|
|
||||||
|
if (!$orderId) {
|
||||||
|
wp_send_json_error(['message' => __('Invalid order ID.', 'wc-licensed-product')]);
|
||||||
|
}
|
||||||
|
|
||||||
|
$order = wc_get_order($orderId);
|
||||||
|
if (!$order) {
|
||||||
|
wp_send_json_error(['message' => __('Order not found.', 'wc-licensed-product')]);
|
||||||
|
}
|
||||||
|
|
||||||
|
// Check if order is paid
|
||||||
|
if (!$order->is_paid()) {
|
||||||
|
wp_send_json_error(['message' => __('Order must be paid before licenses can be generated.', 'wc-licensed-product')]);
|
||||||
|
}
|
||||||
|
|
||||||
|
// Get domain
|
||||||
|
$domain = $order->get_meta('_licensed_product_domain');
|
||||||
|
if (empty($domain)) {
|
||||||
|
wp_send_json_error(['message' => __('Please set the order domain before generating licenses.', 'wc-licensed-product')]);
|
||||||
|
}
|
||||||
|
|
||||||
|
// Generate licenses for each licensed product
|
||||||
|
$generated = 0;
|
||||||
|
$skipped = 0;
|
||||||
|
|
||||||
|
foreach ($order->get_items() as $item) {
|
||||||
|
$product = $item->get_product();
|
||||||
|
if ($product && $product->is_type('licensed')) {
|
||||||
|
$license = $this->licenseManager->generateLicense(
|
||||||
|
$orderId,
|
||||||
|
$product->get_id(),
|
||||||
|
$order->get_customer_id(),
|
||||||
|
$domain
|
||||||
|
);
|
||||||
|
|
||||||
|
if ($license) {
|
||||||
|
// Check if this is a new license or existing
|
||||||
|
$existingLicenses = $this->licenseManager->getLicensesByOrder($orderId);
|
||||||
|
$isNew = true;
|
||||||
|
foreach ($existingLicenses as $existing) {
|
||||||
|
if ($existing->getProductId() === $product->get_id() && $existing->getId() !== $license->getId()) {
|
||||||
|
$isNew = false;
|
||||||
|
break;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
if ($isNew) {
|
||||||
|
$generated++;
|
||||||
|
} else {
|
||||||
|
$skipped++;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
if ($generated > 0) {
|
||||||
|
wp_send_json_success([
|
||||||
|
'message' => sprintf(
|
||||||
|
/* translators: %d: Number of licenses generated */
|
||||||
|
_n(
|
||||||
|
'%d license generated successfully.',
|
||||||
|
'%d licenses generated successfully.',
|
||||||
|
$generated,
|
||||||
|
'wc-licensed-product'
|
||||||
|
),
|
||||||
|
$generated
|
||||||
|
),
|
||||||
|
'generated' => $generated,
|
||||||
|
'skipped' => $skipped,
|
||||||
|
'reload' => true,
|
||||||
|
]);
|
||||||
|
} else {
|
||||||
|
wp_send_json_success([
|
||||||
|
'message' => __('All licenses already exist for this order.', 'wc-licensed-product'),
|
||||||
|
'generated' => 0,
|
||||||
|
'skipped' => $skipped,
|
||||||
|
'reload' => false,
|
||||||
|
]);
|
||||||
|
}
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -3,7 +3,7 @@
|
|||||||
* Plugin Name: WooCommerce Licensed Product
|
* Plugin Name: WooCommerce Licensed Product
|
||||||
* Plugin URI: https://src.bundespruefstelle.ch/magdev/wc-licensed-product
|
* Plugin URI: https://src.bundespruefstelle.ch/magdev/wc-licensed-product
|
||||||
* Description: WooCommerce plugin to sell software products using license keys with domain-based validation.
|
* Description: WooCommerce plugin to sell software products using license keys with domain-based validation.
|
||||||
* Version: 0.3.8
|
* Version: 0.3.9
|
||||||
* Author: Marco Graetsch
|
* Author: Marco Graetsch
|
||||||
* Author URI: https://src.bundespruefstelle.ch/magdev
|
* Author URI: https://src.bundespruefstelle.ch/magdev
|
||||||
* License: GPL-2.0-or-later
|
* License: GPL-2.0-or-later
|
||||||
@@ -28,7 +28,7 @@ if (!defined('ABSPATH')) {
|
|||||||
}
|
}
|
||||||
|
|
||||||
// Plugin constants
|
// Plugin constants
|
||||||
define('WC_LICENSED_PRODUCT_VERSION', '0.3.8');
|
define('WC_LICENSED_PRODUCT_VERSION', '0.3.9');
|
||||||
define('WC_LICENSED_PRODUCT_PLUGIN_FILE', __FILE__);
|
define('WC_LICENSED_PRODUCT_PLUGIN_FILE', __FILE__);
|
||||||
define('WC_LICENSED_PRODUCT_PLUGIN_DIR', plugin_dir_path(__FILE__));
|
define('WC_LICENSED_PRODUCT_PLUGIN_DIR', plugin_dir_path(__FILE__));
|
||||||
define('WC_LICENSED_PRODUCT_PLUGIN_URL', plugin_dir_url(__FILE__));
|
define('WC_LICENSED_PRODUCT_PLUGIN_URL', plugin_dir_url(__FILE__));
|
||||||
|
|||||||
Reference in New Issue
Block a user