Resolve #3402388 "Pages with default"
2 unresolved threads
Closes #3402388
Merge request reports
Activity
15 class JsonapiDefaults implements JsonapiDefaultsInterface { 16 17 /** 18 * {@inheritdoc} 19 */ 20 public function getResourceConfigFromRequest(Request $request, ResourceType $resourceType = NULL): ?JsonapiResourceConfig { 21 $resourceType = !$resourceType ? $request->get(Routes::RESOURCE_TYPE_KEY) : $resourceType; 22 23 if ($resourceType instanceof ConfigurableResourceType) { 24 $relatedField = $request->attributes->get('_on_relationship') 25 ? NULL 26 : $request->attributes->get('related'); 27 $resourceType = static::correctResourceTypeOnRelated($relatedField, $resourceType); 28 29 if ( 30 $resourceType instanceof ConfigurableResourceType We need this check because we re-declare this variable after the first check:
$resourceType = static::correctResourceTypeOnRelated($relatedField, $resourceType);
.Edited by Ihor Chebotarov
49 * The entity type manager. 50 * @param \Drupal\Core\Entity\EntityFieldManagerInterface $field_manager 51 * The entity type field manager. 52 * @param \Drupal\jsonapi\ResourceType\ResourceTypeRepositoryInterface $resource_type_repository 53 * The JSON:API resource type repository. 54 * @param \Drupal\Core\Render\RendererInterface $renderer 55 * The renderer. 56 * @param \Drupal\Core\Entity\EntityRepositoryInterface $entity_repository 57 * The entity repository. 58 * @param \Drupal\jsonapi\IncludeResolver $include_resolver 59 * The include resolver. 60 * @param \Drupal\jsonapi\Access\EntityAccessChecker $entity_access_checker 61 * The JSON:API entity access checker. 62 * @param \Drupal\jsonapi\Context\FieldResolver $field_resolver 63 * The JSON:API field resolver. 64 * @param \Symfony\Component\Serializer\SerializerInterface|\Symfony\Component\Serializer\Normalizer\DenormalizerInterface $serializer changed this line in version 5 of the diff
added 1 commit
- a03776b9 - Fixed 500 error on the pages without default includes
added 2 commits
added 15 commits
-
8fd83869...e7583aec - 5 commits from branch
project:8.x-3.x
- 55ddfc2e - Test caching
- 94498ca8 - Add caching for includes
- f4d57c14 - Fixed var name
- 2f6dd8cb - Fixed code styles
- 7f480034 - Fixed 500 error on the pages without default includes
- c28ea24e - Fixed param types
- 146b7601 - Fixed phpcs error
- 8d8ddc85 - Fixed Kernel test
- 367c8bec - Issue #3402388 by ihor_allin: Solved an issue with the cached response with the wrong include keys
- c6e3d087 - Issue #3402388 by ihor_allin: Fixed issue in the check if the request uses only default includes.
Toggle commit list-
8fd83869...e7583aec - 5 commits from branch
Please register or sign in to reply