The previous code has a bug, in that it assigns the names of the objects, not the objects themselves.
It also has complex looping logic setting and examining $marker which in fact does nothing.
I found that both count() and empty() do not work as expected on the array returned by the function - I have not worked out why, just avoided their use.
Note that this does not change the 'click here to download' file.
Coverage remained the same at 88.771% when pulling 1ea4f67901cee4d97a6f3dfc8643eaa33388b9b9 on myshkin-uk:patch-1 into ea17effa6b532a11c3e6593419653812417f8480 on rackspace:working.
The previous code has a bug, in that it assigns the names of the objects, not the objects themselves.
It also has complex looping logic setting and examining $marker which in fact does nothing.
I found that both count() and empty() do not work as expected on the array returned by the function - I have not worked out why, just avoided their use.
Note that this does not change the 'click here to download' file.