The settings form at admin/settings/amazon_s3 doesn't need to have it's own submit handler, using system_settings_form() will take care of that for you. The fields on the page should also have a #title attribute.

CommentFileSizeAuthor
#1 amazon_s3-config_form_cleanup-1049036-2-eojthebrave.patch1.95 KBeojthebrave

Comments

eojthebrave’s picture

eojthebrave
he/him
Primary language English
Location Minneapolis, MN
commented
Status: Active » Needs review
StatusFileSize
new amazon_s3-config_form_cleanup-1049036-2-eojthebrave.patch1.95 KB

Attached patch removes the custom submit handler in favor of using system_settings_form() and adds a #title property to both the AWS Access Key, and AWS Secret Key form fields.

torgospizza’s picture

torgospizza
he/him
Primary language English
Location Portland, OR
commented
Status: Needs review » Closed (fixed)

Part of this was a duplicate of #591916: SSL certificate problem (form submit handler) but I added the titles, so we can close this. Fixed in CVS, revision 1.1.2.2.