Contrib.social

Make reloadPage() a protected method

Open on Drupal.org →
Created on 2 November 2024, about 2 months ago

Problem/Motivation

The NavigationExtraToolsController class includes the method reloadPage function as a public method. However, this does not need to be called from outside the class, so there's no need for it to be public.

Steps to reproduce

Regiew NavigationExtraToolsController class.

Proposed resolution

Change the declaration of reloadPage from public to protected.

Remaining tasks

  • Make the code change
  • Endure tests still pass

User interface changes

None.

API changes

None.

Data model changes

None.

📌 Task
Status

Active

Version

1.0

Component

Code

Created by

🇮🇪Ireland lostcarpark

Live updates comments and jobs are added and updated live.
  • Novice

    It would make a good project for someone who is new to the Drupal contribution process. It's preferred over Newbie.

Sign in to follow issues

Merge Requests

Comments & Activities

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024