Review of the 7.x-1.x branch: ./embridge_component/components/entermediaasset.inc: all functions should be prefixed with your module/theme name to avoid name clashes. See http://drupal.org/node/318#naming function _webform_defaults_entermediaasset() { function _webform_theme_entermediaasset() { function _webform_edit_entermediaasset($component) { function _webform_render_entermediaasset($component, $value = NULL, $filter = TRUE) { function _webform_display_entermediaasset($component, $value, $format = 'html') { function _webform_delete_entermediaasset($component, $value) { function _webform_analysis_entermediaasset($component, $sids = array()) { function _webform_table_entermediaasset($component, $value) { function _webform_csv_headers_entermediaasset($component, $export_options) { function _webform_csv_data_entermediaasset($component, $export_options, $value) { Drupal Code Sniffer has found some issues with your code (please check the Drupal coding standards). See attachment. This automated report was generated with PAReview.sh, your friendly project application review script. You can also use the online version to check your project. Get a review bonus and we will come back to your application sooner. FILE: ...eview/sites/all/modules/pareview_temp/test_candidate/css/bulkupload.css -------------------------------------------------------------------------------- FOUND 46 ERROR(S) AFFECTING 46 LINE(S) -------------------------------------------------------------------------------- 32 | ERROR | Multiple CSS properties should be listed in alphabetical order 33 | ERROR | Multiple CSS properties should be listed in alphabetical order 48 | ERROR | Multiple CSS properties should be listed in alphabetical order 105 | ERROR | Multiple CSS properties should be listed in alphabetical order 110 | ERROR | Multiple CSS properties should be listed in alphabetical order 118 | ERROR | Multiple CSS properties should be listed in alphabetical order 119 | ERROR | Multiple CSS properties should be listed in alphabetical order 124 | ERROR | Multiple CSS properties should be listed in alphabetical order 145 | ERROR | Multiple CSS properties should be listed in alphabetical order 146 | ERROR | Multiple CSS properties should be listed in alphabetical order 149 | ERROR | Multiple CSS properties should be listed in alphabetical order 150 | ERROR | Multiple CSS properties should be listed in alphabetical order 156 | ERROR | Multiple CSS properties should be listed in alphabetical order 158 | ERROR | Multiple CSS properties should be listed in alphabetical order 159 | ERROR | Multiple CSS properties should be listed in alphabetical order 186 | ERROR | Multiple CSS properties should be listed in alphabetical order 188 | ERROR | Multiple CSS properties should be listed in alphabetical order 189 | ERROR | Multiple CSS properties should be listed in alphabetical order 191 | ERROR | Multiple CSS properties should be listed in alphabetical order 196 | ERROR | Multiple CSS properties should be listed in alphabetical order 197 | ERROR | Multiple CSS properties should be listed in alphabetical order 237 | ERROR | Multiple CSS properties should be listed in alphabetical order 238 | ERROR | Multiple CSS properties should be listed in alphabetical order 254 | ERROR | Multiple CSS properties should be listed in alphabetical order 260 | ERROR | Multiple CSS properties should be listed in alphabetical order 261 | ERROR | Multiple CSS properties should be listed in alphabetical order 268 | ERROR | Multiple CSS properties should be listed in alphabetical order 269 | ERROR | Multiple CSS properties should be listed in alphabetical order 275 | ERROR | Multiple CSS properties should be listed in alphabetical order 281 | ERROR | Multiple CSS properties should be listed in alphabetical order 287 | ERROR | Multiple CSS properties should be listed in alphabetical order 293 | ERROR | Multiple CSS properties should be listed in alphabetical order 295 | ERROR | Multiple CSS properties should be listed in alphabetical order 298 | ERROR | Multiple CSS properties should be listed in alphabetical order 306 | ERROR | Multiple CSS properties should be listed in alphabetical order 307 | ERROR | Multiple CSS properties should be listed in alphabetical order 313 | ERROR | Multiple CSS properties should be listed in alphabetical order 319 | ERROR | Multiple CSS properties should be listed in alphabetical order 326 | ERROR | Multiple CSS properties should be listed in alphabetical order 327 | ERROR | Multiple CSS properties should be listed in alphabetical order 334 | ERROR | Multiple CSS properties should be listed in alphabetical order 337 | ERROR | Multiple CSS properties should be listed in alphabetical order 339 | ERROR | Multiple CSS properties should be listed in alphabetical order 402 | ERROR | Multiple CSS properties should be listed in alphabetical order 403 | ERROR | Multiple CSS properties should be listed in alphabetical order 404 | ERROR | Multiple CSS properties should be listed in alphabetical order -------------------------------------------------------------------------------- FILE: ...eview/sites/all/modules/pareview_temp/test_candidate/css/entermedia.css -------------------------------------------------------------------------------- FOUND 3 ERROR(S) AFFECTING 3 LINE(S) -------------------------------------------------------------------------------- 16 | ERROR | Multiple CSS properties should be listed in alphabetical order 31 | ERROR | Multiple CSS properties should be listed in alphabetical order 46 | ERROR | Multiple CSS properties should be listed in alphabetical order -------------------------------------------------------------------------------- FILE: ...pareview/sites/all/modules/pareview_temp/test_candidate/embridge.module -------------------------------------------------------------------------------- FOUND 14 ERROR(S) AFFECTING 14 LINE(S) -------------------------------------------------------------------------------- 101 | ERROR | Do not use t() in hook_menu() 102 | ERROR | Do not use t() in hook_menu() 111 | ERROR | Do not use t() in hook_menu() 112 | ERROR | Do not use t() in hook_menu() 120 | ERROR | Do not use t() in hook_menu() 121 | ERROR | Do not use t() in hook_menu() 129 | ERROR | Do not use t() in hook_menu() 130 | ERROR | Do not use t() in hook_menu() 139 | ERROR | Do not use t() in hook_menu() 140 | ERROR | Do not use t() in hook_menu() 149 | ERROR | Do not use t() in hook_menu() 150 | ERROR | Do not use t() in hook_menu() 159 | ERROR | Do not use t() in hook_menu() 160 | ERROR | Do not use t() in hook_menu() -------------------------------------------------------------------------------- FILE: ...w_temp/test_candidate/embridge_component/components/entermediaasset.inc -------------------------------------------------------------------------------- FOUND 1 ERROR(S) AFFECTING 1 LINE(S) -------------------------------------------------------------------------------- 200 | ERROR | Inline comments must end in full-stops, exclamation marks, or | | question marks -------------------------------------------------------------------------------- FILE: ...review_temp/test_candidate/embridge_component/embridge_component.module -------------------------------------------------------------------------------- FOUND 1 ERROR(S) AFFECTING 1 LINE(S) -------------------------------------------------------------------------------- 375 | ERROR | Inline comments must end in full-stops, exclamation marks, or | | question marks -------------------------------------------------------------------------------- FILE: .../modules/pareview_temp/test_candidate/embridge_field/embridge_field.css -------------------------------------------------------------------------------- FOUND 2 ERROR(S) AFFECTING 2 LINE(S) -------------------------------------------------------------------------------- 12 | ERROR | Multiple CSS properties should be listed in alphabetical order 30 | ERROR | Multiple CSS properties should be listed in alphabetical order --------------------------------------------------------------------------------