We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
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:
move
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.
request->query['ids']
ids
Second, instead of $this->cakeError there should be thrown an exception, for example BadRequestException.
$this->cakeError
BadRequestException
This is already reported in #278.
I also try to fix this issue and open a pull request with my fix proposal.
The text was updated successfully, but these errors were encountered:
Successfully merging a pull request may close this issue.
I go to issues list, then I select multiple tasks and open context menu in which I select Move and I get:
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 addsids
query param.Second, instead of
$this->cakeError
there should be thrown an exception, for exampleBadRequestException
.This is already reported in #278.
I also try to fix this issue and open a pull request with my fix proposal.
The text was updated successfully, but these errors were encountered: