View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0021710 | mantisbt | ui | public | 2016-09-22 06:11 | 2016-10-02 18:41 |
Reporter | joel | Assigned To | dregad | ||
Priority | normal | Severity | tweak | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Product Version | 2.0.0-beta.3 | ||||
Target Version | 2.0.0-beta.3 | Fixed in Version | 2.0.0-beta.3 | ||
Summary | 0021710: Incorrect display on Bug report confirmation page | ||||
Description | As reported in https://github.com/mantisbt/mantisbt/pull/884#issuecomment-248846990 | ||||
Additional Information | Regression introduced by merge PR https://github.com/mantisbt/mantisbt/pull/884 | ||||
Tags | No tags attached. | ||||
Attached Files | |||||
1 & 2 are easily fixed. The problem with 3 is that the button actually submits a form with the report data, so there are 2 options to address the issue I'd rather go for a), even if it makes for a long URL string, to avoid code duplication. Let me know your thoughts. |
|
@joel Have a look at https://github.com/dregad/mantisbt/tree/f20-21710-oper-success, implements my earlier comments Note: incidentally, this also fixes 0021704 |
|
I Checked and tested your recent changes. it fixed the issue both in https://github.com/mantisbt/mantisbt/pull/884#issuecomment-248846990 and 0021704 Thanks |
|
@dregad Should i continue merging your fix/changes and push it to my PR?, or you want it to create a new PR with your changes here https://github.com/dregad/mantisbt/tree/f20-21710-oper-success Let me know your thoughts? |
|
I'll take care of it, thanks. |
|
I resolve this issue for the 2 display regressions, will follow up on point 3 in 0021704. |
|
MantisBT: master 9882d64f 2016-09-22 03:11 Details Diff |
Fix bug_report.php regressions Commit a7ae02dae5e20ba498520551984be026fc658005 (issue 0021683) introduced 2 regressions when displaying bug report page: - Confirmation message as '0': due to a missing parameter in the html_operation_confirmation() call - Incorrect display of the footer: caused by an extra div tag Fixes 0021710 |
Affected Issues 0021683, 0021710 |
|
mod - bug_report.php | Diff File |