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

Remove functions:

  • element_property()
  • element_properties()
  • element_child()
  • element_children()
  • element_get_visible_children()
  • element_set_attributes()
bash-3.2$ grep element_set_attributes -nr *
core/includes/common.inc:1462:function element_set_attributes(array &$element, array $map) {
bash-3.2$ 
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

a_thakur’s picture

Status: Needs review » Active
a_thakur’s picture

Status: Active » Needs review
FileSize
1.15 KB

Please find the patch which removes the function.

daffie’s picture

Status: Needs review » Reviewed & tested by the community

Looks good to me.

alexpott’s picture

Status: Reviewed & tested by the community » Needs work

Please attached the relevant change record.

alexpott’s picture

Also please merge #2443793: Remove element_property() from common.inc and #2443807: Remove element_get_visible_children() from common.inc and remove the rest of the element_* from common.inc in this issue.

a_thakur’s picture

Patch attached.

a_thakur’s picture

Status: Needs work » Needs review
a_thakur’s picture

Link added to relevant change records as well: https://www.drupal.org/node/2173683

Status: Needs review » Needs work

The last submitted patch, 6: remove_element-2443805-6.patch, failed testing.

a_thakur’s picture

Title: Remove element_set_attributes() from common.inc » Remove element_* from common.inc
Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 6: remove_element-2443805-6.patch, failed testing.

daffie’s picture

a_thakur’s picture

Status: Postponed » Needs review
FileSize
4.11 KB

Usage have been fixed in https://www.drupal.org/node/2226621, please find the patch to delete the functions.

a_thakur’s picture

Assigned: a_thakur » Unassigned
JeroenT’s picture

Status: Needs review » Reviewed & tested by the community

Patch looks good. All tests pass.

ianthomas_uk’s picture

Status: Reviewed & tested by the community » Needs work

element_info and element_info_property should not be removed since they were not deprecated in time (see the comment)

Beta evaluation on the meta, #2205673: [META] Remove all @deprecated functions marked "remove before 8.0"

JeroenT’s picture

Status: Needs work » Needs review
Issue tags: +drupaldevdays
FileSize
3.03 KB

Created new patch that removes the following functions:

  • element_property()
  • element_properties()
  • element_child()
  • element_children()
  • element_get_visible_children()
  • element_set_attributes()
andypost’s picture

Issue summary: View changes
Status: Needs review » Reviewed & tested by the community
Parent issue: » #2205673: [META] Remove all @deprecated functions marked "remove before 8.0"
Related issues: +#2443793: Remove element_property() from common.inc
Mile23’s picture

Does in fact remove:

  • element_property()
  • element_properties()
  • element_child()
  • element_children()
  • element_get_visible_children()
  • element_set_attributes()

Also verified that those functions are marked for deprecation in 8.0.0.

Thus: +1 on RTBC.

catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.0.x, thanks!

  • catch committed b9114f6 on 8.0.x
    Issue #2443805 by a_thakur, JeroenT: Remove element_* from common.inc
    

Status: Fixed » Closed (fixed)

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