Solved

jquery login page - need to load new nagivation

Posted on 2008-06-16
6
2,480 Views
Last Modified: 2010-04-21
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
Comment
Question by:kathys39
[X]
Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
  • 3
  • 3
6 Comments
 
LVL 14

Expert Comment

by:Vel Eous
ID: 21799303
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
 

Author Comment

by:kathys39
ID: 21837510
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
 
LVL 14

Expert Comment

by:Vel Eous
ID: 21837689
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
Get 15 Days FREE Full-Featured Trial

Benefit from a mission critical IT monitoring with Monitis Premium or get it FREE for your entry level monitoring needs.
-Over 200,000 users
-More than 300,000 websites monitored
-Used in 197 countries
-Recommended by 98% of users

 

Author Comment

by:kathys39
ID: 21840934
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
 
LVL 14

Accepted Solution

by:
Vel Eous earned 125 total points
ID: 21842105
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
 

Author Closing Comment

by:kathys39
ID: 31467706
sounds good to me - I'll use jquery elsewhere. thanks for the help.
0

Featured Post

Technology Partners: We Want Your Opinion!

We value your feedback.

Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Password hashing is better than message digests or encryption, and you should be using it instead of message digests or encryption.  Find out why and how in this article, which supplements the original article on PHP Client Registration, Login, Logo…
3 proven steps to speed up Magento powered sites. The article focus is on optimizing time to first byte (TTFB), full page caching and configuring server for optimal performance.
The viewer will learn how to dynamically set the form action using jQuery.
The viewer will learn how to create and use a small PHP class to apply a watermark to an image. This video shows the viewer the setup for the PHP watermark as well as important coding language. Continue to Part 2 to learn the core code used in creat…

627 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question