Skip to content

Add missing docstring parameter#836

Open
LoLei wants to merge 1 commit intoSuiteCRM:hotfixfrom
LoLei:patch-4
Open

Add missing docstring parameter#836
LoLei wants to merge 1 commit intoSuiteCRM:hotfixfrom
LoLei:patch-4

Conversation

@LoLei
Copy link
Contributor

@LoLei LoLei commented Jan 19, 2026

Description

SuiteCRM 8.9 added this class member in NavbarHandler.php:

/**
 * @var array
 */
protected $navbarOverrides;

Commit: a387c6b

However it was missed to be added to the list of constructor docstring parameters.

Motivation and Context

This missing docstring parameter misaligns the list of docstring parameters with the actual constructor parameters.

How To Test This

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Documentation change

Final checklist

  • My code follows the code style of this project found here.
  • My change requires a change to the documentation.
  • I have read the How to Contribute guidelines.

@mattlorimer mattlorimer added Status: Requires Code Review PR 1-3 Complexity Score given to PRs once assessed labels Jan 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR 1-3 Complexity Score given to PRs once assessed Status: Passed Code Review Status: Requires Testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants