Automattic / Automattic/co-authors-plus

Separate out some functions

Open
#396 5 comments 0 reactions 0 assignees View on GitHub
Status: has patch type: enhancement
Dominant language
PHP
Stars
322
Forks
216
Avg merge
7h 50m
Merged PRs (30d)
65

Description

`co-authors-plus.php` is pretty big and unnecessarily so. It should be split up into more manageable chunks.

For example, `filter_ef_calendar_item_information_fields()` and `filter_ef_story_budget_term_column_value()` could move into the `integrations` folder, along with the AMP compat stuff.

Admin UI functionality could be split into a separate file/class too.

Contributor guide

Open the contributing guide

Research direction

Start with co-authors-plus.php and inventory its functions, then inspect the integrations folder. Review filter_ef_calendar_item_information_fields(), filter_ef_story_budget_term_column_value(), the AMP compatibility code, and the admin UI functionality to identify sensible boundaries. Done means these responsibilities are separated into manageable files or classes without changing existing behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
backend
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.