1. configure = admin/config/development/content_type_settings
There is no use of this in Info file, so can we remove it?
2. Can we remove blank space after below lines:
'access arguments' => array('custom content type dashboard'),
'access callback' => 'user_access',
'file' => 'inc/content_dashboard.inc',
3. $form['quick_filter']['Value'] = array(
contentdashboard.inc (Value is added in caps, it should be value)
4. All the private function name should start with module name
eg._get_data function name should be _content_dashboard_get_data
Comments
Comment #2
sandeepscs commentedComment #3
sandeepscs commentedComment #4
swati.nuna commentedComment #6
swati.nuna commentedFixed.
Comment #7
sandeepscs commentedComment #8
sandeepscs commented