@heddn,
Thanks for your detailed review.
1) I have corrected the comments.
2) I have never used hook_theme() actually. So i will implement the same and update here.
3) I actually wanted code readability to be good, which is the reason i have seperated helper_functions, blocks, and menus-permissions via different files.
@klausi, @guptas89, @shipra, @nguerrero:
Thank you for your reviews !
it was fun working in a very well settled review process and i also learned some things from here, which i definetly did not knew before the contribution process.
For example : using placeholders in strings and about sanitization functions + Drupal translations.
ONLY ONE THING PENDING
As heddn mentioned, that hook_theme() implementation will give some colors to this module, But my problem is that i have never used hook_theme().
i google'd around but all i got was that i will have to create a custom TPL file to use hook_theme(). (I am not sure if this is what u meant @heddn)
@heddn : Can you please share any usefull link which will help me in my hook_theme() implementation. i would be really thankful.