ASP.net button onclick send querystring

Question

I have the button:

<asp:Button runat="server" OnClientClick="return ValidateNewMessage();" OnClick="PostComment" ID="AddCommentButton" CssClass="nice-button" Text="Add Your Comment" />

It is on the URL:

blog/42/gregre-re-greg-er-g/comments-6#comments

When I click the button, it takes me to:

blog/42/gregre-re-greg-er-g/comments-6?ID=42&comments=1&page=6

This is making my scripts work strangely, I really need the button to just keep it's URL format and not include the querystring data, is this possible easily? So fater clicking it, the page is still:

blog/42/gregre-re-greg-er-g/comments-6#comments

The post comment method is as follows:

/// <summary>
/// Post a comment
/// </summary>
public void PostComment(object sender, EventArgs e)
{
    CommentResponse CommentResp = CommentCommon.NewComment(NewComment.Text, this.ThisUser.UserID, this.Anchor);

    if (!this.ThisUser.IsLoggedIn)
        CommentResp.Status = CommentError.NotLoggedIn;

    // Error messages
    if (CommentResp.Status == CommentError.CommentsPostedTooQuick)
    {
        DiscussError.Visible = true;
        DiscussErrorMessage.Text = "You are posting comments too quickly";
    }
    else if (CommentResp.Status == CommentError.ExceededCommentsPer3Mins)
    {
        DiscussError.Visible = true;
        DiscussErrorMessage.Text = " You are posting comments too quickly";
    }
    else if (CommentResp.Status == CommentError.NotEnoughChars)
    {
        DiscussError.Visible = true;
        DiscussErrorMessage.Text = "Comment is not long enough";
    }
    else if (CommentResp.Status == CommentError.NotLoggedIn)
    {
        DiscussError.Visible = true;
        DiscussErrorMessage.Text = "You are not logged in";
    }
    else if (CommentResp.Status == CommentError.UnspecifiedError)
    {
        DiscussError.Visible = true;
        DiscussErrorMessage.Text = "Unspecified error.";
    }

    // Posted ok, redirect to last page
    if (CommentResp.Status == CommentError.Success)
    {
        int TotalComments = CommentCommon.CountComments(this.Anchor);
        int TotalPages = (TotalComments + Settings.CommentsPerPage - 1) / Settings.CommentsPerPage;
        Response.Redirect(this.PageNavURL.Replace("$1", TotalPages.ToString()) + "#comments");
    }

}

The problem occurs when the status ISN'T success, when it redirects it works fine.

What does your form definition look like?

Kon
– Kon

2011-07-10 12:52:42 +00:00
Commented Jul 10, 2011 at 12:52 — Kon
– Kon, Commented Jul 10, 2011 at 12:52

Jonathan Wood · Accepted Answer · 2011-07-10 12:56:37Z

1

The following code does the redirect.

Response.Redirect(PageNavURL.Replace("$1", TotalPages.ToString()) + "#comments");

The URL appears to be based on the PageNavURL property. You need to change that property or the code so it results in a different URL.

answered Jul 10, 2011 at 12:56

Jonathan Wood

68.1k86 gold badges308 silver badges542 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Tom Gullen Over a year ago

This isn't called with the problem pages im having. When status == success it works fine. When status doesn't equal sucess, it skews the page URL. That's the problem, not that redirect line.

Jonathan Wood Over a year ago

That's not what your original question said. And that's not what your code says. If there is no redirect, then it's simply posting back to the original URL. What does PageNavURL look like?

Collectives™ on Stack Overflow

ASP.net button onclick send querystring

1 Answer 1

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related