Maybe I missed the (Drupal) announcement, or there was none, but with the release of PHP 7.2 a few days ago, calling assert() with a string parameter is now deprecated. #2853503: Remove all assert('string') calls from Drupal core because deprecated in PHP 7.2 already changed all those calls to just use regular code, so we should do the same.

Comments

drunken monkey created an issue. See original summary.

drunken monkey’s picture

Status: Active » Needs review
Issue tags: -Novice
StatusFileSize
new2.88 KB

OK, the "Novice" tag was well-meant, but …

  • drunken monkey committed b26b3b4 on 8.x-1.x
    Issue #2928944 by drunken monkey: Removed assert() calls with string...
drunken monkey’s picture

Status: Needs review » Fixed

OK, looks good enough. Committed.

Status: Fixed » Closed (fixed)

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