Skip to content

Commit

Permalink
fix typo and update formatting
Browse files Browse the repository at this point in the history
  • Loading branch information
Raphael Levin committed Oct 6, 2023
1 parent 531ad8a commit 24b2c68
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 4 deletions.
2 changes: 1 addition & 1 deletion spec/samples/inline_trackers.js
Original file line number Diff line number Diff line change
Expand Up @@ -234,7 +234,7 @@ export const inlineTrackersParsed = {
'https://www.iab.com/wp-content/uploads/2014/09/iab-tech-lab-6-644x290.png',
htmlResource:
'<!DOCTYPE html>\n <html lang="en">\n <head>\n <meta charset="UTF-8">\n <meta http-equiv="X-UA-Compatible" content="IE=edge">\n <meta name="viewport" content="width=device-width, initial-scale=1.0">\n <title>Document</title>\n </head>\n <body>\n <h1>titre</h1>\n </body>\n </html>',
iframeResource: 'liens vers html',
iframeResource: 'http://example.com/htmlresourcefile.html',
nonlinearClickThroughURLTemplate: 'https://iabtechlab.com',
nonlinearClickTrackingURLTemplates: [
{
Expand Down
2 changes: 1 addition & 1 deletion src/fetcher.js
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ export class Fetcher {
/**
* Fetches a VAST document for the given url.
* Returns a Promise which resolves,rejects according to the result of the request.
* @param {String} url - The url to request the VAST document.
* @param {String} url - The url to request the VAST document.
* @param {Number} wrapperDepth - How many times the current url has been wrapped.
* @param {String} previousUrl - Url of the previous VAST.
* @param {Object} wrapperAd - Previously parsed ad node (Wrapper) related to this fetching.
Expand Down
2 changes: 0 additions & 2 deletions src/parser/vast_parser.js
Original file line number Diff line number Diff line change
Expand Up @@ -359,9 +359,7 @@ export class VASTParser extends EventEmitter {
return new Promise((resolve) => {
// Going one level deeper in the wrapper chain
wrapperDepth++;

// We already have a resolved VAST ad, no need to resolve wrapper

if (!ad.nextWrapperURL) {
delete ad.nextWrapperURL;
return resolve(ad);
Expand Down

0 comments on commit 24b2c68

Please sign in to comment.