@heddn,
Thanks for your detailed review.
1) I have corrected the comments.
2)...

Contribution Date
Technology
Contribution Details

@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.

Contribution Author
Files count
0
Patches count
0