[wp-trac] [WordPress Trac] #63028: PHP Warning caused by media.php and formatting.php

[WordPress Trac]

#63028: PHP Warning caused by media.php and formatting.php
-------------------------------+------------------------------
 Reporter:  isrgrajan          |       Owner:  (none)
     Type:  defect (bug)       |      Status:  new
 Priority:  normal             |   Milestone:  Awaiting Review
Component:  Media              |     Version:  6.7.2
 Severity:  normal             |  Resolution:
 Keywords:  reporter-feedback  |     Focuses:
-------------------------------+------------------------------

Comment (by isrgrajan):

 To simulate the condition where an attachment’s post parent is an array,
 even though in a normal `WP_Post` object it should be a single, scalar
 value:

 {{{
 // Assume $attachment_id is a valid attachment ID.
 $attachment = get_post( $attachment_id );

 // Normally, $attachment->post_parent is a scalar (e.g. an integer).
 // For testing, we deliberately convert it into an array to replicate the
 error.
 $attachment->post_parent = array( $attachment->post_parent );

 // Now, when processing the attachment, this will trigger the "Array to
 string conversion" warning.
 if ( $attachment->post_parent ) {
     // This line will produce a warning because $attachment->post_parent
 is an array.
     $post_parent = get_post( $attachment->post_parent );
 }
 }}}

 A `WP_Post` object's `post_parent` must be a single scalar (typically an
 integer). If it's an array, it triggers PHP warnings and risks
 unpredictable behaviour, compromising site stability.


 Replying to [comment:1 dd32]:
 > > Process media uploads where the attachment's post parent is set as an
 array.
 >
 > Can you provide an example to duplicate that?
 >
 > `$attachment` here should be a `WP_Post` object, which can only have a
 singular scalar ID.
 >

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/63028#comment:3>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform