Needs review
Project:
Spaces
Version:
7.x-3.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
23 Sep 2011 at 10:52 UTC
Updated:
15 Aug 2012 at 13:03 UTC
Jump to comment: Most recent file
When create new dashboard, see error:
Notice: Undefined index: #value in spaces_dashboard_add_validate() (line 29 of /home/tiensi/Documents/Workspace/projects/spaces/trunk/spaces_dashboard/spaces_dashboard.admin.inc).
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 1288676-3-spaces-dashboard-add-validation.patch | 1.04 KB | mr.baileys |
| #1 | spaces3-dashboard-1.patch | 825 bytes | mrsinguyen |
| spaces3-dashboard.patch | 837 bytes | mrsinguyen | |
| spaces-dashbaord.png | 88.3 KB | mrsinguyen |
Comments
Comment #1
mrsinguyen commentedAbove patch not correctly, just upload new patch
Comment #2
mrsinguyen commentedChanged status
Comment #3
mr.baileysI'm able to reproduce this error with current HEAD.
The patch in #1 does not address the real issue though: since the form name is
spaces_dashboard_add, Drupal's default behavior is to invokespaces_dashboard_add_validateon the whole form (and not just for the individual elements where the same function was registered through#element_validate). Validation then fails since the top fieldset does not have the#value-property.Either the
spaces_dashboad_add_validatefunction needs to be rewritten so it loops through the custom dashboard label textfields, or it needs to be renamed so it is not invoked automatically by Drupal on the entire form.Patch attached that implements the second alternative (renaming the element validation function).