View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0026482 | mantisbt | ui | public | 2019-12-18 13:17 | 2020-03-15 15:23 |
Reporter | rogueresearch | Assigned To | atrol | ||
Priority | normal | Severity | major | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Product Version | 2.23.0 | ||||
Target Version | 2.23.1 | Fixed in Version | 2.23.1 | ||
Summary | 0026482: 'View Issue' page fails to populate some fields (ex 'ID') for some projects (but not others) | ||||
Description | We have 2.21.1 on our production server and are testing 2.23.0 on a test server. We noticed a difference viewing some issues between 2.21.1 and 2.23.0. See attached image. In 2.21.1 some fields didn't even get a heading. 2.23.0 seems to have added the heading (good!), but the value is still missing. In the screenshot, I would expect to see ID as "7142". (We never noticed this in 2.21.1, probably just thought it was strange spacing/formatting, but the addition of the headers in 2.23.0 makes it move obvious this has been broken for a while.) Additionally, the 'summary' field doesn't have the issue number prepended in 2.23.0 but does in 2.21.1. See attached image again. It seems to happen for all issues in a particular project, but for no issues in other projects. Not sure what the relevant difference between projects is. The project where it occurs has many custom fields so it could be related to that. | ||||
Tags | No tags attached. | ||||
Attached Files | |||||
This is / was not the stanadard, so I assume you have installed some custom functions and / or plugins on your our production server that have not been activated on your test server. |
|
The only plugins we have are the default ones, plus: Source, SourceSVN, and SourceGitlab (which is installed but not used yet). Prod server uses 2.2 and test server was upgraded to 2.3. Not sure what you mean by "custom functions", but I don't believe we have any. We certainly didn't change any code (well only <https://github.com/mantisbt-plugins/source-integration/pull/261>) The project in question does use many custom fields, all created in Mantis' own web UI. |
|
Check if file config/custom_functions_inc.php exists on the production server. |
|
No such file, only: .htaccess, Web.config, config_inc.php, config_inc.php.sample |
|
It's better to start with the existing problem you have in 2.21.1, as this might be the root cause for the issue you get in 2.23.0. I am not able to reproduce this behavior and none of our users entered a similar problem until now. The following additional information may be useful:
|
|
rogueresearch, You did not provide feedback; I am therefore resolving this issue as "no change required". Feel free to reopen the issue at a later time and provide the requested information. |
|
The feedback will be coming. We use mantis at work, and we've been on xmas vacation recently... Back to work Jan 6, should have time that week to investigate more. |
|
Happy new year! So we found how to trigger this. Basically, several of our projects have customizations in Manage > Manage Configurations > Configuration Report. For example, we configured "bug_view_page_fields" to: array ( (see attached.) If I delete that custom configuration entirely then the 2.23 view page again shows the "ID" field (and a whole bunch of other stuff). So, I'd say 2.21.1 has correct behaviour and 2.23 is buggy: a) it shouldn't show "ID" and other field headings that were removed in customization. |
|
Thanks @rogueresearch I am now able to reproduce the issue. |
|
Great! BTW: those arrays in my last screenshot, do they have to have integer keys like that? If so, should they start at 0 or 1? I've noticed our customizations are inconsistent in that regard. Thanks. |
|
No, they keys are added automatically when entering the arrays |
|
Cool, thanks for the speedy fix! I don't know PHP, but from the look of the diff, the issue is merely cosmetic, correct? I'm trying to decide if this should prevent me from updating to 2.23. Lastly, can PHP 5.4 syntax be used for those arrays? i.e. could I change:
to just: |
|
I am not sure if I understand what you mean with this.
When upgrading to 2.23.0, be aware that there is at least one more regression in 2.23.0 where you might be affected 0026470
No, but you can use this syntax to enter the configuration (as told before, keys are added automatically) |
|
Sorry for the confusion. I'm just wanting to be sure there isn't, for example, a database migration bug here. i.e. if I update from 2.21.1 to 2.23.0, yes, I'll have blank looking fields, but it's just a cosmetic issue. When I later upgrade to 2.23.1 or later, all my data is safe, and the cosmetic bug will also be fixed. If this bug is indeed cosmetic only, I'll probably upgrade to 2.23.0 anyway. |
|