NewsletterModule is really just a wrapper around another control that you could use directly in layout.master

<portal:Subscribe ID="subscribe1" runat="server" />

all the module is doing is setting some properties on the control, but these could also be set declaratively right on the control.

Inside the module it sets properties based on module settings like this:

subscribe1.ShowList = NewsletterShowListSetting;
            subscribe1.IncludeDescriptionInList = NewsletterIncludeDescriptionInListSetting;
            subscribe1.ShowFormatOptions = NewsletterShowFormatOptionsSetting;
            subscribe1.HtmlIsDefault = NewsletterHtmlIsDefaultSetting;
            subscribe1.ShowMoreInfoLink = NewsletterShowmoreInfoLinkSetting;
            subscribe1.ShowPreviousEditionsLink = NewsletterShowPreviousEditionLinksSetting;
            subscribe1.OverrideInputWidth = NewsletterOverrideInputWidthSetting;

but you can also set those properties declaratively right on the control declaration like:

<portal:Subscribe ID="subscribe1" runat="server" ShowList="true" IncludeDescriptionInList="true" HtmlIsDefault="true" ShowMoreInfoLink="true" ShowPreviousEditionsLink="true" />

Best,

Joe

That would be great Joe!

Many thanks!

Hi David,

I guess technically it is a bug since it can be fixed.

Basically the issue is that we are calling Page.Validate(); before we save page settings and this causes it to validate all items on the page.

The solution is to use a validation group and only validate the group, actually we already have the validation group for pagesettings so changing the code like this should solve it.

Page.Validate("pagesettings");

The subscribe control internally already does it correctly so it won't affect other items on a page like this:

Page.Validate("subscribe");

I will fix this, let me know any other pages where it happens as I expect it will happen elsewhere on administrative pages where I wasn't anticipating other things on the page that may have validators. I'll look around in the admin pages myself.

Best,

Joe