Verification link causes an error

Description

Please test linked issue also because this issue causes a slighly different symptom on Evoq.
This issue must be tested on all products: Platform, Basic, Content, Engage

QA Notes:

  • Verify that error below is NOT logged in the log4net or event viewer.

  • Verify that the browser does not show DNN Error as shown in the screenshot in

Step to repro:

  1. Register an account with 'Verified' turned on.

  2. Email is send with verification link in it.

  3. User is already logged in with yellow bar explaining of unverified status.

  4. User clicks on link in email and it fails as described in forum post http://www.dnnsoftware.com/forums/forumid/200/threadid/523706/scope/posts

  5. User logs out and clicks again on verify link in email -> Success and verified.

No error in eventlog but only in Portals_default\Logs logfile:

2015-10-15 20:06:34,006 [MJKServer][Thread:57][FATAL] DotNetNuke.Framework.PageBase - An error has occurred while loading page.
System.Web.HttpException (0x80004005): The Controls collection cannot be modified because the control contains code blocks (i.e. <% ... %>).
at System.Web.UI.ControlCollection.RemoveAt(Int32 index)
at System.Web.UI.ControlCollection.Clear()
at DotNetNuke.Modules.Admin.Authentication.Login. (EventArgs e)
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Page.ProcessRequestMain(Boolean includeStagesBeforeAsyncPoint, Boolean includeStagesAfterAsyncPoint)

QA Test Plan

None

Activity

Show:

Bing WuJanuary 3, 2016 at 7:09 PM
Edited

Validated in platform 8.0.0.757
Validated in Content 8.3.0.285 and Basic
Validated on upgraded scenario with Engage (from 8.2.0)

Kan MaDecember 31, 2015 at 10:15 PM
Edited

Ilpo JuvanderDecember 29, 2015 at 6:29 PM

Note that this fixes the error in 7.4.2.
Version 8.0 has still more on this to be fixed, few lines above this one!

Ilpo JuvanderDecember 29, 2015 at 11:17 AM

Ilpo JuvanderDecember 29, 2015 at 10:26 AM

Error is caused by this: https://github.com/dnnsoftware/Dnn.Platform/commit/046c013eb169e5f9c6a4d912f8b5713124f84789#diff-a366d743febfc06d695862c897e7a6b1

Line 60:
var $loginButton = $('#<%=cmdLogin.ClientID%>');
should be replaced by:
var $loginButton = $("a[id$=DNN_cmdLogin]");

Complete

Details

Assignee

Reporter

Story Size

S

Severity

Show Stopper

Triage

Verified

Reported in Build #

Includes Code Fix

No

QA Story Points

2

Story Points

Components

Sprint

Fix versions

Affects versions

Priority

Created October 15, 2015 at 8:25 PM
Updated December 1, 2016 at 12:13 PM
Resolved December 31, 2015 at 10:18 PM