Problem/Motivation

#2352239: Use base form id for tokens in order to minimize ajax requests on repeated forms (e.g. commerce add-to-cart) paves the road for reducing the number of Ajax requests considerably under some circumstances. This patch introduces the necessary changes for the Ajax component.

Proposed resolution

Remaining tasks

User interface changes

API changes

Comments

znerol’s picture

Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 1: 2352967-authcache-ajax-aggregate-requests.diff, failed testing.

Status: Needs work » Needs review
Colin @ PCMarket’s picture

Following with interest...

Is this patch likely to help speed up ubercart ajax requests also?

znerol’s picture

Is this patch likely to help speed up ubercart ajax requests also?

This patch does not speed up Ajax requests. Instead it makes sure that only one request is triggered for duplicated fragments (i.e. fragments with the same URL).

Together with the other patch referenced as a related issue, this will reduce the number of Ajax requests necessary to retrieve CSRF form tokens when one form is repeated multiple times on a page. This is a common pattern for the add-to-cart form in Drupal Commerce, but it also may apply to Ubercart.

  • znerol committed fa3af28 on
    Issue #2352967 by znerol: Cache results of Ajax requests
    
znerol’s picture

Status: Needs review » Fixed

  • znerol committed b3a9006 on 7.x-2.x
    Revert "Issue #2352967 by znerol: Cache results of Ajax requests"
    
    This...

  • znerol committed 55553ae on 7.x-2.x
    Revert "Revert "Issue #2352967 by znerol: Cache results of Ajax requests...

Status: Fixed » Closed (fixed)

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