Skip to content
This repository was archived by the owner on Mar 4, 2019. It is now read-only.

Commit 3a61744

Browse files
committed
fixing the issues reported by auto code review
1 parent 048a721 commit 3a61744

File tree

3 files changed

+8
-8
lines changed

3 files changed

+8
-8
lines changed

ajax_chain_select.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@
2626
continue;
2727
}
2828
else {
29-
//Disabling all next select elements and setting them to their first value.
29+
// Disabling all next select elements and setting them to their first value.
3030
next_ptr.addClass('form-disabled').find('select').attr('disabled', 'disabled');
3131
if (next_ptr.find('select').find('option') !== undefined) {
3232
next_ptr.find('select').find('option:eq(0)').attr('selected', 'selected');

ajax_chain_select.module

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -67,12 +67,12 @@ function ajax_chain_select_element_ac_select($element, &$form_state, $form) {
6767
'#tree' => TRUE,
6868
'#id' => !empty($element['#id']) ? $element['#id'] : drupal_html_id('edit-' . implode('-', $element['#parents'])),
6969
'#title' => !empty($element['#title']) ? $element['#title'] : array(),
70-
'#array_parents' => array(),
70+
'#array_parents' => array(),
7171
'#element_validate' => array('ajax_chain_select_custom_validate'),
7272
'#states' => !empty($element['#states']) ? $element['#states'] : array(),
7373
'#levels' => $levels,
7474
'#data_callback' => $data_callback,
75-
);
75+
);
7676

7777
$region_element[$element_name]['#attached']['js'][] = array(
7878
'type' => 'setting',
@@ -118,9 +118,9 @@ function ajax_chain_select_element_ac_select($element, &$form_state, $form) {
118118
'#default_value' => !empty($config[$level]['#default_value']) ? $config[$level]['#default_value'] : NULL,
119119
'#title_display' => !empty($config[$level]['#title_display']) ? $config[$level]['#title_display'] : 'before',
120120
'#disabled' => (empty($data)) ? TRUE : FALSE,
121-
);
121+
);
122122

123-
if ($level != $levels - 1) {
123+
if ($level != $levels - 1) {
124124
$region_element[$element_name]['level_' . $level]['#attributes']['level'] = $level;
125125
$region_element[$element_name]['level_' . $level]['#attributes']['class'] = array('ajax-chain-select-select', 'ajax-progress-throbber');
126126
}

ajax_chain_select_sample/ajax_chain_select_sample.module

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -52,11 +52,11 @@ function ajax_chain_select_sample_sample_form($form, &$form_state) {
5252
'#title' => 'Region',
5353
'#config' => $acs_config,
5454
'#levels' => 4,
55-
'#required_levels' => 3,
55+
'#required_levels' => 3,
5656
'#show_throbber' => TRUE,
5757
'#progress_message' => t('Please wait..'),
5858
'#data_callback' => 'ajax_chain_select_sample_fetchdata',
59-
);
59+
);
6060

6161
$form['submit'] = array(
6262
'#type' => 'submit',
@@ -211,4 +211,4 @@ function ajax_chain_select_sample_get_locality_by_city($city_id) {
211211
}
212212

213213
return $localities;
214-
}
214+
}

0 commit comments

Comments
 (0)