| 1 |
<?php |
<?php |
| 2 |
// $Id: coder_review_7x.inc,v 1.44 2009/10/18 22:11:03 snpower Exp $ |
// $Id: coder_review_7x.inc,v 1.45 2009/10/18 23:05:45 snpower Exp $ |
| 3 |
|
|
| 4 |
/** |
/** |
| 5 |
* @file |
* @file |
| 146 |
'#function-not' => '^drupal_set_time_limit$', |
'#function-not' => '^drupal_set_time_limit$', |
| 147 |
'#warning_callback' => '_coder_review_7x_time_limit_warning', |
'#warning_callback' => '_coder_review_7x_time_limit_warning', |
| 148 |
), |
), |
| 149 |
|
// http://drupal.org/node/224333#archive-tar |
| 150 |
|
// http://drupal.org/node/224333#search-api |
| 151 |
|
// http://drupal.org/node/224333#user-1 |
| 152 |
|
|
| 153 |
|
|
| 154 |
// Permissions and Access |
// Permissions and Access |
| 200 |
), |
), |
| 201 |
|
|
| 202 |
// http://drupal.org/node/224333#system_admin_menu_block_access --- Hide empty menu categories with access callback |
// http://drupal.org/node/224333#system_admin_menu_block_access --- Hide empty menu categories with access callback |
| 203 |
|
// http://drupal.org/node/224333#user-roles-api --- Added API functions for creating, loading, updating, and deleting user roles and permissions |
| 204 |
|
|
| 205 |
|
|
| 206 |
// Database |
// Database |
| 255 |
|
|
| 256 |
|
|
| 257 |
// Block |
// Block |
| 258 |
|
// N/A - http://drupal.org/node/224333#hook_block_info_alter --- Added hook_block_info_alter() |
| 259 |
|
|
| 260 |
// http://drupal.org/node/224333#block_optional --- Block module now optional |
// http://drupal.org/node/224333#block_optional --- Block module now optional |
| 261 |
array( |
array( |
| 262 |
'#type' => 'callback', |
'#type' => 'callback', |
| 272 |
'#warning_callback' => '_coder_review_7x_hook_block_op_warning', |
'#warning_callback' => '_coder_review_7x_hook_block_op_warning', |
| 273 |
), |
), |
| 274 |
|
|
|
// http://drupal.org/node/224333#hook_block_list_alter --- Added hook_block_list_alter() |
|
| 275 |
|
|
| 276 |
|
|
| 277 |
// Comment |
// Comment |
| 278 |
// N/A - http://drupal.org/node/224333#comment_load_multiple --- New comment_load_multiple() |
// N/A - http://drupal.org/node/224333#comment_load_multiple --- New comment_load_multiple() |
| 279 |
// N/A - http://drupal.org/node/224333#comment_save_refactor --- comment_save() now supports programatic saving |
// N/A - http://drupal.org/node/224333#comment_save_refactor --- comment_save() now supports programatic saving |
| 280 |
|
// N/A - http://drupal.org/node/224333#comment_presave_hook --- New hook_comment_presave() for comments |
| 281 |
|
|
| 282 |
// http://drupal.org/node/224333#comment_load |
// http://drupal.org/node/224333#comment_load |
| 283 |
array( |
array( |
| 352 |
'#value' => 'admin\/settings\/filters(\/.*?)?', |
'#value' => 'admin\/settings\/filters(\/.*?)?', |
| 353 |
'#warning_callback' => '_coder_review_7x_filter_urls_warning', |
'#warning_callback' => '_coder_review_7x_filter_urls_warning', |
| 354 |
), |
), |
| 355 |
|
// http://drupal.org/node/224333#default-text-formats |
| 356 |
|
// http://drupal.org/node/224333#text-format-permissions |
| 357 |
|
// http://drupal.org/node/224333#filter-formats-parameters |
| 358 |
|
|
| 359 |
|
|
| 360 |
// Taxonomy |
// Taxonomy |
| 425 |
'#value' => '[\s\(]taxonomy_get_related\s*\(', |
'#value' => '[\s\(]taxonomy_get_related\s*\(', |
| 426 |
'#warning_callback' => '_coder_review_7x_related_terms_warning', |
'#warning_callback' => '_coder_review_7x_related_terms_warning', |
| 427 |
), |
), |
| 428 |
|
// http://drupal.org/node/224333#taxonomy-node |
| 429 |
|
|
| 430 |
|
|
| 431 |
// Javascript |
// Javascript |
| 506 |
'#filename' => array('js'), |
'#filename' => array('js'), |
| 507 |
'#warning_callback' => '_coder_review_7x_no_jsenabled_warning', |
'#warning_callback' => '_coder_review_7x_no_jsenabled_warning', |
| 508 |
), |
), |
| 509 |
|
// http://drupal.org/node/224333#jquery_once |
| 510 |
|
// http://drupal.org/node/224333#rename-drupal-to-js |
| 511 |
|
|
| 512 |
|
|
| 513 |
// CSS |
// CSS |
| 515 |
// N/A - http://drupal.org/node/224333#drupal_add_css_inline --- Inline cascading style sheets from drupal_add_css() |
// N/A - http://drupal.org/node/224333#drupal_add_css_inline --- Inline cascading style sheets from drupal_add_css() |
| 516 |
// N/A - http://drupal.org/node/224333#drupal_add_library --- Ability to add multiple JavaScript/CSS files at once |
// N/A - http://drupal.org/node/224333#drupal_add_library --- Ability to add multiple JavaScript/CSS files at once |
| 517 |
// N/A - http://drupal.org/node/224333#drupal_add_js_css_reset --- Ability to reset JavaScript/CSS |
// N/A - http://drupal.org/node/224333#drupal_add_js_css_reset --- Ability to reset JavaScript/CSS |
| 518 |
|
// N/A - http://drupal.org/node/224333#add-css-external --- drupal_add_css() now supports external CSS files |
| 519 |
|
|
| 520 |
// http://drupal.org/node/224333#drupal_add_js_options |
// http://drupal.org/node/224333#drupal_add_js_options |
| 521 |
array( |
array( |
| 523 |
'#value' => '[\s\(]drupal_add_css\s*\(\s*(' . $argex . '\s*,\s*){2,}(' . $argex . ')\s*\)', |
'#value' => '[\s\(]drupal_add_css\s*\(\s*(' . $argex . '\s*,\s*){2,}(' . $argex . ')\s*\)', |
| 524 |
'#warning_callback' => '_coder_review_7x_drupal_add_css_parameters_warning', |
'#warning_callback' => '_coder_review_7x_drupal_add_css_parameters_warning', |
| 525 |
), |
), |
| 526 |
|
// http://drupal.org/node/224333#drupal_add_css_weight |
| 527 |
|
// http://drupal.org/node/224333#class_attribute_array |
| 528 |
|
|
| 529 |
|
|
| 530 |
// Theming |
// Theming |
| 560 |
'#filename' => array('tpl.php'), |
'#filename' => array('tpl.php'), |
| 561 |
'#warning_callback' => '_coder_review_7x_closure_warning', |
'#warning_callback' => '_coder_review_7x_closure_warning', |
| 562 |
), |
), |
| 563 |
// http://drupal.org/node/224333#element_theme_properties --- Element theming properties used by drupal_render() have changed |
// http://drupal.org/node/224333#element_theme_properties |
| 564 |
array( |
array( |
| 565 |
'#type' => 'regex', |
'#type' => 'regex', |
| 566 |
'#source' => 'allphp', |
'#source' => 'allphp', |
| 570 |
), |
), |
| 571 |
|
|
| 572 |
// http://drupal.org/node/224333#drupal_render_children --- Element theme functions should call drupal_render_children() |
// http://drupal.org/node/224333#drupal_render_children --- Element theme functions should call drupal_render_children() |
| 573 |
|
// http://drupal.org/node/224333#theme-links-param --- theme_links() has a new parameter $heading for accessibility |
| 574 |
|
// http://drupal.org/node/224333#theme_changes --- theme() now takes only takes two arguments. |
| 575 |
|
|
| 576 |
|
|
| 577 |
|
|
| 857 |
'#warning_callback' => '_coder_review_7x_actions_synchronize_warning', |
'#warning_callback' => '_coder_review_7x_actions_synchronize_warning', |
| 858 |
), |
), |
| 859 |
// http://drupal.org/node/224333#distauth --- Login validation change for distributed authentication modules |
// http://drupal.org/node/224333#distauth --- Login validation change for distributed authentication modules |
| 860 |
|
// http://drupal.org/node/224333#trigger_overhaul --- Trigger and Actions API overhaul |
| 861 |
|
// http://drupal.org/node/224333#hook_entity_load --- Added hook_entity_load(). |
| 862 |
); |
); |
| 863 |
$review = array( |
$review = array( |
| 864 |
'#title' => t('Converting 6.x modules to 7.x'), |
'#title' => t('Converting 6.x modules to 7.x'), |