Question

I’m working on a patch to submit to the Registration Code module for Drupal. In short, is there a more efficient way to write the code below?

if (module_exists('regcode_voucher')) {
  $cnfg = variable_get('regcode_voucher_display', array('regform' => 'regform'));
  if (empty($cnfg['regform'])) {
    return;
  }
}

It seems like I should be able to reduce it to one if statement with && combining two conditions, but I haven’t found the syntax or the necessary php array function that would allow me to do that.

In case some context helps, the regcode_voucher sub-module allows users to enter their registration code on the user edit page. On our sites, after a “beta” period, we want to simplify the registration form by removing the registration code field; but we’d like users to still be able to enter the code on their account edit page. The code above is part of a patch that allows the regcode’s hook_user changes to be bypassed.

Answer 1

Code looks like good, what efficient do you want? Little changes may be:

if (module_exists('regcode_voucher')) {
  $cnfg = variable_get('regcode_voucher_display', null);
  if ($cnfg) {
    // do your actions
  }
}

And I don’t recommend to merge if…, code should be clear and simpler to understand. If you merge these for optimizing, you win “tiny” milliseconds for real-live processors, but lost your clean code.