- Patch by Steven and me: refactored the form handling of nodes. The node system is now using form_set_error() and friends like the rest of Drupal does. This makes for both a consistent user experience and consistent code. It simplifies the forms and validation code, however, it does change the node API slightly:

    * The _validate hook and the _nodeapi('validate') hook of the node API (1) no longer take an 'error' parameter and (2) should no longer return an error array. To set an error, call form_set_error().

    * The _form hook of the node module no longer takes a form hook and should not worry about displaying errors. Ditto for _nodeapi('form_post') and _nodeapi('form_pre').

1 files changed, 2 insertions, 2 deletions

diff --git a/modules/comment.module b/modules/comment.module
index 0c56e9929..cf7771212 100644
--- a/modules/comment.module
+++ b/modules/comment.module
@@ -316,7 +316,7 @@ function comment_post($edit) {
       $edit['subject'] = truncate_utf8(strip_tags($edit['comment']), 29);
     }
 
-    if (!form_has_errors()) {
+    if (!form_get_errors()) {
       // Check for duplicate comments.  Note that we have to use the
       // validated/filtered data to perform such check.
 
@@ -1255,7 +1255,7 @@ function theme_comment_form($edit, $title) {
     $form .= form_submit(t('Preview comment'));
   }
 
-  if (!form_has_errors()) {
+  if (!form_get_errors()) {
     $form .= form_submit(t('Post comment'));
   }