[serversetup-bwlp-ipxe/minilinux] Add comments, fix minor logic bug

Saving a boot entry didn't properly check the validity of a hook's ID,
which was broken anyways, so the invalid check didn't uncover that the
actual cheking code didn't work either.

1 files changed, 2 insertions, 3 deletions

diff --git a/modules-available/serversetup-bwlp-ipxe/page.inc.php b/modules-available/serversetup-bwlp-ipxe/page.inc.php
index cc5fdbe5..a636665e 100644
--- a/modules-available/serversetup-bwlp-ipxe/page.inc.php
+++ b/modules-available/serversetup-bwlp-ipxe/page.inc.php
@@ -527,7 +527,7 @@ class Page_ServerSetup extends Page
 						if ($he->moduleId === $row['module']) {
 							$he->setSelected($row['data']);
 							$he->checked = 'checked';
-							if ($he->getBootEntry($row['data']) === null) {
+							if (!$he->isValidId($he->getSelected())) {
 								Message::addError('invalid-custom-entry-id', $row['module'], $row['data']);
 							}
 							break;
@@ -848,8 +848,7 @@ class Page_ServerSetup extends Page
 			/** @var BootEntryHook $module */
 			$module = $hook->run();
 			$id = Request::post('selection-' . $type, false, 'string');
-			$entry = $module->isValidId($id);
-			if ($entry === null) {
+			if (!$module->isValidId($id)) {
 				Message::addError('invalid-custom-entry-id', $type, $id);
 				return;
 			}