»My sugarcane farm doesn't work

So i've made a sugarcane farm from bedrock to skylimit. At the top i have a crop hopper and a sell chest.. But my crop hopper doesn't seem to collect any sugarcane. What have i missed?


I've used Cobbelstone as a platform with holes for water.
Then i've placed sand on top of the cobblestone.
Then i've placed water in the holes.
And then i've filled the sand with sugarcane.

 

its a spawner economy, farming is disabled.

 

Maybe you placed the hopper wrong idk

 

wait really??? that explains alot lol