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

Bulk move is not working #279

Open
yaras opened this issue Mar 26, 2016 · 0 comments · May be fixed by #280
Open

Bulk move is not working #279

yaras opened this issue Mar 26, 2016 · 0 comments · May be fixed by #280
Labels

Comments

@yaras
Copy link

yaras commented Mar 26, 2016

I go to issues list, then I select multiple tasks and open context menu in which I select Move and I get:

An Internal Error Has Occurred.
Error: An Internal Error Has Occurred.

After I dug into code I found two issues:

First, move method checks:

  • request->params['issue_id']
  • request->params['url']['ids']
  • request->data['Issue']['ids']

but it does not check request->query['ids'], which should be done as bulk move adds ids query param.

Second, instead of $this->cakeError there should be thrown an exception, for example BadRequestException.

This is already reported in #278.

I also try to fix this issue and open a pull request with my fix proposal.

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

Successfully merging a pull request may close this issue.

2 participants