From ae9c0e71bdaef381b43bc7dcdb5323525a7c6618 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?G=C3=A1bor=20Hojtsy?= <gabor@hojtsy.hu>
Date: Mon, 10 Nov 2008 10:31:06 +0000
Subject: [PATCH] #329660 by pwolanin: node_configure_validate() should be
replaced with a #submit handler to conform to FormAPI rules
---
modules/node/node.admin.inc | 12 ++++--------
1 file changed, 4 insertions(+), 8 deletions(-)
diff --git a/modules/node/node.admin.inc b/modules/node/node.admin.inc
index aaac957d1005..98adfa3a47d3 100644
--- a/modules/node/node.admin.inc
+++ b/modules/node/node.admin.inc
@@ -25,6 +25,7 @@ function node_configure() {
$form['access']['rebuild'] = array(
'#type' => 'submit',
'#value' => t('Rebuild permissions'),
+ '#submit' => array('node_configure_access_submit'),
);
}
@@ -59,18 +60,14 @@ function node_configure() {
'#description' => t('Must users preview posts before submitting?'),
);
- $form['#validate'] = array('node_configure_validate');
-
return system_settings_form($form);
}
/**
- * Form validate callback.
+ * Form button submit callback.
*/
-function node_configure_validate($form, &$form_state) {
- if ($form_state['values']['op'] == t('Rebuild permissions')) {
- drupal_goto('admin/content/node-settings/rebuild');
- }
+function node_configure_access_submit($form, &$form_state) {
+ $form_state['redirect'] = 'admin/content/node-settings/rebuild';
}
/**
@@ -87,7 +84,6 @@ function node_configure_rebuild_confirm() {
function node_configure_rebuild_confirm_submit($form, &$form_state) {
node_access_rebuild(TRUE);
$form_state['redirect'] = 'admin/content/node-settings';
- return;
}
/**
--
GitLab