codysnider / tt-rss

lib/floIcon.php

<?php
// Compatibility shim for hooking up jimIcon to Tiny Tiny RSS.

require_once "jimIcon.php";

class floIconIcon {
        function getImageResource() {

It is generally recommended to explicitly declare the visibility for methods.

                return $this->img;

The property img does not exist on floIconIcon. Did you maybe forget to declare it?

        }
}

class floIcon {
        function readICO($file) {

It is generally recommended to explicitly declare the visibility for methods.

                $jim = new jimIcon();
                $icon = new floIconIcon();
                $icon->img = $jim->fromiconstring(file_get_contents($file));

The property img does not seem to exist on floIconIcon.

                $this->images = array($icon);

The property images does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        }
}
?>

It is not recommended to use PHP's closing tag ?> in files other than templates.