Bug 23272: (RM followup) Fix failing tests
authorMartin Renvoize <martin.renvoize@ptfs-europe.com>
Fri, 13 Sep 2019 10:19:28 +0000 (11:19 +0100)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Fri, 13 Sep 2019 10:21:42 +0000 (11:21 +0100)
commit9515677afa2d6823016b3351fddbc45930c21667
treeb9005570933e177d7e667d98a877a8b75a0347dd
parentce223ed30e4c4308d6585ae13c1da64d356cf7e0
Bug 23272: (RM followup) Fix failing tests

This bug introduced a regression whereby if any authorized value has no
branch limitations then the authorized value administration page would
fail to display at all.

A simple ternary corrects the issue, but I'm now wondering about the
logic of returning 'undef' from 'get_branch_limits'.. perhaps an empty
(or full) resultset might be a better option to prevent us from
requireing ternaries before calling chained methods on such objects.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
admin/authorised_values.pl