Because of this current logic like this in setDisplay():
<?php
$display_id = 'default';
if (!$this->displayHandlers->has($display_id)) {
debug(format_string('set_display() called with invalid display ID @display.', array('@display'=> $display_id)));
return FALSE;
}
?>It will never return false for an invalid display ID, but just return the default.
It is also checking twice the the displayHandlers (displayBag) has the plugin.
We should take this opportunity to just tidy this method up.
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| d8.tidy-setDisplay.patch | 2.27 KB | Idle | FAILED: [[SimpleTest]]: [MySQL] 52,192 pass(es), 10 fail(s), and 0 exception(s). | View details | Re-test |