Skip to content

Commit

Permalink
Merge pull request #96 from theonion/article-position-no-es6
Browse files Browse the repository at this point in the history
refactor es6 syntax
  • Loading branch information
brycedorn authored Nov 15, 2018
2 parents 8cedb9e + 39d1965 commit d96b800
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions src/helpers/TargetingPairs.js
Original file line number Diff line number Diff line change
Expand Up @@ -39,8 +39,10 @@ var TargetingPairs = {
*/
getArticlePosition: function (adContainer, kinjaMeta) {
if (document && adContainer && kinjaMeta.pageType === "permalink" && !(kinjaMeta.post && kinjaMeta.post.isFeatured)) {
var readingListPostIds = Array.from(document.getElementsByClassName('js_reading-list-item')).map(post => post.dataset.postId);

var readingListPostIds = Array.from(document.getElementsByClassName('js_reading-list-item')).map(function(post) {
return post.dataset.postId;
});

if (readingListPostIds.length > 0) {
var readingListItem = adContainer.closest('.js_reading-list-item');
if (readingListItem) {
Expand Down

0 comments on commit d96b800

Please sign in to comment.