HTML error with data-wvstooltip

Yan Morin

A space is missing in sprintf of data-wvstooltip of includes/functions.php. This was fixed in standard version.  Pro version: 1.1.2

Easy patch with just adding a space inside 4 sprintf.

Patch:

@@ -497,7 +497,7 @@
                                                }
                                                
                                                $tooltip_html_attr = '';
-                                               $tooltip_html_attr .= ( $show_tooltip && $tooltip_text && $tooltip_type == 'text' ) ? sprintf( 'data-wvstooltip="%s"', esc_attr( $tooltip_text ) ) : '';
+                                               $tooltip_html_attr .= ( $show_tooltip && $tooltip_text && $tooltip_type == 'text' ) ? sprintf( ' data-wvstooltip="%s"', esc_attr( $tooltip_text ) ) : '';
                                                
                                                $tooltip_image_width = absint( woo_variation_swatches()->get_option( 'tooltip_image_width' ) );
                                                
@@ -627,7 +627,7 @@
                                        }
                                        
                                        $tooltip_html_attr = '';
-                                       $tooltip_html_attr .= ( $show_tooltip && $tooltip_text && $tooltip_type == 'text' ) ? sprintf( 'data-wvstooltip="%s"', esc_attr( $tooltip_text ) ) : '';
+                                       $tooltip_html_attr .= ( $show_tooltip && $tooltip_text && $tooltip_type == 'text' ) ? sprintf( ' data-wvstooltip="%s"', esc_attr( $tooltip_text ) ) : '';
                                        
                                        $tooltip_image_width = absint( woo_variation_swatches()->get_option( 'tooltip_image_width' ) );
                                        
@@ -763,7 +763,7 @@
                                                        $tooltip = false;
                                                }
                                                
-                                               $tooltip_html_attr = ! empty( $tooltip ) ? sprintf( 'data-wvstooltip="%s"', esc_attr( $tooltip ) ) : '';
+                                               $tooltip_html_attr = ! empty( $tooltip ) ? sprintf( ' data-wvstooltip="%s"', esc_attr( $tooltip ) ) : '';
                                                
                                                if ( wp_is_mobile() ) {
                                                        $tooltip_html_attr .= ! empty( $tooltip ) ? ' tabindex="2"' : '';
@@ -839,7 +839,7 @@
                                                $tooltip = false;
                                        }
                                        
-                                       $tooltip_html_attr = ! empty( $tooltip ) ? sprintf( 'data-wvstooltip="%s"', esc_attr( $tooltip ) ) : '';
+                                       $tooltip_html_attr = ! empty( $tooltip ) ? sprintf( ' data-wvstooltip="%s"', esc_attr( $tooltip ) ) : '';
                                        
                                        if ( wp_is_mobile() ) {
                                                $tooltip_html_attr .= ! empty( $tooltip ) ? ' tabindex="2"' : '';

Golam Kibria

Hi Yan,

Thanks a lot for sharing your suggestion. I will share this with our development team.

We really appreciate your efforts.

Thank You