Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Naming convention clash #742

Open
rizaardiyanto1412 opened this issue Feb 9, 2023 · 1 comment
Open

Naming convention clash #742

rizaardiyanto1412 opened this issue Feb 9, 2023 · 1 comment

Comments

@rizaardiyanto1412
Copy link

rizaardiyanto1412 commented Feb 9, 2023

Original discussion:
https://wordpress.org/support/topic/naming-convention-class/

Based on Ola

Checking the code now, I can see there’s series_group taxonomy and there’s series_grouping post type
It was series_group taxonomy label that was changed to Series Category

The plugin is using series_grouping post type in background and if you check where we registered the post type in https://github.com/publishpress/publishpress-series/blob/development/addons/grouping/legacy-includes.php#L189, it was registered as a private post type(user doesn’t need any interaction with it or have reasons to) as it’s not meant to be use by admin but for series_group(Series Category) usage and it doesn’t have to make sense to him. All he had to do is ignore it and no reason to have confusion over it.

It’s a post type been used in conjuction with series_group(Series Category) taxonomy. Check https://github.com/publishpress/publishpress-series/blob/development/addons/grouping/legacy-includes.php#L209 and it description reads ‘Used for associating Series with series categories’

The two, post type and taxonomy are legacy code we inherit and from my quick look into the code now, the purpose for registering the post type is to enable us register the series_grouping taxonomy as we need to associate it to a post type by default and that’s why we registered series_group post type.

@galbaras
Copy link

galbaras commented Feb 9, 2023

The point is that this post type is also associated with the category taxonomy, and this creates confusing in the Rank Math settings, and possibly elsewhere.

Also, since a series is a dimension of posts, isn't it simpler just to make it a taxonomy and link it to posts?

@stevejburge stevejburge added this to the 2.12.0: Bug Fixes milestone May 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants