View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0024877 | mantisbt | bugtracker | public | 2018-10-21 00:30 | 2019-01-02 17:25 |
Reporter | vboctor | Assigned To | vboctor | ||
Priority | normal | Severity | minor | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Product Version | 2.18.0 | ||||
Target Version | 2.19.0 | Fixed in Version | 2.19.0 | ||
Summary | 0024877: IssueNoteAddCommand: reassign_on_feedback doesn't work if reporter is not specified | ||||
Description | The web UI and REST API provide the reporter explicitly, but if not, and the id is retrieved from logged in user, the user id can be a string instead of an integer which causes the following check to fail:
The specific check that fails is: The workaround is to set the reporter explicitly. | ||||
Steps To Reproduce | Here a sample script that reproduces the problem. Note that includes are excluded.
| ||||
Tags | No tags attached. | ||||
@vboctor do you think that 0023712 and https://mantisbt.org/forums/viewtopic.php?f=3&t=25080 are the same issue like this one? |
|
@atrol yes, I have updated the PR to reflect that in the commit message and added a couple of extra defensive fixes. |
|
I have pushed the changes to the source branch but github didn't pick it up in the PR due to semi-outage issues. Changes can be viewed at: |
|
@atrol are you ok with merging this PR? |
|
I approved the PR. |
|
MantisBT: master 183a4a94 2018-10-20 20:32 Details Diff |
Fix IssueNoteAddCommand re-assign on feedback The issue was caused by logged in user id being returned as a string. Fixes 0024877, 0023712 |
Affected Issues 0023712, 0024877 |
|
mod - core/authentication_api.php | Diff File | ||
mod - core/current_user_api.php | Diff File | ||
mod - core/user_api.php | Diff File |