>> drush cr
PHP Parse error:  syntax error, unexpected ')' in /app/web/modules/contrib/user_expire/src/Form/UserExpireSettingsForm.php on line 44
 [warning] Drush command terminated abnormally.
CommentFileSizeAuthor
#3 user_expire_syntax_error.patch513 bytescyoon84

Comments

cyoon84 created an issue. See original summary.

cyoon84’s picture

cyoon84 commented
Status: Needs review » Active
cyoon84’s picture

cyoon84 commented
StatusFileSize
new user_expire_syntax_error.patch513 bytes
cyoon84’s picture

cyoon84 commented
Status: Active » Needs review
greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Title: throws syntax error when clearing the cache » Syntax error on php 7.2 and below, trailing comma in function call
Status: Needs review » Reviewed & tested by the community

Looks like this was introduced in #3095908: Drupal 9 Deprecated Code Report. It's valid syntax in PHP 7.3 and above. I'm not sure what version of PHP this module favors, necessarily. It seems better to support 7.2 if possible.

  • greggles committed 626fb83 on 8.x-1.x authored by cyoon84 
    Issue #3224046 by cyoon84, greggles: Syntax error on php 7.2 and below,...
greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Reviewed & tested by the community » Fixed

Thanks for the patch! Now committed.

Status: Fixed » Closed (fixed)

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