sunnysideup / silverstripe-sharethis

code/api/MyTwitter.php

<?php

namespace SunnysideUp\ShareThis;

use SilverStripe\Core\Injector\Injectable;
use SilverStripe\Control\Session;
use SilverStripe\Core\Config\Config;
use SunnysideUp\ShareThis\MyTwitter;
use SunnysideUp\ShareThis\MyTwitterData;
use SilverStripe\Control\Director;
use SunnysideUp\ShareThis\TwitterOAuth;
use SilverStripe\ORM\ArrayList;
use SilverStripe\ORM\FieldType\DBDatetime;
use SilverStripe\View\ArrayData;

/**
 * @author romain [at] sunnys side up .co.nz + nicolaas [at] sunny side up . co .nz
 * @inspiration: https://github.com/tylerkidd/silverstripe-twitter-feed/
 * @funding: MSO Design (www.msodesign.com)
 *
 **/
class MyTwitter
{
    use Injectable;

    /**
     * @var boolean
     */
    private static $debug = false;

The private property $debug is not used, and could be removed.

    /**
     * @var array
     */
    private static $singletons = [];

    /**
     * @var boolean
     */
    private static $favourites_only = false;

The private property $favourites_only is not used, and could be removed.

    /**
     * @var boolean
     */
    private static $non_replies_only = false;

The private property $non_replies_only is not used, and could be removed.

    /**
     * @var string
     */
    private static $twitter_consumer_key = "";

The private property $twitter_consumer_key is not used, and could be removed.

    /**
     * @var string
     */
    private static $twitter_consumer_secret = "";

The private property $twitter_consumer_secret is not used, and could be removed.

    /**
     * @var string
     */
    private static $titter_oauth_token = "";

The private property $titter_oauth_token is not used, and could be removed.

    /**
     * @var string
     */
    private static $titter_oauth_token_secret = "";

The private property $titter_oauth_token_secret is not used, and could be removed.

    /**
     * @var array
     */
    private static $twitter_config = [

The private property $twitter_config is not used, and could be removed.

        'include_entities' => 'true',
        'include_rts' => 'true'
    ];

    /**
     * returns a DataObjetSet of the last $count tweets.
     * - saves twitter feed to dataobject
     *
     * @param String $username (e.g. mytwitterhandle)
     * @param Int $count - number of tweets to retrieve at any one time
     * @return DataObjectSet | Null

The type SunnysideUp\ShareThis\DataObjectSet was not found. Maybe you did not declare it correctly or list all dependencies?

     */
    public static function last_statuses($username, $count = 1, $useHourlyCache = true)
    {
        if (!$username) {
            user_error("No username provided");
        }

        $sessionName = "MyTwitterFeeds$username".date("Ymdh");

        if (Session::get($sessionName) && $useHourlyCache && !Config::inst()->get(MyTwitter::class, "debug")) {

The method SilverStripe\Control\Session::get() is not static, but was called statically.

            //do nothing
        } else {
            if (empty(self::$singletons[$username])) {
                self::$singletons[$username] = new MyTwitter($username, $count);

The call to SunnysideUp\ShareThis\MyTwitter::__construct() has too many arguments starting with $username.

            }

            $dataObjectSet = self::$singletons[$username]->TwitterFeed($username, $count);

            if ($dataObjectSet && $dataObjectSet->count()) {
                foreach ($dataObjectSet as $tweet) {
                    if (!MyTwitterData::get()->filter(["TwitterID" => $tweet->ID])->count()) {
                        $myTwitterData = new MyTwitterData();
                        $myTwitterData->TwitterID = $tweet->ID;

The property TwitterID does not exist on SunnysideUp\ShareThis\MyTwitterData. Since you implemented __set, consider adding a @property annotation.

                        $myTwitterData->Title = $tweet->Title;
                        $myTwitterData->Date = $tweet->Date;

The property Date does not exist on SunnysideUp\ShareThis\MyTwitterData. Since you implemented __set, consider adding a @property annotation.

                        $myTwitterData->write();
                    }
                }
            }

            Session::set($sessionName, 1);

The method SilverStripe\Control\Session::set() is not static, but was called statically.

        }

        Config::inst()->update(MyTwitterData::class, "username", $username);

The method update() does not exist on SilverStripe\Config\Collections\ConfigCollectionInterface. It seems like you code against a sub-type of SilverStripe\Config\Collections\ConfigCollectionInterface such as SilverStripe\Config\Collections\MemoryConfigCollection.

        return MyTwitterData::get()->filter(["Hide" => 0])->limit($count);
    }


    /**
     * retries latest tweets from Twitter
     *
     * @param String $username (e.g. mytwitterhandle)
     * @param Int $count - number of tweets to retrieve at any one time
     * @return DataObjectSet | Null
     */
    public function TwitterFeed($username, $count = 5)
    {
        if (!$username) {
            user_error("No username provided");
        }

        Config::inst()->update(MyTwitterData::class, "username", $username);

        //check settings are available
        $requiredSettings = [
            "twitter_consumer_key",
            "twitter_consumer_secret",
            "titter_oauth_token",
            "titter_oauth_token"
        ];

        foreach ($requiredSettings as $setting) {
            if (!Config::inst()->get(MyTwitter::class, $setting)) {
                user_error(" you must set MyTwitter::$setting", E_USER_NOTICE);
                return null;
            }
        }

        require_once(Director::baseFolder().'/'.SS_SHARETHIS_DIR.'/third_party/twitter_oauth/TwitterOAuthConsumer.php');

        $connection = new TwitterOAuth(
            Config::inst()->get(MyTwitter::class, "twitter_consumer_key"),
            Config::inst()->get(MyTwitter::class, "twitter_consumer_secret"),
            Config::inst()->get(MyTwitter::class, "titter_oauth_token"),
            Config::inst()->get(MyTwitter::class, "titter_oauth_token_secret")
        );

        $config = Config::inst()->get(MyTwitter::class, "twitter_config");
        $config['screen_name'] = $username;
        $tweets = $connection->get('statuses/user_timeline', $config);
        $tweetList = new ArrayList();

        if (count($tweets) > 0 && !isset($tweets->error)) {
            $i = 0;

            foreach ($tweets as $tweet) {
                if (Config::inst()->get(MyTwitter::class, "favourites_only") && $tweet->favorite_count == 0) {
                    break;
                }

                if (Config::inst()->get(MyTwitter::class, "non_replies_only") && $tweet->in_reply_to_status_id) {
                    break;
                }

                if (Config::inst()->get(MyTwitter::class, "debug")) {
                    print_r($tweet);
                }

                if (++$i > $count) {
                    break;
                }

                $date = new DBDatetime();
                $date->setValue(strtotime($tweet->created_at));
                $text = htmlentities($tweet->text, ENT_NOQUOTES, $encoding = "UTF-8", $doubleEncode = false);

                if (!empty($tweet->entities) && !empty($tweet->entities->urls)) {
                    foreach ($tweet->entities->urls as $url) {
                        if (!empty($url->url) && !empty($url->display_url)) {
                            $text = str_replace($url->url, '<a href="'.$url->url.'" class="external">'.$url->display_url.'</a>', $text);
                        }
                    }
                }

                $tweetList->push(
                    new ArrayData([
                        'ID' => $tweet->id_str,
                        'Title' => $text,
                        'Date' => $date
                    ])
                );
            }
        }

        return $tweetList;
    }
}