javascript confirm dialog not working in for loop C#

Question

0.00/5 (No votes)

See more:

Hi,

I have a javascript confirm dialog that asks the user a certain question. I have the javascript code on the aspx page and calling it from code behind in a loop, the problem I am having is that if a user selects multiple options to transfer some data between tables the confirm dialog only shows up once a transfer has been made.

Please can someone tell me where I am going wrong:

C#

foreach (ListItem Item in listbox1.Items)
{
    if (Item.Selected == true)
    {
        ScriptManager.RegisterStartupScript(this, GetType(), "progress", @"window.onload = AddMediaGroupToExisitingSuffix();", true);
        //do transfer
    }
}

C#

function AddMediaGroupToExisitingSuffix() {
    if (confirm("'do you want to transfer data?'") == true)
        return true;
    else
        return false;
}

[Edit member="Tadit"]
Added pre tags.
[/Edit]

Posted 5-Oct-14 21:16pm

Kats2512

Updated 6-Oct-14 0:34am

Tadit Dash (ତଡିତ୍ କୁମାର ଦାଶ)

v2

Add a Solution

Comments

JR009 6-Oct-14 3:33am

why you are using @"window.onload = AddMediaGroupToExisitingSuffix();" ?

try,

ScriptManager.RegisterStartupScript(this, GetType(), "progress", "AddMediaGroupToExisitingSuffix();", true);

Kats2512 6-Oct-14 3:47am

i tried all, even alert, the window.onload was just before I posted this and none of them seem to go while looping.

Sinisa Hajnal 6-Oct-14 4:01am

It will simply register the same function multiple times, will it execute it multiple times too once the page loads?

You should make javascript / jQuery function client-side which iterates through selected items and checks with the user...but it would be better to ask only once for all the data. By asking for each you will get one of the following: users wil l stop using your page due to irritation OR if they have to use it they will start hitting Yes / Enter without paying attention to it and will start processing even if they really don't need / want to.

Single confirmation and then send the marked data - put yourself in users shoes, how would you feel if you clicked select all on the page with 5-10 items and then had to click yes same number of times?

Kornfeld Eliyahu Peter 6-Oct-14 5:35am

In fact it smart enough to register the same script only once - that the reason it asks for type and key...

Kats2512 6-Oct-14 4:13am

that is what is required, user must confirm multiple times

Sinisa Hajnal 6-Oct-14 6:09am

I'm afraid you'll have to do it client side then. The page loads only once regardless of how many times you loop in code behind. As far as the page is concerned it's just some background processing and you get one PreRender server side and finally on client side one load event.

Kornfeld Eliyahu Peter 6-Oct-14 6:49am

The problem is not with the requirement, but with your design...
You have to assign validation code to each item (it can be done via server side, but the code will run on the client for most cases).
What are you doing is run a script with no connection to any item or event...

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Tadit Dash (ତଡିତ୍ କୁମାର ଦାଶ) · Answer 1 · 2014-10-06T00:56:00

Refer - ScriptManager.RegisterStartupScript Method[^].

Quote:
The script block that is rendered by the RegisterStartupScript method executes when the page finishes loading but before the page's client onload event is raised. Startup script blocks are located at the bottom of the rendered ASP.NET page just before the </form> tag.

I would suggest you to go for Ajax Modal Popups.