Project

General

Profile

Actions
Copy link

Bug #649

closed

PHP header:Location function required the whole URL for redirection

Added by Eric Hou almost 14 years ago. Updated almost 14 years ago.

Status:
Closed
Priority:
Urgent
Assignee:
Eric Hou
Category:
-
Target version:
Appion/Leginon 2.0.0
Start date:
06/10/2010
Due date:
% Done:

100%

Estimated time:
Affected Version:
Show in known bugs:
Workaround:

Description

Some php versions allow to use filename to do redirect from page to page in the same sub directory but some don't.
Please add the whole URL when doing the redirect.

Thanks.

Actions
Copy link
 #1

Updated by Eric Hou almost 14 years ago

  • Assignee set to Eric Hou
Actions
Copy link
 #2

Updated by Eric Hou almost 14 years ago

  • Status changed from New to In Code Review
  • Assignee changed from Eric Hou to Amber Herold
  • % Done changed from 0 to 100

Add new function in the setupUtils.inc

It is very hard to test this unless we know which php version required the whole url for redirection.
Scott, could you test this, because we know your setup required the whole url for redirection.

To Test:
myamiweb/setup/index.php
and click "next" button

it should go to myamiweb/setup/setupBase.php page without any problem.

Thanks.

Actions
Copy link
 #3

Updated by Eric Hou almost 14 years ago

  • Priority changed from Normal to Urgent
Actions
Copy link
 #4

Updated by Scott Stagg almost 14 years ago

Do you want me to use some different version of myamiweb? If so, which? With the version that I installed, when I click next, it doesn't go to setupBase.php but instead stays on index.php and doesn't show any text in the white part of the window.

Actions
Copy link
 #5

Updated by Eric Hou almost 14 years ago

Hi Scott!
If you can check out the myamiweb again from svn co http://emg.nysbc.org/svn/myami/branches/myami-2.0/myamiweb myamiwebtest/

put it in your web dir.
and open your browser : myamiwebtest/setup/index.php

and click the next button.

Thanks.

Eric

Actions
Copy link
 #6

Updated by Scott Stagg almost 14 years ago

It still doesn't work for me when I use the version I checked out as you suggest above. If you would like to call me to work through this, I am available now. My number is 850-645-7872.

Actions
Copy link
 #7

Updated by Eric Hou almost 14 years ago

Scott!
I can call you tomorrow. We have meeting today whole day.

Eric

Actions
Copy link
 #8

Updated by Amber Herold almost 14 years ago

  • Assignee changed from Amber Herold to Eric Hou

Eric, can you add a more descriptive function header on redirect()? Also, should 'distPage' be 'destPage' as in destination page?

Actions
Copy link
 #9

Updated by Eric Hou almost 14 years ago

Hey Scott:
can you double check with you php.ini setup?
http://emg.nysbc.org/projects/appion/wiki/Configure_phpini

double check the following:
error_reporting = E_ALL & ~E_NOTICE & ~E_WARNING

display_errors = On

register_argc_argv = On

short_open_tag = On

I just updated some files, and I tested them on my mac and centos are both worked.
So can you svn update to get the newest code and test it again on your server?

Thanks.

Eric

Actions
Copy link
 #10

Updated by Scott Stagg almost 14 years ago

All the php.ini values are set correctly. I svn updated, and it still doesn't work. I should be in my office if you want to give me a call.

Actions
Copy link
 #11

Updated by Eric Hou almost 14 years ago

  • Status changed from In Code Review to In Test

Just tested new installation.
Should work fine now.

Thanks.

Actions
Copy link
 #12

Updated by Eric Hou almost 14 years ago

  • Status changed from In Test to Closed
Actions
Copy link

Also available in: Atom PDF