When a field is deleted the following error shows:
strict warning: Only variables should be passed by reference in cck/includes/content.crud.inc on line 550.

CommentFileSizeAuthor
#1 cck-n2466551-1.patch769 bytesDamienMcKenna
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Active » Needs review
FileSize
cck-n2466551-1.patch769 bytes

The problem was the return from content_field_instance_read() was being passed directly into array_pop(), whereas it needed to be stored in a variable first.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Related issues: +#1962718: Strict warning: Declaration should be compatible
Michelle’s picture

Michelle
she/her
Primary language English
Location Wisconsin, USA
CreditAttribution: Michelle at Mediacurrent commented
Status: Needs review » Reviewed & tested by the community

I confirmed this got rid of the error.

osopolar’s picture

osopolar
Primary language German
Location 🇩🇪 GER 🌐
CreditAttribution: osopolar commented

I marked issue #2328963: On fields delete from content type shows "PHP Warning : Only variables should be passed by reference" as duplicate, as this one is already RTBC.

Please add credit to rohit.wadhwa too, as he fixed this first.