Using global is bad practice

[jerrywham]

First of all, thank you for this wonderfull and simple app. I love it.

To improve it, I propose you to change global settings for steam and coverart classes to private attribute. It will be more secure and your objects will be more strongs.

So, instead of :

    public function __construct()
    {
        //  get the user's settings
        global $settings;

        //  import the user's settings
        $this->settings = $settings;
       (...)

for the constructor of this two objects, you should used :

    private $settings = array();


    public function __construct(array $settings)
    {

        //  import the user's settings
        $this->settings = $settings;
        (...)

In api/index.php, you have to call these objects with new Steam($settings) and new CoverArt($settings).
And in currentlyPlaying method of CoverArt, new Steam($this->settings) .