Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Remove deprecated getFeedParser method from JFactory #1852

Merged
merged 1 commit into from

2 participants

@elinw

The deprecation notice said that the proxied method would be removed without replacement in 12.3. This is currently proxying a method that does not exist. I considered proxying to JFeedFactory but I really don't see why this is being done at all and the fact that from what I can tell no one has complained about the non existent method means it can go away.

@eddieajau eddieajau merged commit e429676 into joomla:staging
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Mar 19, 2013
  1. @elinw

    Update factory.php

    elinw authored
This page is out of date. Refresh to see the latest.
Showing with 0 additions and 23 deletions.
  1. +0 −23 libraries/joomla/factory.php
View
23 libraries/joomla/factory.php
@@ -355,29 +355,6 @@ public static function getMailer()
}
/**
- * Get a parsed XML Feed Source
- *
- * @param string $url Url for feed source.
- * @param integer $cache_time Time to cache feed for (using internal cache mechanism).
- *
- * @return mixed SimplePie parsed object on success, false on failure.
- *
- * @since 11.1
- * @deprecated 13.3 Use JSimplepieFactory::getFeedParser() instead.
- */
- public static function getFeedParser($url, $cache_time = 0)
- {
- if (!class_exists('JSimplepieFactory'))
- {
- throw new BadMethodCallException('JSimplepieFactory not found');
- }
-
- JLog::add(__METHOD__ . ' is deprecated. Use JSimplepieFactory::getFeedParser() instead.', JLog::WARNING, 'deprecated');
-
- return JSimplepieFactory::getFeedParser($url, $cache_time);
- }
-
- /**
* Reads a XML file.
*
* @param string $data Full path and file name.
Something went wrong with that request. Please try again.