summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--coder_review/includes/coder_review_7x.inc5
-rw-r--r--coder_review/tests/coder_review_7x.test3
2 files changed, 4 insertions, 4 deletions
diff --git a/coder_review/includes/coder_review_7x.inc b/coder_review/includes/coder_review_7x.inc
index ff517db..dc2cf3c 100644
--- a/coder_review/includes/coder_review_7x.inc
+++ b/coder_review/includes/coder_review_7x.inc
@@ -1062,9 +1062,10 @@ function _coder_review_7x_user_delete_warning() {
function _coder_review_7x_hook_user_op_warning() {
return array(
- '#warning' => t('!hook_user() has been split into a number of smaller hooks.',
+ '#warning' => t('!hook_user() has been removed in favour of hook_user_$op functions, for example !hook_user_validate().',
array(
- '!hook_user()' => theme('drupalapi', 'hook_user', '7'),
+ '!hook_user()' => theme('drupalapi', 'hook_user', '6'),
+ '!hook_user_validate()' => theme('drupalapi', 'hook_user_validate', '7'),
)
),
'#link' => 'http://drupal.org/node/224333#remove_op',
diff --git a/coder_review/tests/coder_review_7x.test b/coder_review/tests/coder_review_7x.test
index 50784c9..bbd7cca 100644
--- a/coder_review/tests/coder_review_7x.test
+++ b/coder_review/tests/coder_review_7x.test
@@ -475,8 +475,7 @@ class CoderReviewUpgrade7xTest extends CoderReviewTestCase {
$this->assertCoderReviewPass(' function foo_user_delete($edit, $uid)');
// http://drupal.org/node/224333#remove_op
- $this->assertCoderReviewFail(" function mymodule_user(\$edit, &\$user) {\n if (\$op == 'load') {\n }\n}");
- $this->assertCoderReviewFail(" function mymodule_user(\$edit, &\$user) {\n switch (\$op) {\n case 'load':\n }\n}");
+ $this->assertCoderReviewFail(" function mymodule_user(");
// http://drupal.org/node/224333#logout_path
$this->assertCoderReviewFail(' drupal_goto(\'logout\');');