Skip to content

Consolidate get_post_types_names() helper #433

@ilicfilip

Description

@ilicfilip

Currently we are changing a lot of things related to content tracking & tasks.

In conversation with @aristath we mentioned that Content_Helpers class will probably be remove as not needed anymore.

In parallel we are adding valuable content settings and tasks in following PRs:

For now I have copied Content_Helpers::get_post_names() to class-settings.php in one of the branches so that helper is not removed if Content_Helpers is deleted, but we should find a better place for it once refactor is done (and also use it in all above PRs consistently).

Using just \progress_planner()->get_settings()->get( [ 'include_post_types' ], [ 'post', 'page'] ) gets the saved value properly, but since CPT properties (like public) can be changed dynamically I think it's better to have a helper in place which filters out CPTs which are no longer eligible.

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions