Hello

I just saw that Views Megarow has a security update https://www.drupal.org/node/2726457 . Since I learned from @mglaman to make a module update, I add the patch in first comment.
Have a good coding :)

CommentFileSizeAuthor
#2 commerce_kickstart-views-megarow-security-update-2728847-1.patch458 bytescandelas
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

candelas created an issue. See original summary.

candelas’s picture

candelas CreditAttribution: candelas as a volunteer commented
FileSize
commerce_kickstart-views-megarow-security-update-2728847-1.patch458 bytes

I tested and it works :)

mglaman’s picture

mglaman
Primary language English
Location WI, USA
CreditAttribution: mglaman at Centarro commented

Thanks, we need to run this against Travis, through a GitHub PR. I will create that shortly.

mglaman’s picture

mglaman
Primary language English
Location WI, USA
CreditAttribution: mglaman at Centarro commented

Here is the PR https://github.com/commerceguys/commerce_kickstart/pull/201

Thanks for writing the patch!

  • mglaman committed 72aea1c on 7.x-2.x 
    Issue #2728847 by candelas, mglaman: Views Megarow security update (#201...
mglaman’s picture

mglaman
Primary language English
Location WI, USA
CreditAttribution: mglaman at Centarro commented
Status: Needs review » Fixed

Thanks! (sorry, I ran the merge through GitHub w/ squash and it borked authorship :/ TIL)

candelas’s picture

candelas CreditAttribution: candelas as a volunteer commented

@mglaman don't worry about authorship! It is less than a line... :)))
I don't know how to test with Travis. One more to learn... Next time I do it in github to be more efficient.
Thanks for so much work in CK.

Status: Fixed » Closed (fixed)

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