Skip to content

IBX-10458: Refactored findContentTypes to use proxy objects #639

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

Open
wants to merge 2 commits into
base: ibx-10458-content-type-search-papi
Choose a base branch
from

Conversation

barw4
Copy link
Contributor

@barw4 barw4 commented Aug 22, 2025

🎫 Issue IBX-10458

Related PRs:

#633

Warning

Baseline updated for CI - has to be fixed

Description:

Query coming from findContentTypes:

SELECT c.id AS ezcontentclass_id, c.version AS ezcontentclass_version, c.serialized_name_list AS ezcontentclass_serialized_name_list, c.serialized_description_list AS ezcontentclass_serialized_description_list, c.identifier AS ezcontentclass_identifier, c.created AS ezcontentclass_created, c.modified AS ezcontentclass_modified, c.modifier_id AS ezcontentclass_modifier_id, c.creator_id AS ezcontentclass_creator_id, c.remote_id AS ezcontentclass_remote_id, c.url_alias_name AS ezcontentclass_url_alias_name, c.contentobject_name AS ezcontentclass_contentobject_name, c.is_container AS ezcontentclass_is_container, c.initial_language_id AS ezcontentclass_initial_language_id, c.always_available AS ezcontentclass_always_available, c.sort_field AS ezcontentclass_sort_field, c.sort_order AS ezcontentclass_sort_order, c.language_mask AS ezcontentclass_language_mask, GROUP_CONCAT(a.id) AS a_ids, GROUP_CONCAT(g.group_id) AS g_ids FROM ezcontentclass c LEFT JOIN ezcontentclass_attribute a ON (c.id = a.contentclass_id) AND (c.version = a.version) LEFT JOIN ezcontentclass_classgroup g ON (c.id = g.contentclass_id) AND (c.version = g.contentclass_version) WHERE EXISTS (SELECT g.contentclass_id FROM ezcontentclassgroup ctg LEFT JOIN ezcontentclass_classgroup c_group ON ctg.id = c_group.group_id WHERE (ctg.is_system = :dcValue1) AND (c_group.contentclass_id = c.id)) GROUP BY ezcontentclass_id, ezcontentclass_version, ezcontentclass_serialized_name_list, ezcontentclass_serialized_description_list, ezcontentclass_identifier, ezcontentclass_created, ezcontentclass_modified, ezcontentclass_modifier_id, ezcontentclass_creator_id, ezcontentclass_remote_id, ezcontentclass_url_alias_name, ezcontentclass_contentobject_name, ezcontentclass_is_container, ezcontentclass_initial_language_id, ezcontentclass_always_available, ezcontentclass_sort_field, ezcontentclass_sort_order, ezcontentclass_language_mask LIMIT 25

Explain:

id select_type table type possible_keys key key_len ref rows Extra
1 PRIMARY c_group ALL PRIMARY null null null 27 Start temporary; Using temporary; Using filesort
1 PRIMARY c ref PRIMARY PRIMARY 4 test.c_group.contentclass_id 1
1 PRIMARY ctg eq_ref PRIMARY PRIMARY 4 test.c_group.group_id 1 Using where
1 PRIMARY a ref ezcontentclass_attr_ccid ezcontentclass_attr_ccid 4 test.c_group.contentclass_id 4 Using where; Using index
1 PRIMARY g ref PRIMARY PRIMARY 8 test.c_group.contentclass_id,test.c.version 1 End temporary

For QA:

Documentation:

@barw4 barw4 marked this pull request as ready for review August 22, 2025 12:26
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
8.7% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

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

Successfully merging this pull request may close these issues.

4 participants