Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Send Photo Request test gives parameter 'address' error #798

Closed
ElimAdmin opened this issue Jan 23, 2015 · 4 comments
Closed

Send Photo Request test gives parameter 'address' error #798

ElimAdmin opened this issue Jan 23, 2015 · 4 comments
Assignees
Labels
Status: Confirmed It's clear what the subject of the issue is about, and what the resolution should be. Type: Bug Confirmed bugs or reports that are very likely to be bugs.

Comments

@ElimAdmin
Copy link

Clicking Test on the Photo Request page gives error message: "The parameter 'address' cannot be an empty string. Parameter name: address." Checking the boxes doesn't make a difference.

sendpicreq1


Exception Error Details:
An exception has occurred. Details of this error can be found below:
An error occurred on the Rock RMS site on page:
http://elimrms.org:80/page/316
HttpUnhandledException in System.Web
Message
Exception of type 'System.Web.HttpUnhandledException' was thrown.
Stack Trace
at System.Web.UI.Page.HandleError(Exception e)
at System.Web.UI.Page.ProcessRequestMain(Boolean includeStagesBeforeAsyncPoint, Boolean includeStagesAfterAsyncPoint)
at System.Web.UI.Page.ProcessRequest(Boolean includeStagesBeforeAsyncPoint, Boolean includeStagesAfterAsyncPoint)
at System.Web.UI.Page.ProcessRequest()
at System.Web.UI.Page.ProcessRequestWithNoAssert(HttpContext context)
at System.Web.UI.Page.ProcessRequest(HttpContext context)
at ASP.themes_rock_layouts_fullwidth_aspx.ProcessRequest(HttpContext context)
at System.Web.HttpApplication.CallHandlerExecutionStep.System.Web.HttpApplication.IExecutionStep.Execute()
at System.Web.HttpApplication.ExecuteStep(IExecutionStep step, Boolean& completedSynchronously)
-ArgumentException in System
Message
The parameter 'address' cannot be an empty string. Parameter name: address
Stack Trace
at System.Net.Mail.MailAddress..ctor(String address, String displayName, Encoding displayNameEncoding)
at System.Net.Mail.MailAddress..ctor(String address, String displayName)
at Rock.Communication.Transport.SMTPComponent.Send(Communication communication)
at Rock.Communication.MediumComponent.Send(Communication communication)
at Rock.Communication.Medium.Email.Send(Communication communication)
at RockWeb.Blocks.Crm.PhotoSendRequest.btnTest_Click(Object sender, EventArgs e)
at System.Web.UI.WebControls.LinkButton.OnClick(EventArgs e)
at System.Web.UI.WebControls.LinkButton.RaisePostBackEvent(String eventArgument)
at System.Web.UI.WebControls.LinkButton.System.Web.UI.IPostBackEventHandler.RaisePostBackEvent(String eventArgument)
at System.Web.UI.Page.RaisePostBackEvent(IPostBackEventHandler sourceControl, String eventArgument)
at System.Web.UI.Page.RaisePostBackEvent(NameValueCollection postData)
at System.Web.UI.Page.ProcessRequestMain(Boolean includeStagesBeforeAsyncPoint, Boolean includeStagesAfterAsyncPoint)

@nairdo nairdo added Status: Confirmed It's clear what the subject of the issue is about, and what the resolution should be. Type: Bug Confirmed bugs or reports that are very likely to be bugs. labels Feb 4, 2015
@nairdo nairdo self-assigned this Feb 4, 2015
@nairdo
Copy link
Member

nairdo commented Feb 4, 2015

@ElimAdmin It's because there is a missing From address in the Photo Request Template (under Admin Tools > Communication Settings > Communication Templates). Can you confirm that the issue goes away after you set that value? We'll put safeguards in the block to check for that too.

@ElimAdmin
Copy link
Author

@nairdo Yes - setting those values does resolve this. Maybe even a little more guidance in the error message could help point someone to do this. Thanks.

@nairdo
Copy link
Member

nairdo commented Feb 11, 2015

@ElimAdmin Yes. In that situation this cc47358 change will now show the message "The '{0}' communication template is missing a From address."

nairdo added a commit that referenced this issue Feb 13, 2015
…thod of the SMTPComponent should also check the from address and use the GlobalAttribute setting like it did before the cc47358 commit. (re fixes #798)
@nairdo
Copy link
Member

nairdo commented Feb 13, 2015

Slight change of plans (for the better). This new change now means it will work the way it used to -- even if there is no from address defined in the template it will use the GlobalAttribute's OrganizationEmail value.

@nairdo nairdo closed this as completed in cc47358 Apr 13, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Confirmed It's clear what the subject of the issue is about, and what the resolution should be. Type: Bug Confirmed bugs or reports that are very likely to be bugs.
Projects
None yet
Development

No branches or pull requests

2 participants