LowFile-scopeRuby Plug & play

Move generic helpers to ApplicationHelper

Place cross-controller view helpers in ApplicationHelper (or a shared module) for reuse.

Why this matters

Avoids duplication and inconsistent behavior across views.

Side-by-side examples engineers can pattern-match during review.

Avoid

module OrdersHelper; def currency(v); number_to_currency(v); end end

Prefer

module ApplicationHelper; def currency(v); number_to_currency(v); end end

Bad

duplicate helpers across modules

Good

shared helper in ApplicationHelper

From the same buckets as this rule.

Critical
Avoid Duplicating String Literals
Store repeated string literals in constants or variables to improve maintainability and reduce redundancy.
duplication-complexitymaintainability
Critical
Avoid Redundant `None` Comparisons
Identify cases where `x is None` or `x is not None` is used redundantly when `x` is always or never `None`. This may indicate a deeper logic issue.
duplication-complexityreadability-refactor+1
High
Avoid Functions That Always Return the Same Value
Identify functions that return the same value regardless of input. This may indicate poor design or a logic error.
duplication-complexityreadability-refactor+1