WordPress.org

Ready to get started?Download WordPress

Forums

Facebook Friends Inviter
[resolved] [shortcode] (7 posts)

  1. christonphero
    Member
    Posted 1 year ago #

    Line 348 of facebookinviter.php

    FB.init({
    		    appId:'<?php echo $appid; ?>',
    		    cookie:true,
    		    status:true,
    		    frictionlessRequests:true,
    		    xfbml:true
    		    });
    		    function FacebookInviteFriends()
    		    {
    		    FB.ui({
    		    method: 'apprequests',
    		    title: '<?php echo $title; ?>'
    		    message: '<?php echo $message; ?>'
    		    });
    		    }

    Missing comma after the title element breaks the popup when the shortcode is used, widget works fine. :)

    http://wordpress.org/extend/plugins/facebook-friends-inviter/

  2. christonphero
    Member
    Posted 1 year ago #

    Also, I did notice that I couldn't use single quotes in the message because they get escaped.

    For me personally, I just added to line 322 so that it changed the single quotes back and escaped them:

    $message=esc_attr($message); /*added->*/ $message = str_replace('&#039 ;',"\\'",$message);

    But esc_attr gets rid of double quotes and other things that people besides me would possibly still want.

    Anyways, besides that it works fine! Those were the two problems that prevented me from being able to simply install the plugin and use it. Peace.

  3. funmi omoba
    Member
    Posted 1 year ago #

    Same here, if you are not using the widget, the plugin shortcode will not popup.

  4. leehodson
    Member
    Plugin Author

    Posted 1 year ago #

    I've updated to version 1.0.3 with the missing comma added and a change in sanitization. You can now use double quotes in the widget. Will update again at some point to alter the text input/output fields.

    While updating I noticed the sidebar widget's settings override the shortcode's settings. Will make a few more changes to correct that this week.

    Thank you to both of you :)

  5. leehodson
    Member
    Plugin Author

    Posted 1 year ago #

    I'm marking this as resolved :)

  6. christonphero
    Member
    Posted 1 year ago #

    do it. lol

  7. leehodson
    Member
    Plugin Author

    Posted 1 year ago #

    Haha think I might just ;)

    I'll be pushing out a new update over the next few hours. This update works better with "double quotes" in text fields. Single quotes in the title field stop the popup box popping up so I don't recommend them.

Topic Closed

This topic has been closed to new replies.

About this Plugin

About this Topic

Tags

No tags yet.