...
  foreach ($form_values['restricted_fields'] as $key => $value) {
    if ($value) {
      $fields[] = $key;
    }
  }

...

  variable_set('uc_node_checkout_'. $form_values['type'] .'_restrictions', $fields);

Why not place this info in DB?

Comments

rszrama’s picture

rszrama commented
Version: 5.x-1.x-dev » 6.x-1.x-dev
Category: support » feature
Status: Active » Postponed

It could be placed in the DB... it just wasn't really necessary and was easy to do this way.

rszrama’s picture

rszrama commented
Version: 6.x-1.x-dev » 6.x-2.x-dev
tr’s picture

tr commented
Status: Postponed » Closed (won't fix)

Nothing wrong with keeping these as variables. These values are very static, so it seems more efficient to keeps them in the variable table cache. But if you think storing them in the DB could be more performant, please post a patch and some benchmarks.