Skip to content

Commit

Permalink
Flatten commentHandler
Browse files Browse the repository at this point in the history
  • Loading branch information
westonruter committed Oct 26, 2018
1 parent 4da1a3f commit 69eb395
Show file tree
Hide file tree
Showing 2 changed files with 24 additions and 27 deletions.
2 changes: 2 additions & 0 deletions .jshintignore
Original file line number Diff line number Diff line change
Expand Up @@ -2,3 +2,5 @@ wp-includes/js/service-worker.js
wp-includes/js/service-worker-navigation-routing.js
wp-includes/js/service-worker-precaching.js
wp-includes/js/service-worker-stream-combiner.js
amp/amp-service-worker-offline-commenting.js
wp-includes/js/service-worker-offline-commenting.js
49 changes: 22 additions & 27 deletions amp/amp-service-worker-offline-commenting.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,36 +10,31 @@
// @todo Make sure that 409 etc. error still work as expected.
return response;
} )
.catch( () => {
const bodyPromise = clone.blob();
return bodyPromise.then(
function( body ) {
const request = event.request;
const req = new Request( request.url, {
method: request.method,
headers: request.headers,
mode: 'same-origin',
credentials: request.credentials,
referrer: request.referrer,
redirect: 'manual',
body: body
} );
.catch( () => clone.blob() )
.then( ( body ) => {
const queuedRequest = new Request( event.request.url, {
method: event.request.method,
headers: event.request.headers,
mode: 'same-origin',
credentials: event.request.credentials,
referrer: event.request.referrer,
redirect: 'manual',
body: body
} );

// Add request to queue. @todo Replace when upgrading to Workbox v4!
queue.addRequest( req );
// Add request to queue. @todo Replace when upgrading to Workbox v4!
queue.addRequest( queuedRequest );

const jsonBody = JSON.stringify( { 'error': errorMessages.comment } );
return new Response( jsonBody, {
headers: {
'Access-Control-Allow-Credentials': 'true',
'Content-Type': 'application/json; charset=UTF-8',
'Access-Control-Expose-Headers': 'AMP-Access-Control-Allow-Source-Origin',
'AMP-Access-Control-Allow-Source-Origin': SITE_URL,
'Cache-Control': 'no-cache, must-revalidate, max-age=0'
}
} );
const jsonBody = JSON.stringify( { 'error': errorMessages.comment } );
return new Response( jsonBody, {
headers: {
'Access-Control-Allow-Credentials': 'true',
'Content-Type': 'application/json; charset=UTF-8',
'Access-Control-Expose-Headers': 'AMP-Access-Control-Allow-Source-Origin',
'AMP-Access-Control-Allow-Source-Origin': SITE_URL,
'Cache-Control': 'no-cache, must-revalidate, max-age=0'
}
);
} );
} );
};

Expand Down

0 comments on commit 69eb395

Please sign in to comment.