[webui] Refactor check of section for Image editor

openSUSE · Nov 9, 2017 · 7a48507 · 7a48507
1 parent fd9b095
commit 7a48507
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/api/app/controllers/webui/kiwi/images_controller.rb b/src/api/app/controllers/webui/kiwi/images_controller.rb
@@ -54,12 +54,12 @@ def show
       def edit
         @image.build_description if @image.description.nil?
         @description = @image.description.specification
-        @version =  'x.x.x'
+        @version = 'x.x.x'
         @package_groups = @image.default_package_group
         @author = @image.description.author
         @contact = @image.description.contact
 
-        @is_edit_details_action = params[:section] == 'details'
+        @is_edit_details_action = params[:section] == 'details' || params[:section].nil?
         @is_edit_software_action = params[:section] == 'software'
 
         respond_to do |format|

diff --git a/src/api/app/views/webui/kiwi/_tabs.html.erb b/src/api/app/views/webui/kiwi/_tabs.html.erb
@@ -24,8 +24,8 @@
       link_to_software =  (action_name == 'edit') ? '#' : { controller: 'kiwi/images', action: 'edit', section: 'software' }
     %>
     <%= content_tag('li', link_to('Details', link_to_details, { id: "kiwi_details_trigger"}),
-                    class: "#{'selected' if params[:section] == 'details'}", id: "kiwi_details_tab" ) %>
+                    class: "#{'selected' if @is_edit_details_action}", id: "kiwi_details_tab" ) %>
     <%= content_tag('li', link_to('Software', link_to_software, { id: "kiwi_software_trigger"}),
-                    class: "#{'selected' if params[:section] == 'software'}", id: "kiwi_software_tab" ) %>
+                    class: "#{'selected' if @is_edit_software_action}", id: "kiwi_software_tab" ) %>
   </ul>
 </div>