I just noticed that after saving the admin form, no confirmation message is given to the user (at first I thought something went wrong, so I digged around in the code a little). Again, really minor issue, but thought I'd help out.

I've attached what is pretty much a one-line patch, adding the default confirmation message to ais_admin_settings_submit.

Alternatively, I suppose the form could be wrapped in system_settings_form, but that would probably mean the way (or rather, the naming with wich) the settings are stored would change, which would quite likely mean a large portion of files would have to be rewritten - which seems somewhat overkill for only a confirmation message.

CommentFileSizeAuthor
ais-confirmation-message-0.patch368 bytespixelsmash
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

  • wwarren committed 2fe20bf on 7.x-1.x authored by pixelsmash
    Issue #1969734 by pixelsmash: Confirmation message after saving messages
    
wwarren’s picture

Issue summary: View changes
Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

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