From 78d3e2164116f299effb11d25b1971e6d375366e Mon Sep 17 00:00:00 2001 From: Bernhard Posselt Date: Wed, 22 Oct 2014 01:35:59 +0200 Subject: get rid of simplepie --- utility/picofeedclientfactory.php | 40 +++++++++++++++++++++++++++++ utility/simplepieapifactory.php | 53 --------------------------------------- 2 files changed, 40 insertions(+), 53 deletions(-) create mode 100644 utility/picofeedclientfactory.php delete mode 100644 utility/simplepieapifactory.php (limited to 'utility') diff --git a/utility/picofeedclientfactory.php b/utility/picofeedclientfactory.php new file mode 100644 index 000000000..527d0e13c --- /dev/null +++ b/utility/picofeedclientfactory.php @@ -0,0 +1,40 @@ + + * @author Bernhard Posselt + * @copyright Alessandro Cosentino 2012 + * @copyright Bernhard Posselt 2012, 2014 + */ + + +namespace OCA\News\Utility; + +use \PicoFeed\Config; +use \PicoFeed\Client; + +class PicoFeedClientFactory { + + private $config; + + public function __construct(Config $config) { + $this->config = $config; + } + + + /** + * Returns a new instance of an PicoFeed Http client + * @return \PicoFeed\Client instance + */ + public function build() { + $client = Client::getInstance(); + $client->setConfig($this->config); + return $client; + } + + +} \ No newline at end of file diff --git a/utility/simplepieapifactory.php b/utility/simplepieapifactory.php deleted file mode 100644 index 92cbe18f3..000000000 --- a/utility/simplepieapifactory.php +++ /dev/null @@ -1,53 +0,0 @@ - - * @author Bernhard Posselt - * @copyright Alessandro Cosentino 2012 - * @copyright Bernhard Posselt 2012, 2014 - */ - - -namespace OCA\News\Utility; - - -class SimplePieAPIFactory { - - /** - * Builds a simplepie file object. This is needed because - * the file object contains logic in its constructor which makes it - * impossible to inject and test - * - * @param $url - * @param int $timeout - * @param int $redirects - * @param string $headers - * @param string $useragent - * @param bool $force_fsockopen - * @return \SimplePie_File a new object - */ - public function getFile($url, $timeout=10, $redirects=5, $headers=null, - $useragent=null, $force_fsockopen=false) { - - return new \SimplePie_File($url, $timeout, $redirects, $headers, - $useragent, $force_fsockopen); - } - - - /** - * Returns a new instance of a SimplePie_Core() object. This is needed - * because the class relies on external dependencies which are not passed - * in via the constructor and thus making it nearly impossible to unit test - * code that uses this class - * @return \SimplePie_Core instance - */ - public function getCore() { - return new \SimplePie(); - } - - -} \ No newline at end of file -- cgit v1.2.3