The call to list_themes() is not needed here:

function omega_tools_theme_reset_confirm_submit($form, &$form_state) {
  $themes = list_themes();
  $theme = $form_state['theme'];

  omega_tools_revert_theme_settings($theme);

  $form_state['redirect'] = 'admin/appearance/settings/' . $theme;
}
CommentFileSizeAuthor
#1 1729880-1-omega_tools-unneeded_list_themes.patch1.2 KBpfrenssen
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pfrenssen’s picture

pfrenssen
Location Sofia
CreditAttribution: pfrenssen commented
Status: Active » Needs review
FileSize
1729880-1-omega_tools-unneeded_list_themes.patch1.2 KB

Here's a patch. I took the liberty to provide a docblock as well.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Needs review » Reviewed & tested by the community

Short & sweet.