panda-cms 0.7.4 → 0.7.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 07d84fd0b6bd17f5e5f253a570bae55270227bec0196037960aa3358d09903bd
-  data.tar.gz: c6bc6d0173f89d965a23132bc016bfd3a44e6bd0b2533960cca495fa28e645e3
+  metadata.gz: a11a8249369ac564953d025d4887020ced356b6357a5d8c823580c27e39394cb
+  data.tar.gz: 04d6f8dab1ae48f77414ea10b515739f36a54c0f37b79e82f7d9282305af03a1
 SHA512:
-  metadata.gz: 63b30314a40563ab4cb8f99d516cd30edc3aed7ffe8d1841d4b8760ea6c80c8b253a983ca7cadd473caba240a7b18132cda903b23ed35da845833aef4880b2d4
-  data.tar.gz: 820e1b606f088e8bacd7e03d997812f5a2255941e399e8fe863076a0a1196935c66c6a2aca34d50f9dd69563280cda9d795692ccba6df5af565aefd2b38439e2
+  metadata.gz: eb71f7f01ba585e142738e5d28510c5a0508b69843aab7c5b60fbf5cbaa7bb89419a916df923d0282cd2311a61614d836002635e67b9d6dfadd4bd6dba4e3f81
+  data.tar.gz: a795545501c2633b3e911b93ccd5c302adbb7d9c9f5a0ab6c363196ae4e0d18089a2b99cc3ada4b66e44902c8a5828212251b331e1b8b4d4762dfafef74dfe0f

data/README.md

