Problem/Motivation

Bytes::toInt is used but is deprecated as of Drupal 9.1, toNumber should be used instead.

Steps to reproduce

N/A

Proposed resolution

Remove the deprecation.

Remaining tasks

Remove the deprecation.

User interface changes

N/A

API changes

N/A

Data model changes

N/A

Issue fork search_api_attachments-3202868

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

BramDriesen created an issue. See original summary.

BramDriesen opened merge request !2

bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented
Status: Active » Needs review

Merge request created.

bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented
Title: Bytes::toInt is deprecated » Bytes::toInt() is deprecated

Note that a specific Drupal 9 branch should be created if this is merged since this will break backwards compatibility with Drupal 9.

bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented
Title: Bytes::toInt() is deprecated » Bytes::toInt() is deprecated (D9/D10)
bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented
Issue tags: +Drupal 10 compatibility
izus’s picture

izus commented

adding the change notice : https://www.drupal.org/node/3162663

  • izus committed aedf522 on 8.x-1.x authored by BramDriesen 
    Issue #3202868 by BramDriesen, izus: Bytes::toInt() is deprecated (D9/...
izus’s picture

izus commented
Status: Needs review » Fixed

this is now merged, thanks

  • 6446c5c committed on 8.x-1.x 
    Revert "Issue #3202868 by BramDriesen, izus: Bytes::toInt() is...
izus’s picture

izus commented

i reverted this in 8.x-1.x and merged it 9.0.x

izus’s picture

izus commented

corrected the brancjh name in last command
the new branch is 9.0.x

Status: Fixed » Closed (fixed)

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