if (substr_count($info['php'], '.') < 2) {
  $php_required .= '.*';
}

$php_required is never initialized so the value is either ".*" or "" so the user message reads funny since there is an incorrect value (or none) placed in it.

CommentFileSizeAuthor
#1 894614-php-compatibility.patch710 bytesboombatower

Comments

boombatower’s picture

boombatower commented
Assigned: Unassigned » boombatower
Status: Active » Needs review
StatusFileSize
new 894614-php-compatibility.patch710 bytes
solotandem’s picture

solotandem commented
Status: Needs review » Reviewed & tested by the community

Makes sense.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

#1: 894614-php-compatibility.patch queued for re-testing.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

Marked #806312: This module requires PHP version .* and is incompatible with PHP version as a duplicate of this issue since this one is RTBC.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Good catch, boombatower. Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)

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