Meta issue: #2205673: [meta] Remove all @deprecated functions marked "remove before 8.0"

CommentFileSizeAuthor
#1 remove-update_fetch_data_batch-2306437-1.patch852 bytesa_thakur

Comments

a_thakur’s picture

a_thakur commented
Status: Active » Needs review
StatusFileSize
new remove-update_fetch_data_batch-2306437-1.patch852 bytes

Please find the attached file. The use of the function is already removed.

[ashish@xeon:~/proj/self/d8/data/drupal8]$grep update_fetch_data_batch -r *
core/modules/update/update.fetch.inc:function update_fetch_data_batch(&$context) {

Only the function exists, no usage of the function exists, safe to be removed.

KarlKedrovsky’s picture

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

Searched for any mentions of update_fetch_data_branch and found no other references.

Checked to make sure the patch still applied cleanly.

Looked at all of the lines in the patch checking to make sure all of the comments and code were removed and that no additional lines were added or removed by the patch.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Needs work

Can we add this issue to the relevant change record.

a_thakur’s picture

a_thakur commented
Component: upload.module » update.module
penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
Issue tags: +Needs change record

Can be grouped with #2345371: Remove unused update_parse_xml() and update_fetch_data_batch(). I didn't find a relevant change record, maybe we need a new one.

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
Status: Needs work » Fixed

Oops, sorry, this was already commited as part of #2345371: Remove unused update_parse_xml() and update_fetch_data_batch(). Still needs the change record.

Status: Fixed » Closed (fixed)

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

cilefen’s picture

cilefen commented
Issue tags: -Needs change record