Be sure to open the model before getting relevant information. Issue #…

…423
matiasdelellis · Aug 24, 2023 · ca75eeb · ca75eeb
1 parent ff949b3
commit ca75eeb
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/lib/Controller/SettingsController.php b/lib/Controller/SettingsController.php
@@ -170,6 +170,7 @@ public function setAppValue($type, $value) {
 					$message = $this->l10n->t("The maximum recommended area is %s", $this->getFourByThreeRelation($value));
 					$status = self::STATE_ERROR;
 				}
+				$model->open();
 				$maxImageArea = $model->getMaximumArea();
 				if ($value > $maxImageArea) {
 					$value = $maxImageArea;

diff --git a/lib/Settings/Admin.php b/lib/Settings/Admin.php
@@ -77,7 +77,13 @@ public function getForm() {
 
 		$model = $this->modelManager->getCurrentModel();
 		if (!is_null($model)) {
-			$maxImageRange = strval($model->getMaximumArea());
+			try {
+				$model->open();
+				$maxImageRange = strval($model->getMaximumArea());
+			} catch (\Exception $e) {
+				$resume .= $e->getMessage();
+				$isConfigured = false;
+			}
 		} else {
 			$resume .= $this->l10n->t("It seems you don't have any model installed.");
 			$isConfigured = false;