Skip to content

Commit e716b62

Browse files
committed
fix add function, fix frontend if default is empty
1 parent a0df781 commit e716b62

File tree

2 files changed

+13
-6
lines changed

2 files changed

+13
-6
lines changed

src/plugins/global-settings/color-schemes/color-scheme-picker.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,7 @@ const ColorSchemePicker = props => {
9898
const customColorSchemes = applyFilters( 'stackable.global-settings.global-color-schemes.custom-color-schemes', [] )
9999

100100
// Add a custom color scheme
101-
const handleAddItem = ( scheme = null ) => {
101+
const handleAddItem = ( event, scheme = null ) => {
102102
doAction( 'stackable.global-settings.global-color-schemes.custom-color-schemes.add-color-scheme', scheme, setItemInEdit, saveTimeout )
103103
}
104104

@@ -210,7 +210,7 @@ const ColorSchemePicker = props => {
210210
const confirmDuplicate = window.confirm( __( 'Do you want to duplicate this color scheme?', i18n ) )
211211

212212
if ( confirmDuplicate ) {
213-
handleAddItem( item )
213+
handleAddItem( null, item )
214214
}
215215
}
216216

src/plugins/global-settings/color-schemes/index.php

Lines changed: 11 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -169,14 +169,21 @@ public static function get_color_scheme_properties( $values = null ) {
169169
* @return String
170170
*/
171171
public function add_global_color_schemes_styles( $current_css ) {
172-
$schemes_array = get_option( 'stackable_global_color_schemes' );
173-
$base_default = get_option( 'stackable_global_base_color_scheme' ) != '' ? get_option( 'stackable_global_base_color_scheme' ) : 'scheme-default-1';
174-
$background_default = get_option( 'stackable_global_background_mode_color_scheme' ) != '' ? get_option( 'stackable_global_background_mode_color_scheme' ) : 'scheme-default-2';
175-
$container_default = get_option( 'stackable_global_container_mode_color_scheme' ) != '' ? get_option( 'stackable_global_container_mode_color_scheme' ) : 'scheme-default-1';
172+
$schemes_array = is_array( get_option( 'stackable_global_color_schemes' ) ) ? get_option( 'stackable_global_color_schemes' ) : [];
176173

177174
// Get all color schemes, including custom color schemes if any
178175
$all_color_schemes = apply_filters( 'stackable_global_color_schemes.get_color_schemes', $schemes_array );
176+
177+
if ( ! is_array( $all_color_schemes ) ) {
178+
return $current_css;
179+
}
180+
179181
$this->color_schemes = $this->convert_to_assoc_array( $all_color_schemes );
182+
183+
$base_default = isset( $this->color_schemes[ get_option( 'stackable_global_base_color_scheme' ) ] ) ? get_option( 'stackable_global_base_color_scheme' ) : 'scheme-default-1';
184+
$background_default = isset( $this->color_schemes[ get_option( 'stackable_global_background_mode_color_scheme' ) ] ) ? get_option( 'stackable_global_background_mode_color_scheme' ) : 'scheme-default-2';
185+
$container_default = isset( $this->color_schemes[ get_option( 'stackable_global_container_mode_color_scheme' ) ] ) ? get_option( 'stackable_global_container_mode_color_scheme' ) : 'scheme-default-1';
186+
180187
$styles = array();
181188

182189
if ( isset( $this->color_schemes[$base_default] ) ) {

0 commit comments

Comments
 (0)