Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Missing "comm_path" change to js env variable #4648

Closed
rcubetrac opened this issue Aug 22, 2014 · 3 comments
Closed

Missing "comm_path" change to js env variable #4648

rcubetrac opened this issue Aug 22, 2014 · 3 comments

Comments

@rcubetrac
Copy link

Reported by kevinlam on 22 Aug 2014 01:49 UTC as Trac ticket #1490041

When the "$RCMAIL->set_task" was called in php side, it does not call the "output->set_env" to change the "comm_path" variable to js side. Then, the js side script uses the wrong url to make requests.

Simple Fix:

At rcmail.php::set_task

...
if ($this->output) {
$this->output->set_env('task', $this->task);
@+++
$this->output->set_env('comm_path', $this->comm_path);
}
...

Migrated-From: http://trac.roundcube.net/ticket/1490041

@rcubetrac
Copy link
Author

Comment by @alecpl on 22 Aug 2014 06:53 UTC

Fixed in e35eab5.

@rcubetrac
Copy link
Author

Status changed by @alecpl on 22 Aug 2014 06:53 UTC

new => closed

@rcubetrac
Copy link
Author

Milestone changed by @alecpl on 22 Aug 2014 06:53 UTC

later => 1.0.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant