Skip to content

Remove empty tasksearch_helper.php and its dead load calls#1299

Open
NAME-ASHWANIYADAV wants to merge 1 commit into
openml:developfrom
NAME-ASHWANIYADAV:cleanup/remove-dead-tasksearch-helper
Open

Remove empty tasksearch_helper.php and its dead load calls#1299
NAME-ASHWANIYADAV wants to merge 1 commit into
openml:developfrom
NAME-ASHWANIYADAV:cleanup/remove-dead-tasksearch-helper

Conversation

@NAME-ASHWANIYADAV

Copy link
Copy Markdown

Description

tasksearch_helper.php is a completely empty file - it contains only opening and closing PHP tags (<?php ?>) with no functions, constants, or logic. Despite being empty, it is loaded via $this->load->helper('tasksearch') in two controller constructors, adding unnecessary file I/O on every request.

Changes

  • Delete openml_OS/helpers/tasksearch_helper.php (empty file)
  • Remove $this->load->helper('tasksearch'); from Frontend.php constructor
  • Remove $this->load->helper('tasksearch'); from Backend.php constructor

Verification

  • Searched the entire codebase with grep -rn "tasksearch" - no other references exist (no function calls, no imports, no tests)
  • The helper defines zero functions, so removing it cannot break any callers
  • 3 files changed, 5 deletions, 0 additions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant