Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Why can't you UnRegisterStartupScript?

Seems to me it is a bit wierd that you can do..

Page.ClientScript.RegisterStartupScript(this.GetType(), "KeyName", "alert('changed my mind')", true);

And then later on you can't unregister or stop the javascript from being rendered programatically.

Why would Microsoft do this?

I don't like the work around here.. http://hemant-vikram.blogspot.com/2005/11/unregister-startup-script-workaround.html

And I don't like the option of just re-registering it and having it do nothing..

Thoughts?

like image 488
Lee Englestone Avatar asked Aug 25 '09 14:08

Lee Englestone


3 Answers

It's actually worse than you think - "just re-registering it and having it do nothing" isn't an option either. The register method checks if a script is already registered with that key, and does nothing if it is. Once you call RegisterStartupScript, nothing you can do will stop that script being rendered.

As to why Microsoft did that, I'd guess that the possibility of changing or removing registered scripts was simply forgotten when RegisterStartupScript was first designed. The design choices happened to make it non-trivial to go back and create an unregister method, so they'd now need a good reason to do that.

When you register a script, it's stored in two places within the ClientScriptManager. A ListDictionary allows checks for whether a script is already registered, and an ArrayList stores the actual scripts as they will be rendered. I assume that the ArrayList is used to ensure that scripts are rendered in the order in which they were registered, but it also means that you can't tell which string in the ArrayList belongs to which key.

It wouldn't be terribly hard to equip your own page class with methods for MaybeAddStartupScript(key,script) and ChangedMyMindAboutThatStartupScript(key). Store the keys and scripts in your own dictionary and then in PreRender, register whichever scripts made it this far. It's certainly annoying to have to do it yourself, though.

like image 77
stevemegson Avatar answered Nov 11 '22 05:11

stevemegson


I assume that you'd want to "unregister" the script (which has already been registered) under some condition, like so:

Page.ClientScript.RegisterStartupScript(this.GetType(), "KeyName", "alert('changed my mind')", true);
...
if(condition)
    Page.ClientScript.UnregisterStartupScript(this.GetType(), "KeyName", "alert('changed my mind')", true);

Why can't this simply be changed to:

if(!condition)
    Page.ClientScript.RegisterStartupScript(this.GetType(), "KeyName", "alert('changed my mind')", true);
like image 5
JoshJordan Avatar answered Nov 11 '22 07:11

JoshJordan


After much studying, I found a solution to the problem if you want to only run the registered script once based on an event. Special thanks to @stevemegson's answer here that helped me understand what the problem actually was.

The solution involves the Sys.Application.add_load and remove_load javascript methods.

It works like this. A generic javascript function that first calls the worker function, then removes itself is the first statement, then add_load follows it up by calling that function:

protected void Button1_Click(object sender, EventArgs e)
{
    var script = @"<script language='javascript'>
                       function f(){ MyWorkerFunction(); 
                                     Sys.Application.remove_load(f);}; 
                       Sys.Application.add_load(f);
               </script>";

    ClientScript.RegisterStartupScript(GetType(), 
                                   "MyWorkerFunctionKey", 
                                    script, 
                                    false);
}

This is based on a telerik solution. It references telerik controls, but all the functionality is ASP.NET.

like image 2
crthompson Avatar answered Nov 11 '22 05:11

crthompson