Skip to content
GitLab
Commit 7e884f43 authored by Côme Chilliet's avatar Côme Chilliet
Browse files

:ambulance: fix(core) Use ... operator where it makes sense 

Instead of less readable func_get_args and call_user_func_array

issue #5827
parent ed65985c
Hide whitespace changes
Inline Side-by-side
Showing
with 12 additions and 17 deletions
+12 -17
  • bmortier @bmortier ·
    Maintainer

    SonarQube analysis reported 52 issues

    • :no_entry_sign: 1 critical
    • :warning: 39 major
    • :arrow_down_small: 4 minor
    • :information_source: 8 info

    Top 10 extra issues

    Note: The following issues were found on lines that were not modified in the commit. Because these issues can't be reported as line comments, they are summarized here:

    1. :no_entry_sign: Move this "case default" clause to the end of this "switch" statement. :blue_book:
    2. :warning: Remove the unused function parameter "$headline". :blue_book:
    3. :warning: This function has 8 parameters, which is greater than the 7 authorized. :blue_book:
    4. :warning: Define and throw a dedicated exception instead of using a generic one. :blue_book:
    5. :warning: Review the data-flow - use of uninitialized value. :blue_book:
    6. :warning: Define and throw a dedicated exception instead of using a generic one. :blue_book:
    7. :warning: Define and throw a dedicated exception instead of using a generic one. :blue_book:
    8. :warning: Remove the unused function parameter "$classes". :blue_book:
    9. :warning: Remove the unused function parameter "$entry". :blue_book:
    10. :warning: Reduce the number of returns of this function 4, down to the maximum allowed 3. :blue_book:
    • ... 42 more

    By Ghost User on 2018-05-02T14:11:50 (imported from GitLab)

Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment