Products

Solutions

Resources

Partners

Community

Blog

About

QA

Ideas Test

New Community Website

Ordinarily, you'd be at the right spot, but we've recently launched a brand new community website... For the community, by the community.

Yay... Take Me to the Community!

Welcome to the DNN Community Forums, your preferred source of online community support for all things related to DNN.
In order to participate you must be a registered DNNizen

HomeHomeDevelopment and...Development and...DNN Platform (o...DNN Platform (o...BreadCrumb throwing exceptionBreadCrumb throwing exception
Previous
 
Next
New Post
8/22/2013 4:59 PM
 

Hi all,

I recently encountered an issue with the breadcrumb control of DotNetNuke (to be found under /admin/skins). I am using the DPS payment on the site and when this redirects to the original site it uses a get-variable with the name "userid" this causes an issue with breadcrumb because of the property "ProfileUserId" which is also referring to the very same variable name. (The DPS-UserId is a string whereas the ProfileUserId is of type Int32 which results in a cast exception).

Here is how I fixed it:

        public int ProfileUserId
        {
            get
            {
                var userId = Null.NullInteger;
                if (!string.IsNullOrEmpty(Request.Params["UserId"]))
                {
                    // Original below
                    //userId = Int32.Parse(Request.Params["UserId"]);
                    // *** FIX START ***
                    // Fixed as the original could lead to an exception when the 
                    // request string is not a number (int32)
                    if (!Int32.TryParse(Request.Params["UserId"], out userId))
                        userId = Null.NullInteger;
                    // *** FIX END ***
                    
                }
                return userId;
            }
        }

As the original line (" userId = Int32.Parse(Request.Params["UserId"]); ") could allways result in an exception causing a page not to load, I would suggest to alter this to the provided code above.


Cheers

Nils

 
New Post
8/23/2013 4:29 AM
 
Nils,
please enter the issue and your fix into the public issue tracker at support.dotnetnuke.com, so it can be incorporated into next version of DNN.

Cheers from Germany,
Sebastian Leupold

dnnWerk - The DotNetNuke Experts   German Spoken DotNetNuke User Group

Speed up your DNN Websites with TurboDNN
 
New Post
8/27/2013 8:22 PM
 

I have added this issue here.

Nils

 
Previous
 
Next
HomeHomeDevelopment and...Development and...DNN Platform (o...DNN Platform (o...BreadCrumb throwing exceptionBreadCrumb throwing exception


These Forums are dedicated to discussion of DNN Platform and Evoq Solutions.

For the benefit of the community and to protect the integrity of the ecosystem, please observe the following posting guidelines:

  1. No Advertising. This includes promotion of commercial and non-commercial products or services which are not directly related to DNN.
  2. No vendor trolling / poaching. If someone posts about a vendor issue, allow the vendor or other customers to respond. Any post that looks like trolling / poaching will be removed.
  3. Discussion or promotion of DNN Platform product releases under a different brand name are strictly prohibited.
  4. No Flaming or Trolling.
  5. No Profanity, Racism, or Prejudice.
  6. Site Moderators have the final word on approving / removing a thread or post or comment.
  7. English language posting only, please.
What is Liquid Content?
Find Out
What is Liquid Content?
Find Out
What is Liquid Content?
Find Out