Skip to content

IBX-6554: Prevent Twig errors when location is not defined in Location View #1517

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 7 commits into
base: 4.6
Choose a base branch
from

Conversation

tbialcz
Copy link
Contributor

@tbialcz tbialcz commented Apr 7, 2025

🎫 Issue IBX-6554

Related PRs:

Description:

This PR updates the location_view.html.twig template to prevent runtime errors when location is not passed (e.g. when viewing deleted/trashed assets or content without an accessible location).

@tbialcz tbialcz changed the base branch from main to 4.6 April 7, 2025 07:41
@tbialcz tbialcz added Bug Something isn't working Ready for review labels Apr 7, 2025
Copy link

sonarqubecloud bot commented Apr 7, 2025

@tbialcz tbialcz requested a review from a team April 7, 2025 07:42
@adamwojs adamwojs changed the title IBX-6554: Fixed (location-view), prevent Twig errors when location is not defined IBX-6554: Prevent Twig errors when location is not defined in Location View May 24, 2025
@adamwojs
Copy link
Member

@tbialcz Rebase is needed here

@adamwojs adamwojs requested a review from a team May 24, 2025 08:23
tbialcz added 2 commits June 4, 2025 08:27
… not defined

# Conflicts:
#	src/bundle/Resources/views/themes/admin/content/location_view.html.twig
…om/ibexa/admin-ui into IBX-6554-location-view-null-checks

# Conflicts:
#	src/bundle/Resources/views/themes/admin/content/location_view.html.twig
@tbialcz tbialcz force-pushed the IBX-6554-location-view-null-checks branch 2 times, most recently from 26f260f to 2e515a3 Compare June 16, 2025 11:10
@tbialcz tbialcz force-pushed the IBX-6554-location-view-null-checks branch from 2e515a3 to 0f595d0 Compare June 16, 2025 11:11
@tbialcz tbialcz requested a review from Steveb-p June 16, 2025 11:11
@tbialcz tbialcz requested a review from alongosz June 18, 2025 12:35
@tbialcz tbialcz force-pushed the IBX-6554-location-view-null-checks branch from 22da30e to 4192309 Compare June 20, 2025 05:29
Copy link

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

Successfully merging this pull request may close these issues.

9 participants