Skip to content
This repository has been archived by the owner on Jul 14, 2021. It is now read-only.

CI3 core Updates & Fixes 3.1.11 #1321

Open
wants to merge 10 commits into
base: develop
Choose a base branch
from
Open

Conversation

Reconix
Copy link
Contributor

@Reconix Reconix commented Dec 15, 2019

Added core update + patches
initial basic testing done.

PHP: 7.1.2
sqli:5.6.35

Next tests will be on the latest versions of php 7.2 & 7.3

Added core update + patches
initial basic testing done.

PHP: 7.1.2
sqli:5.6.35

Next tests will be on the latest versions of php 7.2 & 7.3
Added core update + patches
initial basic testing done.

PHP: 7.1.2
sqli:5.6.35

Next tests will be on the latest versions of php 7.2 & 7.3
@Reconix Reconix marked this pull request as ready for review December 15, 2019 22:23
@Reconix Reconix mentioned this pull request Dec 15, 2019
@locvfx
Copy link

locvfx commented Dec 16, 2019

Thank you

Fixed incorrect css parsing when value for url() contains data with parentheses

From: nqxcode
RemoveComments: Whitelist comments like copyright
@GintaWahyudi
Copy link

Added core update + patches
initial basic testing done.

PHP: 7.1.2
sqli:5.6.35

Next tests will be on the latest versions of php 7.2 & 7.3

thanks very much sir,..

@lonnieezell
Copy link
Owner

@Reconix Do you think this is stable and ready to merge?

@Reconix
Copy link
Contributor Author

Reconix commented Dec 17, 2019

PHP: 7.1.2
sql:5.6.35

Are ok but once you update sql to 5.7+ you get that horrible time issue where you've got to set nil.

php 7.2 + still needs proper testing.

@Reconix
Copy link
Contributor Author

Reconix commented Dec 17, 2019

The only other thing I could find, when trying to do a delete all in activities we get this

Error Number: 1292

Truncated incorrect DOUBLE value: 'long-value-here'

UPDATE `bf_activities` SET `deleted` = 1 WHERE `user_id` != 'long-value-here'

Filename: bonfire/bonfire/core/BF_Model.php

Line Number: 659

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants