RssDocsHandler

Complexity

Total Complexity

3

Size/Duplication

Total Lines	26
Duplicated Lines	100 %

Coupling/Cohesion

Components	0
Dependencies	2

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getName()	4	4	1
A	handleCharacterData()	10	10	2

<?php
/*
 You may not change or alter any portion of this comment or credits
 of supporting developers from this source code or any supporting source code
 which is considered copyrighted (c) material of the original comment or credit authors.

 This program is distributed in the hope that it will be useful,
 but WITHOUT ANY WARRANTY; without even the implied warranty of
 MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
*/

/**
 * @category   Class
 * @package    Xml
 * @author     Kazumi Ono (AKA onokazu)
 * @copyright  2000-2016 XOOPS Project (http://xoops.org)
 * @license    GNU GPL 2 or later (http://www.gnu.org/licenses/gpl-2.0.html)
 * @link        http://xoops.org
 * @since       1.0.0
 */
class XoopsXmlRss2Parser extends SaxParser
{
    /**
     * @var array
     */
    private $_tempArr = array();

    /**
     * @var array
     */
    private $_channelData = array();

    /**
     * @var array
     */
    private $_imageData = array();

    /**
     * @var array
     */
    private $_items = array();

    /**
     * XoopsXmlRss2Parser constructor.
     * @param string $input xml document to parse
     */
    public function __construct(&$input)
    {
        parent::__construct($input);
        $this->useUtfEncoding();
        $this->addTagHandler(new RssChannelHandler());
        $this->addTagHandler(new RssTitleHandler());
        $this->addTagHandler(new RssLinkHandler());
        $this->addTagHandler(new RssGeneratorHandler());
        $this->addTagHandler(new RssDescriptionHandler());
        $this->addTagHandler(new RssCopyrightHandler());
        $this->addTagHandler(new RssNameHandler());
        $this->addTagHandler(new RssManagingEditorHandler());
        $this->addTagHandler(new RssLanguageHandler());
        $this->addTagHandler(new RssLastBuildDateHandler());
        $this->addTagHandler(new RssWebMasterHandler());
        $this->addTagHandler(new RssImageHandler());
        $this->addTagHandler(new RssUrlHandler());
        $this->addTagHandler(new RssWidthHandler());
        $this->addTagHandler(new RssHeightHandler());
        $this->addTagHandler(new RssItemHandler());
        $this->addTagHandler(new RssCategoryHandler());
        $this->addTagHandler(new RssPubDateHandler());
        $this->addTagHandler(new RssCommentsHandler());
        $this->addTagHandler(new RssSourceHandler());
        $this->addTagHandler(new RssAuthorHandler());
        $this->addTagHandler(new RssGuidHandler());
        $this->addTagHandler(new RssTextInputHandler());
    }

    /**
     * @param string $name  channel name
     * @param string $value value
     * @return void
     */
    public function setChannelData($name, &$value)
    {
        if (!isset($this->_channelData[$name])) {
            $this->_channelData[$name] = $value;
        } else {
            $this->_channelData[$name] .= $value;
        }
    }

    /**
     * @param string $name channel name
     * @return array|bool
     */
    public function getChannelData($name = null)
    {
        if (isset($name)) {
            if (isset($this->_channelData[$name])) {
                return $this->_channelData[$name];
            }
            return false;
        }
        return $this->_channelData;
    }

    /**
     * @param string $name  image data name
     * @param string $value value
     * @return void
     */
    public function setImageData($name, &$value)
    {
        $this->_imageData[$name] = $value;
    }

    /**
     * @param string $name image data name
     * @return array|bool
     */
    public function getImageData($name = null)
    {
        if (isset($name)) {
            if (isset($this->_imageData[$name])) {
                return $this->_imageData[$name];
            }
            $return = false;
            return $return;
        }
        return $this->_imageData;
    }

    /**
     * @param array $itemarr
     * @return void
     */
    public function setItems(&$itemarr)
    {
        $this->_items[] = $itemarr;
    }

    /**
     * @return array
     */
    public function getItems()
    {
        return $this->_items;
    }

    /**
     * @param string $name
     * @param array  $value
     * @param string $delim
     * @return void
     */
    public function setTempArr($name, &$value, $delim = '')
    {
        if (!isset($this->_tempArr[$name])) {
            $this->_tempArr[$name] = $value;
        } else {
            $this->_tempArr[$name] .= $delim . $value;
        }
    }

    /**
     * @return array
     */
    public function getTempArr()
    {
        return $this->_tempArr;
    }

    /**
     * @return void
     */
    public function resetTempArr()
    {
        unset($this->_tempArr);
        $this->_tempArr = array();
    }
}

/**
 * Class RssChannelHandler
 */
class RssChannelHandler extends XmlTagHandler

PSR1 recommends that each class should be in its own file to aid autoloaders.

{
    /**
     * @return string
     */
    public function getName()
    {
        return 'channel';
    }
}

/**
 * Class RssTitleHandler
 */
class RssTitleHandler extends XmlTagHandler

This class seems to be duplicated in your project.

PSR1 recommends that each class should be in its own file to aid autoloaders.

{
    /**
     * @return string
     */
    public function getName()
    {
        return 'title';
    }

    /**
     * @param XoopsXmlRss2Parser $parser parser
     * @param array              $data
     * @return void
     */
    public function handleCharacterData(SaxParser $parser, &$data)
    {
        switch ($parser->getParentTag()) {
            case 'channel':
                $parser->setChannelData('title', $data);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setChannelData() does only exist in the following sub-classes of SaxParser: XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

                break;
            case 'image':
                $parser->setImageData('title', $data);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setImageData() does only exist in the following sub-classes of SaxParser: XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

                break;
            case 'item':
            case 'textInput':
                $parser->setTempArr('title', $data);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setTempArr() does only exist in the following sub-classes of SaxParser: XoopsThemeSetParser, XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

                break;
            default:
                break;
        }
    }
}

/**
 * Class RssLinkHandler
 */
class RssLinkHandler extends XmlTagHandler

This class seems to be duplicated in your project.

PSR1 recommends that each class should be in its own file to aid autoloaders.

{
    /**
     * @return string
     */
    public function getName()
    {
        return 'link';
    }

    /**
     * @param XoopsXmlRss2Parser $parser parser
     * @param array              $data
     * @return void
     */
    public function handleCharacterData(SaxParser $parser, &$data)
    {
        switch ($parser->getParentTag()) {
            case 'channel':
                $parser->setChannelData('link', $data);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setChannelData() does only exist in the following sub-classes of SaxParser: XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

                break;
            case 'image':
                $parser->setImageData('link', $data);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setImageData() does only exist in the following sub-classes of SaxParser: XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

                break;
            case 'item':
            case 'textInput':
                $parser->setTempArr('link', $data);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setTempArr() does only exist in the following sub-classes of SaxParser: XoopsThemeSetParser, XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

                break;
            default:
                break;
        }
    }
}

/**
 * Class RssDescriptionHandler
 */
class RssDescriptionHandler extends XmlTagHandler

This class seems to be duplicated in your project.

PSR1 recommends that each class should be in its own file to aid autoloaders.

{
    /**
     * @return string
     */
    public function getName()
    {
        return 'description';
    }

    /**
     * @param XoopsXmlRss2Parser $parser parser
     * @param array              $data
     * @return void
     */
    public function handleCharacterData(SaxParser $parser, &$data)
    {
        switch ($parser->getParentTag()) {
            case 'channel':
                $parser->setChannelData('description', $data);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setChannelData() does only exist in the following sub-classes of SaxParser: XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

                break;
            case 'image':
                $parser->setImageData('description', $data);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setImageData() does only exist in the following sub-classes of SaxParser: XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

                break;
            case 'item':
            case 'textInput':
                $parser->setTempArr('description', $data);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setTempArr() does only exist in the following sub-classes of SaxParser: XoopsThemeSetParser, XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

                break;
            default:
                break;
        }
    }
}

/**
 * Class RssGeneratorHandler
 */
class RssGeneratorHandler extends XmlTagHandler

PSR1 recommends that each class should be in its own file to aid autoloaders.

This class seems to be duplicated in your project.

{

    /**
     * @return string
     */
    public function getName()
    {
        return 'generator';
    }

    /**
     * @param XoopsXmlRss2Parser $parser parser
     * @param array              $data
     * @return void
     */
    public function handleCharacterData(SaxParser $parser, &$data)
    {
        switch ($parser->getParentTag()) {
            case 'channel':
                $parser->setChannelData('generator', $data);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setChannelData() does only exist in the following sub-classes of SaxParser: XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

                break;
            default:
                break;
        }
    }
}

/**
 * Class RssCopyrightHandler
 */
class RssCopyrightHandler extends XmlTagHandler

PSR1 recommends that each class should be in its own file to aid autoloaders.

This class seems to be duplicated in your project.

{
    /**
     * @return string
     */
    public function getName()
    {
        return 'copyright';
    }

    /**
     * @param XoopsXmlRss2Parser $parser parser
     * @param array              $data   data
     * @return void
     */
    public function handleCharacterData(SaxParser $parser, &$data)
    {
        switch ($parser->getParentTag()) {
            case 'channel':
                $parser->setChannelData('copyright', $data);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setChannelData() does only exist in the following sub-classes of SaxParser: XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

                break;
            default:
                break;
        }
    }
}

/**
 * Class RssNameHandler
 */
class RssNameHandler extends XmlTagHandler

PSR1 recommends that each class should be in its own file to aid autoloaders.

This class seems to be duplicated in your project.

{
    /**
     * @return string
     */
    public function getName()
    {
        return 'name';
    }

    /**
     * @param XoopsXmlRss2Parser $parser parser
     * @param array $data
     * @return void
     */
    public function handleCharacterData(SaxParser $parser, &$data)
    {
        switch ($parser->getParentTag()) {
            case 'textInput':
                $parser->setTempArr('name', $data);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setTempArr() does only exist in the following sub-classes of SaxParser: XoopsThemeSetParser, XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

                break;
            default:
                break;
        }
    }
}

/**
 * Class RssManagingEditorHandler
 */
class RssManagingEditorHandler extends XmlTagHandler

PSR1 recommends that each class should be in its own file to aid autoloaders.

This class seems to be duplicated in your project.

{

    /**
     * @return string
     */
    public function getName()
    {
        return 'managingEditor';
    }

    /**
     * @param XoopsXmlRss2Parser $parser parser
     * @param array $data
     * @return void
     */
    public function handleCharacterData(SaxParser $parser, &$data)
    {
        switch ($parser->getParentTag()) {
            case 'channel':
                $parser->setChannelData('editor', $data);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setChannelData() does only exist in the following sub-classes of SaxParser: XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

                break;
            default:
                break;
        }
    }
}

/**
 * Class RssLanguageHandler
 */
class RssLanguageHandler extends XmlTagHandler

PSR1 recommends that each class should be in its own file to aid autoloaders.

This class seems to be duplicated in your project.

{

    /**
     * @return string
     */
    public function getName()
    {
        return 'language';
    }

    /**
     * @param XoopsXmlRss2Parser $parser parser
     * @param array $data
     * @return void
     */
    public function handleCharacterData(SaxParser $parser, &$data)
    {
        switch ($parser->getParentTag()) {
            case 'channel':
                $parser->setChannelData('language', $data);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setChannelData() does only exist in the following sub-classes of SaxParser: XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

                break;
            default:
                break;
        }
    }
}

/**
 * Class RssWebMasterHandler
 */
class RssWebMasterHandler extends XmlTagHandler

PSR1 recommends that each class should be in its own file to aid autoloaders.

This class seems to be duplicated in your project.

{
    /**
     * @return string
     */
    public function getName()
    {
        return 'webMaster';
    }

    /**
     * @param XoopsXmlRss2Parser $parser parser
     * @param array $data
     * @return void
     */
    public function handleCharacterData(SaxParser $parser, &$data)
    {
        switch ($parser->getParentTag()) {
            case 'channel':
                $parser->setChannelData('webmaster', $data);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setChannelData() does only exist in the following sub-classes of SaxParser: XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

                break;
            default:
                break;
        }
    }
}

/**
 * Class RssDocsHandler
 */
class RssDocsHandler extends XmlTagHandler

This class seems to be duplicated in your project.

PSR1 recommends that each class should be in its own file to aid autoloaders.

{
    /**
     * @return string
     */
    public function getName()
    {
        return 'docs';
    }

    /**
     * @param XoopsXmlRss2Parser $parser parser
     * @param array $data
     * @return void
     */
    public function handleCharacterData(SaxParser $parser, &$data)
    {
        switch ($parser->getParentTag()) {
            case 'channel':
                $parser->setChannelData('docs', $data);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setChannelData() does only exist in the following sub-classes of SaxParser: XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

                break;
            default:
                break;
        }
    }
}

/**
 * Class RssTtlHandler
 */
class RssTtlHandler extends XmlTagHandler

This class seems to be duplicated in your project.

PSR1 recommends that each class should be in its own file to aid autoloaders.

{
    /**
     * @return string
     */
    public function getName()
    {
        return 'ttl';
    }

    /**
     * @param XoopsXmlRss2Parser $parser parser
     * @param array $data
     * @return void
     */
    public function handleCharacterData(SaxParser $parser, &$data)
    {
        switch ($parser->getParentTag()) {
            case 'channel':
                $parser->setChannelData('ttl', $data);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setChannelData() does only exist in the following sub-classes of SaxParser: XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

                break;
            default:
                break;
        }
    }
}

/**
 * Class RssTextInputHandler
 */
class RssTextInputHandler extends XmlTagHandler

PSR1 recommends that each class should be in its own file to aid autoloaders.

{
    /**
     * @return string
     */
    public function getName()
    {
        return 'textInput';
    }

    /**
     * @param XoopsXmlRss2Parser $parser parser
     * @param array $attributes
     * @return void
     */
    public function handleBeginElement(SaxParser $parser, &$attributes)
    {
        $parser->resetTempArr();

It seems like you code against a specific sub-type and not the parent class SaxParser as the method resetTempArr() does only exist in the following sub-classes of SaxParser: XoopsThemeSetParser, XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

    }

    /**
     * @param XoopsXmlRss2Parser $parser parser
     * @return void
     */
    public function handleEndElement(SaxParser $parser)
    {
        $parser->setChannelData('textinput', $parser->getTempArr());

It seems like you code against a specific sub-type and not the parent class SaxParser as the method getTempArr() does only exist in the following sub-classes of SaxParser: XoopsThemeSetParser, XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setChannelData() does only exist in the following sub-classes of SaxParser: XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

    }
}

/**
 * Class RssLastBuildDateHandler
 */
class RssLastBuildDateHandler extends XmlTagHandler

PSR1 recommends that each class should be in its own file to aid autoloaders.

This class seems to be duplicated in your project.

{
    /**
     * @return string
     */
    public function getName()
    {
        return 'lastBuildDate';
    }

    /**
     * @param XoopsXmlRss2Parser $parser parser
     * @param array $data
     * @return void
     */
    public function handleCharacterData(SaxParser $parser, &$data)
    {
        switch ($parser->getParentTag()) {
            case 'channel':
                $parser->setChannelData('lastbuilddate', $data);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setChannelData() does only exist in the following sub-classes of SaxParser: XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

                break;
            default:
                break;
        }
    }
}

/**
 * Class RssImageHandler
 */
class RssImageHandler extends XmlTagHandler

PSR1 recommends that each class should be in its own file to aid autoloaders.

{
    /**
     * @return string
     */
    public function getName()
    {
        return 'image';
    }
}