@@ -12,7 +12,7 @@ Better websites, on Rails.
 🐼 is grown from our work at [Otaina](https://www.otaina.co.uk), a small group of freelancers. We needed something that could handle websites large and small – but where we could expand it too. We sent our first websites live in March 2024.
 
 ![Gem Version](https://img.shields.io/gem/v/panda-cms) ![Build Status](https://img.shields.io/github/actions/workflow/status/tastybamboo/panda-cms/ci.yml)
-![GitHub Last Commit](https://img.shields.io/github/last-commit/tasty-bamboo/panda-cms) [![Ruby Code Style](https://img.shields.io/badge/code_style-standard-brightgreen.svg)](https://github.com/standardrb/standard)
+![GitHub Last Commit](https://img.shields.io/github/last-commit/tastybamboo/panda-cms) [![Ruby Code Style](https://img.shields.io/badge/code_style-standard-brightgreen.svg)](https://github.com/standardrb/standard)
 
 ## Usage
 
@@ -66,8 +66,43 @@ We welcome contributions.
 
 See our [Contributing Guidelines](https://docs.pandacms.io/developers/contributing/)
 
+### Testing
+
+Panda CMS uses RSpec for testing.
+
+#### Using Fixtures
+
+We encourage using fixtures for tests instead of factories for consistent test data:
+
+1. Create fixture files in `spec/fixtures` named after the model's table (e.g., `panda_cms_pages.yml`)
+2. Define records with unique names and their attributes
+3. Use helper methods to create test templates with mocked file validation
+
+Example fixture format:
+
+```yaml
+# spec/fixtures/panda_cms_pages.yml
+home_page:
+  title: "Home"
+  path: "/"
+  panda_cms_template_id: <%= ActiveRecord::FixtureSet.identify(:page_template) %>
+  status: "active"
+  created_at: <%= Time.current %>
+  updated_at: <%= Time.current %>
+```
+
+Example test using fixtures:
+
+```ruby
+# Access fixture using table name and record name
+page = panda_cms_pages(:home_page)
+expect(page.title).to eq("Home")
+```
+
+When testing models with file validations or complex callbacks, use the helper methods in `spec/models/panda/cms/page_spec.rb` as a reference.
+
 ## License
 
 The gem is available as open source under the terms of the [BSD-3-Clause License](https://opensource.org/licenses/bsd-3-clause).
 
-Copyright © 2024 - 2025, Panda Software Limited.
+Copyright © 2024 - 2025, Otaina Limited.

data/Rakefile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "bundler/setup"
 
 APP_RAKEFILE = File.expand_path("spec/dummy/Rakefile", __dir__)

data/app/builders/panda/cms/form_builder.rb

@@ -1,10 +1,12 @@
+# frozen_string_literal: true
+
 module Panda
   module CMS
     class FormBuilder < ActionView::Helpers::FormBuilder
       include ActionView::Helpers::TagHelper
       include ActionView::Helpers::FormTagHelper
 
-      def label(attribute, text = nil, options = {}, &block)
+      def label(attribute, text = nil, options = {})
         super(attribute, text, options.reverse_merge(class: label_styles))
       end
 
@@ -13,8 +15,11 @@ module Panda
           content_tag :div, class: container_styles do
             label(attribute) + meta_text(options) +
               content_tag(:div, class: "flex flex-grow") do
-                content_tag(:span, class: "inline-flex items-center px-3 text-base border border-r-none rounded-s-md whitespace-nowrap break-keep") { options.dig(:data, :prefix) } +
-                  super(attribute, options.reverse_merge(class: input_styles_prefix + " input-prefix rounded-l-none border-l-none"))
+                content_tag(:span,
+                  class: "inline-flex items-center px-3 text-base border border-r-none rounded-s-md whitespace-nowrap break-keep") do
+                  options.dig(:data, :prefix)
+                end +
+                  super(attribute, options.reverse_merge(class: "#{input_styles_prefix} input-prefix rounded-l-none border-l-none"))
               end + error_message(attribute)
           end
         else
@@ -48,7 +53,7 @@ module Panda
         end
       end
 
-      def select(method, choices = nil, options = {}, html_options = {}, &block)
+      def select(method, choices = nil, options = {}, html_options = {})
         content_tag :div, class: container_styles do
           label(method) + meta_text(options) + super(method, choices, options, html_options.reverse_merge(class: select_styles)) + select_svg + error_message(method)
         end
@@ -147,6 +152,7 @@ module Panda
 
       def meta_text(options)
         return unless options[:meta]
+
         @template.content_tag(:p, options[:meta], class: "block text-black/60 text-sm mb-2")
       end
 
@@ -173,7 +179,8 @@ module Panda
       end
 
       def select_svg
-        @template.content_tag(:svg, class: "pointer-events-none absolute right-3 top-1/2 -translate-y-1/2 text-gray-400", aria_hidden: true) do
+        @template.content_tag(:svg,
+          class: "pointer-events-none absolute right-3 top-1/2 -translate-y-1/2 text-gray-400", aria_hidden: true) do
           @template.content_tag(:path, d: "M9.293 12.95l.707.707L15.657 8l-1.414-1.414L10 10.828 5.757 6.586 4.343 8z")
         end
       end
@@ -208,6 +215,7 @@ module Panda
 
       def error_message(attribute)
         return unless object.respond_to?(:errors) && object.errors[attribute]&.any?
+
         content_tag(:p, class: "mt-2 text-sm text-red-600") do
           object.errors[attribute].join(", ")
         end

data/app/components/panda/cms/admin/heading_component.rb

@@ -29,13 +29,14 @@ module Panda
           output = output.html_safe
           base_heading_styles = "flex pt-1 text-black mb-5 -mt-1"
 
-          if level == 1
+          case level
+          when 1
             content_tag(:h1, output, class: [base_heading_styles, "text-2xl font-medium", @additional_styles])
-          elsif level == 2
+          when 2
             content_tag(:h2, output, class: [base_heading_styles, "text-xl font-medium", @additional_styles])
-          elsif level == 3
+          when 3
             content_tag(:h3, output, class: [base_heading_styles, "text-xl", "font-light", @additional_styles])
-          elsif level == :panel
+          when :panel
             content_tag(:h3, output, class: ["text-base font-medium p-4 text-white"])
           end
         end

data/app/components/panda/cms/admin/panel_component.rb

@@ -4,9 +4,9 @@ module Panda
   module CMS
     module Admin
       class PanelComponent < ViewComponent::Base
-        renders_one :heading, ->(text:, icon: "", level: :panel, additional_styles: "") do
+        renders_one :heading, lambda { |text:, icon: "", level: :panel, additional_styles: ""|
           Panda::CMS::Admin::HeadingComponent.new(text: text, icon: icon, level: level, additional_styles: additional_styles)
-        end
+        }
       end
     end
   end

data/app/components/panda/cms/admin/statistics_component.rb

@@ -4,8 +4,7 @@ module Panda
   module CMS
     module Admin
       class StatisticsComponent < ViewComponent::Base
-        attr_reader :metric
-        attr_reader :value
+        attr_reader :metric, :value
 
         def initialize(metric:, value:)
           @metric = metric

data/app/components/panda/cms/admin/user_activity_component.html.erb

@@ -1,5 +1,7 @@
-<% if user.is_a?(Panda::CMS::User) %>
+<% if user.is_a?(Panda::CMS::User) && time %>
   <%= render Panda::CMS::Admin::UserDisplayComponent.new(user: user, metadata: "#{time_ago_in_words(time)} ago") %>
+<% elsif user.is_a?(Panda::CMS::User) %>
+  <%= render Panda::CMS::Admin::UserDisplayComponent.new(user: user, metadata: "Not published") %>
 <% elsif time %>
   <div class="text-black/60"><%= time_ago_in_words(time) %> ago</div>
 <% end %>

data/app/components/panda/cms/admin/user_activity_component.rb

@@ -4,9 +4,7 @@ module Panda
   module CMS
     module Admin
       class UserActivityComponent < ViewComponent::Base
-        attr_accessor :model
-        attr_accessor :time
-        attr_accessor :user
+        attr_accessor :model, :time, :user
 
         # @param model [ActiveRecord::Base] Model instance to which the user activity is related
         # @param at [ActiveSupport::TimeWithZone] Time of the activity

data/app/components/panda/cms/code_component.rb

@@ -17,15 +17,19 @@ module Panda
         @options[:id] ||= "code-#{key.to_s.dasherize}"
         @editable = editable
 
-        raise BlockError.new("Key 'code' is not allowed for CodeComponent") if key == :code
+        raise BlockError, "Key 'code' is not allowed for CodeComponent" if key == :code
       end
 
       def call
         # TODO: For the non-editable version, grab this from a cache or similar?
-        block = Panda::CMS::Block.find_by(kind: KIND, key: @key, panda_cms_template_id: Current.page.panda_cms_template_id)
+        block = Panda::CMS::Block.find_by(kind: KIND, key: @key,
+          panda_cms_template_id: Current.page.panda_cms_template_id)
 
         if block.nil?
-          raise Panda::CMS::MissingBlockError.new("Block with key #{@key} not found for page #{Current.page.title}") unless Rails.env.production?
+          unless Rails.env.production?
+            raise Panda::CMS::MissingBlockError, "Block with key #{@key} not found for page #{Current.page.title}"
+          end
+
           return false
         end
 
@@ -58,7 +62,7 @@ module Panda
 
       def is_embedded?
         # TODO: Check security on this - embed_id should match something?
-        request.params.dig(:embed_id).present?
+        request.params[:embed_id].present?
       end
     end
   end

data/app/components/panda/cms/menu_component.rb

@@ -23,24 +23,25 @@ module Panda
 
         @menu_items = @menu_items.order(:lft).map do |menu_item|
           if is_active?(menu_item)
-            menu_item.define_singleton_method(:css_classes) { styles[:default] + " " + styles[:active] }
+            menu_item.define_singleton_method(:css_classes) { "#{styles[:default]} #{styles[:active]}" }
           else
-            menu_item.define_singleton_method(:css_classes) { styles[:default] + " " + styles[:inactive] }
+            menu_item.define_singleton_method(:css_classes) { "#{styles[:default]} #{styles[:inactive]}" }
           end
 
           menu_item
         end
 
         # TODO: Surely don't need this but Current.page isn't working in the component
-        if @render_page_menu
-          @current_page = Panda::CMS::Page.find_by(path: @current_path)
-          @page_menu_styles = page_menu_styles
-        end
+        return unless @render_page_menu
+
+        @current_page = Panda::CMS::Page.find_by(path: @current_path)
+        @page_menu_styles = page_menu_styles
       end
 
       def is_active?(menu_item)
         return true if @current_path == "/" && active_link?(menu_item.page.path, match: :exact)
         return true if menu_item.page.path != "/" && active_link?(menu_item.page.path, match: :starts_with)
+
         false
       end
 

data/app/components/panda/cms/page_menu_component.rb

@@ -3,31 +3,29 @@
 module Panda
   module CMS
     class PageMenuComponent < ViewComponent::Base
-      attr_accessor :page
-      attr_accessor :menu_item
-      attr_accessor :styles
+      attr_accessor :page, :menu_item, :styles
 
       def initialize(page:, start_depth:, styles: {}, show_heading: true)
         @page = page
 
-        unless @page.nil?
-          start_page = if @page.depth == start_depth
-            @page
-          else
-            @page.ancestors.find { |anc| anc.depth == start_depth }
-          end
+        return if @page.nil?
 
-          menu = start_page&.page_menu
-          return if menu.nil?
+        start_page = if @page.depth == start_depth
+          @page
+        else
+          @page.ancestors.find { |anc| anc.depth == start_depth }
+        end
 
-          @menu_item = menu.menu_items.order(:lft)&.first
+        menu = start_page&.page_menu
+        return if menu.nil?
 
-          @show_heading = show_heading
+        @menu_item = menu.menu_items.order(:lft)&.first
 
-          # Set some default styles for sanity
-          @styles = styles
-          @styles[:indent_with] ||= "pl-2"
-        end
+        @show_heading = show_heading
+
+        # Set some default styles for sanity
+        @styles = styles
+        @styles[:indent_with] ||= "pl-2"
       end
 
       def render?

data/app/components/panda/cms/rich_text_component.rb

@@ -12,9 +12,7 @@ module Panda
 
       KIND = "rich_text"
 
-      attr_accessor :editable
-      attr_accessor :content
-      attr_accessor :options
+      attr_accessor :editable, :content, :options
 
       def initialize(key: :text_component, text: "Lorem ipsum...", editable: true, **options)
         @key = key
@@ -27,7 +25,8 @@ module Panda
       def before_render
         @editable &&= params[:embed_id].present? && params[:embed_id] == Current.page.id && Current.user.admin?
 
-        block = Panda::CMS::Block.find_by(kind: "rich_text", key: @key, panda_cms_template_id: Current.page.panda_cms_template_id)
+        block = Panda::CMS::Block.find_by(kind: "rich_text", key: @key,
+          panda_cms_template_id: Current.page.panda_cms_template_id)
         raise ComponentError, "Block not found for key: #{@key}" unless block
 
         block_content = block.block_contents.find_by(panda_cms_page_id: Current.page.id)
@@ -65,7 +64,7 @@ module Panda
                     # Ensure the content is properly structured
                     {
                       "time" => parsed["time"] || Time.current.to_i * 1000,
-                      "blocks" => parsed["blocks"].map { |block|
+                      "blocks" => parsed["blocks"].map do |block|
                         {
                           "type" => block["type"],
                           "data" => block["data"].merge(
@@ -73,7 +72,7 @@ module Panda
                           ),
                           "tunes" => block["tunes"]
                         }.compact
-                      },
+                      end,
                       "version" => parsed["version"] || "2.28.2"
                     }
                   else

data/app/components/panda/cms/text_component.rb

@@ -25,10 +25,10 @@ module Panda
         content_tag(:span, @content, @options, false) # Don't escape the content
       rescue
         if !Rails.env.production? || is_defined?(Sentry)
-          raise Panda::CMS::MissingBlockError.new("Block with key #{@key} not found for page #{Current.page.title}")
-        else
-          false
+          raise Panda::CMS::MissingBlockError, "Block with key #{@key} not found for page #{Current.page.title}"
         end
+
+        false
       end
 
       #
@@ -45,11 +45,10 @@ module Panda
       def before_render
         @editable &&= params[:embed_id].present? && params[:embed_id] == Current.page.id
 
-        block = Panda::CMS::Block.find_by(kind: KIND, key: @key, panda_cms_template_id: Current.page.panda_cms_template_id)
+        block = Panda::CMS::Block.find_by(kind: KIND, key: @key,
+          panda_cms_template_id: Current.page.panda_cms_template_id)
 
-        if block.nil?
-          return false
-        end
+        return false if block.nil?
 
         block_content = block.block_contents.find_by(panda_cms_page_id: Current.page.id)
         plain_text = block_content&.content.to_s

data/app/constraints/panda/cms/admin_constraint.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Panda
   module CMS
     class AdminConstraint
@@ -11,7 +13,8 @@ module Panda
       end
 
       def current_user(request)
-        User.find_by(id: request.session[:user_id])
+        user_id = request.session[:user_id]
+        User.find_by(id: user_id)
       end
     end
   end

data/app/controllers/panda/cms/admin/dashboard_controller.rb

@@ -1,19 +1,23 @@
+# frozen_string_literal: true
+
 require "groupdate"
 
 module Panda
   module CMS
-    class Admin::DashboardController < ApplicationController
-      before_action :set_initial_breadcrumb, only: %i[show]
-      before_action :authenticate_admin_user!
+    module Admin
+      class DashboardController < ApplicationController
+        before_action :set_initial_breadcrumb, only: %i[show]
+        before_action :authenticate_admin_user!
 
-      # GET /admin
-      def show
-      end
+        # GET /admin
+        def show
+        end
 
-      private
+        private
 
-      def set_initial_breadcrumb
-        add_breadcrumb "Dashboard", Panda::CMS.route_namespace
+        def set_initial_breadcrumb
+          add_breadcrumb "Dashboard", Panda::CMS.route_namespace
+        end
       end
     end
   end

data/app/controllers/panda/cms/admin/forms_controller.rb

@@ -38,8 +38,6 @@ module Panda
           add_breadcrumb "Forms", admin_forms_path
         end
 
-        private
-
         # Only allow a list of trusted parameters through
         # @type private
         # @return ActionController::StrongParameters

data/app/controllers/panda/cms/admin/my_profile_controller.rb

@@ -19,7 +19,8 @@ module Panda
         # @return void
         def update
           if current_user.update(user_params)
-            redirect_to edit_admin_my_profile_path, flash: {success: "Your profile has been updated successfully."}
+            flash[:success] = "Your profile has been updated successfully."
+            redirect_to edit_admin_my_profile_path
           else
             render :edit, locals: {user: current_user}, status: :unprocessable_entity
           end

data/app/controllers/panda/cms/admin/pages_controller.rb

@@ -38,7 +38,8 @@ module Panda
           page.path = nil if page.path.blank?
 
           # Set the full path before validation if we have a parent
-          if page.parent && page.parent.path != "/" && page.path.present?
+          if page.parent && page.parent.path != "/" && page.path.present? && !page.path.start_with?(page.parent.path)
+            # Only prepend parent path if it's not already included
             page.path = page.parent.path + page.path
           end
 

data/app/controllers/panda/cms/admin/posts_controller.rb

@@ -39,8 +39,7 @@ module Panda
 
           if @post.save
             Rails.logger.debug "Post saved successfully"
-            flash[:success] = "The post was successfully created!"
-            redirect_to edit_admin_post_path(@post.admin_param), status: :see_other
+            redirect_to edit_admin_post_path(@post.admin_param), notice: "The post was successfully created!"
           else
             Rails.logger.debug "Post save failed: #{@post.errors.full_messages.inspect}"
             flash.now[:error] = @post.errors.full_messages.join(", ")

data/app/controllers/panda/cms/admin/sessions_controller.rb

@@ -27,9 +27,7 @@ module Panda
 
             # Always create the first user as admin, regardless of what our settings look like
             # else we can't ever really login. :)
-            if !create_as_admin
-              create_as_admin = true if !create_as_admin && Panda::CMS::User.count.zero?
-            end
+            create_as_admin = true if !create_as_admin && Panda::CMS::User.count.zero?
 
             if user_info["first_name"] && user_info["last_name"]
               firstname = user_info["first_name"]
@@ -55,9 +53,9 @@ module Panda
             return
           end
 
-          if !user.admin?
+          unless user.admin?
             # User can't be found with this email address or can't login
-            Rails.logger.error "User ID #{user.id} attempted admin login, is not admin." if user && !user.admin
+            Rails.logger.error "User ID #{user.id} attempted admin login, is not admin."
             redirect_to admin_login_path, flash: {error: t("panda.cms.admin.sessions.create.error")}
             return
           end

data/app/controllers/panda/cms/admin/settings/bulk_editor_controller.rb

@@ -1,36 +1,43 @@
+# frozen_string_literal: true
+
 module Panda
   module CMS
-    class Admin::Settings::BulkEditorController < ApplicationController
-      before_action :set_initial_breadcrumb, only: %i[new]
+    module Admin
+      module Settings
+        class BulkEditorController < ApplicationController
+          before_action :set_initial_breadcrumb, only: %i[new]
 
-      def new
-        @json_data = BulkEditor.export
-      end
+          def new
+            @json_data = BulkEditor.export
+          end
 
-      def create
-        begin
-          debug_output = BulkEditor.import(params[:site_content])
-        rescue JSON::ParserError
-          redirect_to admin_settings_bulk_editor_path, flash: {error: "Error parsing content; are you sure this update is valid? Reverting..."}
-          return
-        end
+          def create
+            begin
+              debug_output = BulkEditor.import(params[:site_content])
+            rescue JSON::ParserError
+              redirect_to admin_settings_bulk_editor_path,
+                flash: {error: "Error parsing content; are you sure this update is valid? Reverting..."}
+              return
+            end
 
-        # Grab the latest content back so it's all formatted properly
-        @json_data = BulkEditor.export
+            # Grab the latest content back so it's all formatted properly
+            @json_data = BulkEditor.export
 
-        if debug_output[:error].empty? && debug_output[:warning].empty? && debug_output[:success].empty?
-          redirect_to admin_settings_bulk_editor_path, flash: {success: "No changes were found!"}
-        else
-          @debug = debug_output
-          render :new, flash: {warning: "Please review the output below for more information."}
-        end
-      end
+            if debug_output[:error].empty? && debug_output[:warning].empty? && debug_output[:success].empty?
+              redirect_to admin_settings_bulk_editor_path, flash: {success: "No changes were found!"}
+            else
+              @debug = debug_output
+              render :new, flash: {warning: "Please review the output below for more information."}
+            end
+          end
 
-      private
+          private
 
-      def set_initial_breadcrumb
-        add_breadcrumb "Settings", admin_settings_path
-        add_breadcrumb "Bulk Editor", "#"
+          def set_initial_breadcrumb
+            add_breadcrumb "Settings", admin_settings_path
+            add_breadcrumb "Bulk Editor", "#"
+          end
+        end
       end
     end
   end

data/app/controllers/panda/cms/admin/settings_controller.rb

@@ -1,19 +1,23 @@
+# frozen_string_literal: true
+
 module Panda
   module CMS
-    class Admin::SettingsController < ApplicationController
-      before_action :set_initial_breadcrumb, only: %i[index show]
-      before_action :authenticate_admin_user!
+    module Admin
+      class SettingsController < ApplicationController
+        before_action :set_initial_breadcrumb, only: %i[index show]
+        before_action :authenticate_admin_user!
 
-      def index
-      end
+        def index
+        end
 
-      def show
-      end
+        def show
+        end
 
-      private
+        private
 
-      def set_initial_breadcrumb
-        add_breadcrumb "Settings", admin_settings_path
+        def set_initial_breadcrumb
+          add_breadcrumb "Settings", admin_settings_path
+        end
       end
     end
   end

data/app/controllers/panda/cms/application_controller.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Panda
   module CMS
     class ApplicationController < ::ActionController::Base
@@ -37,11 +39,14 @@ module Panda
       end
 
       def authenticate_user!
-        redirect_to root_path, flash: {error: "Please login to view this!"} unless user_signed_in?
+        redirect_to main_app.root_path, flash: {error: "Please login to view this!"} unless user_signed_in?
       end
 
       def authenticate_admin_user!
-        redirect_to root_path, flash: {error: "Please login to view this!"} unless user_signed_in? && current_user.admin?
+        return if user_signed_in? && current_user.admin?
+
+        redirect_to admin_login_path,
+          flash: {error: "Please login to view this!"}
       end
 
       # Required for paper_trail and seems as good as convention these days

data/app/controllers/panda/cms/errors_controller.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Panda
   module CMS
     class ErrorsController < ApplicationController
@@ -5,7 +7,8 @@ module Panda
 
       def show
         exception = request.env["action_dispatch.exception"]
-        status_code = exception.try(:status_code) || ActionDispatch::ExceptionWrapper.new(request.env, exception).status_code
+        status_code = exception.try(:status_code) || ActionDispatch::ExceptionWrapper.new(request.env,
+          exception).status_code
 
         render view_for_code(status_code), status: status_code
       end
@@ -17,7 +20,7 @@ module Panda
       private
 
       def view_for_code(code)
-        supported_error_codes.fetch(code) { "404" }
+        supported_error_codes.fetch(code, "404")
       end
 
       def supported_error_codes