Error message when I click go back one page after log out

phpcoder101 · #1 (**permalink**) November 5th, 2009, 05:41 PM

First I log in and log out from admin.

Then when I click go back button of a browser, I get the following error messages.

Also when I visit /index.php/admin/dashboard directly without logging-in I get the same error messages.

Could anyone tell me how to fix the problem please?

Thanks in advance.

Quote:

A PHP Error was encountered

Severity: Notice

Message: Undefined index: userid

Filename: admin/dashboard.php

Line Number: 8

A PHP Error was encountered

Severity: Warning

Message: Cannot modify header information - headers already sent by (output started at C:\xampp\htdocs\codeigniter_shopping\system\librar ies\Exceptions.php:166)

Filename: helpers/url_helper.php

Line Number: 539

myerman · #2 (**permalink**) November 6th, 2009, 09:54 AM

Your dashboard controller should start like this:

Code:

class Dashboard extends Controller {
  function Dashboard(){
    parent::Controller();
    session_start();
    
	if (isset($_SESSION['userid']) && $_SESSION['userid'] < 1){
    	redirect('welcome/verify','refresh');
    }
  }

what is your email address so I can send you the ZIP archive I have?

phpcoder101 · #3 (**permalink**) November 6th, 2009, 05:22 PM

Thanks. I changed all the files in admin folders, then it works.

When I click "go back" button, it automatically log-in.

However I think once, you log-out, it should require log-in again rather taking you to admin again. I think it is more secure.

If I want to do it, what I should do it?

okada.shin[at]google[dot]com

Thanks in advance.

myerman · #4 (**permalink**) November 6th, 2009, 05:43 PM

confused...isn't it already knocking you out to welcome/verify if you aren't logged in?

phpcoder101 · #5 (**permalink**) November 7th, 2009, 03:44 AM

I tested with FF and chrome.

I loged in and loged out.

Then when I click "Go Back" button, I can go to admin still.

It should ask to log in again.

I have the following codes in all the files in the admin folder as you suggested.

Code:

  
...
...
parent::Controller();
    session_start();
    
	if (isset($_SESSION['userid']) && $_SESSION['userid'] < 1){
    	redirect('welcome/verify','refresh');
    }

phpcoder101 · #6 (**permalink**) November 7th, 2009, 04:30 AM

All the line in admin files have this line.

Code:

if ($_SESSION['userid'] < 1){
redirect('welcome/verify','refresh');

I changed to the following.

Code:

if (!isset($_SESSION['userid']) || $_SESSION['userid'] < 1){
	$this->session->set_flashdata('error',"You must log in!");  
    	redirect('welcome/verify','refresh');

Then it does not take you admin any more and take you to index page with flash message "You must log in".

Need to download code?

Navigation