It's just a simple code style fix.
I also removed some stupidity where it was setting $permisions to default to NULL, and then if it was null then set it to the default array. Since there are no additional parameters, this is pointless.
Also, checkPermissions becomes private simply because it doesn't need to be protected (no one should use it).

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Status: Needs review » Needs work

The last submitted patch failed testing.

boombatower’s picture

Title: Remove the initial _ in the function name of drupalCreateRole in drupal_web_test_case.php » SimpleTest: Clean-up drupalCreateUser()
Assigned: dmitrig01 » boombatower
Status: Needs work » Needs review
FileSize
1.46 KB

The permissions need to be defined on drupalCreateUser() then, you can't just move the default in drupalCreateRole()...which is why the tests failed.

boombatower’s picture

Priority: Minor » Normal
Dave Reid’s picture

dmitrig01’s picture

Status: Needs review » Reviewed & tested by the community

Read through code, looks good, bot passes.

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.