document.getElementById is null or not an object

mohdazeemuddin ahmed

16y10.4k.NET

Hi Every one

i created dynamic radio button and i add OnClick event on that and i write java script for enable and disable the button depending upon some status the event is firing from the radio button and i able to check the status when i try to enable or disable the button i am getting the error document.getelementById is null or not an object i will show you the code which is am using

 function RadioEvent(Status,Button)
    {
//       var Button1='<%=btnAccept.ClientID %>';
//       aler(document.getElementById(Button1).Text);
       if(Status=="Request Accepted")
       {
       document.getElementById(Button).Enabled = true;
//            document.getElementById('btnAccept').disabled=true;
       //document.getElementById(Button1).setAttribute('Enabled','true');
     
        //document.getElementById(Button).disabled = true;
           //alert(Status);
            //document.getElementById('btnDecline').Enabled=false;
        //document.getElementById('btnAccept').Enabled = true;
       }
       else
       document.form1.btnDecline.Enable=true;
       //document.getElementById(Button1).removeAttribute('Enabled');
        //document.getElementById(btnDecline).disabled = false;
      
    }

the above is my java script and i am calling event like this

 MyRadioButton.Attributes.Add("onclick", "RadioEvent('" + Status + "','btnDecline');");

please correct me where i am doing mistake

6 Replies

Know the answer? Post it — somebody with the same question will find it here.

mohdazeemuddin ahmed16y ago
No body knows where i am doing mistake ok give me suggestion at-least how to achieve my task i will show you one image of table and its control which i am creating dynamically the image is shown below

i want to disable or enable this two button depending upon status column when user check on radio button please give me the solution to achieve my task i tried the suggestion of Mr.Amit and Mr.Sam but i got the same error please its very urgent

thanks in advance
0
mohdazeemuddin ahmed16y ago
Thanks For Your Reply's i tried as Amit say's but it still giving same error. i don't need ".Text" for testing i write i need to only Enable or Disable the button depending upon status condition if they is any other way please tell me i will try that also .

To Jaish Mathewsir i am creating radio button dynamically so i can't use your Suggest way but thanks for your reply
0
Jaish Mathews16y ago
But you can reduce the code by totally removing the getElementById by passing the control itself during run time like below, using "this' keyword.

<input type ="radio" id="r1" onclick="RadioEvent('status',this);" />
<input type ="radio" id="r2" onclick="RadioEvent('status',this);" />

function RadioEvent(Status, Button) {
Button.disabled = true;
}
0
Sam Hobbs16y ago
Why are you using "Text" and "Enabled" as properties? They are not properties. The line you have that is:

document.getElementById(btnDecline).disabled = false;

Will work, but you need to use quotes for btnDecline, as Amit says.

Also note that if you want the text in a button, use:

getAttribute("value")
0
mohdazeemuddin ahmed16y ago
Thanks for your reply i try that also but its not working

thanks
0

Amit Choudhary16y ago

hi friend,

if(Status=="Request Accepted")
       {
       document.getElementById(Button).Enabled = true;

Here Button should be inside quotes like this:

document.getElementById('Button').Enabled = true;

Please mark as answer if it helps.