kate522/wordpress-theme-aksrt
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(admin): harden hero/banner round-trip and operator cross-links

Browse Source 
Align enabled-meta semantics with frontend, allow truly empty titles on
update, use router navigation for manager deep-links, and document fixed
skeleton ownership in module schema copy.
This commit is contained in:
kate522
2026-07-13 22:31:12 +08:00
parent dcc82a39e7
commit 24c135aeec
6 changed files with 64 additions and 23 deletions
Download Patch File Download Diff File Expand all files Collapse all files
inc/classes/admin/class-aksrt-admin-page-display-controller.php
+8 -11
View File
@@ -94,7 +94,8 @@ class AKSRT_Admin_Page_Display_Controller extends AKSRT_Admin_Controller_Base {
'link_target' => get_post_meta($post->ID, '_link_target', true) ?: '_self',
'text_align' => get_post_meta($post->ID, '_text_align', true) ?: 'left',
'device_visibility' => get_post_meta($post->ID, '_device_visibility', true) ?: 'all',
'enabled' => get_post_meta($post->ID, '_enabled', true) === '1',
// Match frontend aksrt_get_banners(): missing _enabled means enabled.
'enabled' => get_post_meta($post->ID, '_enabled', true) !== '0',
'show_text' => get_post_meta($post->ID, '_show_text', true) !== '0',
'show_meta' => get_post_meta($post->ID, '_show_meta', true) === '1',
'kind' => get_post_meta($post->ID, '_aksrt_kind', true) ?: 'hero',
@@ -102,7 +103,7 @@ class AKSRT_Admin_Page_Display_Controller extends AKSRT_Admin_Controller_Base {
'text_size_pc' => get_post_meta($post->ID, '_text_size_pc', true) ?: '',
'text_size_m' => get_post_meta($post->ID, '_text_size_m', true) ?: '',
'text_parallax' => get_post_meta($post->ID, '_text_parallax', true) === '1',
'layers' => is_string($layers_json) && $layers_json !== '' ? json_decode($layers_json, true) : [],
'layers' => is_string($layers_json) && $layers_json !== '' ? (json_decode($layers_json, true) ?: []) : [],
];
}, $posts);
return rest_ensure_response(['code' => 'success', 'data' => $banners]);
@@ -113,6 +114,7 @@ class AKSRT_Admin_Page_Display_Controller extends AKSRT_Admin_Controller_Base {
$kind = $this->normalize_banner_kind($params['kind'] ?? 'hero');
$title = sanitize_text_field($params['title'] ?? '');
$post_id = wp_insert_post([
// Title is optional in admin UI; keep a non-empty WP title for list readability.
'post_title' => '' !== $title ? $title : ('banner' === $kind ? 'Banner Slide' : 'Hero Slide'),
'post_type' => 'aksrt_banner',
'post_status' => 'publish',
@@ -139,7 +141,7 @@ class AKSRT_Admin_Page_Display_Controller extends AKSRT_Admin_Controller_Base {
update_post_meta($post_id, '_text_size_m', sanitize_text_field($params['text_size_m'] ?? ''));
update_post_meta($post_id, '_text_parallax', !empty($params['text_parallax']) ? '1' : '0');
if (isset($params['layers']) && is_array($params['layers'])) {
update_post_meta($post_id, '_layers', wp_json_encode($params['layers']));
update_post_meta($post_id, '_layers', wp_json_encode(array_values($params['layers'])));
}
return rest_ensure_response(['code' => 'success', 'data' => ['id' => $post_id], 'message' => __('幻灯片已创建', 'aksrt')]);
@@ -158,13 +160,8 @@ class AKSRT_Admin_Page_Display_Controller extends AKSRT_Admin_Controller_Base {
$params = $this->get_json_params($request);
$kind = $this->normalize_banner_kind($params['kind'] ?? get_post_meta($id, '_aksrt_kind', true) ?: 'hero');
// Optional title: empty is allowed (UI shows “未命名”); do not 422.
$title = sanitize_text_field($params['title'] ?? '');
if ($title === '') {
$existing = get_the_title($id);
$title = is_string($existing) && $existing !== ''
? $existing
: ('banner' === $kind ? 'Banner Slide' : 'Hero Slide');
}
wp_update_post([
'ID' => $id,
@@ -180,8 +177,8 @@ class AKSRT_Admin_Page_Display_Controller extends AKSRT_Admin_Controller_Base {
update_post_meta($id, '_link_target', in_array($params['link_target'] ?? '', ['_blank', '_self'], true) ? $params['link_target'] : '_self');
update_post_meta($id, '_text_align', in_array($params['text_align'] ?? '', ['left', 'center', 'right'], true) ? $params['text_align'] : 'left');
update_post_meta($id, '_device_visibility', in_array($params['device_visibility'] ?? '', ['all', 'pc', 'mobile'], true) ? $params['device_visibility'] : 'all');
update_post_meta($id, '_enabled', !empty($params['enabled']) ? '1' : '0');
update_post_meta($id, '_show_text', !isset($params['show_text']) || !empty($params['show_text']) ? '1' : '0');
update_post_meta($id, '_enabled', !array_key_exists('enabled', $params) || !empty($params['enabled']) ? '1' : '0');
update_post_meta($id, '_show_text', !array_key_exists('show_text', $params) || !empty($params['show_text']) ? '1' : '0');
update_post_meta($id, '_show_meta', !empty($params['show_meta']) ? '1' : '0');
update_post_meta($id, '_text_size_pc', sanitize_text_field($params['text_size_pc'] ?? ''));
update_post_meta($id, '_text_size_m', sanitize_text_field($params['text_size_m'] ?? ''));