ResourceResponseFactory to return a CacheableResourceResponse if request is cacheable

In my opinion, we should create a new branch 2.x to provide full compatibility with Drupal 9.1.x+ and improve phpdocs to return correct instance.

We don't need a new branch, Drupal 9 isn't supported. No longer blocked.

The code already has:

    // \Drupal\jsonapi\ResourceResponse no longer implements
    // CacheableResponseInterface in Drupal 9.1.
    // Drupal\jsonapi\CacheableResourceResponse has ben added for cacheable
    // responses. Keep compatibility with Drupal < 9.1.
    // See https://www.drupal.org/node/3163310
    $document = new JsonApiDocumentTopLevel($data, $includes, $links, $meta);
    $response = class_exists('\Drupal\jsonapi\CacheableResourceResponse') ?
      new CacheableResourceResponse($document, $response_code, $headers) :
      new ResourceResponse($document, $response_code, $headers);

From #3187549: Become compatible with Drupal 9.1 →

I think we can use this ticket to bump to 9.1 support minimum.

MR needs review. CacheableResourceResponse is only used if the request is cacheable, like Drupal core.

Thanks, everyone!

Automatically closed - issue fixed for 2 weeks with no activity.