Fix theme buttons to use standard WordPress URLs and behavior

This commit is contained in:
Marcus Quinn
2025-03-17 01:05:48 +00:00
parent dabc9a3eee
commit 1878ebb98b
2 changed files with 18 additions and 72 deletions

View File

@ -301,87 +301,26 @@ jQuery(document).ready(function($) {
$('.theme-actions .install-now').off('click'); $('.theme-actions .install-now').off('click');
$('.theme-actions .activate-now').off('click'); $('.theme-actions .activate-now').off('click');
// Install theme // Install theme - use standard WordPress behavior
$('.theme-actions .install-now').on('click', function(e) { $('.theme-actions .install-now').on('click', function(e) {
e.preventDefault(); // We're not preventing default here - let the standard WordPress installer handle it
// Just add the updating message
var $button = $(this); var $button = $(this);
var slug = $button.data('slug'); var slug = $button.data('slug');
var themeName = $button.data('name') || slug;
console.log('Installing theme:', slug);
$button.addClass('updating-message').text('Installing...'); $button.addClass('updating-message').text('Installing...');
console.log('Installing theme using standard WordPress URL:', $button.attr('href'));
// Use the WordPress core update API // The rest will be handled by WordPress core
wp.updates.installTheme({
slug: slug,
success: function(response) {
console.log('Theme installed successfully:', response);
$button.removeClass('updating-message').addClass('updated-message').text('Installed!');
setTimeout(function() {
// Replace the button with an activate button
var $parent = $button.closest('.theme-actions');
$button.remove();
// Create a proper activation link
var activateUrl = response.activateUrl ||
wp.updates.adminUrl + 'themes.php?action=activate&stylesheet=' + slug +
'&_wpnonce=' + wp.updates.data.nonce;
var $activateButton = $('<a class="button button-primary activate-now" href="' + activateUrl + '">Activate</a>');
$activateButton.attr('data-slug', slug);
$activateButton.attr('data-name', themeName);
$activateButton.attr('data-nonce', wpAllstars.nonce);
$parent.prepend($activateButton);
// Re-initialize the event handlers
initThemeHandlers();
}, 1000);
},
error: function(response) {
console.error('Theme installation failed:', response);
$button.removeClass('updating-message').text('Install');
alert(response.errorMessage || 'Failed to install theme. Please try again.');
}
});
}); });
// Activate theme // Activate theme - use standard WordPress behavior
$('.theme-actions .activate-now').on('click', function(e) { $('.theme-actions .activate-now').on('click', function(e) {
e.preventDefault(); // We're not preventing default here - let the standard WordPress activation handle it
// Just add the updating message
var $button = $(this); var $button = $(this);
var url = $button.attr('href');
var slug = $button.data('slug'); var slug = $button.data('slug');
console.log('Activating theme:', slug, 'with URL:', url);
$button.addClass('updating-message').text('Activating...'); $button.addClass('updating-message').text('Activating...');
console.log('Activating theme using standard WordPress URL:', $button.attr('href'));
// Use standard link-based activation like plugins // The rest will be handled by WordPress core
$.ajax({
url: url,
type: 'GET',
dataType: 'html',
success: function(response) {
console.log('Theme activation successful');
$button.removeClass('updating-message').addClass('updated-message').text('Activated!');
setTimeout(function() {
// Replace the button with an active button
var $parent = $button.closest('.theme-actions');
$button.remove();
$parent.prepend('<button type="button" class="button button-disabled" disabled="disabled">Active</button>');
// Optionally reload the page to show the active theme
// window.location.reload();
}, 1000);
},
error: function(jqXHR, textStatus, errorThrown) {
console.error('Theme activation error:', textStatus, errorThrown);
$button.removeClass('updating-message').text('Activate');
alert('Failed to activate theme. Please try again or activate from the Themes page.');
}
});
}); });
} }
}); });

View File

@ -39,7 +39,14 @@ if (!defined('ABSPATH')) {
<?php esc_html_e('Activate'); ?> <?php esc_html_e('Activate'); ?>
</a> </a>
<?php else: ?> <?php else: ?>
<a href="#" <?php
// Use the standard WordPress theme installation URL format
$install_url = wp_nonce_url(
admin_url('update.php?action=install-theme&theme=kadence'),
'install-theme_kadence'
);
?>
<a href="<?php echo esc_url($install_url); ?>"
class="button button-primary install-now" class="button button-primary install-now"
data-slug="kadence" data-slug="kadence"
data-name="Kadence"> data-name="Kadence">