Although drupal 6 reached end of life, not everyone has updated to Drupal 7 or 8 and still needs the Drupal 6 site.

These fixes will allow to update at least the php version to 5.6. See also #2411093: PHP strict warnings: Declaration of ... should be compatible with ...

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

osopolar created an issue. See original summary.

osopolar’s picture

Status: Active » Needs review
FileSize
2.05 KB
osopolar’s picture

Issue tags: +PHP 5.6
rjerome’s picture

I see that #2411093 hasn't been committed yet, so the question is does this patch break existing code?

Unfortunately, I can't test this right now because I'm currently running php 7 right now, and Drupal 6 won't even bootstrap with PHP 7 :-(

osopolar’s picture

I think this should not break existing code, but haven't tested jet. Before committing this you should wait until #2411093: PHP strict warnings: Declaration of ... should be compatible with ... is committed (to views 6.x-2.x and 6.x-3.x) – not sure if this will ever happen.

Anyway, I think this patch may help others who also need to update to a maintained php56 version.

Liam Morland’s picture

Status: Needs review » Closed (outdated)

This version is no longer maintained. If this issue is still relevant to the Drupal 7 version, please re-open and provide details.