Please, consider adding the following single line to uc_attribute.module, line 1289, inside function _uc_attribute_alter_form():

$attr_type = ''; // <<<<<<
switch ($attribute->display) {
  ...
}

Apart from being good PHP practice to initialize variables in such cases, it also allows other modules to add new attribute types, without giving a warning later in line 1303 where this variables is used.

Comments

longwave’s picture

Category: feature » bug
Priority: Normal » Minor
longwave’s picture

Status: Active » Fixed

Fixed in both branches.

djg_tram’s picture

Thanks.

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

Issue summary: View changes

Missing comma