/**
 * Class RssUrlHandler
 */
class RssUrlHandler extends XmlTagHandler

PSR1 recommends that each class should be in its own file to aid autoloaders.

{
    /**
     * @return string
     */
    public function getName()
    {
        return 'url';
    }

    /**
     * @param XoopsXmlRss2Parser $parser parser
     * @param array $data
     * @return void
     */
    public function handleCharacterData(SaxParser $parser, &$data)
    {
        if ($parser->getParentTag() === 'image') {
            $parser->setImageData('url', $data);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setImageData() does only exist in the following sub-classes of SaxParser: XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

        }
    }
}

/**
 * Class RssWidthHandler
 */
class RssWidthHandler extends XmlTagHandler

PSR1 recommends that each class should be in its own file to aid autoloaders.

{
    /**
     * @return string
     */
    public function getName()
    {
        return 'width';
    }

    /**
     * @param XoopsXmlRss2Parser $parser parser
     * @param array $data
     * @return void
     */
    public function handleCharacterData(SaxParser $parser, &$data)
    {
        if ($parser->getParentTag() === 'image') {
            $parser->setImageData('width', $data);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setImageData() does only exist in the following sub-classes of SaxParser: XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

        }
    }
}

/**
 * Class RssHeightHandler
 */
class RssHeightHandler extends XmlTagHandler

PSR1 recommends that each class should be in its own file to aid autoloaders.

{
    /**
     * @return string
     */
    public function getName()
    {
        return 'height';
    }

    /**
     * @param XoopsXmlRss2Parser $parser parser
     * @param array $data
     * @return void
     */
    public function handleCharacterData(SaxParser $parser, &$data)
    {
        if ($parser->getParentTag() === 'image') {
            $parser->setImageData('height', $data);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setImageData() does only exist in the following sub-classes of SaxParser: XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

        }
    }
}

/**
 * Class RssItemHandler
 */
class RssItemHandler extends XmlTagHandler

PSR1 recommends that each class should be in its own file to aid autoloaders.

{
    /**
     * @return string
     */
    public function getName()
    {
        return 'item';
    }

    /**
     * @param XoopsXmlRss2Parser $parser parser
     * @param array              $attributes
     * @return void
     */
    public function handleBeginElement(SaxParser $parser, &$attributes)
    {
        $parser->resetTempArr();

It seems like you code against a specific sub-type and not the parent class SaxParser as the method resetTempArr() does only exist in the following sub-classes of SaxParser: XoopsThemeSetParser, XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

    }

    /**
     * @param XoopsXmlRss2Parser $parser parser
     * @return void
     */
    public function handleEndElement(SaxParser $parser)
    {
        $items = $parser->getTempArr();

It seems like you code against a specific sub-type and not the parent class SaxParser as the method getTempArr() does only exist in the following sub-classes of SaxParser: XoopsThemeSetParser, XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

        $parser->setItems($items);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setItems() does only exist in the following sub-classes of SaxParser: XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

    }
}

/**
 * Class RssCategoryHandler
 */
class RssCategoryHandler extends XmlTagHandler

PSR1 recommends that each class should be in its own file to aid autoloaders.

This class seems to be duplicated in your project.

{
    /**
     * @return string
     */
    public function getName()
    {
        return 'category';
    }

    /**
     * @param XoopsXmlRss2Parser $parser parser
     * @param array $data
     * @return void
     */
    public function handleCharacterData(SaxParser $parser, &$data)
    {
        switch ($parser->getParentTag()) {
            case 'channel':
                $parser->setChannelData('category', $data);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setChannelData() does only exist in the following sub-classes of SaxParser: XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

                break;
            case 'item':
                $parser->setTempArr('category', $data, ', ');

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setTempArr() does only exist in the following sub-classes of SaxParser: XoopsThemeSetParser, XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

                break;
            default:
                break;
        }
    }
}

/**
 * Class RssCommentsHandler
 */
class RssCommentsHandler extends XmlTagHandler

