Open yaras opened 8 years ago
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.
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,
movemethod 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 addsidsquery param.Second, instead of
$this->cakeErrorthere 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.