Apply suggestions from code review

Co-authored-by: Hypolite Petovan <hypolite@mrpetovan.com>

diff --git a/src/Module/Admin/Storage.php b/src/Module/Admin/Storage.php
index 04c200429a65303d7b4ac4e58f0e52bb6a252503..0df11aae81cb06b8bbea2452411c0dbc4d4f87d6 100644 (file)
--- a/src/Module/Admin/Storage.php
+++ b/src/Module/Admin/Storage.php
@@ -129,7 +129,6 @@ class Storage extends BaseAdmin
                                'prefix' => $storage_form_prefix,
                                'form'   => $storage_form,
                                'active' => $current_storage_backend instanceof ICanWriteToStorage && $name === $current_storage_backend::getName(),
-                               'is_writable' => DI::config()->isWritable('storage', 'name'),
                        ];
                }
 
@@ -147,6 +146,7 @@ class Storage extends BaseAdmin
                        '$noconfig'              => DI::l10n()->t('This backend doesn\'t have custom settings'),
                        '$form_security_token'   => self::getFormSecurityToken("admin_storage"),
                        '$storagebackend_ro_txt' => !DI::config()->isWritable('storage', 'name') ? DI::l10n()->t('Changing the current backend is prohibited because it is set by an environment variable') : '',
+                       '$is_writable' => DI::config()->isWritable('storage', 'name'),
                        '$storagebackend'        => $current_storage_backend instanceof ICanWriteToStorage ? $current_storage_backend::getName() : DI::l10n()->t('Database (legacy)'),
                        '$availablestorageforms' => $available_storage_forms,
                ]);