I've found very minor bug when using language xml file,
in page \trunk\NSurveyAdmin\UserControls\SecurityAddInOptionsControl.ascx
regarding the link button DisableAddInLinkButton
the following function is called from Page_Load function:
private void LocalizePage()
{
DeleteButton.Text = ((PageBase)Page).GetPageResource("DeleteButton");
InsertAddInHyperLink.Text = ((PageBase)Page).GetPageResource("InsertAddInHyperLink");
if (SecurityAddIn != null)
{
if (SecurityAddIn.Disabled)
{
AddInDisabledLabel.Text = ((PageBase)Page).GetPageResource("AddInDisabledLabel");
DisableAddInLinkButton.Text = ((PageBase)Page).GetPageResource("EnableAddInLinkButton");
}
else
{
DisableAddInLinkButton.Text = ((PageBase)Page).GetPageResource("DisableAddInLinkButton");
}
}
}
When the SecurityAddIn is null the function never get trough the first if condition,
therefor never getting to DisableAddInLinkButton.Text = ((PageBase)Page).GetPageResource("DisableAddInLinkButton");
which translate the DisableAddInLinkButton text so the text remain always "Disable".
I think you should add another line before first if so the function will look like this:
private void LocalizePage()
{
DeleteButton.Text = ((PageBase)Page).GetPageResource("DeleteButton");
InsertAddInHyperLink.Text = ((PageBase)Page).GetPageResource("InsertAddInHyperLink");
DisableAddInLinkButton.Text = ((PageBase)Page).GetPageResource("DisableAddInLinkButton");
if (SecurityAddIn != null)
{
if (SecurityAddIn.Disabled)
{
AddInDisabledLabel.Text = ((PageBase)Page).GetPageResource("AddInDisabledLabel");
DisableAddInLinkButton.Text = ((PageBase)Page).GetPageResource("EnableAddInLinkButton");
}
//else
//{
// DisableAddInLinkButton.Text = ((PageBase)Page).GetPageResource("DisableAddInLinkButton");
//}
}
}
And then you can remove the else option in the function because it is redundant...
Please let me know what you think...
Eran.