From 315e31c5cb1c42d87bc50acb470421dc6ffc3198 Mon Sep 17 00:00:00 2001 From: Peter Yates Date: Wed, 13 Jan 2021 11:13:24 +0000 Subject: [PATCH] Always call super on initialize https://github.com/rubocop-hq/ruby-style-guide/issues/809 --- .../containers/character_count.rb | 2 +- lib/govuk_design_system_formbuilder/containers/check_boxes.rb | 3 ++- lib/govuk_design_system_formbuilder/containers/radios.rb | 3 ++- lib/govuk_design_system_formbuilder/elements/error_summary.rb | 4 ++-- lib/govuk_design_system_formbuilder/elements/null.rb | 4 +++- lib/govuk_design_system_formbuilder/elements/submit.rb | 3 ++- 6 files changed, 12 insertions(+), 7 deletions(-) diff --git a/lib/govuk_design_system_formbuilder/containers/character_count.rb b/lib/govuk_design_system_formbuilder/containers/character_count.rb index 5ab7e471..cb861dd6 100644 --- a/lib/govuk_design_system_formbuilder/containers/character_count.rb +++ b/lib/govuk_design_system_formbuilder/containers/character_count.rb @@ -2,7 +2,7 @@ module GOVUKDesignSystemFormBuilder module Containers class CharacterCount < Base def initialize(builder, max_words:, max_chars:, threshold:) - @builder = builder + super(builder, nil, nil) fail ArgumentError, 'limit can be words or chars' if max_words && max_chars diff --git a/lib/govuk_design_system_formbuilder/containers/check_boxes.rb b/lib/govuk_design_system_formbuilder/containers/check_boxes.rb index 06793939..ac580654 100644 --- a/lib/govuk_design_system_formbuilder/containers/check_boxes.rb +++ b/lib/govuk_design_system_formbuilder/containers/check_boxes.rb @@ -2,7 +2,8 @@ module GOVUKDesignSystemFormBuilder module Containers class CheckBoxes < Base def initialize(builder, small:, classes: nil) - @builder = builder + super(builder, nil, nil) + @small = small @classes = classes end diff --git a/lib/govuk_design_system_formbuilder/containers/radios.rb b/lib/govuk_design_system_formbuilder/containers/radios.rb index dd472cf5..2d9ca118 100644 --- a/lib/govuk_design_system_formbuilder/containers/radios.rb +++ b/lib/govuk_design_system_formbuilder/containers/radios.rb @@ -4,7 +4,8 @@ class Radios < Base include Traits::Hint def initialize(builder, inline:, small:, classes:) - @builder = builder + super(builder, nil, nil) + @inline = inline @small = small @classes = classes diff --git a/lib/govuk_design_system_formbuilder/elements/error_summary.rb b/lib/govuk_design_system_formbuilder/elements/error_summary.rb index 597f8cdf..0618fe9d 100644 --- a/lib/govuk_design_system_formbuilder/elements/error_summary.rb +++ b/lib/govuk_design_system_formbuilder/elements/error_summary.rb @@ -4,8 +4,8 @@ class ErrorSummary < Base include Traits::Error def initialize(builder, object_name, title, link_base_errors_to:) - @builder = builder - @object_name = object_name + super(builder, object_name, nil) + @title = title @link_base_errors_to = link_base_errors_to end diff --git a/lib/govuk_design_system_formbuilder/elements/null.rb b/lib/govuk_design_system_formbuilder/elements/null.rb index 0492c2d0..51cb08cb 100644 --- a/lib/govuk_design_system_formbuilder/elements/null.rb +++ b/lib/govuk_design_system_formbuilder/elements/null.rb @@ -1,7 +1,9 @@ module GOVUKDesignSystemFormBuilder module Elements class Null < Base - def initialize; end + def initialize + super(nil, nil, nil) + end def html nil diff --git a/lib/govuk_design_system_formbuilder/elements/submit.rb b/lib/govuk_design_system_formbuilder/elements/submit.rb index 16787a2d..05695072 100644 --- a/lib/govuk_design_system_formbuilder/elements/submit.rb +++ b/lib/govuk_design_system_formbuilder/elements/submit.rb @@ -4,9 +4,10 @@ class Submit < Base using PrefixableArray def initialize(builder, text, warning:, secondary:, classes:, prevent_double_click:, validate:, disabled:, &block) + super(builder, nil, nil) + fail ArgumentError, 'buttons can be warning or secondary' if warning && secondary - @builder = builder @text = text @prevent_double_click = prevent_double_click @warning = warning