Win99 ph login register.Makakuha ng libreng 700pho sa bawat deposito

Resolved

(@daymobrew)

Notice: Undefined index: tc_slider_default_height in /web/wp-content/themes/customizr/inc/class-fire-init.php on line 415

It is not checked whether $_options[‘tc_slider_default_height’] is set before checking if it is zero.

My suggested fix:

--- class-fire-init.orig.php	2015-01-27 09:23:50.000000000 +0000
+++ class-fire-init.php	2015-02-12 16:42:33.438350100 +0000
@@ -412,7 +412,7 @@
       */
       function tc_set_slider_img_height( $_default_size ) {
         $_options = get_option('tc_theme_options');
-        if ( 0 == $_options['tc_slider_default_height'] )
+        if ( !isset( $_options['tc_slider_default_height'] ) || 0 == $_options['tc_slider_default_height'] )
           return $_default_size;

         $_default_size['height'] = esc_attr( $_options['tc_slider_default_height'] );

Viewing 7 replies - 1 through 7 (of 7 total)

Rocco Aliberti
(@d4z_c0nf)

9 years, 9 months ago

Hi daymobrew,
with which customizr version you get this notice?
The thing is that .. that “index” should always exists, it stays in the default options..

Thread Starter daymobrew
(@daymobrew)

9 years, 9 months ago

3.2.17.
I am using it on a Multisite installation.

Thread Starter daymobrew
(@daymobrew)

9 years, 9 months ago

And I have WP_DEBUG set to true.

Rocco Aliberti
(@d4z_c0nf)

9 years, 9 months ago

Thanks.
We’ll it’s weird, thanks for reporting it, and for providing a patch.

Rocco Aliberti
(@d4z_c0nf)

9 years, 9 months ago

Hi again,
sorry I was wrong. I see what you meant now ??
Basically this happens when you check on Replace the default image slider’s height, but you haven’t changed the new slider’s height yet.
Can you confirm?

Anyway feel free, if you want, to push pull-requests on github
??

I’ll do for this now, quoting this post in its comment.

Thread Starter daymobrew
(@daymobrew)

9 years, 9 months ago

I didn’t set the slider height but the customiser shows it as 500px.
See: https://imgur.com/9IoJz1m

I will submit my patch via a pull-request when I get a chance.

Thread Starter daymobrew
(@daymobrew)

9 years, 9 months ago

Fixed by @d4z_cOnf in github.

Viewing 7 replies - 1 through 7 (of 7 total)

The topic ‘Notice: Undefined index: tc_slider_default_height’ is closed to new replies.

Tags