fix: refresh storage issues when language changes

agama-project · Sep 27, 2024 · f305cf0 · f305cf0
1 parent f3e5331
commit f305cf0
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 7 deletions.
diff --git a/service/lib/agama/storage/manager.rb b/service/lib/agama/storage/manager.rb
@@ -179,6 +179,7 @@ def actions
       # @param locale [String] new locale
       def locale=(locale)
         change_process_locale(locale)
+        update_issues
       end
 
     private

diff --git a/service/lib/agama/storage/proposal.rb b/service/lib/agama/storage/proposal.rb
@@ -45,11 +45,6 @@ def initialize(config, logger: nil)
         @on_calculate_callbacks = []
       end
 
-      # List of issues.
-      #
-      # @return [Array<Issue>]
-      attr_reader :issues
-
       # Whether the proposal was already calculated.
       #
       # @return [Boolean]
@@ -192,6 +187,21 @@ def guided_settings
         strategy_object.settings
       end
 
+      # List of issues.
+      #
+      # @return [Array<Issue>]
+      def issues
+        return [] unless storage_manager.probed?
+
+        updated = []
+        if proposal&.failed? || @calculate_error.is_a?(Y2Storage::NoDiskSpaceError)
+          updated << failed_issue
+        end
+        updated << failed_issue if proposal.failed?
+        updated.concat(strategy_object.issues)
+        updated
+      end
+
     private
 
       # @return [Agama::Config]
@@ -232,13 +242,14 @@ def calculate_agama_from_json(storage_json)
       def calculate
         return false unless storage_manager.probed?
 
-        @issues = []
+        @calculate_error = nil
 
         begin
           strategy_object.calculate
           @issues << failed_issue if proposal.failed?
         rescue Y2Storage::Error => e
-          handle_exception(e)
+          @calculate_error = e
+          raise e
         end
 
         @issues.concat(strategy_object.issues)