Update php requirement in mailchimp.info from 5.5 to 5.4 [#2825998]

Problem/Motivation

The 1.0.5 release of the PHP library for v3 of the MailChimp API reduced the minimum php version requirement from 5.5 to 5.4. The mailchimp.info file mentions php 5.5 as the minimum version.

Proposed resolution

Let's update the mailchimp.info file to reflect the correct minimum php version requirement.

Remaining tasks

Write a patch
Review
Commit

User interface changes

None.

API changes

None.

Data model changes

None.

Comment	File	Size	Author
#2	mailchimp-update_php_requirement-2825998-2.patch	648 bytes	idebr
#2	7.x-4.x: PHP 5.5 & MySQL 5.5, D7 Skipped, branch did not pass for 4 weeks PHP 5.6 & MySQL 5.5, D7 Skipped, branch did not pass for 4 weeks

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

9 November 2016 at 10:51

idebr created an issue. See original summary.

Comment #2

idebr CreditAttribution: idebr at One Shoe commented 9 November 2016 at 10:55

Status:

Active

» Needs review

File	Size
mailchimp-update_php_requirement-2825998-2.patch	648 bytes
7.x-4.x: PHP 5.5 & MySQL 5.5, D7 Skipped, branch did not pass for 4 weeks PHP 5.6 & MySQL 5.5, D7 Skipped, branch did not pass for 4 weeks

Attached patch updates the mailchimp.info file to reflect the correct minimum php version requirement.

Comment #3

DrupalDavie CreditAttribution: DrupalDavie as a volunteer commented 17 November 2016 at 17:21

I followed the Mailchimp upgrade instructions to replace the module (7.x-4.x), replace the library (v1.0.5), and a run database update. But I get an error message that I cannot update because Mailchimp requires at least PHP 5.5.

UPDATE: I selected the 5.5 version in the software section of cPanel, and was able to update.

Comment #4

juankvillegas CreditAttribution: juankvillegas commented 30 November 2016 at 20:00

Based on the latest release announcement, the required PHP version is 5.4, so the .info file should be updated.

Comment #5

Bès CreditAttribution: Bès at Happyculture commented 20 December 2016 at 15:43

Status:

Needs review

» Reviewed & tested by the community

I changed the php requirement and test the module on a php 5.4.45 environment. No error so far.
Code inspection with phpstorm didn't show major issue too.

Seems good to me.

Comment #6

idebr CreditAttribution: idebr commented 26 January 2017 at 14:45

Status:	Reviewed & tested by the community	» Closed (duplicate)
Related issues:		+#2846462: This module requires PHP version 5.5.* and is incompatible with PHP version 5.4.40

This was fixed in issue #2846462: This module requires PHP version 5.5.* and is incompatible with PHP version 5.4.40