.length locks all the code

Question

I need to know if the user is logged in through Javascript (jQuery) to run some commands (purely aesthetic). The quickest alternative I could find was to create a hidden input that is only placed on the page if the user is logged in (verified through PHP) and check by jQuery if that input exists. But when I put the code below within a $(document).ready() along with other functions, none of them works anymore, all my jQuery goes to bag. Follows the code:

$(document).ready(function()
{
   .
   .
   .
   if ($("#estoulogado").length)
   {
      $("#email").prop('disabled', true);
      if ($("#estoulogado").attr('faoulo') == "fabrica")
      {
        $("#lojas").prop('disabled', true);
        $("#selectdepartamentos").prop('disabled', true);
        $("#nome").prop('disabled', true);
      };
      else if ($("#estoulogado").attr('faoulo') == "loja")
      {
        $("#fabrica").prop('disabled', true);
        $("#selectlojas").prop('disabled', true);
      };
   };
   .
   .
   .
});

Opening the console (as suggested by Renan) I found a syntax error on this line:

$("#email").prop('disabled', true);

The error is as follows: Uncaught SyntaxError: Unexpected token (

No matter what I put inside the if, all my code hangs, it doesn’t work. Why does this happen? I also accept a better idea than this to check if the user is logged in.

Answer 1

The mistake Unexpected token ( means that there is a pair of poorly enclosed parentheses in the code. Two considerations:

• It was initially indicated in the question that a parenthesis was missing from the code. You fixed it, but if you still get the same error it’s because there are more pairs of poorly closed parentheses. This is in the code that you didn’t show;

• The error does not occur in the instruction the console tells you. $("#email").prop('disabled', true); is syntactically correct.

Some suggestions:

• Get more out of your IDE. Most mark syntax errors visually to make it easier to fix;

• From the most nested expressions to the most external ones, delete pairs of parentheses with whatever is inside them. In the end it ends up leaving what was not closed.

Good luck!

Answer 2

We’re missing a ) at the end of the code

Do so

$(document).ready(function()
{
   .
   .
   .
   if ( $("#estoulogado").length )
   {
   .
   .
   .
   }
   .
   .
   .
});

Answer 3

I found the mistake. I am posting as an answer because I asked the question as a guest and when completing my registration I can no longer comment on the question because it no longer appears as mine.

Before I wanted to thank Renan for helping me, if it were not for him I think I would not have found the mistake, because I would still be thinking that the problem was the .length. But since I’m a new user and the question no longer belongs to me, I can’t give you reputation.

So let’s make a mistake.

The problem was in the semicolon after the keys of the if within the if. How there is a else after it, there couldn’t be a semicolon there.

Thanks for the help guys.