Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
PHP 7.1 introduced warnings about invalid strings in arithmetic. Memory limit calculation in MigrationBase constructor is performing multiplication on non-numeric string (e.g. 128M) which causes the following notice to be reported:
Notice: A non well formed numeric value encountered in MigrationBase->__construct() (line 425 of .../sites/all/modules/contrib/migrate/includes/base.inc).
Comment | File | Size | Author |
---|---|---|---|
#2 | non-well-formed-numeric-value-2868877-1.patch | 405 bytes | vaish |
Comments
Comment #2
vaish CreditAttribution: vaish at CivicSolar, Inc. commentedThis patch strips non-numeric character from the memory limit value before arithmetic calculations are performed on it.
Comment #3
vaish CreditAttribution: vaish at CivicSolar, Inc. commentedComment #4
vaish CreditAttribution: vaish at CivicSolar, Inc. commentedComment #5
camerongreen CreditAttribution: camerongreen commentedHaving the same issue, and this patch fixes it.
Comment #6
oadaeh CreditAttribution: oadaeh as a volunteer and at Flatt and Sons commentedThe patch in #2 works for me, too.
Comment #7
applicity_sam CreditAttribution: applicity_sam as a volunteer commentedPatch in #2 works for me.
Comment #8
jcisio CreditAttribution: jcisio as a volunteer and at Axess Open Web Services commentedRTBC again. Trivial patch.
Comment #10
pifagorComment #11
pifagorComment #12
oadaeh CreditAttribution: oadaeh as a volunteer and at Flatt and Sons commented@pifagor testing and reviewing patches are contributions, too.
Comment #13
pifagorSure. Sorry, i copied text here - "Git command".
Comment #14
oadaeh CreditAttribution: oadaeh as a volunteer and at Flatt and Sons commentedYou should only need to check the boxes next to the names under "Commit message" of those who helped and Save to update the contributions. It will update the message in "Git command," but it won't affect the actual commit message.
Comment #15
pifagorThank, you!
Comment #16
oadaeh CreditAttribution: oadaeh as a volunteer and at Flatt and Sons commentedБудь ласка.
Comment #17
heddnNo need to move things into closed (fixed) status. The issue queue will do that after 2 weeks after an issue is marked fixed.