Javascript Functions nestled inside an If statement which is inside another Function
up vote
-1
down vote
favorite
I have the following code (I am pasting a smaller excerpt for brevity):
lockButton.onclick = function() { navBarSecKeyFunction(); }
function navBarSecKeyFunction()
{
document.getElementById('input-group').classList.toggle('show');
if (document.getElementById("navSecKey").value != "")
{
setTheKey();
}
else
{
return false;
}
}
However, the setTheKey() function never gets called? I know the if logic works because if I replace setTheKey() with something like alert("this works") then I do get an alert. I am not sure what I am doing wrong and how to make the call to setTheKey()
javascript
asked Nov 21 at 19:32
mindw
1
|
show 6 more comments
up vote
-1
down vote
favorite
I have the following code (I am pasting a smaller excerpt for brevity):
lockButton.onclick = function() { navBarSecKeyFunction(); }
function navBarSecKeyFunction()
{
document.getElementById('input-group').classList.toggle('show');
if (document.getElementById("navSecKey").value != "")
{
setTheKey();
}
else
{
return false;
}
}
However, the setTheKey() function never gets called? I know the if logic works because if I replace setTheKey() with something like alert("this works") then I do get an alert. I am not sure what I am doing wrong and how to make the call to setTheKey()
javascript
asked Nov 21 at 19:32
mindw
1
3
If you can put an alert above thesetTheKey();call and have the alert happen, that function must also be called. Are you sure something unexpected isn't happening in that function after its called?
– Carcigenicate
Nov 21 at 19:35
1
Yeah, we need to seesetTheKey()- how are you so sure it isn't being executed? Add aconsole.log()to the first line ofsetTheKey()and see if it dumps.
– Nunchy
Nov 21 at 19:36
Sorry, thenavSecKeyis an input box and theifstatement checks if there is data in the input box then its supposed to callsetTheKey()and if its empty it will return false. As I mentioned, theifstatement works because I can definitely check if thenavSecKeyinput box has values or not. Hope that helps!
– mindw
Nov 21 at 19:36
@mindw Post the functionsetTheKey()
– Ryan Wilson
Nov 21 at 19:37
1
Is functionsetTheKey()in the same .js file asnavBarSecKeyFunction()?
– Ryan Wilson
Nov 21 at 19:40
|
show 6 more comments
up vote
-1
down vote
favorite
up vote
-1
down vote
favorite
I have the following code (I am pasting a smaller excerpt for brevity):
lockButton.onclick = function() { navBarSecKeyFunction(); }
function navBarSecKeyFunction()
{
document.getElementById('input-group').classList.toggle('show');
if (document.getElementById("navSecKey").value != "")
{
setTheKey();
}
else
{
return false;
}
}
However, the setTheKey() function never gets called? I know the if logic works because if I replace setTheKey() with something like alert("this works") then I do get an alert. I am not sure what I am doing wrong and how to make the call to setTheKey()
javascript
asked Nov 21 at 19:32
mindw
1
I have the following code (I am pasting a smaller excerpt for brevity):
lockButton.onclick = function() { navBarSecKeyFunction(); }
function navBarSecKeyFunction()
{
document.getElementById('input-group').classList.toggle('show');
if (document.getElementById("navSecKey").value != "")
{
setTheKey();
}
else
{
return false;
}
}
However, the setTheKey() function never gets called? I know the if logic works because if I replace setTheKey() with something like alert("this works") then I do get an alert. I am not sure what I am doing wrong and how to make the call to setTheKey()
javascript
javascript
asked Nov 21 at 19:32
mindw
1
asked Nov 21 at 19:32
mindw
1
asked Nov 21 at 19:32
mindw
1
asked Nov 21 at 19:32
mindw
1
asked Nov 21 at 19:32
mindw
1
1
3
If you can put an alert above thesetTheKey();call and have the alert happen, that function must also be called. Are you sure something unexpected isn't happening in that function after its called?
– Carcigenicate
Nov 21 at 19:35
1
Yeah, we need to seesetTheKey()- how are you so sure it isn't being executed? Add aconsole.log()to the first line ofsetTheKey()and see if it dumps.
– Nunchy
Nov 21 at 19:36
Sorry, thenavSecKeyis an input box and theifstatement checks if there is data in the input box then its supposed to callsetTheKey()and if its empty it will return false. As I mentioned, theifstatement works because I can definitely check if thenavSecKeyinput box has values or not. Hope that helps!
– mindw
Nov 21 at 19:36
@mindw Post the functionsetTheKey()
– Ryan Wilson
Nov 21 at 19:37
1
Is functionsetTheKey()in the same .js file asnavBarSecKeyFunction()?
– Ryan Wilson
Nov 21 at 19:40
|
show 6 more comments
3
If you can put an alert above thesetTheKey();call and have the alert happen, that function must also be called. Are you sure something unexpected isn't happening in that function after its called?
– Carcigenicate
Nov 21 at 19:35
1
Yeah, we need to seesetTheKey()- how are you so sure it isn't being executed? Add aconsole.log()to the first line ofsetTheKey()and see if it dumps.
– Nunchy
Nov 21 at 19:36
Sorry, thenavSecKeyis an input box and theifstatement checks if there is data in the input box then its supposed to callsetTheKey()and if its empty it will return false. As I mentioned, theifstatement works because I can definitely check if thenavSecKeyinput box has values or not. Hope that helps!
– mindw
Nov 21 at 19:36
@mindw Post the functionsetTheKey()
– Ryan Wilson
Nov 21 at 19:37
1
Is functionsetTheKey()in the same .js file asnavBarSecKeyFunction()?
– Ryan Wilson
Nov 21 at 19:40
3
3
If you can put an alert above the
setTheKey(); call and have the alert happen, that function must also be called. Are you sure something unexpected isn't happening in that function after its called?– Carcigenicate
Nov 21 at 19:35
If you can put an alert above the
setTheKey(); call and have the alert happen, that function must also be called. Are you sure something unexpected isn't happening in that function after its called?– Carcigenicate
Nov 21 at 19:35
1
1
Yeah, we need to see
setTheKey() - how are you so sure it isn't being executed? Add a console.log() to the first line of setTheKey() and see if it dumps.– Nunchy
Nov 21 at 19:36
Yeah, we need to see
setTheKey() - how are you so sure it isn't being executed? Add a console.log() to the first line of setTheKey() and see if it dumps.– Nunchy
Nov 21 at 19:36
Sorry, the
navSecKey is an input box and the if statement checks if there is data in the input box then its supposed to call setTheKey() and if its empty it will return false. As I mentioned, the if statement works because I can definitely check if the navSecKey input box has values or not. Hope that helps!– mindw
Nov 21 at 19:36
Sorry, the
navSecKey is an input box and the if statement checks if there is data in the input box then its supposed to call setTheKey() and if its empty it will return false. As I mentioned, the if statement works because I can definitely check if the navSecKey input box has values or not. Hope that helps!– mindw
Nov 21 at 19:36
@mindw Post the function
setTheKey()– Ryan Wilson
Nov 21 at 19:37
@mindw Post the function
setTheKey()– Ryan Wilson
Nov 21 at 19:37
1
1
Is function
setTheKey() in the same .js file as navBarSecKeyFunction()?– Ryan Wilson
Nov 21 at 19:40
Is function
setTheKey() in the same .js file as navBarSecKeyFunction()?– Ryan Wilson
Nov 21 at 19:40
|
show 6 more comments
active
oldest
votes
active
oldest
votes
active
oldest
votes
active
oldest
votes
active
oldest
votes
draft saved
draft discarded
Thanks for contributing an answer to Stack Overflow!
- Please be sure to answer the question. Provide details and share your research!
But avoid …
- Asking for help, clarification, or responding to other answers.
- Making statements based on opinion; back them up with references or personal experience.
To learn more, see our tips on writing great answers.
Some of your past answers have not been well-received, and you're in danger of being blocked from answering.
Please pay close attention to the following guidance:
- Please be sure to answer the question. Provide details and share your research!
But avoid …
- Asking for help, clarification, or responding to other answers.
- Making statements based on opinion; back them up with references or personal experience.
To learn more, see our tips on writing great answers.
draft saved
draft discarded
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53419326%2fjavascript-functions-nestled-inside-an-if-statement-which-is-inside-another-func%23new-answer', 'question_page');
}
);
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
3
If you can put an alert above the
setTheKey();call and have the alert happen, that function must also be called. Are you sure something unexpected isn't happening in that function after its called?– Carcigenicate
Nov 21 at 19:35
1
Yeah, we need to see
setTheKey()- how are you so sure it isn't being executed? Add aconsole.log()to the first line ofsetTheKey()and see if it dumps.– Nunchy
Nov 21 at 19:36
Sorry, the
navSecKeyis an input box and theifstatement checks if there is data in the input box then its supposed to callsetTheKey()and if its empty it will return false. As I mentioned, theifstatement works because I can definitely check if thenavSecKeyinput box has values or not. Hope that helps!– mindw
Nov 21 at 19:36
@mindw Post the function
setTheKey()– Ryan Wilson
Nov 21 at 19:37
1
Is function
setTheKey()in the same .js file asnavBarSecKeyFunction()?– Ryan Wilson
Nov 21 at 19:40