• Status: Solved
  • Priority: Medium
  • Security: Public
  • Views: 2489
  • Last Modified:

jquery login page - need to load new nagivation

I'm using php and jquery for a login page - this works well.  However, after I check to see if the user has entered a correct userid and password, I want to replace my current navigation with a new one (with extra items).  Below is the code - everything works until the part where I load a new nagivation panel into my left column.  Can anyone see where I have messed up?  And is this the easiest/best way to accomplish what I am trying to do (give logged in users extra choices?).  


$("#login_form").submit(function()
	{	$("#msgbox").removeClass().addClass('messagebox').text('Validating....').fadeIn(1000);
		//check the username exists or not from ajax
		$.post("ajax_login.php",{ user_name:$('#username').val(),password:$('#password').val(),rand:Math.random() } ,function(data)
        {
		  if (jQuery.trim(data) =='yes') //if correct login detail
		  {
		  	$("#msgbox").fadeTo(200,0.1,function()  //start fading the messagebox
			{ 
			 //add message and change the class of the box and start fading
			$(this).html('Logging in.....').addClass('messageboxok').fadeTo(900,1,				
			function()
			{ 
			  	$("#leftcolumn").load('secure-index.php'); 	// this does not work - 		   
			}); // end function
			.......
 
Here is secure-index.php:
 
	<div class="innertube"></div>
	<div class="arrowlistmenu">
	<h3 class="menuheader expandable">Parents and Fans</h3>
	<ul class="categoryitems">
		<li><a id="imp-dates" href="#">Important Dates</a></li>
		<li><a id="btn-slide" href="#">Game Schedules</a></li>
         .....
	</ul>
 
	
 
	<h3 class="menuheader expandable">Football Program</h3>
	<ul class="categoryitems">
		<li><a id="field-nav" href="#">Purchase an Ad</a></li>
		<li><a id="field-nav" href="#">Purchase a Player Ad</a></li>
	</ul>
 
		<h3 class="menuheader expandable">Admin</h3>
	<ul class="categoryitems">
		<li><a id="admin_one" href="#">Admin One</a></li>
		<li><a id="update_news" href="#">Update News</a></li>
	</ul>
	</div> <!-- end arrowlistmenu -->

Open in new window

0
kathys39
Asked:
kathys39
  • 3
  • 3
1 Solution
 
Vel EousResearch & Development ManagerCommented:
Have you closed off all the braces correctly or have you just excluded them from the code example for speed?

As for is this the best method, I think that is a matter of opinion.  Mine would be to say no.

The way I do it (and a few other mainstream apps) is to have a single file with all my menu items in it.  When a user views a page my PHP determines if the user is authenticated and if so displays the additional items, if not they are ignored:
<ul>
<li>item 1</li>
<li>item 2</li>
<?PHP
// if user is authenticated and only if
if ( isset ( $_SESSION['authed'] ) )
{
?>
<li>Admin</li>
<?PHP
}
?>
</ul>

Open in new window

0
 
kathys39Author Commented:
I haven't closed off braces just to save space.

Your solution makes much more sense.  However, I have a question.  My navigation is in a div called "leftcolumn".    When the user logs in successfully I want other options to appear in the navigation div ("leftcolumn".  this is what I have when the user is successful:

        $(this).html('Logging in.....').addClass('messagebook').fadeTo(900,1,                        
      function()
      {
      $("#leftcolumn").load('secure.php');

This is secure.php:

How do I get it to reload my index.php page, which inside it has your code:

            <?php
            // if user is authenticated and only if
            if ( isset ( $_SESSION['u_name'] ) )
            {
                  ?>
                  <li><a id="add_news" href="#">Add News</a></li>
                  <li><a id="admin_three" href="#">View Player Data</a></li>
                  <?php
            }
            ?>

<?php session_start();
// if session is not set redirect the user
if(empty($_SESSION['u_name']))
	header("Location:index.php");	
 
//if logout then destroy the session and redirect the user
if(isset($_GET['logout']))
{
	session_destroy();
	header("Location:index.php");
}	
 
echo "<a href='secure.php?logout'><b>Logout<b></a>";
echo "<div align='center'>You Are inside secured Page</a>";
 
?>

Open in new window

0
 
Vel EousResearch & Development ManagerCommented:
Rather than having a function to load the new list items when the user logs in, have the left col already loaded but only showing the "non admin" list items.  When they do log in, you login script redirects the users back to your index page where the session test script returns true and displays all list items.
0
The new generation of project management tools

With monday.com’s project management tool, you can see what everyone on your team is working in a single glance. Its intuitive dashboards are customizable, so you can create systems that work for you.

 
kathys39Author Commented:
That is what I tried originally and I cannot get the new/admin items to load.  

Here is what I have in my login.html file (see below).

In the ajax_login.php file, if the userid and passwd are ok I set this:
    $_SESSION['u_name']=$user_name;

Now in my index.php file I check to see if this is set correctly (it is as I've echo'd it out), and if so I display the new items.  Problem is the new items don't display!!

       <?php
      $session_name = $_GET["session"];
      writeLog ("session name is ");
      writeLog ($session_name);
      ?>
.....
    if ( isset ( $session_name) )
            {
                  writeLog("adding extras");
                  ?>
                  <li><a id="add_news" href="#">Add News</a></li>
                  <li><a id="admin_three" href="#">View Player Data</a></li>
                  <?php
            }
            ?>      


I can't figure out if the problem is the jquery accordian menu....
login.html file:
 
$().ready(function() {
	$("#login_form").submit(function()
	{
		//remove all the class add the messagebox classes and start fading
		$("#msgbox").removeClass().addClass('messagebox').text('Validating....').fadeIn(1000);
		//check the username exists or not from ajax
		$.post("ajax_login.php",{ user_name:$('#username').val(),password:$('#password').val(),rand:Math.random() } ,function(data)
        {
		  if (jQuery.trim(data) =='yes') //if correct login detail
		  {
		  	$("#msgbox").fadeTo(200,0.1,function()  //start fading the messagebox
			{ 
			 //add message and change the class of the box and start fading
			$(this).html('Logging in.....').addClass('messageboxok').fadeTo(900,1,				
			function()
			{ 
			$(this).text('loading...');
			// sessions stuff
	 		$(this).text('loading...');
			document.location='index.php?'; 

Open in new window

0
 
Vel EousResearch & Development ManagerCommented:
I've never used jquery I'm afraid so can't comment on the code.

Why not just do something like:
<?PHP
 
// login.php
session_start();
 
if ( $_POST )
{
	$user = $_POST['username'];
	$pass = $_POST['password'];
	
	$sql = mysql_query ( "SELECT * FROM users WHERE username = '$user' AND password = '$pass'" );
	
	if ( mysql_num_rows ( $sql ) === 1 )
	{
		$_SESSION['u_name'] = $user;
		header ( "Location: index.php" );
	}
}
 
?>
 
<?PHP
 
// index.php
session_start();
?>
 
<ul>
	<li>List Item 1</li>
    <li>List Item 2</li>
    <li>List Item 3</li>
    <?PHP
	if ( isset ( $_SESSION['u_name'] ) )
	{
	?>
    <li>Admin List Item 1</li>
    <li>Admin List Item 1</li>
    <?PHP
	}
	?>
</ul>

Open in new window

0
 
kathys39Author Commented:
sounds good to me - I'll use jquery elsewhere. thanks for the help.
0

Featured Post

The new generation of project management tools

With monday.com’s project management tool, you can see what everyone on your team is working in a single glance. Its intuitive dashboards are customizable, so you can create systems that work for you.

  • 3
  • 3
Tackle projects and never again get stuck behind a technical roadblock.
Join Now