Skip to content

IBX-9813: Cacheable request query parameters#3033

Open
adriendupuis wants to merge 22 commits into5.0from
cacheable-query-params
Open

IBX-9813: Cacheable request query parameters#3033
adriendupuis wants to merge 22 commits into5.0from
cacheable-query-params

Conversation

@adriendupuis
Copy link
Contributor

@adriendupuis adriendupuis commented Feb 2, 2026

Question Answer
JIRA Ticket IBX-9813
Versions 4.6, 5.0
Edition Experience, Commerce

About cacheable_query_params block config and ibexa_append_cacheable_query_params() twig function.

Previews:

As ibexa_append_cacheable_query_params(block) is not needed until some custom frontend blocks are created with a cacheable_query_params property, I didn't add it to the sidebar.html.twigfiles used in docs/templating/render_content/render_page.md and tutorials/page_and_form_tutorial/2_prepare_the_landing_page.md

Checklist

  • Text renders correctly
  • Text has been checked with vale
  • Description metadata is up to date
  • Redirects cover removed/moved pages
  • Code samples are working
  • PHP code samples have been fixed with PHP CS fixer
  • Added link to this PR in relevant JIRA ticket or code PR

@adriendupuis adriendupuis mentioned this pull request Feb 2, 2026
7 tasks
@adriendupuis adriendupuis marked this pull request as ready for review February 3, 2026 13:55
@adriendupuis adriendupuis requested a review from barw4 February 3, 2026 13:55
adriendupuis and others added 2 commits February 3, 2026 16:01
Co-authored-by: Marek Nocoń <mnocon@users.noreply.github.com>
@adriendupuis adriendupuis requested review from barw4 and mnocon February 4, 2026 09:07
Copy link
Contributor

@mnocon mnocon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

Please remember that this feature is released and public only for 4.6 now - it will be included in the next v5 release (5.0.6)

@adriendupuis adriendupuis added the Wait with merge PRs that shouldn't be merged instantly label Feb 4, 2026
@adriendupuis
Copy link
Contributor Author

adriendupuis commented Feb 4, 2026

  • Already cherry-picked into branch 4.6 for v4.6.27
  • Waiting for v5.0.6 to merge into branch 5.0

@github-actions
Copy link

github-actions bot commented Feb 4, 2026

code_samples/ change report

Before (on target branch)After (in current PR)

code_samples/page/pagefield_layout.html.twig


code_samples/page/pagefield_layout.html.twig

docs/content_management/field_types/field_type_reference/pagefield.md@72:``` html+twig
docs/content_management/field_types/field_type_reference/pagefield.md@73:[[= include_file('code_samples/page/pagefield_layout.html.twig') =]]
docs/content_management/field_types/field_type_reference/pagefield.md@74:```

001⫶<div>
002⫶ {# The required attribute for the displayed zone #}
003⫶ <div data-ibexa-zone-id="{{ zones[0].id }}">
004⫶ {# If a zone with [0] index contains any blocks #}
005⫶ {% if zones[0].blocks %}
006⫶ {# for each block #}
007⫶ {% for block in blocks %}
008⫶ {# create a new layer with appropriate ID #}
009⫶ <div class="landing-page__block block_{{ block.type }}" data-ibexa-block-id="{{ block.id }}">
010⫶ {# render the block by using the "Ibexa\\Bundle\\FieldTypePage\\Controller\\BlockController::renderAction" controller #}
011⫶ {# location.id is the ID of the Location of the current content item, block.id is the ID of the current block #}
012⫶ {{ render_esi(controller('Ibexa\\Bundle\\FieldTypePage\\Controller\\BlockController::renderAction', {
013⫶ 'locationId': locationId,
014⫶ 'blockId': block.id,
015⫶ 'versionNo': versionInfo.versionNo,
016⫶ 'languageCode': field.languageCode
017⫶ }, ibexa_append_cacheable_query_params(block))) }}
018⫶ </div>
019⫶ {% endfor %}
020⫶ {% endif %}
021⫶ </div>
022⫶</div>

Download colorized diff

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

Labels

Wait with merge PRs that shouldn't be merged instantly

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants