Skip to content
Snippets Groups Projects
Commit 66fc9757 authored by Prasad's avatar Prasad
Browse files

Merge branch 'fixLoginForMigration' into 'master'

fix login in migration page

In the migration page, there is an authentication check to allow user to login. However, the `Migration_Extract_Action` requires login by default.

Since the check was done before authentication check, this causes an authentication failure whether the username and password is correct or not.

Making login not required by `Migration_Extract_Action` will allow the authentication.

This commit also fixes the login issue for `Migration_Extract_Action` since there are something missing in the `$_SESSION`

See merge request !854
parents 370dc2fa 11ca6116
No related branches found
No related tags found
No related merge requests found
......@@ -10,6 +10,9 @@
class Migration_Extract_Action extends Vtiger_Action_Controller {
public function loginRequired() {
return false;
}
public function process(Vtiger_Request $request) {
global $root_directory, $log;
@session_start();
......@@ -30,6 +33,7 @@ class Migration_Extract_Action extends Vtiger_Action_Controller {
$userid = $user->retrieve_user_id($userName);
$_SESSION['authenticated_user_id'] = $userid;
$_SESSION['app_unique_key'] = vglobal('application_unique_key');
header('Location: index.php?module=Migration&view=Index&mode=step1');
} else {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment