Users not showing up on electoral list after adding

druojajay - January 28, 2009 - 04:37
Project:Decisions
Version:5.x-1.1
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

Hi,

I just tested this on two different Drupal 5 installations (5.2 and 5.10). Same problem.

When I try to add users to the electoral list on a new decision in a fresh install, nothing happens.

The selected user (the dropdown works) does not appear in the table.

I tried toggling the "Use electoral list by default:" setting. Don't know what else to do.

Thanks!

dru

#1

luddet - March 4, 2009 - 16:38
Status:active» needs review

There are two form functions that are not named correctly.
The form id for submitting new electors is "decisions_electoral_list_form", so drupal is expecting to find two functions, decisions_electoral_list_form_validate() and decisions_electoral_list_form_submit().

Fix:

  1. open decisions.module
  2. Find line 925:
    function decisions_electoral_list_validate($form_id, $form_values) {
  3. Change to:
    function decisions_electoral_list_form_validate($form_id, $form_values) {
  4. find line 947:
    function decisions_electoral_list_submit($form_id, $form_values) {
  5. Change to:
    function decisions_electoral_list_form_submit($form_id, $form_values) {

There is also a duplicate of the form code that can be removed. Lines 845 to and including 872 can be removed entirely since the actual form is delivered through the function call on line 873.

Sorry about not providing a proper patch file, but I didn't have the time and tools at the moment.

#2

anarcat - April 1, 2009 - 23:01
Status:needs review» fixed

Fix committed, thanks.

#3

System Message - April 15, 2009 - 23:10
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.