The errors for required site information needed for site registry display one at a time. This means that the first time I go to set that up I am told I need to set my slogan. I go do that and then return to site registry. Now I'm told that I need to set my mission statement and I go back to settings again, etc. All errors should be reported at the same time so I can fix them all at once. This meant changing the form_set_error field name to different names so they can all display simultaneously. Note that the existing field name (drupal_directory) isn't a real field anyway.
Since site name is required and comes with a default, it only needs to print an error regarding site name being set to Drupal. I also removed the email error since it is required on the site information page and at install.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | drupal_site_registry_errors_172444-2-really.patch | 2.46 KB | add1sun |
| #1 | drupal_site_registry_errors_172444-2.patch | 2.31 KB | add1sun |
Comments
Comment #1
add1sun commentedpatch attached
Comment #2
add1sun commentedCrud, I had removed the link for the site name error in all my mucking. New patch attached.
Comment #3
fgmLooks good to me.
However, it remains in conflict with http://drupal.org/node/66241
Comment #4
add1sun commentedYes, I know but this is the one I had time to fix and while I believe the other issue should be addressed, if it isn't, I wanted this one in at the least.
Comment #5
AjK commentedComment #6
AjK commentedCommitted, although I suspect much of this stuff will go soon anyway, have to see how the other issues pan out.
Comment #7
(not verified) commented