Views 3.21 contains fixes for 3 security vulnerabilities. It also includes some of the patches Panopoly is using for PHP 7+ compatibility.

CommentFileSizeAuthor
#5 panopoly_core-views-321-update-3039961-5.patch856 bytescboyden
#2 panopoly_core-views-321-update-3039961-2.patch739 bytescboyden

Comments

cboyden created an issue. See original summary.

cboyden’s picture

cboyden commented
Assigned: cboyden » Unassigned
Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new panopoly_core-views-321-update-3039961-2.patch739 bytes

See patch to update panopoly_core.make with latest Views release.

bkosborne’s picture

bkosborne
Primary language English
Location New Jersey, USA
commented
Status: Needs review » Needs work

Those two patches still need to be applied. While it looks like their issues have been resolved/fixed, their code is not in 3.21 version of views. 3.21 is identical to 3.20 except for the security updates that were released.

bkosborne’s picture

bkosborne
Primary language English
Location New Jersey, USA
commented

I see that the php 7.2 compatibility patch won't apply to 3.21... it needs to be re-rolled.

cboyden’s picture

cboyden commented
Status: Needs work » Needs review
StatusFileSize
new panopoly_core-views-321-update-3039961-5.patch856 bytes

Thanks, here's a re-roll that includes an updated version of the Views patch from #2977851: PHP 7.2: count() on non-Countable in views_many_to_one_helper.

  • dsnopek committed f0dc47e on 7.x-1.x 
    Update Panopoly Core for Issue #3039961 by cboyden, bkosborne: Update to...
dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented
Status: Needs review » Fixed

Thanks @cboyden & @bkosborne!

I tested this manually a little bit, and then committed. The automated tests are running here:

https://travis-ci.org/panopoly/panopoly/builds/506452957

Once this comes back good, I'll release!

Status: Fixed » Closed (fixed)

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