1. variable_get always returns 1000, because it's looking for the non-existent key "max_count". Correct key is "ecard_max_count".

2. The error message informs the user that they are not allowed to send to more than $count addresses, where $count is the number of addresses they entered. It should use $max_count, otherwise the user has to guess what the maximum number of recipients is.

3. The error message should inform the user exactly how many addresses they will have to remove from the list.

Patch against CVS DRUPAL-6--2 attached.

CommentFileSizeAuthor
maxcount.patch1.11 KBmmccollow

Comments

kars-t’s picture

#640678 by mmccollow: Fixed Multiple problems in ecard_form_validate().

This one of those "never worked" things. I am amazed that I miss such things again and again ;)

Thank you for the patch! Commited to 6--1 and 6--2

kars-t’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.