PSR1 recommends that each class should be in its own file to aid autoloaders.

{
    /**
     * @return string
     */
    public function getName()
    {
        return 'comments';
    }

    /**
     * @param XoopsXmlRss2Parser $parser parser
     * @param array $data
     * @return void
     */
    public function handleCharacterData(SaxParser $parser, &$data)
    {
        if ($parser->getParentTag() === 'item') {
            $parser->setTempArr('comments', $data);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setTempArr() does only exist in the following sub-classes of SaxParser: XoopsThemeSetParser, XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

        }
    }
}

/**
 * Class RssPubDateHandler
 */
class RssPubDateHandler extends XmlTagHandler

PSR1 recommends that each class should be in its own file to aid autoloaders.

This class seems to be duplicated in your project.

{
    /**
     * @return string
     */
    public function getName()
    {
        return 'pubDate';
    }

    /**
     * @param XoopsXmlRss2Parser $parser parser
     * @param array $data
     * @return void
     */
    public function handleCharacterData(SaxParser $parser, &$data)
    {
        switch ($parser->getParentTag()) {
            case 'channel':
                $parser->setChannelData('pubdate', $data);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setChannelData() does only exist in the following sub-classes of SaxParser: XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

                break;
            case 'item':
                $parser->setTempArr('pubdate', $data);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setTempArr() does only exist in the following sub-classes of SaxParser: XoopsThemeSetParser, XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

                break;
            default:
                break;
        }
    }
}

/**
 * Class RssGuidHandler
 */
class RssGuidHandler extends XmlTagHandler

PSR1 recommends that each class should be in its own file to aid autoloaders.

{
    /**
     * @return string
     */
    public function getName()
    {
        return 'guid';
    }

    /**
     * @param XoopsXmlRss2Parser $parser parser
     * @param array $data
     * @return void
     */
    public function handleCharacterData(SaxParser $parser, &$data)
    {
        if ($parser->getParentTag() === 'item') {
            $parser->setTempArr('guid', $data);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setTempArr() does only exist in the following sub-classes of SaxParser: XoopsThemeSetParser, XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

        }
    }
}

/**
 * Class RssAuthorHandler
 */
class RssAuthorHandler extends XmlTagHandler

PSR1 recommends that each class should be in its own file to aid autoloaders.

{
    /**
     * @return string
     */
    public function getName()
    {
        return 'author';
    }

    /**
     * @param XoopsXmlRss2Parser $parser parser
     * @param array $data
     * @return void
     */
    public function handleCharacterData(SaxParser $parser, &$data)
    {
        if ($parser->getParentTag() === 'item') {
            $parser->setTempArr('author', $data);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setTempArr() does only exist in the following sub-classes of SaxParser: XoopsThemeSetParser, XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

        }
    }
}

/**
 * Class RssSourceHandler
 */
class RssSourceHandler extends XmlTagHandler

PSR1 recommends that each class should be in its own file to aid autoloaders.

{
    /**
     * @return string
     */
    public function getName()
    {
        return 'source';
    }

    /**
     * @param XoopsXmlRss2Parser $parser parser
     * @param array $attributes
     * @return void
     */
    public function handleBeginElement(SaxParser $parser, &$attributes)
    {
        if ($parser->getParentTag() === 'item') {
            $parser->setTempArr('source_url', $attributes['url']);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setTempArr() does only exist in the following sub-classes of SaxParser: XoopsThemeSetParser, XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

        }
    }

    /**
     * @param XoopsXmlRss2Parser $parser parser
     * @param array $data
     * @return void
     */
    public function handleCharacterData(SaxParser $parser, &$data)
    {
        if ($parser->getParentTag() === 'item') {
            $parser->setTempArr('source', $data);

It seems like you code against a specific sub-type and not the parent class SaxParser as the method setTempArr() does only exist in the following sub-classes of SaxParser: XoopsThemeSetParser, XoopsXmlRss2Parser. Maybe you want to instanceof check for one of these explicitly?

        }
    }
}