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.

Comments

add1sun’s picture

Status: Active » Needs review
StatusFileSize
new2.31 KB

patch attached

add1sun’s picture

Crud, I had removed the link for the site name error in all my mucking. New patch attached.

fgm’s picture

Looks good to me.

However, it remains in conflict with http://drupal.org/node/66241

add1sun’s picture

Yes, 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.

AjK’s picture

Project: Drupal core » Site Network
Version: 6.x-dev »
Component: drupal.module » Code
AjK’s picture

Status: Needs review » Fixed

Committed, although I suspect much of this stuff will go soon anyway, have to see how the other issues pan out.

Anonymous’s picture

Status: Fixed » Closed (fixed)