View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0026590||mantisbt||installation||public||2020-01-10 13:44||2020-01-12 07:54|
|Summary||0026590: Wording of admin/check/index.php results could be clarified with a prefix|
When I run admin/check/index.php I get the attached results.
Maybe it's just me, but in both cases, the wording of the first line does not make it clear if it's:
I think this would be much clearer with a little prefix, like:
|Tags||No tags attached.|
The check's description is always stating the expectation (or at least it should be, I did not go back and check every single test).
I hear your argument, but I'm not sure I is really worth the effort to update the all the individual checks' descriptions... And I'm not sure that updating the function printing the test row would be the correct approach, as there are probably a few cases where Checking if or whatever prefix we pick, would not be applicable.
Thanks for the confirmation that it's stating the expectation.
I did come to that conclusion eventually, but for quite a while I thought this was showing me a list of errors, not a list of (failed) expectations. i.e. I thought it was telling me that
I don't know the codebase, but perhaps there is a way to add the prefix without changing every individual check's description...?
That's basically what I meant with _I'm not sure that updating the function printing the test row would be the correct approach, as there are probably a few cases where Checking if or whatever prefix we pick, would not be applicable._
In clear, there are functions that perform the check and print the results, check_print_test_row() and check_print_test_warn_row() so changing those would only work if ALL checks are written the same way.
|2020-01-10 13:44||rogueresearch||New Issue|
|2020-01-10 13:44||rogueresearch||File Added: MantisChecks.png|
|2020-01-10 18:06||dregad||Status||new => feedback|
|2020-01-10 18:06||dregad||Note Added: 0063421|
|2020-01-10 18:37||rogueresearch||Note Added: 0063422|
|2020-01-10 18:37||rogueresearch||Status||feedback => new|
|2020-01-11 17:09||dregad||Note Added: 0063426|
|2020-01-12 07:54||atrol||Category||ui => installation|