+global\FacebookCallback::send_message()

Complexity

Conditions	20
Paths	12974

Size

Total Lines	114
Code Lines	75

Duplication

Lines	21
Ratio	18.42 %

Importance

Changes

0

<?php
    public static function send_message(
        $to = "me",
        $message,

Parameters which have default values should be placed at the end.

        $link = "",
        $otherVariables = array()
    ) {
        //FACEBOOK
        if ($to instanceof Member) {
            $to = $to->FacebookUsername;
        }
        $facebook = self::get_facebook_sdk_class();
        if ($facebook) {
            $user = $facebook->getUser();
            //get email data that does not go to GRAPH:
            if (isset($otherVariables["senderEmail"])) {
                $senderEmail = $otherVariables["senderEmail"];
                unset($otherVariables["senderEmail"]);
            } elseif ($sender = Member::currentUser()) {
                $senderEmail = $sender->Email;
            }

            //start hack
            $message = trim(strip_tags(stripslashes($message)));
            //end hack
            $postArray = array(
                'message' => $message,
                'link' => $link,
            );
            if (count($otherVariables)) {
                foreach ($otherVariables as $key => $value) {
                    $postArray[$key] = $value;
                }
            }
            if ($user) {
                if (empty($otherVariables["Subject"])) {

This code seems to be duplicated across your project.

                    $subject = substr($message, 0, 30);
                } else {
                    $subject = $otherVariables["Subject"];
                }
                //------------- SEND EMAIL TO START DIALOGUE ---
                //BUILD LINK
                $emailLink = "https://www.facebook.com/dialog/feed?"
                        ."to=".$to."&"
                        ."app_id=".self::get_facebook_id()."&"
                        ."link=".urlencode($link)."&"
                        ."message=".urlencode($message)."&";
                //FROM
                if (isset($otherVariables["redirect_uri"])) {
                    $emailLink .= "redirect_uri=".urlencode(Director::absoluteURL("/").$otherVariables["redirect_uri"])."&";
                } elseif (isset($otherVariables["RedirectURL"])) {

This code seems to be duplicated across your project.

                    $emailLink .= "redirect_uri=".urlencode(Director::absoluteURL("/").$otherVariables["RedirectURL"])."&";
                } else {
                    $emailLink .= "redirect_uri=".urlencode(Director::absoluteURL("/"))."&";
                }
                if (isset($otherVariables["pictureURL"])) {

This code seems to be duplicated across your project.

                    $emailLink .= "picture=".urlencode(Director::absoluteURL("/").$otherVariables["pictureURL"])."&";
                }
                if (isset($otherVariables["description"])) {

This code seems to be duplicated across your project.

                    $emailLink .= "description=".urlencode($otherVariables["description"])."&";
                } elseif ($message) {
                    $emailLink .= "description=".urlencode($message)."&";
                }
                if (isset($otherVariables["name"])) {
                    $emailLink .= "name=".urlencode($otherVariables["name"])."&";
                }
                if (isset($otherVariables["caption"])) {

This code seems to be duplicated across your project.

                    $emailLink .= "caption=".urlencode($otherVariables["caption"])."&";
                } elseif (isset($otherVariables["Subject"])) {
                    $emailLink .= "caption=".urlencode($otherVariables["Subject"])."&";
                }
                $from = Email::getAdminEmail();
                //TO
                //SUBJECT
                $subject = _t("FacebookCallback.ACTION_REQUIRED", "Action required for").": ".$subject;
                //BODY
                $body =
                    _t("FacebookCallback.PLEASE_CLICK_ON_THE_LINK", " Please click on the link ")
                    ." <a href=\"".$emailLink."\" target=\"_blank\">"._t("FacebookCallback.OPEN_FACEBOOK", "open facebook")."</a> ".
                    _t("FacebookCallback.TO_SEND_A_MESSAGE_TO_FRIEND", "to send a message to your friend. ").
                    _t("FacebookCallback.DIRECT_LINK", " You can also send the link directly to your friend: ").$link;
                //BCC
                $bcc = Email::getAdminEmail();

$bcc is not used, you could remove the assignment.

                //SEND
                $email = new Email(
                    $from,
                    $senderEmail,

The variable $senderEmail does not seem to be defined for all execution paths leading up to this point.

                    $subject,
                    $body
                );
                $email->send();
                // We have a user ID, so probably a logged in user.
                // If not, we'll get an exception, which we handle below.
                if (1 == 2) {
                    if ($to instanceof Member) {
                        $to = $to->FacebookUsername;
                    }
                    try {
                        $ret_obj = $facebook->api('/'.$to.'/feed', 'POST', $postArray);

$ret_obj is not used, you could remove the assignment.

                        //SS_Log::log($ret_obj, SS_Log::NOTICE);

59% of this comment could be valid code. Did you maybe forget this after debugging?

                        return $body;
                    } catch (FacebookApiException $e) {

The class FacebookApiException does not exist. Did you forget a USE statement, or did you not list all dependencies?

                        // If the user is logged out, you can have a
                        // user ID even though the access token is invalid.
                        // In this case, we'll get an exception, so we'll
                        // just ask the user to login again here.
                        SS_Log::log($user."---".$e->getType()."---".$e->getMessage()."---".$to."---".$message."---".$link."---".$otherVariables."---".print_r($user, 1).print_r(Member::currentUser(), 1), SS_Log::NOTICE);
                    }
                }
            } else {
                SS_Log::log("tried to send a message from facebook without being logging in...", SS_Log::NOTICE);
            }
        }
        return false;
    }