Review comments

We should consider updating this module for adhl 3.0
I think that it's okay to have todo's as reminders in the code as some may also represent stuff that could be fixed if time or that there exists tickets on it in the issue tracker.

@kasperg what is your take on this ?