Closed (fixed)
Project:
Addresses
Version:
6.x-1.0-alpha2
Component:
Code
Priority:
Minor
Category:
Bug report
Assigned:
Reporter:
Created:
19 Sep 2008 at 22:27 UTC
Updated:
5 Oct 2008 at 15:02 UTC
Jump to comment: Most recent file
The address & additional fields are both represented in the admin/settings/address table. There I can say I want to show the address, but hide the additional. Yet, the code says otherwise. I would suggest adding an if() for the additional or remove it from the table so you do not have the option to change additional one way or the other. I agree, however, that it should not be shown if the address is not shown.
// Adds the Streets and the Additional fields
if (!empty($fields['street'])) {
$form['street'] = array(
'#type' => 'textfield',
'#title' => t('Street'),
'#default_value' => isset($values['street']) ? $values['street'] : '',
'#size' => 50,
'#maxlength' => 255,
'#required' => ($fields['street'] == ADDRESSES_FIELD_REQUIRED)
);
$form['additional'] = array(
'#type' => 'textfield',
'#title' => t('Additional'),
'#default_value' => isset($values['additional']) ? $values['additional'] : '',
'#size' => 50,
'#maxlength' => 255
);
}
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | addresses-fix-additional.6.4-1.x.patch | 1.27 KB | AlexisWilke |
Comments
Comment #1
AlexisWilke commentedOkay, there is a patch. I checked and it works great for me. Just in case, there is the code inline.
Comment #2
brmassa commentedAlexis,
patch commited on CVS
regards,
massa
Comment #3
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.