Support » Plugin: Simple Social Media Share Buttons - Social Sharing for Everyone » SSB breaks the page if it contains an explicit excerpt

  • Resolved simonetel

    (@simonetel)


    Hello,

    the plugin creates broken HTML tags when a post contains an explicit excerpt.

    The problem lies in this function:

    
    	function get_excerpt_by_id( $post_id ) {
    
    		if ( ! $post_id ) {
    			return;
    		}
    			// Check if the post has an excerpt
    		if ( has_excerpt() ) {
    				$excerpt_length = apply_filters( 'excerpt_length', 35 );
    				return trim( get_the_excerpt() );
    		}
    

    […]

    The return line should be:

    return esc_html( trim( wp_strip_all_tags( get_the_excerpt() )));

    I hope you can fix it in your repository, so that updating the plugin won’t break my site again 🙂

Viewing 3 replies - 1 through 3 (of 3 total)
  • Plugin Support Sharaz Shahid

    (@sharaz)

    Hi @simonetel,

    Thanks for updating us for this issue. We will test and fix it in our upcoming release.

    Best regards,
    Team SSB

    Plugin Support Sharaz Shahid

    (@sharaz)

    Hi there,

    I have tested it with multiple explicit experts and its working fine. Could you please share one excerpt data which breaks your layout. Also please double check that all html tags are close properly

    here’s an example:

    
    Con grande soddisfazione comunichiamo che RCS MediaGroup ha scelto <strong>Exelis <a href="https://www.exelis.it/prodotti-servizi/lilium/" target="_blank" rel="noopener">Lilium</a></strong> per gestire i processi di acquisizione della pubblicitĂ  classificata e delle necrologie.
    Lilium è la piattaforma web che nasce dall’esperienza maturata dalla nostra azienda nel supporto di tutti flussi di lavorazione necessari ad acquisire, comporre e valorizzare i contenuti tipicamente raccolti dallo “sportello” delle concessionarie.
    

    Since you just add this text to the description, you’re bound to create broken code:

    
    <meta property="og:description" content="Con grande soddisfazione comunichiamo che RCS MediaGroup ha scelto <strong>Exelis <a href="https://www.exelis.it/prodotti-servizi/lilium/" target="_blank" rel="noopener">Lilium</a></strong> per gestire i processi di acquisizione della pubblicitĂ  classificata e delle necrologie.
    Lilium è la piattaforma web che nasce dall’esperienza maturata dalla nostra azienda nel supporto di tutti flussi di lavorazione necessari ad acquisire, comporre e valorizzare i contenuti tipicamente raccolti dallo “sportello” delle concessionarie." />
    

    (quotes are unbalanced and tags should not appear there)

    get_excerpt_by_id() in your code only strip tags if has_excerpt() is false. In both cases, I’d add esc_html() just to be on the safe side, anyway.

    • This reply was modified 5 months, 2 weeks ago by  simonetel.
    • This reply was modified 5 months, 2 weeks ago by  simonetel.
Viewing 3 replies - 1 through 3 (of 3 total)
  • You must be logged in to reply to this topic.